Fix edit patrolling in multi Diffs

In multi diffs we had a "wrong" value for rc_last_oldid
set (not the directly previous revision but the one
selected by the user). Due to that there weren't any
unpatrolled recentchanges rows found. As that is cached
in memcached these rows stay unpatrollable forever.

Bug: 49019
Change-Id: I5d6111032dba7d102e30d85718639697f44c036e

mediawiki.legacy.ajax: Pass JSHint

* Added closure so that "mw.util.wikiScript" doesn't access
  global "mw" directory but uses local reference to global
  "mediaWiki".
* Keps names as-is, as such disabled camelcase rule locally
  for this file. This code will be wrap-deprecated and removed
  over the next 2 releases.
* Turned into local cross-references and exposed those, that
  way the individual methods don't all call implied globals but
  are ensured to be referring to the original methods as they
  were defined in the same scope.
* Fixed clashing 'e' variable name.
* Removed unexpected "delete x" statement. Does nothing and
  causes a lint warning.
* Replaced "delete request" with "request = undefined". The
  purpose of the statement is to explicitly dereference the
  XHR object in older IE browsers. However "delete" is not
  allowed for local varables, setting to undefined has the
  same effect.

* Moved debug_mode and request_type down to the other exposures.

Change-Id: Ib45ed9bd8c6f4ebdb060a527a63eaf59d2839493

Merge "Update messages.inc and rebuild MessagesEn.php."

doc: doxygen expands tabs to 4 spaces instead of 8

In doxygen, code snippets (and possibly over places) have their
tabulations expanded to 8 spaces which is a bit too much. The TAB_SIZE
configuration parameter let us set the number of spaces, down to 4 which
makes code examples easier to read IMHO.

Change-Id: I62b417496f4e5ec70e80919c28747f1bd44a7bd6

Update messages.inc and rebuild MessagesEn.php.

Change-Id: I567f5f96c7b21dbc166e9cf4ac02078dd9023ea0

test: inject autoloader in Maintenance class

We used to inject the test autoloader from the global space after the
Maintenance class has been initialized.  That prevents us from using
tests classes (such as Mock) during the Maintenance setup.

This patch move the TestsAutoLoader.php require() at the beginning of
finalSetup() which would let use Mock classes.

Change-Id: Ia402eafae8407d1516e3d200ac97539e3681fdc6

LinkCache singleton can now be altered

This patch adds the ability to destroy the LinkCache singleton or to
replace the instance with a different object.  Note the $instance static
variable is now at the class level instead of at the method level.

Change-Id: Ib110ad061868834a52a6bac651976b3ffab4a6ce

Throw an exception in ScopedCallback for invalid callbacks.

Change-Id: I303dec64e55df0b1de73c6db6577bfb005cbe464

Cleaned up some Article comments.

Change-Id: Iad1a56179345044aff8c144b8b72cdaaffa778c4

remove leftover @embed directive

Change-Id: I38ef973b458fd45f7bc6c768e6e097c521071656

Merge "Migrate Extension:PostEdit interface to core"

Fix bad copy-paste error in deprecated method

Instead of passing the conditions to the update method, it was passing the write values.

Bug: 48782
Change-Id: I3d67e2b8f1d80865a21799873d7ca239db6e9678

Merge "WebRequest::getIP() cleanups."

Merge "Do not show empty parenthesis on log entry with no block flags"

Merge "Fixed EZConnect string regex in OracleInstaller class."

Merge "Update Parsoid-only tests to reflect shortened typeof"

Migrate Extension:PostEdit interface to core

As requested by the VisualEditor team, this change migrates the post-edit
confirmation notice from Extension:PostEdit to core.

This entails porting the 'postedit-confirmation' en/qqq message from the
extension's i18n file (the message key remains the same) to the appropriate
message files and augmenting mediawiki.action.view.postEdit.js to not only
signal post-edit state but also act on it by displaying the confirmation.

Bug: 48276
Change-Id: I01cfc0630c3a505af82922844b5e70c1d61f3c1d

Merge "Delete RELEASE-NOTES-1.21"

Do not show empty parenthesis on log entry with no block flags

When nothing is stored in the database, the explode in
LogPage::formatBlockFlags will explode a empty string to a array with a
empty string as only value. Changed the count(array) check to empty
string check.

Bug: 47035
Change-Id: I879169599a2a24b35e26d4a23ab8202f267d8b00

Fixed trailing slash removal code.

Change-Id: I152dcec5cc8f4c417e3503f4b8fd3fdd8bc4fda5

