Clean some phpcs errors in Preferences.php

Change-Id: I315e562c5f85e464a8ca61be6a47e2a468156a55

UserArray: Avoid * and double keys in select

Use User::selectFields to get all fields instead of *,
also use array_unqiue to make the sql nicer by having only single value
in the IN statement.

Change-Id: I0a4888be20f84f2955e82ced621aed526cc83f20

(bug 40889) Follow-up 6f1d4db3: Fix the upload form when radio buttons are shown.

The previous change had broken the upload form if URL uploads or
some other possible upload source types are enabled - i.e. in
the case when there are radio buttons. The "incorrect" label ID
was used by radiobutton.

Change-Id: I7c15fd4250b6a2d26af2cc0373da5ac6c884d690

Replace deprecated notifyRC2UDP() with notifyRCFeeds().

Change-Id: I54f6ce861ee0ecc6121df76e52ca3de16dd141fa

Use canonical casing for Xml::option (no code changes)

Per Umherirrender.

Change-Id: I520d71cc9969fe823122565faea9e725f96972f4
Follow-up: d0c0bad56d8ff3fe

Merge "Removed excess title validity check"

Make visibleMetadataField protected

Related patch sets:
* https://gerrit.wikimedia.org/r/#/c/99429/ Extension PagedTiffHandler

Change-Id: I0441fdf1ce1e06688f16c62e1993bb05911b45bc

Only show 'Show extended details' for file metadata having extra rows

When a file only has default visible metadata, there is no need
to allow extending the optional metadata, because there are none.

Also added comments in ImagePage.php and shared.css because this
system is scattered all over and each piece seems disconnected
(the shared.css block was incorrectly documented as only being
used to initially hide rows, wheareas in practice that never
happens and instead the collapsed state is toggled on click).

Change-Id: I595fed425135c1e8dd341258633217428bcb4154

Fix outdated comment in DefaultSettings

wgCopyUploadTimeout is used now.

Follow up to Ia85a97434c14adcdaafc0802cbe0530bfa57a435

Change-Id: I9838f3f13274f4276b6a51a50afdc64fa34554e2

Merge "Document the wikipage.content hook"

Document the wikipage.content hook

There is some text in the mw.hook documentation section, but that's
really intended to document mw.hook.

Change-Id: I2afb74a1124123220b26b50b3df64dd7bae80208

Add method to get entire text layer. This could be useful for search

Change-Id: I22f5fc47aef3cf362cdf630980deea48fe531d45

Merge "startup.js: log current time as global 'mediaWikiLoadStart'"

Merge "Blacklist Google Glass web browser from JS"

Merge "Fix WikiPage::prepareContentForEdit's default format handling"

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

Fix WikiPage::prepareContentForEdit's default format handling

If some caller passes null for the $serialization_format parameter,
they're going to break the in-memory caching of the prepared edit even
though it's likely that the format being used is the same as the format
being used by everything else.

It seems sensible to cache null as the default format for the content
object to avoid this.

Bug: 57026
Change-Id: I4ddcf6e388636e6a6397b664b8683ad774a09a04

Localisation updates from https://translatewiki.net.

Change-Id: Idd97bcf1748297b4633435fd73530d7bc14e2707

Fix TempFSFile creation

A new TempFSFile should be instantiated via the factory rather than
using the object constructor.

Change-Id: Icd059566087c6a644a0d0ed69abfe13b88b38ee4

Merge "Fix TextContent::preSaveTransform() return type"

Merge "Use TempFSFile in UploadFromUrl"

Fix TextContent::preSaveTransform() return type

TextContent::preSaveTransform should return an object of the same class
as $this, not a WikitextContent.

Apparently this was never noticed because most code paths rtrim the
input and TextContent::preSaveTransform returns $this if the rtrim is a
no-op on the input.

Bug: 58451
Change-Id: I9f8f78d52448d1ba055a654a12ff4fad86756110

Merge "Fix error handling in File::createThumb."

