Merge "Move ApiQueryUserInfo::getBlockInfo() to ApiBase"

Merge "rcfeed: Replace usage of deprecated class RCFeedEngine (in comments)"

Merge "NS_MEDIA also always has page_id 0"

Merge "MovePage methods need to run safety checks"

Merge "Don't pass Config to NamespaceInfo"

Merge "Update GenderCache to use NamespaceInfo"

Merge "Deprecate the Http class"

Merge "Update DefaultPreferencesFactory to use NamespaceInfo"

Update DefaultPreferencesFactory to use NamespaceInfo

Depends-On: I552264714de7227965f978f20f932f41cf96042e
Change-Id: I0568e7f4c7223208001f378a03fef395df8bff69

Merge "Improve rollback tests setup by extracting repeating logic to HistoryPage object"

Update GenderCache to use NamespaceInfo

Change-Id: I0c2997bf8764249c26251d0f63f3ea0a0b272a7e

Deprecate the Http class

All methods got moved to HttpRequestFactory or MWHttpRequest or dropped.

I made the return value of the new HttpRequestFactory::request/get/post
methods null on error instead of false, so that when we drop PHP 7
support, we can use a "?string" return value. This could theoretically
change behavior of code that was switched from the old Http methods, but
probably won't. I kept the old behavior for the deprecated methods.

I changed the default value of $wgHTTPProxy from false to ''. This way
it should be usable directly without a trivial wrapper method. For the
benefit of anyone who might have set it to false in LocalSettings.php, I
also recommend casting to string just in case.

Http::$httpEngine is deprecated. Eventually it will be removed along
with the curl and PHP engines, leaving only the Guzlle engine.

I also added deprecation of MWHttpRequest::factory, which occurred in
1.31, to the release notes for 1.34. Now hopefully we can hard-deprecate
it in another couple of versions.

Bug: T214390
Change-Id: I2a316a758d793857f248bd251b90f5e9a6440e3a

Don't pass Config to NamespaceInfo

Change-Id: Ie43e6108c6b9bcb666b1dece055e0df689e2ec42

MovePage methods need to run safety checks

A move method that doesn't check for things like moving a page on top of
itself or moving to a namespace with a different content model does not
seem like it's what callers would expect, and not what Title::moveTo
ever did. If there's a caller that really wants this behavior, we can
make moveUnsafe public.

I also made the $reason and $createRedirect parameters to move()
optional to match Title::moveTo() behavior. However, I made $reason
default to null instead of '', to distinguish between an empty edit
reason provided by the user and no edit reason provided at all (e.g., a
move done internally without specific user request).

Depends-On: I971e619eb76c4474fe037fad258f9c496717bf41
Change-Id: I6ddcc9f34a48f997ae39b79cd2df40dd2cc10197

Move ApiQueryUserInfo::getBlockInfo() to ApiBase

ApiBase directly uses this method causing a cyclic dependency between
ApiBase and ApiQueryUserInfo

Change-Id: I84ed21641c44b2f65ebe1980b0893d1846db3b34

Localisation updates from https://translatewiki.net.

Change-Id: I7fb884c03375b5c918f8e4b7f5309bbbe70c89ad

Merge "Permissions: Declare the $nsInfo property and document it in constructor"

Move getPrevious/NextRevision logic out of Title

They belong in RevisionStore. This change removes the dependency on
Title for these methods and will assist in porting more code to
LinkTarget.

At the same time, deprecate the Title parameter to
RevisionLookup/RevisionStore getPreviousRevision/getNextRevision, and
add a $flags parameter to match the functionality of the Title versions.
Since code search turned up no callers that passed a Title outside core,
this variant is immediately hard-deprecated. The Title methods
themselves are only soft-deprecated.

Change-Id: I76bc6fd6ee1a9f35b5f29fa640824fb5da3bb78e

Permissions: Declare the $nsInfo property and document it in constructor

Was added in commit 1f7ab9c but was never documented nor declared before its
usage.

Change-Id: Iea833f9ffb6ac210251912d35cdebf9bb064d610

