Merge "adding data-attribute to sister-search sidebar results."

Merge "Hide Pig Latin language entirely if $wgUsePigLatinVariant is false"

adding data-attribute to sister-search sidebar results.

Adding source identifier (iwprefix) data-attribute
to interwiki search sidebar results.

Change-Id: I9e11a368fe4f5d79d81503fb4db1cc1649723839

Hide Pig Latin language entirely if $wgUsePigLatinVariant is false

Previously, even if $wgUsePigLatinVariant was false, the language
would show up on Special:Preferences (and some other places) as
'en-x-piglatin - Igpay Atinlay'.

Follow-up to d8375bee241b4d134aff901c7d072aa1374c860b.

Change-Id: I08faacabca87c04299c7b535be8df1770e0a37ac

API: Deprecate diff and parse parameters in ApiQueryRevisionsBase

The 'rvdifftotext', 'rvdifftotextpst', 'rvdiffto', 'rvexpandtemplates',
'rvgeneratexml', 'rvparse', and 'rvprop=parsetree' parameters to
prop=revisions are deprecated, as are the similarly named parameters to
prop=deletedrevisions, list=allrevisions, and list=alldeletedrevisions.

Use action=compare, action=parse, or action=expandtemplates instead.

Bug: T164106
Change-Id: I6deaf550a4b96f0d35488bea7bfd6637d6d1c36a

Merge "Fix call to makeContent in SpecialChangeContentModel"

Merge "Add namespace names for Atikamekw (atj)"

Merge "RCFilters: Always replaceState the URL"

Merge "Make DatabasePostgres::serverIsReadOnly() check default_transaction_read_only"

Add namespace names for Atikamekw (atj)

Bug: T167714
Change-Id: I5e98c80a8365707d73ed7e259466ab57cca7261b

build: Remove Travis CI allow_failure for Postgres

Now passing!

Follows-up 7399a3ec01.

Bug: T75174
Change-Id: I9cdd0aefa955f2bd91945969f6066ab67cda0d5f

mediawiki.storage: Document 'null' return from get()

Follows-up 7daab75414d. Forgot to document the full return scope
of HTML Storage#get, which is 'string or null'.