Fixed EZConnect string regex in OracleInstaller class.

The Oracle connect string was valid only if it contained alphanumerics,
underscore and dot. Some new schmemes makes uses of slashes and
EZConnect has the concept of server types (pooled, dedicated, shared)
which we now validate.

The long regex is now in OracleInstaller::checkConnectStringFormat()
(flagged with @since 1.22). The patch provides a bunch of very basic
tests to test out the regex.

Change-Id: Ie3a0af9801bfdbc9129298be07e1676145a1607a

Update Parsoid-only tests to reflect shortened typeof

https://gerrit.wikimedia.org/r/#/c/64217/ cleans up the RDFa types by dropping
the Object/ prefix and renaming mw:Object/Template to the more general
mw:Object/Transclusion. This patch updates the Parsoid-only tests to reflect
this change.

Change-Id: I474c2cf672beb239d0ac482043ccdfd4681d64fb

Delete RELEASE-NOTES-1.21

1.21 is now released

Change-Id: Ic7519a3a81e33b2212e0c0d28d2a85f0eb7ff39e

Merge "Darker labels in new login and create acct forms"

Merge "benefits icon part of background, not float"

Merge "Compress PNGs used in new forms"

WebRequest::getIP() cleanups.

* Always treat the first XFF IP from cache proxies as sane
  even if it is a private IP (useful for things like labs wmf).
* Make sure IP::canonicalize() gets called if the IP is selected
  from the XFF chain (this matches getRawIP()).
* Altered and expanded unit tests.

bug: 48919
Change-Id: I350aca72c7a96ba3ec727324800612fc84e0e7a4

Merge "Tooltips for HTMLCheckMatrix"

Tooltips for HTMLCheckMatrix

Generates tooltips on the columns labels of HTMLCheckMatrix.
Bug: 47094

Change-Id: I9670ecdc2cb553df07ee06e3b83b003382471d27

Merge "Micro optimization when fetching a magic from cache"

Micro optimization when fetching a magic from cache

The Language::getMagic() methods would request a full copy of the magic
words array from the cache.  Instead use LocalisationCache::getSubitem()
to only fetch the magic word we are interested in.

I do not think that makes that much of a difference, but that is
certainly nicer.  Note that getSubItem() returns null on miss instead of
the previous 'false', that still works for the !is_array().

Change-Id: I02824fd313eeecb5e5b26cb79d7b5549f6648f91

Merge "tests: resources tests were not being run"

tests: resources tests were not being run

The ResourcesTest class under ./resources/ were not registered anywhere.
This patch move it under the `structure` testsuite and phaseout the, now
empty, `resources` directory.

Change-Id: I53410e1dc83263c4c541f4dc278a9e616265d7b1

Merge "Renaming files to follow name conventions"

Merge "Add setting wgLogAutopatrol to toggle logging of autopatrol actions"

Add setting wgLogAutopatrol to toggle logging of autopatrol actions

Per bug 47415, this is needed for Wikidata and could be useful for
other wikis, as well.

Bug: 47415
Change-Id: Ic999454d001c38dea08746d1e8184f0163cb7330

Update linktrails for Hindi and Odia

For Hindi(hi) use Devanagari and Extended Devanagari Unicode range, but
exclude danda characters(0964 and 0965)

For Odia(or) use Odia Unicode range.

Bug: 36966

Change-Id: Ib1b233d227f33e77c212e67eee2aea64357e55ba

Merge "tests: group structures tests in their own directory"

Merge "overhaul maintenance/mwdocgen.php"

Merge "Merge 'passwordreset-pretext' and 'passwordreset-text' messages"

Merge "doc: various updates"

Merge "Customise linktrail for Gujarati (gu)"

Darker labels in new login and create acct forms

Bug: 47777
Change-Id: I01dd960d673a37d519ae8805db1fbbcb5a4306c9

Merge "Add TitleQuickPermissions hook to Title::checkQuickPermissions"

Add new parameter $patrolFooterShown to hook ArticleViewFooter

Some extensions like PageTriage will be able to use this new
parameter to determine whether to show its own curation toolbar.

Change-Id: I07e85aa37edc994c5a2bf9d011976b91ff01ab14

Merge "Fixed OracleUpdater::doPageRestrictionsPKUKFix prefix"

Fixed OracleUpdater::doPageRestrictionsPKUKFix prefix

Previously OracleUpdater::doPageRestrictionsPKUKFix had the
database prefix hard-coded into the SQL query. This removes
the hard-coded value and replaces it with the actual configured
prefix.

