Merge "Avoid an infinite redirect in $wgSecureLogin handling"

MWException: Remove deprecated methods

No callers in core or extensions.

Change-Id: I3e2dc615bcd23838cf096625a767a7cf50fea140

Merge "Update OOjs UI to v0.1.0-pre (80f1797a5c)"

Update OOjs UI to v0.1.0-pre (80f1797a5c)

New changes:
9e4d2c6 Remove @abstract from classes which can be instantiated
0b43fbe Button accessibility
f61558d Add confirmation dialog
f52f1f2 Localisation updates from https://translatewiki.net.
d6849d5 build: Update jscs and jshint config
d0a5670 ConfirmationDialog clean up
513cb9d Auto-hide menus by default

Local changes:
Add the confirmation dialog messages to MW core's list in Resources.php

Bug: 50955
Change-Id: I67329820a631f46ed537d93398f35456f0e58679

Localisation updates from https://translatewiki.net.

Change-Id: I7826e5f6fdb2e4e2af623e8fdfc44323469aebac

Merge "Linker.php: Do not double escape accesskey in tooltip"

Merge "Implement mediawiki.cookie module"

Use FOR UPDATE in prior LocalFile timestamp check

* This covers the case with a snapshotted transaction outside of lock()/unlock()

Change-Id: I8639c2e51cb918f452dc1576681d6224fb4fb2c0

Remove unused variables from SpecialAllpages::showToplevel()

Follow-up If750cad676 (71fe7c5)

Change-Id: I95bf953b0eb742e94ddff1b6adf6d784fb5dc87c

Implement mediawiki.cookie module

Provides functonality similar to WebRequest#getCookie and WebResponse#setcookie.

Wraps $.cookie and automatically takes care of wgCookiePrefix etc.

Bug: 49156
Change-Id: I217ef258aecf1acd335e2cea56ae08b22541c7d4
Co-Author: Matthew Flaschen <mflaschen@wikimedia.org>
Co-Author: Timo Tijhof <krinklemail@gmail.com>

Avoid an infinite redirect in $wgSecureLogin handling

If, for some reason, (e.g. https://gerrit.wikimedia.org/r/#/c/133205/ )
HTTPS requests are not recognised as such and WebRequest::getProtocol()
unconditionally returns "http", use the fromhttp parameter to avoid an
infinite redirect loop.

Change-Id: Iefbf679ef995f5e347e2b9c3f1f5a5b22055e107

Merge "qunit: Make use of 'config' and 'messages' shortcuts in module setup"

qunit: Make use of 'config' and 'messages' shortcuts in module setup

While calling mw.config and mw.messages manually works fine (the sandbox
covers mw.config and mw.messages in general, not just when using the
shortcut), let's consistently use the shortcuts.

Also clean up the extra local variable in some cases.

Change-Id: I9ac467ad87816c7dc681b62e9b085b643024c15e

Merge "MWException: Don't send headers multiple times"

Merge "Made LocalFile avoid duplicate (name,timestamp) pairs"

Made LocalFile avoid duplicate (name,timestamp) pairs

* Various bits of code seem to assume uniqueness already

bug: 65264
Change-Id: Ib00fdbe87a79296c640cd69d74928236ce7a4aee

Merge "Revert "Simplify thumbnail styling to have consistency with mobile implementation""

Revert "Simplify thumbnail styling to have consistency with mobile implementation"

Causes significant breakage for videos, author not around this week to address concerns.

This reverts commit b0012d2aeb6a5ef8172223c58ceb8fa5c78c5138.

Change-Id: If1d7ba81b47b9351427111a9060ea9893c87c96e

MWException: Don't send headers multiple times

Ideally we wouldn't have sent headers yet, but in some weird
code paths where we output stuff raw (like Export) we might
have already sent them.

All depends on when the failure happened.

Should silence all of the "headers already sent" being seen
from exports.

Change-Id: I12e0532e93d30b2255f73a9d0e017c73e30c3e28

Merge "Improve a bit the code of SpecialAllpages::showChunk()"

Merge "Installer: Minor tweaks to UI"

Merge "jquery.accessKeyLabel: Replace hardcoded brackets and space"

Merge "Make phpcs-strict pass on includes/ (6/~10)"

Merge "Allow floating point values for $wgJobBackoffThrottling"

Merge "Update documentation and method visibility for SkinTemplate"

Localisation updates from https://translatewiki.net.

Change-Id: I2be71c60446c0de60944b9a1439ec52e4c99e129

Merge "Update the API credits"

Merge "Improved rate limit log to mention IP"

Improve a bit the code of SpecialAllpages::showChunk()

- Early return when including the page, makes one indentation
  level less for the rest of the method
- Directly put top and bottom links in an array, so that it is
  easier to see where they are going to be displayed
- Group the HTML generation for the top form at the end of the
  method again for better readability

Change-Id: I70c174a4c6363b2303cb5110782c5a3375640f2d

jquery.accessKeyLabel: Replace hardcoded brackets and space

Since If914e26f2af23f04c46f011ad8a276a9d3190583 the brackets of the
accesskey are localized, so do it also on the client side.

Also added a comment to match php and js.
Adjust tests to work on wikis with changed messages.

Bug: 65010
Change-Id: Id9908b9efcb923f0b740e2c8fd77d1ef676b489f

Installer: Minor tweaks to UI

- add a tooltip to the "help" links to make it clear they'll expand in place
- don't use an IMG tag for the download link;
  instead, apply a css background to the A element, so that on hover
  there isn't an underlined space between the link text and the image;
- grammar fix to sqlite help message: replace a colon with a semicolon.

Change-Id: Iacf5ed57d983a21d87467c848823607032901cec

Merge "Revert "Make it possible to install extensions using Composer""

Merge "API: Fix documentation for ApiBase::require*OneParameter"

Require one of page id or page title as params to ApiRollback

Also made the code flow easier to understand by moving
calls to ApiBase::extractRequestParams up in the stack.

Change-Id: I346c6ae4fc0b48c5a794c3d621018baf9bde6b9e
Follows-up: I9c4e4fa

API: Fix documentation for ApiBase::require*OneParameter

The existing doc blocks for all three of these functions is entirely
wrong.
* ApiBase::requireOnlyOneParameter
* ApiBase::requireMaxOneParameter
* ApiBase::requireAtLeastOneParameter

Change-Id: Id0dd86be3cbfa813a43f918c7d780c7ee2afc09e

Update the API credits

Put Brad Jorsch (anomie) as lead developer (as he's the one
who does most of the review work etc. these days).

Made Yuri's lead developer time frame end in 2013 (as he
last commited to the api in 2013 AFAIS).

