Merge "Make WikiPage::doDeleteArticle more robust"

Make WikiPage::doDeleteArticle more robust

When it becomes impossible to load the content of a page due to some
error or misconfiguration, we still want to be able to delete that
page. This change makes WikiPage::doDeleteArticle more robust by catching
any exceptions that may be thrown while trying to load the page content
during the deletion process.

See T128466 for context.

Change-Id: I19f2d16850a3c1af5b504a70a27b9bf1330bc68d

Merge "Floor slave lag before display"

Merge "resourceloader: Add test coverage for DerivativeResourceLoaderContext"

Merge "Use DB_SLAVE in TemporaryPasswordPrimaryAuthenticationProvider"

database: Clean up profiling code in DatabaseBase

This cuts down on conditionals and ScopedCallback use.

Change-Id: Ie478c613b062e45120cdd626f9fb9de09594c638

resourceloader: Add test coverage for DerivativeResourceLoaderContext

* Re-order method tests to match source class.
* Add @covers.
* Add tests to make coverage 100%.

Change-Id: I46bdb66bc08120a82d44208a39d2dd6b7de18de1

Move HTTP 304 check from performRequest to ViewAction

* Follow-up to 8b141886edebc
* The method is now called after the setCdnMaxage() call
  in performAction.
* Allow any CDN urls for the title now, check $wgDebugToolbar,
  and allows caching redirects. The multi-step redirect case does
  not cache however, for simplicity.
* Removed now-unused code in Article that calculated $timestamp.

Change-Id: Ic4f4e3a79d7d386c2f15ca5b11dddf5c57ff9e9f

Localisation updates from https://translatewiki.net.

Change-Id: I1c626bebfe8884c968a8e76e151496f224f90cea

Merge "mediawiki.api.messages: Fix typo in comments"

mediawiki.api.messages: Fix typo in comments

Change-Id: I2d51a982408f54ea62e38d7b3da99af4b73541c6

Rename EditButtonPublishNotSave to EditSubmitButtonLabelPublish

Per legoktm.

Change-Id: I4fbe54876095378a476399bf5e336c8df3e91e14

Merge "Directly use getSession for specialPreferencesSaveSuccess"

Merge "Fix shutdown() code duplication in LBFactory"

Merge "Remove Block::purgeExpired() call from block query API"

Remove Block::purgeExpired() call from block query API

This avoids master queries on HTTP GET. Filter out rows on the
fly and allow for pruning to only happen when blocks do.

Change-Id: Idd25b2f8a789ba191fde87e997910e78d67c7021

Fix shutdown() code duplication in LBFactory

Merge duplicate shutdown() methods from LBFactory subclasses
to the base class, as cronProt is also set there too.

Change-Id: I4050b3469f61b1f4173d5841ff2497f3f30e1382

Merge "Set the WRITE_SYNC flag for SessionBackend writes"

Use DB_SLAVE in TemporaryPasswordPrimaryAuthenticationProvider

Barring ChronologyProtector failure (which itself falls back
to waitForReplication) on account creation, this should be fine.

Change-Id: I94be86c8f56e9e98a9d06814b33a9f08a9228db6

Merge "ApiResult: Remove double space in error message"

ApiResult: Remove double space in error message

Change-Id: I5888d617ab9aebe5ae1fe4da6873639a81f60fc3

Add notes in release notes for supporting xiz and xit in time parser function

Change-Id: I0bec049a23337933a4a2c06b72260d7399c7cd4c

Merge "introduce {{#time: xiz}} for days passed in the year"

Directly use getSession for specialPreferencesSaveSuccess

Change-Id: If9017f6c42f4c8f697504e3b92cee8bfb907b8e7

Merge "Memcached PECL: Allow using binary protocol"

Set the WRITE_SYNC flag for SessionBackend writes

Session writes should be seen in all datacenters so users do not
randomly end up logged on the next page view on a GET request
after login. Sticky DC cookies help, but not for cross domain
redirects or page views.

Change-Id: Id533fa1b867680e6386060efa4878ad1b4638c18

Localisation updates from https://translatewiki.net.

Change-Id: Ic246c3d4e15143a000363bcfb64921cc88e1ae58

Merge "Fix database class doc typos"

Fix database class doc typos

Change-Id: I3e8c4e31326cb4f1a6b427348b42975fba86e62b

Merge "mw.loader: Minor clean up and optimisations"

Merge "Add new UsersPagerDoBatchLookups hook to allow for extensions to modify UsersPager::doBatchLookup()'s rigid behavior"

Localisation updates from https://translatewiki.net.

Change-Id: I430efe1c74261562782b5814c0ed609a340ff88c

Merge "Enable $wgResourceLoaderStorageEnabled by default"

