Change visibility of some methods in Parser and update docs accordingly

Change-Id: Ibe9d817325b4abafe137cd3f2fc6ccc25740cf58

Merge "Various documentation updates for includes/parser/"

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

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()"

wikibits deprecations: Do not offer another deprecated function

Since I211593629ca23540dcd5a93b11834155f4f04883 the module
mediawiki.util has deprecated function which was mention here.

Change-Id: I7e4031f859392aeeb61344ceaf78d2b92badd2a8

Merge "Remove isset() check for $wgLocalTZoffset in Language.php"

Remove useless variable from Title::newFromText()

Instead of setting and returning it directly, we
might as well return null directly...

Change-Id: Icbde20fe0f6a064dd58018d6543b03def7203a84

Fix PHPUnit version to 3.7.35

"MediaWiki tries to require PHPUnit files directly,
but the phpunit/phpunit PEAR channel now falls back to the PHAR
method (install a single file, /usr/bin/phpunit, with all classes
packed inside it)." (from the bug report)

Bug: 64597
Change-Id: Ib93b32473ab1a96ea97d90c15fd3675e8ec9843c

Remove isset() check for $wgLocalTZoffset in Language.php

Do not use isset() to check for null; furthermore this variable
can no longer be null at this point now, so the check is useless.

Also remove the initial defintion of $minDiff since this was the
only case where it could be not set afterwards and fix spacing
after the comment.

Change-Id: I6e62ce1217e6e8750acb411673903fc3bde57841

Merge "Add space between two feed links"

Add space between two feed links

When more than one feed link was given, the links was concat without a
space, which looks odd. Adding now a space, between each link in the
toolbox.

Bug: 36356
Change-Id: Iaf61c88dde0a92c475a60c005e13222eb7618eef

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

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

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

Pass phpcs-strict on includes/poolcounter/

Change-Id: If10d03e66bf617f6eb2d9ec9f43d6d55c5c5d784

Add $user param to SpecialPageFactory::getRestrictedPages

This avoids the use of $wgUser

Change-Id: I5cc85391710f6e6967d78825fc8a3eda0e15a1a1

Pass phpcs-strict on includes/profiler/

Change-Id: Ic9058a650ea0735b71b3725703d76cc62b5c9fb1

Pass phpcs-strict on includes/rcfeed/

Change-Id: I197b4963a29fc5c13d71cd820ab780dedbc30b99

Pass phpcs-strict on includes/resourceloader/

Change-Id: I5c63e0612c2aae240abcdbf21da9cdadd80dee31

Merge "Pass phpcs-strict on includes/upload"

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

Pass phpcs-strict on includes/upload

Change-Id: I80eaa22398227a4edb7160d560b625102c7436ac

Pass phpcs-strict on includes/revisiondelete/

Change-Id: I0622c0156925c938b10cc1eccb4b26988452f57d

Remove underscore from classes RevDel_*

Part of program to remove underscores from class names. Checked core and
675 extensions for occurrences. All uses are in core in core are updated
in this patch. There are no uses in Gerrit extensions.

Change-Id: Ibad9afb9c696da894dab6b34ea14fb62642573ad

Merge "Declare visibility on class properties of CdbPHP.php"

Fix incorrect comment

Fix a comment incorrectly referring to user pages instead of category
pages. Follow-up to Ic93616a54c8e98e3dc71daee3c92c466d64daffc

Change-Id: I74533658c32647f77e3d8d54a05e9835e1c37952

Merge "SpecialUserlogin: Error out when attempting to create a username with a '#'"

SpecialUserlogin: Error out when attempting to create a username with a '#'

"Best practices"? Ain't never heard of it.

Bug: 64960
Change-Id: I88c479cea2bc9d2eab882e0ee8ebcbe2d1dd125e

Localisation updates from https://translatewiki.net.

Change-Id: I4ddccb026c78b4e0a7b29cda1d91ec7548f14cac

Merge "Allow mobile to reduce image quality"

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

Allow mobile to reduce image quality

http://www.mediawiki.org/wiki/Requests_for_comment/Reducing_image_quality_for_mobile

Per above RFC, this patch implements the core changes required to
specify quality reduction of JPEG via URL. To test, make sure your
setup uses 404-based thumb generation. Vagrant supports it by adding
"multimedia" role:  $ vagrant add-role multimedia && vagrant provision
* Pick any thumbnail jpeg URL that exists on the test wiki, e.g.
  http://.../images/thumb/4/49/Img.jpg/400px-Img.jpg
* check that basic scaling works by altering 400
* add quality parameter qlow- right before px:
  http://.../images/thumb/4/49/Img.jpg/qlow-400px-Img.jpg

Change-Id: I930ea06be6d302ffc8832d12b251422a9f1b3e75

Pass phpcs-strict on includes/search/

Change-Id: Ia38af607b1f2a8e49f60f1a659a14b801e7dcda0

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

Merge "Ignore CodeSniffer checks on HTML template files"

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

Requested by bot runners, who often accumulate huge watchlists
and have trouble resetting them.

Requires POST and a token to prevent accidents or abuse.
Same token as individual or list watching.

Bug: 65058
Change-Id: Id8476a1ba8b2661cf6451cacf3121d5c2d0675cf

Pass phpcs-strict on includes/site/

Change-Id: Ibc3f29e21dd5bcfd5ea641a58f8d3f641948741d

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

Change-Id: Ib4522f22551ed5501e1ab7767b8f481fe6cd72d2

Merge "ChangesListSpecialPage: Use Html instead of Xml in makeLegend()"