Merge "Reorder SpecialRecentChanges::webOutput"

Merge "Allow registration of Actions using a callback that returns an Action instance"

Give grep a chance to find the usages

Change-Id: I7fc00b055b21bbbbab209a1b1d23f23172862a0b

Merge "Call $wgContLang->findVariantLink() in {{PAGESINCATEGORY: }}"

Call $wgContLang->findVariantLink() in {{PAGESINCATEGORY: }}

Corrects inconsistent return value for number of pages in a
category when language variants are in use.

Change-Id: I27668ba348c45bc34b264f8771e91e58a9920552

Merge "vector/screen.css: Remove some unnecessary rules"

vector/screen.css: Remove some unnecessary rules

* There is no label inside div#simpleSearch.
* Inside div#simpleSearch, input and input#searchInput are the
  same in practice (there is a <input type=submit>, but it is hidden).

Change-Id: I1d9657a5ab608d1a54314dd7c96144af96c53d59

Allow registration of Actions using a callback that returns an Action instance

Basically implementing what Brion suggested on wikitech

This allows for injecting dependencies while still only loading the actual class when needed.

Simple example:

$wgActions['epundo'] = function( Page $page, IContextSource $context = null ) use ( $differ ) {
$undoAction = new \EducationProgram\UndoAction( $page, $context );
$undoAction->setDiffer( $differ );
return $undoAction;
};

Change-Id: I6c0f4022f1df1ebaf9cd1a5fe4bd362d0ecc0d62

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I5e7da5913a10c7f75cdf223f2166cc7873134949

jquery.suggestions: Add some comments

Change-Id: I0706e1f847f8ebf3d48cbb40a88dd6f6d487d2e6

Merge "No spaces after (casts)"

No spaces after (casts)

Also removed some unnecessary ones. I think I've caught them all.

The spaceless version already appears in core ~300 times (after
accounting for false positives when grepping). Some consistency would
be nice.

Change-Id: I607655b5f4366e66dc78730d5fd2f57ed8776cae

Merge "Use canonical class name, remove unused globals."

Merge "Add a deferrable update class for callback/closure udpates"

Merge "Make mwCustomEditButtons a no-op and mark it as deprecated"

Merge "Parse limitation warnings as 'text' and in user language"

Add a deferrable update class for callback/closure udpates

Change-Id: Ifdf2a0937df50eb2f04d514b5cc2ef39a54ebe8f

Removed "requires JavaScript" from preference texts

Bug: 44947
Change-Id: I15e34206437cd55f1033d2cf76f377a2505aaffc

Fix typo in property name

Change I270bde41 introduced a typo to RecentChanges.php, referring to
'$this->mExtras' when '$this->mExtra' was intended. This caused the IRC
log action comment to be omitted from the IRC RC feed.

Bug: 53720
Change-Id: I7f29ba8392861e00a37a2196833154bac41298b4

Merge "Improve wfEscapeWikiText"

SECURITY: Fix extension detection with 2 .'s

Fix IE6 extension detection, to prevent XSS in the API response.

Bug: 52746
Change-Id: If58a81b62209b36bd143a1766a8a20c9c34dbe3b

SECURITY: Prevent tokens in jsonp mode

Add checks to token-returning functions to prevent returning tokens in
jsonp mode. This affects action=tokens, action=login,
action=createaccount, and action=query&list=deletedrevs.

Also, remove the "gettoken" parameter to action=block and
action=unblock, which has been deprecated since 1.20.

Bug: 49090
Change-Id: Ibeaa5c72d8084585092b15935a3f5709104bf7f7

SECURITY: Prevent FPD on exceptions in load.php

Sanitize error messages in ResourceLoader if $wgShowExceptionDetails is
false.

Bug: 46332

Change-Id: Ia14ae21972192d291cb86dce65568e9e8b4674f7

Improve wfEscapeWikiText

The following are now correctly escaped:
* Space at the start of a line
* Start-of-line characters after \r
* Magic links such as "RFC 123" with non-space whitespace
* URIs that don't use "://", such as "urn:foo"
* Double-underscore magic words

Bug: 53658
Change-Id: Ie8803a03b20e3fd7e5cc7e55829f24bb86f60b17

Merge "Hide server IP addresses from DB error pages"

Localisation updates from translatewiki.net.

Change-Id: I5dc3f7f303f05290e4b29dceecc69d7347e65e51