Change-Id: I640ddf251340214b4520d3072e22dfe6b54815fb

Allow use of ApiRollback using pageid instead of title

Change-Id: I9c4e4fa59af0b84416d950f1428da821318f85fc

Remove user id from Special:Preferences

Dating back all the way to r15, it serves no useful purpose
as users typically do not need to know their own user id.
If they really want to, they can easily look it up using the
API.

Bug: 58626
Change-Id: Ib146abb8b579bc9a3dac601aff8e2818b2ed49e4

Revert "Add API action=watch 'entirewatchlist' param to bulk-clear watchlist."

Unbounded modification = replication lag. And this function is specifically intended for clearing very large watchlists which would easily escalate to pageable lags of several minutes.

This reverts commit 66b07ea38133992f98bb158c95103b720fe958ec.

Change-Id: Ie34be712884407e671d715e56bda3bff4a7e78f4

Localisation updates from https://translatewiki.net.

Change-Id: I4f58769751a4a439ab93f7d22c427c49857d4c22

Merge "Remove messages.inc, rebuildLanguage.php, writeMessagesArray.inc"

Make phpcs-strict pass on includes/ (6/~10)

Change-Id: I566183b5d660a55bb3b2aa7186aaed5355ead2c6

Merge "Add two more categories to Special:TrackingCategories"

Add two more categories to Special:TrackingCategories

Bug: 63795
Change-Id: I303fd3faa00aa0d3ca8931bea0d7c5ce5a263184

Remove messages.inc, rebuildLanguage.php, writeMessagesArray.inc

They are not being used anymore since the messages lists are in JSON
format, no idea why they were still here. Especially messages.inc is
just confusing everyone adding or removing messages.

Change-Id: If104ca1609e122b6871a97521065916a26734516

Update documentation and method visibility for SkinTemplate

Change-Id: I7c23e6c9f5189378fd1d33230820e8fd157a4750

Merge "Make update.php file executable"

Release notes for c0c39640e472332

Change-Id: I008dea87357344cf9fc9d530cd0e84edcaba78f9

Preprocessor_DOM::newPartNodeArray should check that loadXML succeeded

If something manages to get invalid UTF-8 into
Preprocessor_DOM::newPartNodeArray, or anything else that somehow is
invalid XML, it should handle it in the same way that
Preprocessor_DOM::preprocessToObj does rather than having something
further down the line blow up on a PPNode_DOM with a null node.

Bug: 65081
Change-Id: Ic24db455808106e17d49a11e41df33ec170f1206

Merge "tyop fxi"

SpecialAllpages: Remove fancy index functionality

Causes more trouble than it's worth.

Bug: 58051
Bug: 65159
Change-Id: If750cad67690df301608951b72f89ae18851e71e

Merge "PPAccum_Hash -> PPDAccum_Hash"

Replace complex for loop for foreach, count and break