Change-Id: If60f23af7fd238d4c91ebf25b095749345a2718d

Merge "Oracle: DRCP and persistent connections support"

Oracle: DRCP and persistent connections support

Adds $wgDBOracleDRCP, when set to true, enables persistent connection
with DRCP on Oracle.

Requires Oracle DB 11gR1 or above. More informations about DRCP can be
found at:
http://www.oracle-base.com/articles/11g/database-resident-connection-pool-11gr1.php

Change-Id: I2be7120a2ebec39d866b79be62ac715bbc3738a9

(bug 48294) API: Fix chunk upload async mode

I915c1678 fixed single-chunk chunked uploads for the common case, but
didn't store the data for async uploads under the correct filekey. This
fixes that.

Bug: 48294
Change-Id: Idbeab2fd0d801974b610198dcf02c81798f65a0d

Merge "Make 'subnet' feature of $wgRateLimits work with IPv6"

Merge "Newlines should be converted to spaces for IRC feed"

Renaming files to follow name conventions

And renamed the inner class name.

Change-Id: I2ed94a61214439d5c70d04bd1dbddd68754b595e

overhaul maintenance/mwdocgen.php

Now based on Maintenance class.
Reinstate {{EXCLUDE}} which got removed by 8199984

This is roughly back compatible.

bug: 34436
Change-Id: Ic1a2b2c29718054ec740b2ffb90350af700a6a58

wfIniGetBool: reduce strtolower() calls

We were calling strtolower() up to three times. As a micro optimization,
call it once around ini_get() and do the logical checks against the
variable containing the lower case output.

Change-Id: I16c149fbb9de84d7b6f3e68da06de208c5572b7c

Allow patrolling page creations from Special:RecentChanges

In case we got a page creation which is yet unpatrolled and
recent changes patrolling is enabled, the user probably rather
wants to patrol the whole page (first revision) instead of seeing
a patrollink for the current revision.

Follow up to I1e24733c

Bug: 48928
Change-Id: I9f89f1d44852d4c03a67ac622d5cf4d86453dce3

revert last 'ru' l10n update

The russian localisation update made in 4455865 are apparently wrong.  A
side effect is also the 'mainpage' message that got changed and which is
breaking tests in master.

Change-Id: I99bcda82f524ddde25a860dc5c08a77a31acd604

Localisation updates from translatewiki.net.

Change-Id: I9832bfa9e077c7f71e95a64d615dd83b98676124

Merge "Localisation updates from translatewiki.net."

Merge 'passwordreset-pretext' and 'passwordreset-text' messages

It is confusing to have in the same two messages which can be interpreted
with different meanings. When there is more than one way of resetting a
password, two messages are being show to the user:
- 'passwordreset-pretext' saying to fill only one of the fields below
- 'passwordreset-text' saying to *complete* the form, which people
  can interpret as "fill all the fields below"

This already got changed at the English Wikipedia, see
https://en.wikipedia.org/wiki/Wikipedia:MediaWiki_messages/Archive_5#Special:PasswordReset_improvements
I think it's worth changing MediaWiki too.

Change-Id: Ia2f0c889959d11396411ba44ae595ffad5d498d2

Merge "Check if API enabled before loading dependent JS modules"

benefits icon part of background, not float

The separate icon floating left made the benefit text not request a
width, so the benefits column width was determined by the h2 "made by
people like you" heading and a short h2 could clip big numbers.

Now the icon is part of the background for the text div, and the column
width is the larger of h2 heading and benefits.

Bug: 48909
Change-Id: Iaf5fb1ccb4f50d34cad8a29b2bdd8f5b267ef008

Check if API enabled before loading dependent JS modules

Affects whether these modules are loaded:

* mediawiki.searchSuggest ($wgEnableAPI only)
* mediawiki.page.watch.ajax ($wgEnableAPI, $wgEnableWriteAPI,
  'writeapi' right)
* mediawiki.page.patrol.ajax (same as above)

Checking of $wgUseAjax has not been removed where it was
already present, in case some users have set the variable
to false to disable these specific features.

Bug: 30213
Change-Id: If2ec219cfbb94e7c9718c58b9b54a508d0e0c656

wikibits: Pass JSHint

Change-Id: I95897e01b339c16abdd12190129d2b343be43d30

wikibits: Wrap jsMsg in mw.log.deprecate