Hide server IP addresses from DB error pages

Error details are now omitted if both $wgShowHostnames and
$wgShowSQLErrors are false. WMF wikis have the former set to true;
users of those wikis can still include the details in bug reports.

Fixing this for DBConnectionError was more or less straightforward.
However, in the case of DBQueryError, this involved splitting the
error message into multiple smaller messages (killing a raw HTML
message in the process).

Note that this is an incomplete fix: information disclosure is,
for now, still possible from DBUnexpectedError exceptions (are
these even supposed to go into the exception log?) or exceptions
that occur from within the exception handler. Yet this is still
an improvement.

Bug: 26811
Change-Id: I1756b296d5e8d1d22511a3c3b58b5bb0dd025fec

Merge "Actually use $ignoreInvalidDB in UserRightProxy::getDB()"

Merge "Add/update comments for grep"

Make mwCustomEditButtons a no-op and mark it as deprecated

Bug: 50310
Change-Id: I59d081b4f2581cea25485c69ef2933b94681e82a

Localisation updates from translatewiki.net.

Change-Id: Ibd05bfafcc2d7bbe5e5a7248625da61f5aaed83c

Merge "Add converted namespace names as aliases to avoid confusion."

Merge "Remove dead code"

Localisation updates from translatewiki.net.

Change-Id: I4e6557cfbd773ac932c5e218adcfe7e7e6207b78

Merge "Also redirect if prefixed https cookie is preset"

Merge "Make morenotlisted more grammar-friendly"

Make morenotlisted more grammar-friendly

It's nice to be able to have different translations compared to
moredotdotdot, but normally "More not listed..." is not worth
the grammatical hassles and is not one-size-fits-all.
<https://translatewiki.net/wiki/Thread:Support/MediaWiki:Morenotlisted_too_general>

Change-Id: Ica649d7630106143204c3495c544025439bb1921

Add email recipient username as a parameter to emailuser confirmation

Just as optional parameter, strictly following the bug summary, should be enough.
I sometimes open many tabs, forget who I emailed and have to check my email,
but unless someone thinks this common I don't think it's worth changing
the actual text of the confirmation adding more clutter.

Bug: 44668
Change-Id: Icf0ac18150847424728121e614849f87bee2e0e7

Use login-throttled properly everywhere

Actually proper usage would probably be to split the messages.
Followup to Id385be840f340476fbe7a818ff1d05154fe86d68;
spotted by Shirayuki.
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Login-throttled/en

Change-Id: I565971a90fc383327a51d717a80013cb7a3e9c9b

Add missing bug number for I458fb688

Bug: 14431
Change-Id: I193abd91ce89971d40297c22d386e0ac3c453b88

Merge "No spaces within (casts)"

No spaces within (casts)

Per the code conventions ([[mw:CC/PHP]]).

This file appears to be the only outlier.

Change-Id: I634d9927174737ffc99976a97a41ce04c842fc32

Merge "Delete docs/upload.txt"

Delete docs/upload.txt

"This document describes how the current uploading system is build up and how
custom backends can be built. (At least someday it will)."

It says this since 2009 (r55604). I see no need to keep that red herring here.

Change-Id: I70d20bf21527bee26bf244381a9d75d96856dd52

docs/scripts.txt: Remove docs about deleted files

trackback.php has been removed in 1.18, redirect.php sometime sooner.

This highlights the state of disarray these docs are in…

Change-Id: I8115ca5649e82537a638b2cd3775717e43099124

Add/update comments for grep

Change-Id: I6402a593872480cac4b76492957771529c1adeb8

Remove dead code

Accidentally left in dabbbc7d2a88ceb9efce5742af05e848dbcaa66e

Change-Id: I58e65ca7ad23d00992f66f3bcee879e662b62ef0

Also redirect if prefixed https cookie is preset

After we switch to not using prefixes, still respect prefixed cookies
for users who are currently logged in.

Bug: 53538
Change-Id: I950e36e10420209fca28c41c438c363a3f332c14

Merge "SpecialPrefixindex formatting methods are now protected"

Localisation updates from translatewiki.net.

Change-Id: Id582eb69175c2916c81c1819a00ece57a64645f9

Use canonical class name, remove unused globals.

Change-Id: I94cf491e4c1c793f455fd597b383f8133b0aace3

Support search update for MySQL search engine

Since change I2063fe0 SearchMySQL does not do a search update anymore.