Merge "Made RepoGroup use ProcessCacheLRU"

Fix error handling in File::createThumb.

Was checking for null, but transform always returns either
false or an object.

createThumb isn't used in core, but some extensions use it.

Bug: 58436
Change-Id: Icaf5088a193dc1d9b9c365f92722bef1f9c1dba6

Update oojs to v1.0.6

Code:
 http://krinkle.github.io/oojs/releases/oojs-1.0.6.js

Release notes:
 https://github.com/wikimedia/oojs/blob/v1.0.6/History.md

Change-Id: I75e814385f697d92fdd1d22b3a0dbec3e0fe397e

Merge "mediawiki.mixins: Remove -o and -moz vendor prefixes for transition"

Update FSRepo to avoid the deprecated FileBackend construction style

Change-Id: Iaa8d57e22750582eb95e4abab7444ff291807416

Use TempFSFile in UploadFromUrl

Make it a little harder for the developer to leak files in the temp
directory by managing the temporary file with a TempFSFile instance.

Change-Id: I8bafc3a6e6212781d9e2e4f8f370ce7c446f1fd5

Merge "Removed stray ! in isQueueDeprioritized()"

Removed stray ! in isQueueDeprioritized()

Change-Id: I7a4b7f6205350dd61cb37a6cd9b57d40cd74e6c8

Localisation updates from https://translatewiki.net.

Change-Id: I256c74a670f5576e22e7ee6c9d3eefdade64c788

Merge "Avoid calling protected File field"

SpecialWantedcategories: Show more current information when in cached mode

Usability improvements I thought up when trying to through nearly
500 entries on this page on pl.wp.

* Strike items on the list if the category was created or emptied
  since the last update
* Show current number of items if it's different from the original one

Change-Id: Ifcc534f19fb60732bcd40f75ddca390106ebe28e

Avoid calling protected File field

bug: 58333
Change-Id: I0bd4a5fe9687c4261ca0f57e30f723e8bf2589ac

Add ApiBeforeMain hook

$wgExtensionFunctions is called before the RequestContext is really
available (i.e. getTitle() is null, accessing the User object may fail
because other extensions' UserLoadFromSession hooks don't expect to be
called that early, etc). The index.php code path has BeforeInitialize to
take action after RequestContext is available but before much else
happens, but the API doesn't (yet) have anything similar.

Bug: 58380
Change-Id: I0a167faaa4166da381194c53373d82e038848924

Merge "Make (Title|User)Array implements Countable"

Make (Title|User)Array implements Countable

count( $results ) always returned 1, now expected number

Bug: 58377
Change-Id: I7d6b3bc1b09b46fe60cb5a74c25f9b28fffa5b73

Merge "Always give sane output for Title::getUserCaseDBkey()"

Merge "mediawiki.js: Minor clean up of comments and code"

mediawiki.js: Minor clean up of comments and code

* Put ternary on one line or separated over 3 lines.
* Whitepace inside parentheses (e.g. catch statement).
* Avoid trailing line comments.
* Capitalise description of parameters for consistency and to
  have it better stand apart from the variable name.
* Phrase primary method block description as a single line
  in the imperative mood.
* Avoid shortened urls inside documentation.
  (updated Modernizr ref to a release instead of git hash).
* Using store.enabled as boolean (null being falsey).
* Merge subsequent if-statements having the same logical purpose
  and body.

Change-Id: Ia1db9e54d528b2799187d80d2e3e22e31f74fb6b

IcuCollation: Add CollationCkb subclass for Sorani Kurdish

ICU does not currently support Sorani Kurdish / Central Kurdish
language ('ckb'). CollationCkb uses the same collation rules as
Persian / Farsi ('fa'), but different characters for digits.

For use at ckb.wikipedia, which currently has 'uca-fa' collation
deployed as a workaround.

Added the MW language used for transforming digits to cache key for
first-letters data, in addition to the ICU locale.

Bug: 55630
Change-Id: I7d7f007592ede952859c5c9556b9ea5084b90e89