Don't require Title for getTimestampFromId

3e36ba655e3a added an option for passing a page ID to this method of
Revision, and e03787afd91c switched it to a Title and made it mandatory.
This behavior propagated to the method in RevisionStore.  As far as I
can tell, the parameter does not help anything, but it can add a
database query to get the page ID if it's not cached, and impedes
conversion to LinkTarget. I can't figure out any reason to not
completely drop it. I've noted it as deprecated but still supported it
for now for compatibility -- I found one extension that does pass it.
(It's ignored, though, which theoretically would be a behavior change if
someone was passing a Title that didn't match the revision ID.)

While I was at it, I added the method to RevisionLookup, although it's
only used in later patches. Properly I should move that piece to a later
patch, but it didn't seem worth the effort.

I didn't change the Revision method, because the whole Revision class is
deprecated anyway.

Change-Id: I26ef5f2bf828f0f450633b7237d26d888e2c8773

Convert WatchedItem and friends to UserIdentity

I wasn't able to port some places that rely on isAllowed, getOption, or
related methods.

This adds isRegistered() to UserIdentity, which works like
User::isLoggedIn() but with a better name.

I also cleaned up User mocks in WatchedItemQueryServiceUnitTest in the
course of debugging test failures when switching them to
UserIdentityValue instead of mock Users where possible. They now specify
explicitly which methods are allowed to be called on their User objects,
which I believe is good practice for mocks (and unfortunately PHPUnit
makes it awkward).

Bug: T207972
Depends-On: I883d506197a011fe4c102b72df4d9deb58ab5ca2
Change-Id: Iadbf7bc31a496899dbef44e49065ff89f37aea89

NS_MEDIA also always has page_id 0

Change-Id: I0047e5f8e367fe3e8433e715ea56616a6f8dab85

Merge "Add UserOptionsUpdateJob class"

Merge "Mock IDatabase::class instead of DatabaseMysqli::class"

Localisation updates from https://translatewiki.net.

Change-Id: Iaffb949f59706ee6ca0d61ca8a3c9487be0d5a31

Merge "Avoid/Replace usage of deprecated `wfMemcKey()` function"

Add UserOptionsUpdateJob class

This can help avoid DB master queries on HTTP GET requests

Change-Id: I6fa2d53d96509d88f5d3e1ff18f3ca1de8fa4bb1

Merge "generatePhpCharToUpperMappings.php: Write output directly to json file"

Merge "HTMLForm: Improve method interface for collapsible forms"

Merge "Make JSON styling available on mobile"

Make JSON styling available on mobile

Styling works fine on mobile and is only loaded
on JSON pages. Without the styling the page is
much harder to read.

Change-Id: I68a7a02f42ec4352969d6e24ba7b0b1f85374523

generatePhpCharToUpperMappings.php: Write output directly to json file

Change-Id: Ic19e5c9899f19d9b6ebf4a65a81100670514aa14

Merge "docs/kss/package.json: Update Gerrit /r/p/ link to /r/"

HTMLForm: Improve method interface for collapsible forms

Also minor tweaks:
* Fix @since tags, this did not make it into 1.33
* Fix usage of FieldsetLayout 'group' in OOUIHTMLForm
* Documentation changes

Follow-up to 2896e87a1007202f5416fe506c691652b27c4ac9,
per my post-merge review on that commit.

Change-Id: Ib93375cde19730a46e4929878d6e472d3ac8f631

Merge "Make generatePhpCharToUpperMappings.php a proper maintenance script"

Merge "SearchInputWidget: Replace pushPending hack"

SECURITY: LogPager: Don't STRAIGHT_JOIN when using log_search

We'll hope MariaDB doesn't trigger T221458 in that situation.

Bug: T222324
Signed-off-by: Scott Bassett <sbassett@deploy1001.eqiad.wmnet>
Signed-off-by: James D. Forrester <jforrester@wikimedia.org>
Change-Id: I06364e9d0bce45bd97b2ec837d1f479feff76699

Make generatePhpCharToUpperMappings.php a proper maintenance script

This allows us to use Title for converting to upper case which
will respect any compatibility fixes added later.

Bug: T219279
Change-Id: I746487df12e4628f1e37b33b7cc3cce597853596

SearchInputWidget: Replace pushPending hack

Use the showPendingRequest config option instead of setting
a prototype method to false.

Bug: T222329
Change-Id: I0e3176141c63ed9a849326c2f9a5a26ffc2b273f

Merge "Mock IDatabase::class instead of Database::class"

Mock IDatabase::class instead of Database::class

Change-Id: I960bfdd5c9738c201a1be3ccaae05efc3d176ea8

Avoid/Replace usage of deprecated `wfMemcKey()` function

This global function was deprecated in 1.30 and is replaced with
the use of `ObjectCache::getLocalClusterInstance()->makeKey()`.

Change-Id: Ic08b53111be4374a973e08c2ed68224bfa922fa8

Don't pass Config to service constructors

We don't want to depend on the entire site configuration when we only
need a few specific settings.

This change additionally means that these services no longer see a live
version of the settings, but rather a copy. This means in tests you
really do have to call overrideMwServices() if you want services to pick
up your config changes.

ResourceLoader and SearchEngineConfig will need more work to port,
because they expose their member Config in a getter, and the getter is
actually used.

Parser and NamespaceInfo are also relatively complicated, so I split
them into separate patches.

Tested with 100% code coverage. \o/

Depends-On: If6534b18f6657ec1aba7327463f2661037f995b3
Change-Id: I1a3f358e8659b49de4502dc8216ecb6f35f4e02a

Merge "build: Upgrade grunt-banana-checker from 0.6.0 to 0.7.0 and make pass"

Merge "build: Upgrade grunt from 1.0.3 to 1.0.4"

Merge "Hide uploads link on IP contributions page"

Merge "Remove `-moz` vendor prefixes"

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: I31fb1c0676b6f32bbdde280afb98d3ce6c85854a

Merge "Fix class name handling in DeprecationHelper"

Remove `-moz` vendor prefixes

These are applying only on Firefox < 33, which is mostly limited
to Firefox 3(.6?) on Windows XP and are currently used by 0.1% of our
users, which translates to little or no impact on users of affected
scripts.
Let's remove those for better code maintainability.

Bug: T222222
Change-Id: I9bcebb31036cc93759ff9167b8774d58cf768f8e

Merge "shell: annotate return types"

Merge "Introduce a BlockManager service"

Merge "Update an old comment that predated IPv6"

Merge "Fix WikiPage::commitRollback error message"

Update an old comment that predated IPv6

Change-Id: Ia9968eab3793f275cd7ca6a35cfd5b0bcc9604c5

Mock IDatabase::class instead of DatabaseMysqli::class

Change-Id: I4c15c32b17e1e5a70ad59b25f976a2fc0e7907fe

Hide uploads link on IP contributions page

If IPs cannot upload files, hide link to list of files
which they uploaded.

Also simplify the code: User::getID() should always return int,
so $id !== null was always true. And add references to some tasks.

Bug: T220674
Change-Id: Ia9e1516393d6e85858cceb02bb09405148adaf51

Merge "Default $wgActorTableSchemaMigrationStage to SCHEMA_COMPAT_NEW"

Merge "ContribsPager: Fix slow queries"

Fix WikiPage::commitRollback error message

The user got accidentally mixed up in Ifd23bc1cd6.

Bug: T222179
Change-Id: I4d45f68caf160f001420cf46ffcd2b46cc2ddac1

ContribsPager: Fix slow queries

When ContribsPager is using an auxiliary table like ip_changes or
revision_actor_temp for the main action of the query, we already had
code in place to let it use the auxiliary table's denormalized timestamp
field for the ordering. What we didn't have was code to let it also use
the auxiliary table's denormalized timestamp field for *continuation*.

With the schema defined in tables.sql, the simplest thing to do would be
to be to add a redundant JOIN condition between rev_timestamp and the
denormalized timestamp field which would be enough to allow
MySQL/MariaDB to propagate the continuation conditional on rev_timestamp
to the denormalized timestamp field.

Unfortunately many Wikimedia wikis have rev_timestamp defined
differently from table.sql (P8433), and that difference is enough to
break that propagation. So we need to take a more difficult route,
restructuring the code tell IndexPager to explicitly use the
denormalized fields for ordering and continuation.

On the plus side, since we're doing that anyway we can get rid of the
code mentioned in the first paragraph.

Bug: T221380
Change-Id: Iad6c0c2f1ac5e1c610de15fe6e85a637c287bcd8

Default $wgActorTableSchemaMigrationStage to SCHEMA_COMPAT_NEW

Probably good to start testing this in CI sometime soon.

This also updates a bunch of tests that were forcing an older stage to
force SCHEMA_COMPAT_NEW instead, or to test both ways (until a future
patch removes the _OLD version).

Bug: T188327
Change-Id: Icb9b55cb9d754f2d30d6883005658b9670834756

build: Upgrade grunt-banana-checker from 0.6.0 to 0.7.0 and make pass

Change-Id: I4c604ab50caeffc5738c719c17f19e769082376f

build: Upgrade grunt from 1.0.3 to 1.0.4

Change-Id: I9bfada10337dbab4cb5c77348f5f0df6cfab40a7

Localisation updates from https://translatewiki.net.

Change-Id: I550dfb385ac3897d4d4cbcfc412f78a9579d1707

Merge "Fix @return documentation for MediaWikiTestCase::editPage()"

Merge "Block: Type hint Block constructor, follow-up on I37ab858494a173c6982bb"

Merge "Add missing IDatabase type hints to all doAtomicSection() calls"

Block: Type hint Block constructor, follow-up on I37ab858494a173c6982bb

Since 1.26, both options array and individual arguments were accepted but
was later removed in I37ab858494a173c6982bb (support for multiple args was
dropped) and **only** array of options is accepted from 1.33 since the use
of individual arguments all disappeared, see usage below;

Usage
=====

https://codesearch.wmflabs.org/search/?q=new%20Block%5C(&i=nope&files=&repos=

Bug: T220656
Change-Id: Id5266400def9fafbc89d5bdcdb63aed2a63da34d

Improve rollback tests setup by extracting repeating logic to HistoryPage object

This also removes the helper method waitForScriptsToBeReady() and uses the more
general waitForModuleState() method directly.

Bug: T220479
Change-Id: I6be81ad68720449ef03b6a60b4b329040f236c72

Add missing IDatabase type hints to all doAtomicSection() calls

Change-Id: I11061c358013fc67ec7a8ded54aeddcb7ff4910b

Fix @return documentation for MediaWikiTestCase::editPage()

Change-Id: Iec6ec28256b014a2c745413786d0ba5e8d5839d7

Merge "Replace pauses in rollback tests w/ dynamic waits"

Merge "Use POST to submit wikitext to mw.api.parse"

rcfeed: Replace usage of deprecated class RCFeedEngine (in comments)

Doing the replacement for code logic for where it's used is breaking
compatibility, the first change set did this but as adviced, let's wait
for a while until the class is removed.

Change-Id: I3cc0593080f0d51b380c9cdbabc646459e6798e1

Replace pauses in rollback tests w/ dynamic waits

Bug: T220479
Change-Id: I95cf06a4d6a677ca14b56f11f5c6bd98aa0abd05

Merge "HTMLFormFieldWithButton: Don't pass empty string as 'id'"

HTMLFormFieldWithButton: Don't pass empty string as 'id'

The button must have an ID to be able to be infused. If the given ID
is null, that's fine, OOUI will generate one. But passing an empty
string apparently disables that.

Bug: T222013
Change-Id: I96e1f838385c5539ed246d2ee7107cd037a5f338

docs/kss/package.json: Update Gerrit /r/p/ link to /r/

Bug: T218844
Change-Id: Ie503c5f7dc3bd89d983c1b368e25a4d185ae617e

Merge "User: Remove/Kill usage of deprecated User::makeGroupLinkWiki()"

User: Remove/Kill usage of deprecated User::makeGroupLinkWiki()

This method was deprecated in 1.29 and usage of this method can no
longer be found.

Usage
=====

https://codesearch.wmflabs.org/search/?q=makeGroupLinkWiki&i=nope&files=&repos=

Bug: T220656
Change-Id: Iff5ef6666e8ec3e7060ddef8f60362206f4306d0

Merge "jobqueue: make JobQueueDB stricter about broken job_params fields"

Merge "objectcache: replace debugLog() calls with faster debug() calls in MemcachedPeclBagOStuff"

Merge "objectcache: micro-optimize determineKeyClassForStats() by using the limit option to explode()"

Localisation updates from https://translatewiki.net.

Change-Id: I29daa267c536e8dc57b524e4a9232c606364b9cb

Merge "User: Remove deprecated methods ::getGroupPage() & ::makeGroupLinkHTML()"

User: Remove deprecated methods ::getGroupPage() & ::makeGroupLinkHTML()

These methods were deprecated in 1.29 and usage can no longer be found so,
removing ahead of 1.34 cut.

Usage
=====

* User::getGroupPage() -
  https://codesearch.wmflabs.org/search/?q=%5B%5E%3E%5D(User)%3A%3AgetGroupPage%5C(&i=nope&files=&repos=

* User::makeGroupLinkHTML() -
  https://codesearch.wmflabs.org/search/?q=%5B%5E%3E%5D(User)%3A%3AmakeGroupLinkHTML%5C(&i=nope&files=&repos=

Bug: T220656
Change-Id: I4e48eb351883b044269d77a8baaf7b5030138c34

Merge "mw.widgets.TitleWidget: Update icons, use OOUI where possible"

Introduce a BlockManager service

This introduces a minimal BlockManager service, for getting blocks
that apply to a User.

Move the part of User::getBlockedStatus that checks for the blocks
into BlockManager::getUserBlock, and move the related helper
methods from User to BlockManager.

Hard deprecate or remove these helper methods, and move to private
methods in the BlockManager:
* User::getBlockFromCookieValue
* User::isLocallyBlockedProxy
* User::inDnsBlacklist

Soft deprecate these helper methods, and move to public methods in
the BlockManager:
* User::isDnsBlacklisted

Add tests to cover the methods moved to BlockManager.

Bug: T219441
Change-Id: I0af658d71288376735cebe541215383b56bb72e5

Merge "Clarify documentation of minor and notminor API parameters"

mw.widgets.TitleWidget: Update icons, use OOUI where possible

Bug: T222079
Change-Id: I7ff1ca9410236b7404556cb0b90cab3340ee7b0b

Merge "Deprecate User::isBlocked()"

Merge "editpage: Make TextConflictHelper::toEditContent private"

Merge "Avoid usage of deprecated wfGetMainCache() function"

SpecialUserrights: Improve ::userCanChangeRights() method logic

Bug: T37674
Change-Id: I22aa9ff72aec175b0f86ebcf48cd7716f3328b01

Clarify documentation of minor and notminor API parameters

Change-Id: I9f55dd179ea7efcc618120da4e18b7708903a5ff

Avoid usage of deprecated wfGetMainCache() function

This function was deprecated in 1.32 and replaced with the use of
ObjectCache::getLocalClusterInstance().

Change-Id: Id50700a6a7110888b12c4ee1a2100134173760a0

GlobalFunctions: Remove deprecated `wfMakeUrlIndexes()` function

This function was hard deprecated in 1.33 and per code search, usage
is no longer there, probably completely replaced?

Usage
=====

https://codesearch.wmflabs.org/search/?q=%5CbwfMakeUrlIndexes%5Cb&i=nope&files=&repos=

Bug: T42485
Change-Id: I01642a8207f582762428f5a7ac33830066268b2e

Merge "jobqueue: Follow-up for fc5d51f12936ed (added GenericParameterJob)"