We may want to consider changing the 'false' return to 'null', so
that the concept of "localStorage disabled" is entirely hidden
behind the regular return values (null in case of #get).

Change-Id: I6bed62457d2e64831ec8241e16f7e8c85c6531a4

Merge "Localisation updates from https://translatewiki.net."

Merge "RCFilters: Add edit tags drop down"

Localisation updates from https://translatewiki.net.

Change-Id: If38ca86942d9591ab9f86f8584de5ad0d136a27f

RCFilters: Add edit tags drop down

Fetches the tags from the wiki and displays them as additional
filters for RCFilters.

Bug: T159942
Bug: T161650
Bug: T164130
Change-Id: I7bfa99cd5aeb34b6c7de74c15aac158ee40eac2f

Merge "Set flush mode in DatabasePostgres::reportQueryError()"

API: Split description messages into summary + additional text

To fix T123930, it would be very handy to have the summary for each
module without any of the additional explanatory text.

Bug: T166411
Change-Id: I778bab2b87971875aed4c804341ad59c1c6e021f

Merge "Add media type based filtering to Special:NewFiles"

Merge "Allow filtering on newbies in Special:NewFiles"

Merge "Show dimensions in TraditionalImageGallery"

Merge "Add | to error message about invalid change tag characters"

Fix call to makeContent in SpecialChangeContentModel

ContentHandler::makeContent expects serialized content as the first
parameter, not native data.

Change-Id: Ibf1825a191d10b4dd8b9a5cee95b02b2b91f3d92

Add missing dependencies to module 'mediawiki.api'

Change-Id: I62f6e75d840d07a4ae6676fa88c6b655aa575bba

Make DatabasePostgres::serverIsReadOnly() check default_transaction_read_only

Change-Id: If6088c15271a827fedc6a585e070bab6c1acc371

Set flush mode in DatabasePostgres::reportQueryError()

Bug: T167927
Change-Id: Ib02608cf33d369939a9e587cebfc736336a1e979

Fix typo "developement" in MockFSFile comment

Change-Id: If5beaff9bceea44a7e2d4d893b82a294e61afc84

Merge "objectcache: Forward MultiWriteBagOStuff::makeKey to primary backend"

Add | to error message about invalid change tag characters

Pipe itself is already not allowed.

Bug: T92956
Change-Id: Icb98b5e8040fa91685adcdfdb46e1173162f2b9e

Fix enhanced RC data attribute sanitizing

We push 'class' in the attribute array so the hook
can manipulate it, so it needs to be added to the attribute
whitelist as well.

Broken in I6dd006d0b1b0fd35c0020f0f9eea9113eca30b35.

Bug: T167922
Bug: T167535
Change-Id: Ic24400382a9dcbb990e12dfddae4ab7db14553cc

Merge "New language variant 'en-x-piglatin' for easier variant testing"

Merge "RCFilters: Add 'views' concept and a namespace view to RCFilters"

Merge "Replace deprecated $wgDummyLanguageCodes in WebInstallerLanguage.php"

Localisation updates from https://translatewiki.net.

Change-Id: I6dd2ce2c8fab01838c84bc016d04192e397fb3c9

Merge "Use strtr() for clarity"

RCFilters: Always replaceState the URL

Allow users to hit 'back' button to actually go back to their
previous page, rather than use 'back' button to undo their filter
toggling before they're allowed to go back to their previous
page.

Bug: T167919
Change-Id: I171799bd5c139addebcccc4799003f040c3f119b

Use strtr() for clarity

Change-Id: I35e7530c13fee028ea95a56df1bd2714a610db6e

RCFilters: Add 'views' concept and a namespace view to RCFilters

Enhanced RCFilters: Add the ability to filter by namespaces to RCFilters.
🎉 🎁 🎊

- Add the ability to separate groups of filters by 'views'
- Add the first views as 'default' (for predefined filters)
  and 'namespace' as the list of namespaces.
- Add 'nsinvert' to namespace group
- Allow highlighting namespaces
- Allow searching on either view, depending on prefix
- Add a way to switch views by typing prefix, clicking the
  'Namespaces' button or clicking a tag (either namespace
  or filter tag, changes the view accordingly, and adds
  or removes the prefix from the input to stay consistent)
- Add an optional wrapper text for tags, so we can represent
  them with their respective prefixes and (if needed) with
  a special message for inverted state.
- Add unit tests and make pass
- Bonus: Fix issue with URL not updating (and not being updated)
  the inverted and highlight enabled states.

Bug: T159942
Bug: T163521
Bug: T164130
Change-Id: I7e83f0800cbeb289dfd3461c1c5a197c053147ca

Merge "RCFilters: Make frontend URL follow backend rules and add 'urlversion=2'"

Merge "ChangesListSpecialPage: Add urlversion and make urlversion=2 ignore defaults"

Merge "Require $key in msg() functions"

Merge "Introduce MessageLocalizer interface for exposing msg() method"

RCFilters: Make frontend URL follow backend rules and add 'urlversion=2'

The backend always merges the query with wiki/user defaults before
it gives us data. The frontend, though, initially assumed that the
state is given strictly by the URL parameters (especially after the
URL shorening commit). This made it so that the frontend state is
incompatible with backend state.

However, always merging frontend state with user/wiki defaults can
produce inconsistencies between URLs in the same wiki, preventing
users from sharing them -- and making it potentially break if ever
a wiki default changes.

The solution is to add 'urlversion=2' to all RCFilters-generated
URLs and have the backend recognize this parameter as 'do not
merge with defaults'.

When RCFilters frontend loads, it checks whether the parameter
exists; if it doesn't, it merges whatever it sees with the defaults
just like the backend, then it transforms the URL to represent the
correct full state, and adds 'urlversion=2' to the URL parameters,
making it consistent across accounts and through time for the
next time it will load.

This means several new behaviors over the 'short url' commit:
- Accessing Special:RecentChanges directly (no query) will result
  in one of two things:
  -- If there is a saved query that's set to default:
     The system will load that saved query "straight forward" (as
     if the user clicked that option from the menu) causing, also,
     an ajax re-request from the server (since the server does not
     yet know about saved queries or their potential for being
     the default state.)
  -- If there is no saved query default: The system will load
     user/wiki defaults (like the backend does) and then fix the
     url to represent this state fully (with parameters showing the
     actual state of the filters.
  -- Both cases will also result in adding 'urlversion=2' to
     the end result URL.
- Accessing Special:RecentChanges?urlversion=2 (without any other
  parameters) will result in loading a completely empty filter set
  in RCFilters. We assume that 'urlversion=2' does not load defaults
  even if it is the only parameter in the URL.
- Accessing Special:RecentChanges?hideX=1 (parameter set without
  urlversion=2) will result in the front end taking the requested
  parameters, merging them with user/wiki default (reproducing what
  the backend does) and then adding urlversion=2 to the URL.

In all cases except for the default-saved-query-load case, the initial
load will **not** re-request data from the backend. The backend needs
to adjust to respect urlversion=2 as well (will come in an upcoming
commit) so the state and expectation of both the front- and back-end
are the same.

This commit also factors out URL handing to a separate class (UriProcessor)
and adds unit tests for it.

Bug: T166907
Bug: T166972
Bug: T166974
Change-Id: I0eed3bc0d4fa4810b6301b535c75b6bfbc8b4a5b

objectcache: Forward MultiWriteBagOStuff::makeKey to primary backend

Similar to what WANObjectCache and CachedBagOStuff are already doing.

Also add missing tests for WANObjectCache (similar to those for CachedBagOStuff).

Bug: T167465
Change-Id: I1a0c9324726aa6a1b221def985773b1b819181fd

ParserCache: Delete old-style key when saving

It was noticed that disk usage on the parser cache machines was
increasing since shortly after wmf.4 was redeployed everywhere on the
9th. One theory is that I7fb9ffca9 causes this by making reparses for an
existing old-style cache entry start writing the new-style key where
they would previously have overwritten the old-style key. On that
theory, let's delete that old-style key (that should now be useless) on
save.

I'm assuming here that firing a blind delete for keys that probably
don't exist in the cache (i.e. every new edit) isn't going to hurt
anything. If that's not the case, we'd need to check existence before
deleting.

Bug: T167784
Change-Id: Ie5efb05722cb7da2a90da195a1f244468177175d

Merge "Linked HTML representation of a format to formatted output"

Linked HTML representation of a format to formatted output

Added Hyperlink to non-HTML formatted output from the HTML
representation of the output

Bug: T123905
Change-Id: Id55c524a29f679b8e06d4e3d05bd50e8685fb830

Require $key in msg() functions

Note: calling msg() with no parameter was never supported,
doing this on a RequestContext for example would result in:
PHP Warning:  Missing argument 1 for wfMessage() ...followed
by a bunch of fallout.

So this patch only formally declares what was already a
requirement in reality.

Change-Id: I1864afb8bcc641698689828914949a06506d8f3a

Merge "Fix Postgres support"

Merge "Skin: Don't use parser cache in getCachedNotice()"

Skin: Don't use parser cache in getCachedNotice()

Just use wfGetCache( CACHE_ANYTHING ) which should be sufficient for
most cases.

Change-Id: Ic97549c9649d0cc1938773b10e26f6e8f819c7fa

DB: Add join conditions to selectField, selectFieldValues, and insertSelect

selectField() and selectFieldValues() are trivial, they just need to
pass it through to select(). In fact, selectFieldValues() was already
doing it, just no one ever updated IDatabase.

insertSelect() is a little more work. nativeInsertSelect() was
originally written as largely a copy-paste of select() and has since
gotten well out of sync. Now that we have selectSQLText(), we should be
able to just use that. DatabasePostgres's implementation can wrap the
parent implementation instead of being another copy-paste, but
DatabaseOracle seems to still need to be special.

Change-Id: I0e6a9e6daa510639d3212641606047a5db96c500

ParserOptions: Fix handling of 'editsection'

The handling of the 'editsection' option prior to I7fb9ffca9 was
unusual: it was included in the cache key, but the getter didn't ever
flag it as "used". This was overlooked in I7fb9ffca9.

This fixes the handling to restore that behavior. It's no longer
considered to be a real parser option, so changing it won't make
isSafeToCache() fail while reading it won't flag it as 'used'.

But to keep Wikibase working (see T85252), if 'editsection' is supplied
in $forOptions optionsHash() will still include it in the hash so
whatever Wikibase is doing by forcing that doesn't break. The hash when
it is included is the same as was used in I7fb9ffca9 to reuse keys.

Once optionsHashPre30() is removed, Wikibase should be changed to use
some other method to fix T85252 so we can remove that hack from
optionsHash().

Change-Id: I77b5519c5a1122a1fafbfc523b77b2268c0efeb1

Make Titles with an unknown namespace ID refer to Special:Badtitle.

Without this patch, Title::getPrefixedText() would return ":Foo"
if the namespace was unknown, potentially creating a misleading
link to the main namespace. With this change, getPrefixedText()
will return something like "Special:Badtitle/NS12345:Foo".

Note that round trip behavior is broken either way.

Bug: T165149
Change-Id: I0d491a2b58ff45f207f83ee62ca6e7e6ffbf790a

Merge "Hide <style> tags from Tidy"

Merge "RCFilters: Send form info even for empty checkboxes"

Merge "LoadMonitor: Group some logging parameters"

Merge "API: Restore ability for dieStatus() to be passed a non-fatal Status"

RCFilters: Send form info even for empty checkboxes

Make sure the url params override previous ones in case they are
empty, make sure we still send them through the fieldset form with
an empty value, rather than not send them at all (and then they are
being re-merged into the existing URL and become essentially 'sticky')

Change-Id: I294dba17c5a5b792f5bf5064a76dedefa4d12557

LoadMonitor: Group some logging parameters

Change-Id: Iefd59db547d799a02edc9f733cadb7668176d4cd

Remove "@author Umherirrender" annotations

Also remove my name from i18n files I am not translate

Bug: T139301
Change-Id: If806324cd07e662f5e995e11cf899b5b934cd79b

Merge "Update magic words and special page aliases for Arabic and Egyptian Arabic"

Merge "ApiParse: Fetch content with parsetree"

Localisation updates from https://translatewiki.net.

Change-Id: I3aa05ab2b7d18f6d134e4b395e2a50526cba5630

ApiParse: Fetch content with parsetree

I0079e10a assumed callers were correctly passing $getContent to
getParsedContent(). Apparently they weren't.

Bug: T167826
Change-Id: I85cbdfb5d0580252c1df23a0f27c8052f1d65dc3

Hide <style> tags from Tidy

Some versions of html-tidy (e.g. the one currently in use on WMF wikis)
will try to move all <style> tags in the body into the head, effectively
removing them for our purposes. We need to avoid that for TemplateStyles.

Bug: T167349
Change-Id: I133776d16f366cad73ed30af0e5a665fdf9f5ed9

Update magic words and special page aliases for Arabic and Egyptian Arabic

It is requested by User:Meno25 to update magic words and special page aliases for Arabic and Egyptian Arabic languages.

Bug: T143880
Change-Id: I5701ab523aea6c9a933586c86a478c4dbcc8cf7a

API: Restore ability for dieStatus() to be passed a non-fatal Status

This ability was accidentally removed in Iae0e2ce3.

Bug: T167690
Change-Id: I79cc9f2f8e75c3cf4e0942d17faafa84955aa020

Make Special:PageData accept two-part subpage

Bug: T163923
Change-Id: I9fc55fd1eb956871fc5dd3e104c6786d380ef445

Merge "Disable addHelpLink when including special pages"

Merge "ApiParse: Clean up parsing code"

ChangesListSpecialPage: Add urlversion and make urlversion=2 ignore defaults

This allows us to build reliable URLs that will be consistent over time
and between users (because the defaults depend on preferences,
extension presence, etC) by using urlversion=2 and specifying the
state of each filter relative to the empty state (i.e. nothing enabled).

urlversion defaults to 1, and urlversion=1 maintains the current
behavior where the parameters are interpreted relative to the
user's default state.

Bug: T166906
Change-Id: Iaf33c14e3f909092d96453e78016814aa417673a

New language variant 'en-x-piglatin' for easier variant testing

Guarded by the $wgUsePigLatinVariant variable, off by default.

Pig Latin is a language game where words in English are altered
according to the following rules:

* Words starting with a vowel have a '-way' suffix appended.
* Words starting with a consonant have the initial consonants (or 'qu'
  group) moved to the end and an '-ay' suffix appended.

https://en.wikipedia.org/wiki/Pig_Latin

* Added 'en-x-piglatin' as a language name.
* Added 'en' to LanguageConverter::$languagesWithVariants.
* Added LanguageEn class and its corresponding EnConverter which
  provides one-way translation from English to Pig Latin.
* Some minor internal changes in code that assumed that English
  doesn't have a language class or converter.

Bug: T45547
Depends-On: I1d9691c784032669979f8109c9a5f65cbf4122c9
Change-Id: I7fa2d85d6364958c5138366e8b4504a2697a8731

Merge "Sync up with Parsoid parserTests.txt"

Merge "RELEASE-NOTES: OutputPage::enableTOC was removed"

Localisation updates from https://translatewiki.net.

Change-Id: I11b382a06772ebb9f908ba543f2276a8cf6e1a8b

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit 108eed81b3eb78b77ade5ba5daac71fb43fff6de

Change-Id: Ib2b31f008adaf16866de16ef963bc58d6cabb088

Merge "Delete CologneBlue messages"

Delete CologneBlue messages

They are moved to the CologneBlue repo.

Bug: T159921
Depends-On: Ide5c96b418f6abfb4244efa777c57ec2ffa9f087
Depends-On: Ib46d62e2464d7292482e8ea63ce187987104c59d
Depends-On: I522a52e548b88d0904f6890ddc7d0adb5d54eba2
Change-Id: Ib262d89e4485dfff2ef7ee9bbaa1152604626c4c

Merge "Allow CategoryPageView to set $mCategoryViewerClass"

Merge "Remove X-Content-Dimensions header"

Merge "Log when transactions affect many rows in TransactionProfiler"

Merge "Add $wgMaxJobDBWriteDuration setting for avoiding replication lag"

Merge "Allow skins to control style of mediawiki.hlist"

Remove X-Content-Dimensions header

Reverts 84e4d7508893, and parts of cdfe08439c3b and 4511f6fa9fa0.

Bug: T150741
Bug: T167034
Change-Id: I39cfcf2cb231b6dfef569968fba6f473da258916

FileDeleteForm: Create log entry even if the page didn't exist

Deleting a file with no description page should still produce a log
entry.

Bug: T167712
Change-Id: Icc7d9d12365d38faa502689ceefb4d1ee5f60210

Merge "Do not try to parse empty argument in getErrorsOrWarnings in OOUI"

Merge "RCFilters: Retain extra url params when comparing url equivalency"

RELEASE-NOTES: OutputPage::enableTOC was removed

Follows-up c7e00974c7eb1 (T130632).

Also announced on Wikitech:
<https://lists.wikimedia.org/pipermail/wikitech-l/2017-June/088301.html>

Change-Id: I9dde207d2314993b9e88e6bb000b0a11b37799ad

API: Allow deleting files without corresponding pages

Such a deletion doesn't currently produce a log entry, but that's a
different bug that also occurs via the web UI.

Bug: T167693
Change-Id: If6e751aa28960243db49ac9b81fe518edba11bd5

ApiParse: Clean up parsing code

Now that ParserOptions->isSafeToCache() exists, use it where necessary.
This also moves the use inside the makeParserOptions() method so other
callers can pick it up as well.

Then pass the flag as $forceParse into WikiPage::getParserOutput()
instead of duplicating the logic in several cases, and generally clean
up the logic in the module to let WikiPage decide when to use the cache
in more cases.

Change-Id: I0079e10a40997e4a3b59ac21ef6c92246a147736

Remove unused and unnecessary imports

Change-Id: I26e623a4e4ba965c07670369a90c8a95185ea1e4

Move HttpAccept* to libs

Bug: T163923
Change-Id: I4984f6b77843669950afeff82351827cd416a80d

Use "target" instead "title" as the param name in SpecialPageData

Using "title" confuses mediawiki and causes redirect loops

Bug: T163923
Change-Id: I29e0e6a61703cc04397205a3f7f38df5906622e1

RCFilters: Retain extra url params when comparing url equivalency

The extra params should also dictate whether the URL should change,
so we update parameters that are external to RCFilters' model, like
limit and from, etc.

Bug: T167551
Change-Id: I21e35e7871abbc57634231b6330cc1fad8299860

Merge "Start a very basic version of Special:PageData"

Merge "Move HttpAcceptNegotiator and HttpAcceptParser from Wikibase to core"

Do not try to parse empty argument in getErrorsOrWarnings in OOUI

Bug: T167644
Change-Id: I0dcb9bd51ad0cecbdab868fed056b4077d1c9fa8

Introduce MessageLocalizer interface for exposing msg() method

Bug: T162594
Change-Id: I2d9af9fc05264bffda675af46f656038cd72d0db

Merge "SpecialNewimages: Do not add the module when the special page is included"