Do not localize content model name inside of english exception messages

Bug: T143889
Change-Id: I2d8004123ca6ae94219bd7880beb1c274ce13417

Merge "API: Some raw message cleanup"

Merge "Rollback: Catch error when unable to load latest revision data"

Merge "Linker: Get rid of "shortcuts" in image linking functions"

Rollback: Catch error when unable to load latest revision data

Ideally this never happens, but it can, so fail gracefully instead
of just throwing a 'call to a member function on non-object' message

Change-Id: Iac8be9c7994aa3e74ed3c0eecff211524037165e

mw.loader: Minor clean up and optimisations

* mw.loader.state: Simplify code by removing redundant branch that could set
  the value to what it already was. Also remove the condition for it since
  re-setting it should never happen in practice.
  If it does happen, it's a harmless no-op.

* mw.loader.store.set: Document use cases for descriptor being partial.
  On a most page views, this branch runs for 'site.styles', 'json', 'user',
  'dom-level2-shim', 'mediawiki.hidpi', 'user.options', and 'user.tokens'.
  Aside from group=user modules, the others are skipped or style modules.

* register: Make faster and reduce GC overhead
  (MacBookPro, Chrome canary 52, Vagrant, 566 registered modules)
  Measured using Timeline (JS Profile) and Profile (Allocation Profile).

  - Use typeof instead of isFunction().
  - In resolveIndexedDependencies:
    - Use a for-loop instead of $.each(). This uncovered a JSHint warning for
      making $.map() functions in a loop. Move the function out and re-use it.
    - Use a for-loop instead of $.map().
  - Don't create an empty dependencies array in the registry only to
    dereference it in the next statement.

  Time for register() down from 5-10ms to 3-4ms.
  Memory for register() down
   from 153KB (88% in register, 12% in resolveIndexedDependencies/each/map)
   to 120KB (100% in register, no mention of resolveIndexedDependencies)
  GC down from 350KB to 240KB.

* Remove use of isFunction(). We're not interested in its edge cases of
  native functions in old IE and regex false-positive in Android 2.3.
  For plain user-specified functions, typeof will return 'function'
  in all supported browsers.

Change-Id: I43fc7c7f54d0c279b659d6fddd21673de48f4bc2

Merge "Make database ping() method avoid starting transactions"

Localisation updates from https://translatewiki.net.

Change-Id: I3b13581197c9b68ffa28e156f47311b6a26d741d

Merge "resources: Deprecate module "jquery.appear""

Make database ping() method avoid starting transactions

Also use the $ignoreErrors flag to simplify the code.

Change-Id: Ieb685366b35487abcd618eb73cf67f4c266a70c7

Merge "Fix docs for the HtmlPageLinkRenderer hooks"

Fix docs for the HtmlPageLinkRenderer hooks

The $text parameter wasn't correctly documented here,
as far as I can tell.

Change-Id: I335d1a59739d80f9f904872e3135f0aae7535956

Merge "ApiUpload: Better handle unreasonably large metadata in 'imageinfo'"

Merge "Clean up array() syntax in docs, part IV"

ApiUpload: Better handle unreasonably large metadata in 'imageinfo'

Bug: T143993
Change-Id: I1fcdbca9981dd034572eeb32070d574cf97a132f