Bug: 53408
Change-Id: I7ccd033c13f4dd622dde7ac97c56f68474780ef2

Merge "Normalize IPv6 addresses in XFF chains"

Merge "Remove prefix from forceHTTPS cookie"

Normalize IPv6 addresses in XFF chains

Otherwise, we're at proxy's mercy for using the same form of IPs.

Change-Id: I090947958bf699ee6d8a9009c9e4a6d11da23f2c

Merge "Fill in defaults for omitted parameters to RecentChange::sendToUDP"

Remove prefix from forceHTTPS cookie

Match CentralAuth, and set appropriate expiration too.

Bug: 53538
Change-Id: I3c16ff92781e1a72346058ae3838d8fc47019d55

Fill in defaults for omitted parameters to RecentChange::sendToUDP

Not all callers of RecentChange::sendToUDP pass all the parameters -- which is
to be expected, since the sendToUDP's signature specifies the empty string as
default. It is not correct to leave the values empty, though, since they are
used to construct the logging URL. When the parameters are unset, the correct
behavior should be to fill in defaults from $wgRC2UDP*, as was the case prior
to I270bde418. This patch reverts to the old, correct behavior.

Change-Id: I4570473ae7382e602292e58a41e440325a65a275

Merge "doc: Remove repetitive "Give grep a chance" phrase from comments"

Localisation updates from translatewiki.net.

Change-Id: I9a1d9d6c7444e0e746a3071520a124a9937633d3

doc: Remove repetitive "Give grep a chance" phrase from comments

* Follows-up b2e2b2e016.
* Minor clean up of surrounding documentation comments.
* Fixed missing keys for messages in WebInstallerPage

Change-Id: Iaa692064262f3c0e10cfa5e4b1ec8c86e5d02362

Merge "Make FakeConverter more realistic"

SpecialPrefixindex formatting methods are now protected

namespacePrefixForm() and showPrefixChunk() accepted additional
arguments to slightly alter the form such as hiding redirects.  The
argument has been removed in favor of class property just like
stripprefix introduced earlier and the method have been made protected
since there is no point in calling them out of the special page scope.

Change-Id: I55728fd2634f8a935a033052dcce3c7247cb1aa3

Merge "Efficiently reset null user tokens"

Merge "Add Special:[All]MyFiles as alias for Special:[All]MyUploads"

Merge "Added a getLazyConnectionRef() function to load balancer"

Efficiently reset null user tokens

Allow resetting null tokens (users affected by bug 41586), and improve
the overall performance of the script for large wikis.

Adds "nulls" flag to indicate that only users with bad tokens get reset.

Bug: 41586
Change-Id: I524607d3e54aa1c3703019cca46b69558847182e

[new gallery]. Centre really skinny images above their caption.

Change-Id: I193ee3b984635e72352872e148c1ea7768d6cc5f

Add Special:[All]MyFiles as alias for Special:[All]MyUploads

I've been typing the wrong one all day. When I mentioned it on
irc, Odder seemed to think making an alias would be a good idea.

Change-Id: I4373caf65495b1bd773758d880b1f5c03cfbfe2a

Merge "Add help message to prefershttps"

Merge "Allow listing old files in Special:ListFiles. Add Special:AllMyUploads"

Allow listing old files in Special:ListFiles. Add Special:AllMyUploads

This solves the problem of new users on commons wanting a list of
their files, but something like cropbot modifying it and taking
it off Special:MyUploads.

I'm worried this is a bit hacky to make TablePager work with
two queries. If anyone has any suggestions on how to do this
in a less hacky way, please say.

Some notes:
* This totally removes any revdeleted entries instead
  of dealing with them. Some future iteration can do selective
  deletion.
* Should maybe add links to old versions of files description
  page somewhere. Not sure where in ui that would fit (The date maybe)
  (by old file description page I mean something like flagged revs
  filetimestamp parameter)
* The latest version column should perhaps have "latest" and "old"
  instead of "yes" or "no"
* This is slightly different from the suggestion on the bug report
  as it shows all revisions (instead of say just first revisions).
  I think showing all revisions makes more sense for the "where
  are my uploads" use case. Second of all, the checkbox is not on
  by default.

Bug: 30607
Change-Id: I9e58db1f212e3bb361316c05ef32d4b9c31c6490

Merge "Make variant selection menu toggleable by keyboard"

Add help message to prefershttps

The preference requires logging back in again to set the new cookies.