No release notes since it was already deprecated (in 1.17 to
mw.util.jsMessage and in 1.20 to mediawiki.notify, though we
don't fallback to that directly since it is not compatible).

Change-Id: I359c94929e58363d5b38f410a60b456111bb0897

wikibits: Replace checkboxShiftClick globals with dummies

They have been deprecated in favour of jquery.checkboxShiftClick for
many releases now (since MediaWiki 1.17) and aren't used anywhere in
MediaWiki core (or extensions that I know of).

Though removal is justified, given the slow pace at which the tour[1]
is progressing and the amount of garbage I still find on a daily basis,
I'm replacing them with dummies instead and marking the dummies as
deprecated for easier tracking.

[1] https://meta.wikimedia.org/wiki/User:Krinkle/Le_Tour_de_Wik%C3%AD/2011_Resource_Walker

Change-Id: I72aac6111b9be974f160fecad8312c49c9460c5b

wikibits: Wrap UA and accesskey globals in mw.log.deprecate

Change-Id: I026d3d3110fe1580733324d50e8a4c8b04798234

Merge "new Create account contributors icon"

Merge "Give skin more control of OutputPage modules"

Merge "mediawiki.searchSuggest: Use mw.Api instead of jQuery.ajax"

Merge "Allow setting the ID of the main table in HTMLForm"

Allow setting the ID of the main table in HTMLForm

- Added new method HTMLForm::setTableId() to allow to set the ID
  of the main table (or the outermost <div> element depending on
  the format).
- Changed the second parameter of HTMLForm::displaySection()
  to take the full ID so that the one set with setTableId()
  will not get prefixed with "mw-htmlform-"

Note: If someone has a better idea for the name of that method
(since it also applies to <div> elements), please tell me it :)

Change-Id: Id01bd0fa44512b0320f3c3ff79627e8a9bccd5bb

Merge "De-duplicate refreshLinks2 base jobs on insert."

Give skin more control of OutputPage modules

Remove addDefaultModules from OutputPage
Instead only enforce mediawiki.page.startup

Add a method getDefaultModules which groups modules
by type allowing a skin to tweak

Change-Id: I89d529f0378d90af0fe0a5adea5d5dbdca83a86e

ApiHelp: Avoid header for example, when there no examples

When writing a new extension the source:

public function getExamples() {
return array(
//TODO
);
}

would produce a empty "Example:" header on the help page. Avoid this, by
changing the condition to check for truthy instead of false and empty
string.

Change-Id: I67ecacbbac66b97ed96412abf79b49aebe0ebdbf

Customise linktrail for Gujarati (gu)

Release notes will follow with another commit when this is merged.

Bug: 48798
Change-Id: I872a9f141f64a664bc3743fcff5f036634445ba0

mediawiki.searchSuggest: Use mw.Api instead of jQuery.ajax

Change-Id: I5e12371bf7e0a6eaeb2507b8942773f63281139b

Merge "ResourceLoader: Prevent prefixless modules from overwriting others"

ResourceLoader: Prevent prefixless modules from overwriting others

Expanding prefixless modules should not overwrite the result set
of expanded modules.

Change-Id: Ic6e0e1c9e3fe7a585d34de21c532ee5ac3e7fdc5

Localisation updates from translatewiki.net.

Change-Id: I488ebb99fba311a78fc879933606f797a1f640ad

Add autocreate to addNewUserLogEntry docs.

Change-Id: I40bdd510dbbc32d32ceee39b54bb4c66ace0ccfc

Merge "Avoid changing the global on special pages"

Fieldset legends is broken.

HTMLForm constructor didn't use messagePrefix if context is null.

Bug: 48650
Change-Id: I26367bd597f873838c8cf45d75800661d37c0f95

Moved job queue aggregator classes to a subdirectory.

Change-Id: Iabbf566b05d44e19770db7b4519abfe00e4a8d90

Add Release notes for I1e24733c

Change-Id: I271cf65b74ff9101914670b3e0c70241b5638020

Merge "Updated profiling table definition."

Merge "Enable fallback languages when retrieving messages"

Enable fallback languages when retrieving messages

The core function behind wfMessage() (MessageCache->get()) did not
apply the language fallback chain to on-wiki messages.

This patch has changed the behavior to iterate over all possible
languages, first checking on-wiki and then checking the CDB cache,
until it finds the message. Note that fallback languages never
take precedence over the actual requested language.

This patch was taken from the following changes and then
adjusted to fix issues that caused bug 46579.