IcuCollation: Sort digits under localised digits' headings

Previously both '1' and '۱' ("DIGIT ONE" and "EXTENDED ARABIC-INDIC
DIGIT ONE") were sorted under '1' heading, regardless of collation
locale.

Now they will be both sorted under localised heading name (transformed
using Language#formatNum), for example '1' for 'uca-en' collation or
'۱' for 'uca-fa' collation.

Bug: 55630
Change-Id: I0b745d955a6e72f53873c95648087aa5f90a8852

Merge "Fix QueryPage transaction plan"

Better warning for Message object

Trying to track down the source of the production warnings

Change-Id: Iba666af3e84651fde338764d0f9d47a38b6b54e9

mediawiki.mixins: Remove -o and -moz vendor prefixes for transition

-moz is needed for Firefox <= 15.0, -o is needed for Opera <= 12.0 [1].
Usage of those browsers is negligible [2].

[1] http://caniuse.com/css-transitions
[2] http://stats.wikimedia.org/wikimedia/squids/SquidReportClients.htm

Change-Id: I43bcf1efacb87ab61b78140fe85bd846d791b54a

Always give sane output for Title::getUserCaseDBkey()

Sometimes (e.g. via Title::makeTitle), the usercase db key is
not set. If that happens we should fallback to the normal db key.

This fixes file redirects on repos where initialCapital is set
to false.

Change-Id: I38bc4ebeb41b06f5c718cf5f64da158e6c1a9f49

Merge "CSSMin: Correctly format 'url()' values with parentheses etc."

Merge "CSSMin: Change behavior for missing files"

Merge "CSSMin: Fix remapOne() for URLs that are proto-relative or have query part"

CSSMin: Correctly format 'url()' values with parentheses etc.

Introduce new static function, CSSMin::buildUrlValue.

Actually using such values in CSS does not work well because the
URL_REGEX is nowhere near good enough. :(

Change-Id: I04a7078dd0087bcb461fa5e5168c870d37c255f4

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

CSSMin: Change behavior for missing files

We would previously return the path to the local file on the
filesystem, which is useless in all cases and possibly a security
issue in some. Now we return the URL at which the file would be
accessible had it existed.

Also reordered the code around that part to make the control flow
clearer and added a test.

Change-Id: I1d5befb2ea385ae4d316c5d8c5d1fc092b64c4ff

Localisation updates from https://translatewiki.net.

Change-Id: I105dee236f2f05f91d0ffc4ae931c3012b48d938

CSSMin: Fix remapOne() for URLs that are proto-relative or have query part

Bug: 58338
Change-Id: I836a2c054ae3edc07895b2388f4ec8663223347a

Changed TIME_ONLY to NAME_AND_TIME_ONLY in FileRepo

* This now returns a (final DB key, timestamp) map for each file.
  This makes proper redirect handling easier for callers.
* Also fixed the case where ucfirst normalization is different in the
  repo versus the local wiki.
* Also added $flags to RepoGroup::findFiles() as expected.

Change-Id: I978a1cc0a0589179c46e477cfd06c8bff4c08fa9

Blacklist Google Glass web browser from JS

This will prevent things like collapsing sections on the Google Glass
web browser, and other interactive extras that really don't work well
on the limited UI (although the browser supports JS the interaction
model doesn't let you do much usefully, so a static page is better).

bug: 56008
Change-Id: I406d8131d4a8b3b98328e6c624433b67a18db645

Merge "Allow customizing classes inside of diffs"

Allow customizing classes inside of diffs

Not every implementation might want them

Change-Id: Id186b71760d59be7e61458bf1f62965a0f2f826c

Tweaked FileRepo::findFiles() behavoir to match LocalRepo

* This seems more intuitive and deals with an API bug

bug: 45930
Change-Id: I5e265bb9c6063306669cc4014eeb17cfda86436b

Add new message key to maintenace/messages.inc

Followup to Ica447a3b6f08422fd3c7452a5bd87d509dad9870

Change-Id: Ifed3c19dc1733c6454358f82ccd68dfd1f6c1e9a

Merge "Preferences: Improve accessibility of the JS tabs of Preferences"

Merge "Send QueryPage recache queries to a separate query group"

Revert "Module storage: randomly choose between Function and $.globalEval"

Did not get a chance to deploy this.

This reverts commit a20333cc4da04775db04b33407c88b4ccb5e5e74.

Change-Id: I6937d54897477f1942a8a9c312324a39d2a3868d

Fix QueryPage transaction plan

The COMMIT in updateSpecialPages.php was unmatched and just logged an
error. We would like the stale results to still be presented to the user
during the hours of query execution time, so doing a DELETE in
autocommit mode before the main query starts does not seem appropriate.
And holding a master transaction open for hours, with a lock on
querycache, is certainly not the right way to do it.

So, move the DELETE to after the completion of the main query, and wrap
a transaction around the updates to querycache and querycache_info so
that the user always sees a consistent populated UI. Remove the
unmatched COMMIT from updateSpecialPages.php.

Change-Id: I27c22b96f43a1064eb17a0c6a1c56d1f4a2dff9a

Send QueryPage recache queries to a separate query group

This essential feature was broken 3 years ago, in r78786. The group
connection was being used only for a call to the archaic
Database::fetchObject().

Change-Id: Ia4c76c88ecc77eb663a535b0625a1509882a35c7

Merge "Added --wikidb param to sql.php"

Merge "Module storage: randomly choose between Function and $.globalEval"

Added --wikidb param to sql.php

* This is useful for updating DBs that are not actually a that of a wiki.
  For example a global blocking or central auth DB may not correspond to
  any single wiki. Trying to use --wiki will fail with het deploy

Change-Id: I232182b79c9ca6421789f19771ba427498fff713

Merge "Add "nocreate" and "createonly" options to edit.php"

Merge "tests: beginning of tests for DjVu files"

Add "nocreate" and "createonly" options to edit.php

Change-Id: I2101360a934b8b9a92094c44429e3e2314c7fba3

tests: beginning of tests for DjVu files

Change-Id: I453b2b1a1c4999150a570e22170a9c630c2ef265

Merge "filebackend: cleaned up the FileBackend constructor"

Merge "API: Add iiprop=canonicaltitle"

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

Merge "Deleting a page and then immediately create-protecting it caused a PHP Fatal error"

Localisation updates from https://translatewiki.net.

Change-Id: I78257c1482d2a8a5a36693082c3228a63d0f4b8f

Merge "Remove problem forced index"

Merge "identify LogPager queries for load balancing"

Preferences: Improve accessibility of the JS tabs of Preferences

This enables keyboard accessibility of the tabs, by only allowing
focus on one tab element using tabindex=0 and tabindex=-1 for the
other tabs. Navigation between tabs is then handled by using the left
and right arrow keys. This is the advised methodology.

We also add tab, tabpanel and tablist roles to improve accessibility
for assistive technology, while overriding the implicit tablist role
of the li element with 'presentation' to make sure we don't have mixed
semantics of lists and tabs.

We keep track of:
aria-selected:   If this tab is currently selected
aria-controls:   Which tabpanel is controlled by this tab
aria-labelledby: Which tab is the label for this tabpanel
aria-hidden:     If this tabpanel is (not) visible

Tested using VoiceOver. Should also work with JAWS 14.

Change-Id: Ica447a3b6f08422fd3c7452a5bd87d509dad9870

Canonicalize text and update docs of messages using 'loginreqlink'

Since 'loginreqlink' is being used as a part of patchwork messages we
should try to keep the grammar the same in all of its uses.

"You have to log in" / "You must log in" / "Please log in" were all
used with various frequencies; I changed (hopefully) all uses to
"Please log in".

Two messages receiving 'loginreqlink' as a parameter do not currently
use it: 'changeemail-no-info' and 'resetpass-no-info'. This is
intentional, as they are less "positive" than the rest and thus should
not link to the login form.

Updated and cross-referenced qqq of 'loginreqlink' and messages using it.

Change-Id: Ie245486051d6fbc72a596ba825bfd5d9e3a163bf

Module storage: randomly choose between Function and $.globalEval

V8 disables certain optimizations for eval()'d code because scope resolution is
tricky. The same is not true for code compiled via Function(), which always
runs in global scope. Let's see if Function() is faster.

This patch makes ResourceLoader randomly use either Function() or $.globalEval.
The choice is recorded in a boolean `useFunction` property on mw.loader.store
so its value can be logged with load timing measurements. The logging will be
done in WikimediaEvents.

Bug: 58259
Change-Id: I7183778cb65c421ee19dcd61ee1dc0085f86bf10

Remove problem forced index

bug: 58256
Change-Id: I48484b39f99d92f51e531324870236b00b481553

Remove problem forced index. Bug 57175.

Change-Id: I94e0a743ccad0bad9a3a0d4d52b23abc0ef405b2

identify LogPager queries for load balancing

Change-Id: Ie89cd56700c3d6cbe19457b5b64576a71b3a9876

Merge "API: Add image sizes and default user prefs to meta=siteinfo"

Merge "Removed 'Disable browser page caching" user preference"

Merge "Status::getHTML should actually return HTML"

Merge "Don't assume test user has ID 1 in SpecialPageTest"

Localisation updates from https://translatewiki.net.

Change-Id: I8c000b7b635a9e3b964cdae08a3f5c87b14fa610

API: Add image sizes and default user prefs to meta=siteinfo

Bug: 58217
Change-Id: I33fc64681bc6924324d254b676686f3a6dee2769

Merge "Restore compatibility with curl < 7.16.2"

Merge "Split includes/SpecialPage.php"

Status::getHTML should actually return HTML

Currently it only returns wikitext with templates expanded (like
Message's text() method).

Bug: 45844
Change-Id: I24b5b098f15d0a4194817f31f63e37be1179aae6

Split includes/SpecialPage.php

Separated base classes for special pages (most of them abstract) and
several actual special pages, which were previously both in the same
file.

Added license headings, @ingroup SpecialPage and updated class doc
comments; no other changes.

Most classes are now in separate files (the list of changes can be
seen by diffing AutoLoader.php), with three exceptions:
* SpecialListAdmins and SpecialListBots (redirects to
  Special:ListUsers with a parameter) joined their friend in
  includes/specials/SpecialListusers.php.
* SpecialMycontributions, SpecialMypage, SpecialMytalk,
  SpecialMyuploads and SpecialAllMyUploads were moved into a new file
  includes/specials/SpecialMyRedirectPages.php – each of them is
  literally ten lines of code and any changes are likely to touch them
  all, separate files seem impractical.
* RedirectSpecialArticle and SpecialRedirectToSpecial are in one file
  with their parent class RedirectSpecialPage (on a side note, I filed
  bug 58215 to rename them to something less silly).

Change-Id: Ida87238401b182924dbe169a6278588bc2fbecfd

Restore compatibility with curl < 7.16.2

Follows-up 58f71c7e9eca, which assumed that the CURLOPT_CONNECTTIMEOUT_MS
constant was defined.

Bug: 58178
Change-Id: Id31a3ec56e4c7cac7d3edb2653c87be467d1a471

Added a TIME_ONLY option to FileRepo::findFiles to use less RAM

* Also renamed $possFile to just $file in method

Change-Id: Idb497901b9a7bd96c828267769a66efa282c5918

API: Add iiprop=canonicaltitle

With recent changes to the filerepo backends, prop=imageinfo will now
return information for the target file when file redirects are queried.
To make this more transparent to clients, add an iiprop to return the
canonical title for the imageinfo data.

Change-Id: I62e368ffe09512f1af6677994f1ccca776b97651

Merge "Formatting fixes in includes/htmlform/*"