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

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"

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

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

Fix HISTORY hint in RELEASE-NOTES-1.24

Was missed for RELEASE-NOTES-1.23 and copied over to 1.24
Needs update, when 1.23 is copied into HISTORY.

Change-Id: I0d7f154e622052149c1c37a5847bec9cae44a891

Always link to diff in watchlist feed

Remove option to link to diffs in watchlist feed instead of the page itself, which was disabled by default.
This makes it consistent with the contributions feed when Change-Id: Ie33e2ba90e7a7566d0e0925220b8fd12b1928513 is merged.
For these changes feeds it makes sense to always link to the actual diffs.

See also bug 10268.

Change-Id: I0917844219be773ce24b4c6bac9b7d0d5a4b922d

Remove $wgExtraSubtitle

Remove $wgExtraSubtitle, per Change-Id: Ic5121b820f0ce87222e292c447e9ce0014186d76

Change-Id: I20e710a6978f1893ee84a6219464337745b7d2ce

Merge "Rename variable _AdminPassword2 to _AdminPasswordConfirm for clarity"

Adds the SidebarBeforeOutput hook

Allows to modify sidebar abstract representation just before its output

Required by change Ib4014253016db1c3d6b624be9ebbdaf452115145

Change-Id: I170e7253825c8dab5cad38e6b0ba59f28572efbf

Merge "Remove B/C classes WatchlistEditor and IPBlockForm"

Merge "Release notes for ContentGetParserOutput hook"

Merge "$wgEnotifUseRealName: Check, if realname is non-empty"

Remove B/C classes WatchlistEditor and IPBlockForm

There was a rename of these classes in 1.18
For backward compatibility the old name was extending the new one.

Change-Id: I8251fb00ced5c3b474f29b245d89d0d3fb2c14da

Release notes for ContentGetParserOutput hook

Followup to: I979e2438689648ba4c959d8083197ef14ce524e8

Change-Id: I25bba1f4e66c2f530eecc3207e8e7ac92f63d143

Rename variable _AdminPassword2 to _AdminPasswordConfirm for clarity

Change-Id: Ica04efb07014d0e7c91ae69c1a7ba5e00c0b16c2

Merge "Fix HISTORY hint in RELEASE-NOTES-1.23"

$wgEnotifUseRealName: Check, if realname is non-empty

When $wgEnotifUseRealName is true and no realname is given, the email
notification will contain a empty string, so checking the realname first

Change-Id: I1b76d9eae8ada8ca2eee8fa93d7119fbff44269d

Merge "Add --supplementary flag to generateJsonI18n"

Merge "Adds an --extension option to generateJsonI18n"

Link to diff in contribution feed

Link to the actual differences in the API contributions feed.

Related to bug 10268 (which is for watchlist, which already has this as an option disabled by default).

Change-Id: Ie33e2ba90e7a7566d0e0925220b8fd12b1928513

Add --supplementary flag to generateJsonI18n

Recursively searches the DonationInterface extension for i18n files and
performs default conversion on the lot.

Usage:
php maintenance/generateJsonI18n.php \
--extension DonationInterface --supplementary

Change-Id: I8e61c3d0f34394c0d766cf9532df9ac221ce7405

Merge "Add Page Existance checking maintenance script"

Adds an --extension option to generateJsonI18n

Usage:
php maintenance/generateJsonI18n.php --extension WikiLove
Performs the default conversions in $IP/extensions/WikiLove/, replacing
the WikiLove.i18n.php file with a backwards-compatibility shim and porting
messages to the WikiLove/i18n/ directory.

This patch has the side-effect of making the second cmdline argument in
the ordinary usage optional.  When omitted, we will make the default guess
as to $jsondir, that it should live in the same directory as the .i18n.php file.

Change-Id: I6246d2c4eb7327d6dcc503421b71ffdcaf01769c

Merge "Introduce ContentGetParserOutput hook"

Add Page Existance checking maintenance script

Change-Id: I0121f9b4d67d6dd2d27f5719d61c70683f1bae66

Remove $wgSessionStarted

This variable is useless since there is the check "session_id() !== ''"
to do the same, that works correctly if the session has been started
during the request.

Change-Id: I5081e79e30e01c97aa0e59b106cc75e9d1ba951a

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

Change-Id: I9502370c6b53f5a5c8cffc3d15516c560e709e64

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

Change-Id: Ide979e5b948115afcec59d929516411c2e8346ae

Merge "Restored method for clearing a watchlist in web UI, and rebuilt code."

Fix HISTORY hint in RELEASE-NOTES-1.23

Was not updated the last time when RELEASE-NOTES-1.22 was deleted and
moved to HISTORY.

Change-Id: Iaec2b60091cd813c28bf9d0f35c444446023df1b

Restored method for clearing a watchlist in web UI, and rebuilt code.

People have been asking about this, as raw can cause problems for those with
massive lists (upwards of 1 million).

Bug: 13250
Change-Id: I17dde71858fa65532fbe5d9c2ee2a65a39b2aa87

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

Merge "Remove underscore from classes RevDel_*"

Merge "Add deprecation notices to recent s/var/public/ property changes."

Merge "wikibits deprecations: Do not offer another deprecated function"

Add deprecation notices to recent s/var/public/ property changes.

See change I309cf46891223c5f1520e97b04d4ed315a300e7a for more
information.

Change-Id: I9fe5270f490657450ba2fcdb329abf66736f38c3

Introduce ContentGetParserOutput hook

This hooks allows extensions to override the normal model-specific
rendering of page content. A typical use would be to provide syntax
highlighting for pages that contain scripts. In that sense,
ContentGetParserOutput is a generalization of the old ShowRawCssJs
hook.

This, together with I76412f9d, is a (hopefully) fixed version of the
reverted Ibfb2cbefea44.

Change-Id: I979e2438689648ba4c959d8083197ef14ce524e8

Merge "Remove useless variable from Title::newFromText()"