* Change-Id: Iaaf6ccebd8c40c9602748c58c3a5c73c29e7aa4d
- Author: Matthew Walker <mwalker@wikimedia.org>
- (cherry picked from commit d434bfcf3bbab05660ed8f798a4622487dd8ba56)
* Change-Id: Ib607a446d3499a3c042dce408db5cbaf12fa9e3d
- Author: Mormegil <mormegil@centrum.cz>
- (cherry picked from commit 1b8cb8dc3119bfb12d86d2f044018dc12553939b)

Bug: 1495
Bug: 46579
Change-Id: I420457863eeb79824698d06abc7784032b267af2

Merge "Language::convertPlural: check if matching form exists"

Updated profiling table definition.

Change-Id: I20e92f2bc9b57a9c06cbcdfd30370fedf3c0e2f7

[JobQueue] Allow using gzip on larger jobs in JobQueueRedis.

Change-Id: I74ebdcfa0d3f2bcdc429394f7873fa25b7eb0f58

Localisation updates from translatewiki.net.

Change-Id: Ie17b0ec177b58826a9fa3e8131d29ffdb75dc8dd

Ensure title in NS_FILE before showing archived file

Avoid an MWException "X is not a valid file title." with invalid
input such as index.php?title=Special:Undelete&title=X&file=Y .

Change-Id: Ie915726fdde8e772a63713d3eb9da232d9fd83bd

Language::convertPlural: check if matching form exists

It is possible that only explicit plural forms are specified, and
therefore, it is possible that none match. However, handling of
explicit forms came after the count( $forms ) check, so input such
as {{PLURAL:|1=}} would trigger a "PHP Notice: Undefined offset: -1".

Change-Id: I8494de8ceb9e0cfff7203c69c21f02b3731275af
Follows-Up: I50eb0c6d1c02ca936848d310de625ed1fe43d91a

Newlines should be converted to spaces for IRC feed

Instead of just removing them, which was causing reasons to show up like this:
"Automatically blocked by abuse filter.Description of matched rule: Long-term Chinese spambots"
While showing like this on the web pages:
"Automatically blocked by abuse filter. Description of matched rule: Long-term Chinese spambots"

Change-Id: I058e4b446aaa42b4dfe3369f101191f36b6f6978

Merge "Rename $usableSkins to $allowedSkins"

Rename $usableSkins to $allowedSkins

This change is entirely in local scope.
The function is still called getUsableSkins()
so this change won't break anything.
But the variable name and the updated documentation comment
make it clearer what the purpose of this function is,
for anyone reading the code or reading the autogenerated docs.

This change also includes minor changes to documentation
in skin-related files (typos and small edits for clarity).

Change-Id: I4220408f4d3b64cf87be4ad1af4b72d3a4524922

Merge "Patrol overhaul phase 1: Remove rcid parameters"

Patrol overhaul phase 1: Remove rcid parameters

I've changed the logic in Article::showPatrolFooter to be able
to fetch the recent changes id and to only show the patrol link
in case the change hasn't yet been patrolled.

In case recentchanges patrolling is enabled this will try to
create a patrol link for the revision the user is currently
viewing. If only new page patrolling is enabled it tries to
create a patrol link for the first revision of the page.

Furthermore I've removed the passing around of &rcid parameters
within MediaWiki as those had several issues (some even security
related) and were only a workaround to protect the DB from some
queries, which is no longer needed.

This has already been partly implemented in a different manner in
r45778 but had to be reverted in r46542 due to performance issues.
This version shouldn't cause such issues as I'm only adding one or
two indexed database queries per page view.

I've written this new version of the patch with mostly
performance in mind and even tested the database queries it uses
against the replicated databases of enwiki on the toolserver. I'm
pretty sure this can't be implemented any faster without creating
a new index on the recentchanges table.

As I was on it I've implemented RecentChange::isInRCLifespan which
checks whether the given timestamp is new enough to may have a RC
row. That way we can avoid some DB queries for timestamps which are
older than the max RC age.

Fixes bugs:
(bug 15936) New page's patrol button should always be visible
(bug 35810) ! N pages non-patrol-able
(bug 36641) Patrol page link shows on non-existent revs

Change-Id: I1e24733cafbfdc51b7a5a9a1c1baf948e760fe1a

RecentChange: deprecate newFromCurRow and loadFromCurRow

This is ancient code related to the no-longer existing cur table,
which doesn't seem to be fully correct anymore and is only called
from the (deprecated) Oversight extension.

Change-Id: I0b40e3fb17ec670c5c85c8ee0dd2e8b74d72e548