Change-Id: I2602205eec752c927ac6df81545e69f5c9e70ee4

Merge "Rephrase the installer's config-welcome message"

Merge "Make restoreprefs crystal clear about the meaning of *all*"

Sort namespace aliases by id in appendNamespaceAliases for pretty output

Bug: 45382
Change-Id: I8850eac676f219482d380a0c4a9cad1009285c68

Fix the message "seconds-ago"

follow up to Ie667088010e24eb6cb569f9e8e8e2553005223eb

Change-Id: If3ddcb2c9d294c45673b905d70dc9ee9073f88dd

Localisation updates from translatewiki.net.

Change-Id: I6bd3ed5448364b1a8a0bb9f06f104289c81851ce

Merge "Do not show empty metadata section on svgs"

Merge "Make thumb.php give http redirects if given a file redirect"

Make thumb.php give http redirects if given a file redirect

See bug 22390 for some of the background. Basically there's two
reasons why we want this:
* Hotlinkers to commons. We don't want to break their links when
we move a file (Wikimedia blog is a prominent hotlinker)
* Cached entires. On wikis using commons files, a file move doesn't
trigger HTMLCacheUpdate jobs for the downstream wiki (it only
triggers it on commons). This means that these pages will still
use the old version of the html (with img tags that have the old
url) until the next edit or purge action. But these urls won't
work as soon as the file is moved.

I'm pretty confident that redirects will work at this point
in the thumb stack, as the redirects for long file names work
fine (provided no thumb exists in swift with the wrong name).

Change-Id: I40350121cf902f66fcbd3cf788335a988fa7ee33

Rebuild MessagesEn.php

Change-Id: Icf903bcf0027f08fcf06214a5fbfcc4fdfc5d0b6

Merge "Remove inconsistent font-family declarations"

Merge "Change wording on message per feedback from translatewiki."

Merge "Add url parameter to trigger autogenerated gallery type."

Merge "Rename '$wgStreamLoggers' => '$wgRCEngines'"

Merge "Make it slightly easier for extensions to hook into page protection."

Merge "Make last row of new gallery not be huge."

Make it slightly easier for extensions to hook into page protection.

Split parts of doUpdateRestrictions() into several more concise functions.
The biggest drawback is increasing $dbw->encodeExpiry() calls times 3. This is
not much of a problem given that's a very inexpensive function and it is only
called very few times ($limit holds 2 values in current codebase)

Added $reason to the ProtectionForm::save hook, so that other code hooking into
and adding their own settings into page protection form can also process the
reason for the protection change.

Change-Id: I879290ed83e4e47e9561d4c352fbd50c07d7e18a

Rename '$wgStreamLoggers' => '$wgRCEngines'

The name '$wgStreamLoggers' captures the direction in which I'd like to see
this evolve, but for the time being it is a misnomer, giving the false
impression that it is more general than it in fact is. I did not bother
deprecating the old name since it was only merged to master yesterday.

Also renamed: RecentChange::getStreamEngine -> RecentChange::getEngine (for
consistency with the change to the config var name).

Change-Id: I0d7b08e591e897b24528ea0981a6fcb93cfe8831

Merge "For new gallery, make nolines mode have caption centered."

Localisation updates from translatewiki.net.

Change-Id: Ia1c22e488c88b2b614563af0afb704c01fc077eb

Fixup coding style of a18d9856f7

Change-Id: Ie2e85af64aee164d6ddb8f6a10ead983ffc02ac5

Merge "[New gallery] ignore perrow option in this mode, as doesn't make sense."

Merge "Clean up parser test's InterwikiLoadPrefix hook"

Enhanced RC: Add a "view changes since my last visit" link

It's only shown if there are any unviewed changes and if as least one
of previous versions was viewed already.

* Added new message 'enhancedrc-since-last-visit' for link text
* Added 'enhancedrc-history' to replace 'hist' in enhanced RC context

Bug: 51901
Change-Id: Id74fe88bb2a1879651935ac56e9cdcb2e54b331f

Merge "wpUndidRevision should be passed along during 'preview' and 'diff'"

Provide a JSON recent changes feed.

This introduces a new configuration variable, $wgRCFeeds, which allows the user
to configure multiple destinations for RC notifications. It also allows the
notification format to be customized. Two formats are included by default: the
older IRC format and a new JSON format.

Change-Id: I270bde418a82985c94372ac4579100435b6ee026