Change-Id: If1d3193f9579ce6b0d3c790b234d56ea1fe73237

tyop fxi

Change-Id: I05cd37b3dab7d0eeffb4df81f906fd1e881b7f9f

Make phpcs-strict pass on includes/ (5/~10)

Change-Id: I259f3f11cfc22f3ed1693f9ebd5bd80491b8a6e8

Make phpcs-strict pass on includes/ (4/~10)

Change-Id: I2767205a7059adfb8fa090a82abd332e518f2590

Make phpcs-strict pass on includes/ (3/~10)

Change-Id: Ibf86d03b5479c47ee72c0dafea6777ef8178fe68

Make phpcs-strict pass on includes/ (2/~10)

Change-Id: I59fa9af7b16e0a5a4eb8a5cc764a605b18137316

Make phpcs-strict pass on includes/ (1/~10)

Change-Id: Ib51381a2261d064988ba2f39b71c0252f2458faf

Localisation updates from https://translatewiki.net.

Change-Id: I8e7928ea4e57fcc0f2fc2c09fa8bed3a05cdf55d

Pass phpcs-strict on includes/objectcache/

Change-Id: Ib99f2e34255df0cc846727b134d1788aec48529d

Pass php-strict for OutputPage

Change-Id: Ia5d03277fff543ca672dc8efc7eaaeca4888ac23

Merge "Update parameter documentation for removeHTMLtags()"

Merge "Change visibility of some methods in Parser and update docs accordingly"

Update parameter documentation for removeHTMLtags()

Change-Id: I7c0530e5b7d0d2001153d69fa9b9525f661dd5c5

Merge "Remove incorrect return type for getGoodLinkID()"

Change visibility of some methods in Parser and update docs accordingly

Change-Id: Ibe9d817325b4abafe137cd3f2fc6ccc25740cf58

PPAccum_Hash -> PPDAccum_Hash

Change-Id: I79eac91cc3ff4ca515d15d8fa08d492b5cdcaa1b

Merge "Various documentation updates for includes/parser/"

Remove incorrect return type for getGoodLinkID()

Change-Id: Ib9c0dcf5104fdc5f364e0b49d539b63bf2eb046c

Various documentation updates for includes/parser/

Change-Id: I16dd3a792cc83f8c80b3652d42c055730f6d177a

Remove unused includes/utils/ConfEditor.php

It was previously used by LocalisationUpdate. After the rewrite, it was no
longer used, but in an obsolete tests file. This has been removed in
I112c7fd396c47827999c in LocalisationUpdate.

Change-Id: I8baab87c1a0c5df73b0904e4ac34b621d0d72778

Merge "Simplify thumbnail styling to have consistency with mobile implementation"

Simplify thumbnail styling to have consistency with mobile implementation

Remove undocumented !important rule - !important is bad.

Bug: 65160
Change-Id: I21d6856d51478395cbe59e86e2f3aec495a74a38

thumb.php: support an optional "px" width suffix

A large percentage (40-50%) of the 500s that are emitted in production
are for a single URL,
https://commons.wikimedia.org/w/thumb.php?f=Crystal_Clear_action_viewmag.png&width=21px

The reason this fails is because thumb.php expects width to be "21", not
"21px", and it currently tries to fetch (and generate) the "21pxpx"
thumb size, which is obviously an invalid size. (an invalid size
shouldn't result in a 5xx but rather to a 4xx. though; that's a separate
bug that needs to be fixed).

This URL is embedded by a gadget, Gadget-searchbox-js, that is copied in
a lot of our wikis, including a big one, frwiki. mwgrep reveals that
there are a bunch of other URLs in various Gadgets that have width
values with "px" in them, so this presumably worked at some point in the
past.

While we could in theory fix all those URLs in these dozens of gadgets
across wikis to not suffix width with "px", this sounds like a herculean
effort and we're probably better off adding this compatibility branch to
thumb.php that strips the "px" suffix, if existent.

Change-Id: I3a00c9634b1c6af49fb8503cc3ff4cafdaff6b43

Revert "Make it possible to install extensions using Composer"

This reverts commit d6e69d774.

MediaWiki extensions are by definition part of the MediaWiki software
ecosystem, and could therefore be managed using a specialized solution;
The same is not true of PHP packages at large: we're not in a position to
change how the PHP community at large manages dependencies. To the extent
that we have a choice, we should use interfaces like composer.json to
solve for the problem of integration with the outside world.

Change Ib125bea00 made the opposite choice, compromising our ability to
express how MediaWiki relates to external software components in exchange
for superficial gains in convenience of managing MediaWiki extensions.
(I consider the gains superficial because they do not leverage the fact
that extensions share MediaWiki's code -- a property that should be
exploited to provide an extension management solution that is
MediaWiki-aware, providing, for example, a uniform configuration
management interface.)