introduce {{#time: xiz}} for days passed in the year

Discussed in the community

Change-Id: I4d5f638d7f0635df2a0a253a3b3bfe278a0aed0e

Clean up array() syntax in docs, part IV

Change-Id: If626409a93d31bf90c054c9bf7ba44a78ea9a621

Remove patch-pl-tl-il-unique.sql

In 1.24 (b8c038f6784ef082), this update from 1.15 (see T18645,
640a6d311e91ccdc) was undone in tables.sql. However, it was not
removed from MysqlUpdater, so update.php would still make the
indices UNIQUE, even when run on a new MediaWiki installation.

In its place, added patch-pl-tl-il-nonunique.sql, which recreates
the indices to match the current definitions from tables.sql.

Bug: T78513
Change-Id: Ic82ca06c8e7051bcd1e0a2a3d5d251e297a9142f

Merge "SQLite syntax checker: Fix support for multiple files"

resources: Deprecate module "jquery.appear"

Bug: T143965
Change-Id: Ib610a624cdce082def3de5efc30ed8721432a7f8

jquery.ui.position: Mark as deprecated

Bug: T142418
Change-Id: Ie6a65e8cfb69f9858c0b04d6f24c307ffcebb5a5

SQLite syntax checker: Fix support for multiple files

Change-Id: I285965189e4186bc557b8ef86168fb666384e5a8

Merge "Update and broaden @covers for PreprocessorTest"

Update and broaden @covers for PreprocessorTest

This class is intended to be an integration test of both preprocessor
implementations and their helper classes.

Change-Id: Iefbd6d8828bbc3278503a0f85efd7d1230a9d66c

Merge "NewParserTest: tear down globals before returning early"

Merge "EditPage: Don't show create label for the button on new sections"

Merge "RELEASE-NOTES: Add release note for 74fa607"

Merge "resourceloader: Make 'site.styles' actually in group 'site'"

NewParserTest: tear down globals before returning early

teardownGlobals() was called at the end of testParserTest(), but not
when returning early or throwing a "skipped" exception. This caused a
test failure when VisualEditor tests were run after parser tests, due to
$wgThumbLimits having the value set in parser tests.

Change-Id: I12d9365813fc51c15f6649084c373f5b7ccfac26

Merge "LoginSignupSpecialPage: Restore old display of 'createacct-helpusername'"

Merge "Add urls from various adware to the CSP false positive list"

Add urls from various adware to the CSP false positive list

URLs are based on spam that comes into the Wikimedia log files,
based on testing on elwiki.

Change-Id: Iee86633abaae86c55764365042681bda1f2304be

resourceloader: Make 'site.styles' actually in group 'site'

Before 93ed259cf and 7bb7315d4, 'site' and 'site.styles' module were one module
loaded in two parts. One part via the general queue (not in a separate request),
and the other part in the stylesheet queue (in a separate request).

This was achieved by hacking the stylesheet queue handler to hardcode the group
to be 'site' for a module named 'site' (later 'site.styles' in c20a76ee)
eventhough the module as a whole was not actually in that group.

Now that these modules are separated, this hack is no longer needed
and the module can be registered as being in group 'site' properly.

Change-Id: If193f556be211749b115f3808ee46536b578fb22

LoginSignupSpecialPage: Restore old display of 'createacct-helpusername'

Note that this message is empty by default. You'll have to create it
on your wiki to see effects of this change.

Bug: T143923
Change-Id: I509c13775426b8d51f65c95af49dffb84ccaa0f4

Localisation updates from https://translatewiki.net.

Change-Id: Ia68c69742fbb7eae69e4160e923ec7fa459f7b67

Merge "Linker: Avoid Title in normaliseSpecialPage()"

Add new UsersPagerDoBatchLookups hook to allow for extensions to modify UsersPager::doBatchLookup()'s rigid behavior

Currently UsersPager::doBatchLookup() assumes that group data comes *only*
from one place, the (local) user_groups DB table. But this isn't correct
when an extension like [[mw:Extension:GlobalUserrights]] is installed.
With the current master version of the GlobalUserrights ext. installed
under MW 1.27, only the *local* groups are shown on Special:ListUsers.
Even if you have a global group called 'staff' and you go to the
[[Special:ListUsers/staff]] page, it *will* display the correct list of
users, but the group data in parentheses is wrong; it's either 1) missing
(if the user is only a member of a global group but not any local groups)
or 2) incorrect in that it omits global group membership(s) entirely.

With this hook, an extension such as GlobalUserrights is able to query an
additional source of user group data (such as the global_user_groups table
in $wgSharedDB in the case of the GlobalUserrights ext.) and have this
data stored in the cache to ensure that global group data shows up as it
should.

Change-Id: Ied2c0f2d5738cf96a66a9672182345d630285639

Merge "debug: Interpolate log messages"

Merge "debug: Merge mediawiki.debug.init module into mediawiki.debug"

Merge "debug: Don't show git branch if on a detached HEAD"

mediawiki.inspect: Output size in bytes used in local storage

The current version only outputs the total size formated
human readable (kb, mb etc). Lets make it also output the size
in plain bytes so it can be formated depending on language.

Bug: T143770
Change-Id: Idda77fdae4e49a7cf46cce2fcd23c2fb2fe43f9d

Merge "mw.htmlform: Don't refer to OO.ui if it might not be loaded"

mw.htmlform.ooui.styles: Remove vertical margin from OOjs UI HTMLForms

Originally added in b524a4333f9c1b42bf6ac0f6caa3055bd8acf84a,
this is no longer needed for framed forms (with setWrapperLegend())
after e5bb1f68cf4154c1bc4389a4404808ebddb5cef9 in OOjs UI and was
never needed for non-framed forms at all.

Change-Id: If4b4b8776080a00685bbcb9026a3d5eb89e4167d

Merge "HTMLFormField: Don't display empty popup in OOUI mode if empty 'help' is given"

Merge "RestbaseVirtualRESTService: Support production URL layout"

Linker: Get rid of "shortcuts" in image linking functions

This code is already hard to read, and using abbreviations definitely
doesn't help.

Change-Id: I681630b66f95319a521ee2aec3be5b678122b73b

Merge "Remove redundant UNIQUE from rev_page_id"

Linker: Avoid Title in normaliseSpecialPage()

Now that SpecialPage::getTitleValueFor() exists, use it so that we can
avoid returning a Title object uncessarily.

Change-Id: I5bf605baf2ecad62e189421a2e5c556cc0b6f6ac

Remove redundant UNIQUE from rev_page_id

The PRIMARY KEY is rev_id, so this is a waste and
makes the index slower to maintain (e.g. no change
buffering).

Bug: T142725
Change-Id: I63f817656ff5e62aa27caf607d70353cc99eb349

mw.htmlform: Don't refer to OO.ui if it might not be loaded

Dependencies are loaded dynamically here, before this code runs.
If there are no OOjs UI HTMLForm fields on the page, then OOjs UI
is not loaded, and the attempt to check if we're working with a
OO.ui.Widget would cause an exception. Invert the test to check
for jQuery, which is always loaded.

Follow-up to 89107070d14187915e760f8543579ec4d784620f.

Bug: T143850
Change-Id: Ibaa5a15fe367ed7470e3e8176e52b13442aeb985

Merge "PreprocessorTest: test both implementations"

PreprocessorTest: test both implementations

* Instead of only testing the configured preprocessor, test each in turn.
* Fix a test error when testing Preprocessor_Hash by removing <equals>
  tags -- only Preprocessor_Hash emits them, but they have no effect on
  the expansion.

Change-Id: I596f6b66fc636b767c447af3450556bfebe28241

Merge "Cleanups to SqlBagOStuff"

Merge "Optimize ChronologyProtector writes for multi-DC case"

Merge "Fix AuthManagerSpecialPage submit button logic"

Fix AuthManagerSpecialPage submit button logic

Bug: T143840
Change-Id: I7fe442f5044c3dce27f89d83b8003dc6e4170296

Cleanups to SqlBagOStuff

* Keep track of the custom LoadBalancer when it makes one.
* Use the custom LoadBalancer to wait for slaves if one
  was used, rather than the main singleton.
* Only wait on the slaves in the LoadBalancer if the
  main DBs are being used.

Change-Id: I11de814306c44f27e0c33b08b5921c0fd4cdc24f

Merge "resourceloader: Improve coverage in ResourceLoaderTest.php"

resourceloader: Improve coverage in ResourceLoaderTest.php

* Fix signature of makeLoaderSourcesScript() to match
  the change in behaviour since e103ba265.

* Consistently order providers before the test.

* Simplify testRegisterValid() and remove needless @depends.

* Remove unused private method stripNoflip().

Coverage:

* Expand test coverage for register().

* Add tests for getModuleNames().

* Add tests for getModule().

* Expand test coverage for addSource().
  (case of invalid array)

* Expand test coverage for makeLoaderImplementScript().
  (case of unwrapped user script, and case of invalid scripts)

* Add tests for makeLoaderSourcesScript().

Change-Id: Ibca3e486fcd3664f171f135327a0f340ee6da9ee

Clear SiteStats process cache after DB update

This fixes a phpunit test error, wherein {{NUMBEROFFILES}} would give
the number of files in the host wiki, not in the temporary database,
when Scribunto was installed, due to a Scribunto phpunit data provider
calling SiteStats::pages().

Change-Id: Ic0d021a72addaa2a13a6b94fd34dccc423de3a8f

Merge "API: Log when too many values are passed for a multi-valued parameter"

EditPage: Don't show create label for the button on new sections

It turns out that the helpful $this->isNew is true on new section edits,
whether or not the page exists, so instead we have to fall back on the
$this->mTitle->exists() call that partially populates isNew anyway.

Change-Id: I7d7cdecda51c6ab4b05496f0b1420cb0264fcccb

debug: Interpolate log messages

Change-Id: Ib61a50863022db796d69264bf738fb0b22f776e9

debug: Merge mediawiki.debug.init module into mediawiki.debug

It just calls mw.Debug.init() on document ready, which can easily be
done inside mediawiki.debug.

Change-Id: Ifaa5a4884c9e430241a4bff51454bb96f468cd61

debug: Don't show git branch if on a detached HEAD

The short SHA1 is already shown with the MediaWiki version, and
displaying the full SHA1 again adds 40+ more characters. Mostly this
annoys me as it causes the toolbar to wrap earlier when making my
browser window smaller.

Change-Id: If7474a090466de708d65cef6dcc30a36df56621f

Merge "Add LBFactory::beginMasterChanges() for doing DBO_TRX rounds"

Localisation updates from https://translatewiki.net.

Change-Id: If5f82b2d32ab93592dd54cf21cb5d36c52315936