The cost of that change are manifest in bug 64597: we lost the ability to
express a dependency on PHPUnit in the way that the PHPUnit upstream
recommends.

The problem that change  Ib125bea00 set out to solve is that modifying
composer.json to express dependencies on extensions means having a local
diff that conflicts with the code in version control. This issue is
actually discussed by the Composer documentation. In short, Composer's
developers recommend that if your production environment uses a cvs to
version code, you should maintain your own clone of the upstream
repository and version your composer.json that way. More detailed
discussion of these issues can be found at
<https://getcomposer.org/doc/05-repositories.md#vcs> and
<https://getcomposer.org/doc/faqs/why-can't-composer-load-repositories-recursively.md>.
I also note that the Composer documentation makes many references
to monolog as an exemplary Composer project, and it too maintains a
composer.json in the repository root.

Change-Id: I3e7c668ee32401e731120cfa9f96986fd8fde8f4

Merge "CSS style to make HTML5 <wbr> work on IE 8-11"

CSS style to make HTML5 <wbr> work on IE 8-11

(Note that using the zero-width space directly is roughly equivalent;
see https://en.wikipedia.org/wiki/Zero-width_space )

Bug: 65155
Change-Id: I53f62fdb26cec65a3f4a5bea1bb9b1d7a9cb9c13

Don't use $this for docuementing hook parameters

It is a reserved keyword, so this only confuses people.

Change-Id: I6553bd93ac6f72be42493a0c3ca59c63d4a97e18

Extend AbortEmailNotification hook to access log type etc.

Bug: 42458
Change-Id: Ifc3935f031c49f9b49ae76c5a8af8334e70ca2b1

Merge "Remove $wgSessionStarted"

Merge "jquery.suggestions: Hide the suggestions list asynchronously"

Merge "Fixed doc typo"

Merge "Several tweaks to the install.php script"

Fixed doc typo

Change-Id: I4c81dc62c241e1c32cbda996323d082c8b2d48a3

Fix most CodeSniffer issues in includes/parser/

Remaining are the classes containing underscores and possibly a few other
issues that will be addressed soonish.

Change-Id: Icf56374c71afc134420ebbcfecf12dcb29dc9564

Several tweaks to the install.php script

- Fix description of the "admin" option, which says there's a default,
  "WikiSysop", when this isn't the case.
- Use "MediaWiki" instead of "Don't care" as the default value
  for the "name" argument
- make the "name" argument optional, so that its default value
  can actually work as such
- test for getOption's default parameter value (null)
  rather than providing one (false) and testing for it
- fix indication of default value for --dbpath
  and document where it comes from
- format the default for --confpath
  the same way as other defaults are displayed
- add description
- use trim() for --passfile, as is done for --dbpassfile
- use single quotes where possible
- mention in the documentation where the default values come from
- various minor text adjustments
- bonus: typo fix in a comment in Maintenance.php

Change-Id: Icbc0b02cfbf09b3f97ed86f4a68c3b35a9c839c8

Declare visibility on class properties of includes/parser/

Change-Id: If03a9bd5eb83be4d15f54e73f49f42540fb7d5fc

Merge "Pass phpcs-strict on includes/profiler/"

Merge "Pass phpcs-strict on includes/specials/ (2/3)"

Merge "Pass phpcs-strict on includes/specials/ (3/3)"

Merge "Update html5 specs link in comment"

Make *.sh files use LF

Someone setting up MediaWiki-Vagrant on Windows had an issue with
a carriage return in a shell file sha-bang, which this should fix.

Change-Id: I190a6a6f80233b4a0ac9fd6c34d1b8af9bd6f17c

Merge "Fix return type documentation for getCacheTTL()"

mediawiki.api.watch: Use postWithToken to recover from an invalid token

Bug: 65157
Change-Id: I7a4282438eb9eb4a66b1c2ac286951bd3b6080e0

Merge "Adds the SidebarBeforeOutput hook"

Fix return type documentation for getCacheTTL()

Change-Id: I57e85351503897f8f8febb2fb49942a2ddd251e6

Remove ruleset for obsolete class .rimage

Change-Id: I0f20b54002729baa1f8d0e249c4ea08db6cd5dc6

Remove unnecessary rule display:-moz-inline-box

Gecko now supports display:inline-block directly.

Change-Id: I6591212c371f03c766a1296f2520bf09fe43e286

Remove tabindex from Special:Block

There is a autofocus on the Target field, which sets the start of
tabbing and than all the other fields are tabbed in the correct native
order. No need for extra tabindex order.
Tested with IE11 and FF28

Bug: 61257
Change-Id: Ic0d7365447733a23fa87c1d9b91c0f4f0f61bb39