Added testStaleSet() WAN cache test

Change-Id: Ib8565c48af09af0df1809e877fde9d4d6c7e7038

Merge "resourceloader: Vary module_deps on language (in addition to skin)"

Merge "Fix Memcached key decode"

Merge "resourceloader: Store relative instead of absolute paths in module_deps"

resourceloader: Vary module_deps on language (in addition to skin)

Re-use the md_skin database field for now as to not need a
schema change, beause this table is going away soon (T113916).

Bug: T113868
Change-Id: I7c7546ec58fd9be0447604989b908dd2084b0fe3

resourceloader: Store relative instead of absolute paths in module_deps

Make paths stored in the module_deps table relative to $IP. This ensures that when
the MediaWiki install path changes and/or if the location of the extension or skins
directory changes, that ResourceLoader's internal model is still accurate.

Previously when these paths change, ResourceLoader would exhibit various bugs.

1. Unable to detect changes in the module (if the directory no longer exists).
2. Point #1 is usually preceeded by one last cache invalidation as the content hash
   of the file path changes (from a valid hash to an empty string).
3. Unnecessary cache invalidation (if both old and new directories exist). This
   happens when a file is both an entry point (in the 'scripts' or 'styles' array)
   and also a file dependency. At first they are de-duplicated by array_unique.
   But after the disk path changes, the next check will result in the old path
   being fetched from module_deps, and the new path from the live configuration.
   This causes two changes that result in needless cache invalidation:
   - The hash list contains one more item (T111481).
   - The hash list contains both the old and new version of a file.
     (or even alternate versions, e.g. when a change is backported to the old
     wmf branch; it also affects wikis on the new branch due to the stale
     file path still in the database).

It seems unusual to move a MediaWiki install, and usually we recommend third
parties to run update.php if site administrators do move their wiki. However
Wikimedia's deployment system implicitly moves the MediaWiki install continously
from e.g. "/srv/mediawiki/php-1.26wmf5" to "/srv/mediawiki/php-1.26wmf6".

This caused virtually all ResourceLoader caching layers to get invalidated every
week when another wmf-branch is deployed, thus breaking these file paths, which
changes the version hash, which then invalidates the cache.

Bug: T111481
Change-Id: I173a9820b3067c4a6598a4c8d77e239797d2659c

Remove b/c dependency of mediawiki.widgets upon the DateInputWidget

The only uses in gerrit were MediaWiki's ForeignStructuredUpload,
updated in 6318b4af, and the UploadWizard extension, updated in
f4632c01.

Change-Id: Ic404d9fb07008cdc2393e20cf92e7ec173e06473

Merge "ForeignStructuredUpload: Depend on mediawiki.widgets.DateInputWidget"

Merge "mediawiki.api.upload: Fix jsduck indentation error"

ForeignStructuredUpload: Depend on mediawiki.widgets.DateInputWidget

Change-Id: I1ba1b5eb415edb7de8d41429199658e9df21cb5c

Merge "FauxResponse: Honor cookie options, and other cleanup"

Merge "Bump oyejorge/less.php to 1.7.0.9"

mediawiki.api.upload: Fix jsduck indentation error

Follows-up 4738b66e754.

The incorrect indentation caused the second line of this description
to be rendered as a <pre> block.

Change-Id: I99b9718234073ed4e7ae02396ff8eb54f87fe1aa

FauxResponse: Honor cookie options, and other cleanup

We have this nice class for unit testing cookie-setting, but the
cookie-setting method ignores all the parameters! Fix that. Also provide
accessors to check the entire set of cookie data, and the set cookies as
a whole.

While this does change the semantics of FauxRequest::getcookie() in that
the name now needs to be prefixed, no extension in Gerrit uses this
method so we should be fine.

Also clean up the case of the setCookie and getCookie methods while
we're at it. Since PHP method names are case-insensitive, this doesn't
even break compatibility with extensions.

Change-Id: Ib44a074bf9796bc0b470d557e39465792f399d30

Merge "Split DateInputWidget & CalendarWidget into a separate ResourceLoader module"

Bump oyejorge/less.php to 1.7.0.9

Fixes leading whitespace in oyejorge/less.php/lib/Less/Version.php

Change-Id: Ifbc353b583fc182c0dc297ccacc8102ed9279c59

Split DateInputWidget & CalendarWidget into a separate ResourceLoader module

...as the first step to splitting up the 'mediawiki.widgets' module.

For backwards-compatibility, it is a dependency of 'mediawiki.widgets'
until extensions that use the widget can be updated.

Also create the `mediaWiki.widgets` object in mediawiki.js instead of
having an entire file just for that one line.

Bug: T108733
Change-Id: Ife4cfba278dad063a3609929ff87eda7c877d589

Merge "Refactored global state out of EnqueueableDataUpdate"

Merge "Removed old fake slave hacks used for debug log testing"

Refactored global state out of EnqueueableDataUpdate

* Follow-up 25a44aa3e4fa
* The static DataUpdate::enqueueUpdates() method will
  deal with JobQueueGroup construction instead

Change-Id: Id7741a770cee81fdc3f61804b7574ccf737bc338

Update OOjs UI to v0.12.10

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.12.10/History.md

Change-Id: I048794c2c3e66e11762598ed97db31afc00cba0c

Merge "More namespace translations for Extremeñu (ext)"

Merge "Actually declare the base LoadMonior::clearCaches"

Merge "Removed $options from RecentChange::newFromConds"

Localisation updates from https://translatewiki.net.

Change-Id: I91a42f708cd83f7dd52a91641088148add7f9a83

Avoid paranoid IDEA error on $conn->getSlavePos() call

Change-Id: I07ca17ad3a434496a3cc14112ad361457c096b4b

Merge "Work around IE9's broken handling of .styleSheet.cssText"

Work around IE9's broken handling of .styleSheet.cssText

IE9 drops @media queries from .styleSheet.cssText when that property
is read. Therefore we can't trust it to contain what we put there
previously.

Bug: T108727
Change-Id: Ic4f553966e48e3fc608a47ef193df923227ceb96

User::incEditCountImmediate(): Add explicit IS NOT NULL check

Within the UPDATE query, if user_editcount IS NULL, user_editcount+1 will
also be NULL, and the change becomes a no-op. So in MySQL, the number of
affected rows would be zero, which is the number of rows that actually
changed.

However, other DBMSs (e.g. PostgreSQL, SQLite) do count no-op changes,
meaning the code would not initialize user_editcount if it were NULL.
Explicitly checking for NULL should ensure consistent behavior across
database types.

Also, if the CLIENT_FOUND_ROWS flag is set when connecting to MySQL, the
server returns "Rows matched:" instead of "Changed:" as the affected row
count. This change would be necessary if MediaWiki, like Drupal, is
changed to use that flag <https://www.drupal.org/node/805858>.

Change-Id: Idac160bae56adc5c5d17f8558c55d87000019741

Merge "Remove $wgUseLinkNamespaceDBFields"

More namespace translations for Extremeñu (ext)

Change-Id: I1bdadbbeb7a416e1047d5961a3579a367aae1be8

Remove $wgUseLinkNamespaceDBFields

Marked as may removed with 1.25, 1.27 should be okay than.

Change-Id: I4ccd7dfe79fa555946cab64dd1eb179a3fc8291d

Removed $options from RecentChange::newFromConds

* The FORCE INDEX statements are not needed

Change-Id: Ia6f386140e88ad01fc35a10b00dbcb0d1eb934ea

Make it more obvious that users can remove their email address

* Add help message to new email address field indicating that the field
  should be left blank if the user want to remove their email address.
  and the consequences of no email address it.
* Update changeemail related messages.

Bug: T98799
Change-Id: Ie92e393207ee30811fd30e25e72b9e523d95fd62

Merge "Add mw.ForeignStructuredUpload.BookletLayout"

Merge "mediawiki.jqueryMsg: Allow more than one thing inside a wikilink"

Merge "Fix plural in mergehistory-done message"

Merge "Fix incomplete PHPDoc tags in ApiBase"

Fix plural in mergehistory-done message

Follow up to I9931bf48182bd41ff32bdac1798336319f233341

Change-Id: Ia4eb3bef3a11faef654922fe05c0d9ef98dc9b90

mw.widgets.TitleOptionWidget: Prevent page navigation by clicking labels

However, allow opening the link in a new tab.

The same code is used in other components, e.g. jquery.suggestions.

Follow-up to ba358947bfb81b58d64489063bd4be816d1a46da.

Change-Id: Ie6840a20c1cc1a10fb1fb74717982dbe7e331d36

Fix incomplete PHPDoc tags in ApiBase

Change-Id: Icb388723e51c8b07fee3900b98795cfe623942ac

Add mw.ForeignStructuredUpload.BookletLayout

Bug: T91717
Change-Id: Ibf283fd185e191ed6ac8668efe6409f11e73f439

mediawiki.jqueryMsg: Allow more than one thing inside a wikilink

So that [[foo|bar {{SITENAME}}]] becomes parseable.

Change-Id: Idf825e4df24cb9a10cac136811936e54e8fce7b6

Merge "mw.widgets.DateInputWidget: Actually display the indicator"

Merge "SpecialMergeHistory: add redirect=no on target parameter on success message"

mw.widgets.DateInputWidget: Actually display the indicator

We were only passing stuff around...

Change-Id: I0d494cffae0c26a6b04ffe102e9e9d6f3c5f5c81

mw.widgets.DateInputWidget: Actually enforce date requiredness

Also, never set 'invalid' flag before the user interacts
with the widget.

Change-Id: I1a00fcbd9d2f675487e821316a3b5ece6845e343

SpecialMergeHistory: add redirect=no on target parameter on success message

Removes mergehistory-success message and introduce mergehistory-done
so that fullurl doesn't have to be added to each translation message.

Bug: T85650
Change-Id: I9931bf48182bd41ff32bdac1798336319f233341

Merge "mw.widgets.DateInputWidget: Implement $overlay config option"

Merge "OOjs UI: Backport d190bc51e60195eed81f147e9324c9ec21c9e68c"

Merge "Remove blue border from DateInputWidget calendar"

Merge "Fallback and namespace names changes for Asturianu (ast)"

Merge "Use variable documentation in Preprocessor_DOM.php"

Merge "Give instructions on removing email address from account"

Actually declare the base LoadMonior::clearCaches

This fixes IDEA errors in LoadBalancer::clearLagTimeCache

Change-Id: Ibd863ef6bc670a29f65821e3f3cd9d721a652b9a

Removed old fake slave hacks used for debug log testing

* Any modern automatic tests would use mocking methods

Change-Id: I5fbd8ebe7090f75bdcccc7a2848ae88f34a64512

Fallback and namespace names changes for Asturianu (ast)

* Set $fallback to Spanish (es), as Asturianu (ast) is a language spoken
solely in Spain. If a message is missing it'd be far more easier for local
users to understand Spanish rather than English.

* Added namespace gender aliases for asturianu.

Change-Id: Iafad07bae88bff5e6c39e5759f2ef4e5efcade1e

Merge "Clarified WAN cache docs by using "datacenter" consistently"

Merge "Restructure /resources/src/mediawiki.page/"

Merge "WatchAction: Clean up redundant getFormFields() and show()"

Restructure /resources/src/mediawiki.page/

Follows-up 0bfdd927.

Change-Id: I65b9343ea002c332323ededf4e328e4463faf698

Restructure /resources/src/mediawiki.api/

Re-do Ifbb0f6751 in a smaller scope as a first step.

Change-Id: I346f3587d3bfeaf0fe3467cd1f4dcf2d134ecc08

mw.widgets.DateInputWidget: Implement $overlay config option

Unlike in other widgets with popups, the one here (CalendarWidget) is
focusable. As the text input and calendar are not in DOM order when
using $overlay, we fix up focus transitions between them manually.

Bug: T112676
Change-Id: I504f81f843e8328f609824d44f276296645d5dd1

WatchAction: Clean up redundant getFormFields() and show()

* Make FormAction::getFormFields not abstract. In most cases this will just
  be an empty array. This is in prep for RollbackAction as well.

* Remove redundant show() in WatchAction. This used to do custom stuff,
  but after 77cdf1919 it does exactly the same as the parent FormAction::show.

* Don't add 'redirectparams' hidden field if there were no custom query parameters
  (e.g. plain index.php?title=..&action=..)

Change-Id: Ia7f9bb0367c49a23179e9fefa9f529fa8aef8f52

Clarified WAN cache docs by using "datacenter" consistently

Change-Id: I448c57cebd0481809fc17a7c2ac698c73b17dcba

Merge "mediawiki.widgets: Cleanup TitleWidget styles and improve API"

OOjs UI: Backport d190bc51e60195eed81f147e9324c9ec21c9e68c

Required for mw.widgets.DateInputWidget when used with '$overlay'
config option, new in I504f81f843e8328f609824d44f276296645d5dd1.

Change-Id: Ib494c58b9dcaecba62e1aaf501880b85452b7826

Merge "WatchAction: Require POST for index.php action=watch"

Merge "build: Bump various devDependencies to latest"

Added $opts to WANObjectCache::set() to detect snapshot lag

* This can avoid some stale write race conditions
* Made use of this option in a few key places
* HOLDOFF_TTL was also bumped

Change-Id: I83505a59cce0119e456191c3100f7d97bc86bdbf

Merge "Include additional analytics in Special:Search"

Merge "Fixed sanity cache clear in User::saveSettings()"

WatchAction: Require POST for index.php action=watch

The GET variant was already rarely used because our frontend enchances these links
with a click handler that uses AJAX to make a POST request to the API.
The index.php url, nor its token, were used for the majority of users.

Simplify this by stripping the 'token' query from these urls and requiring a POST
request for index.php?action=watch and unwatch.

* FormAction: Actually set a proper '<form action>' instead of letting HTMLForm
  default to a confusing title path (e.g. /wiki/Pagename). Article path should
  not be used for POST requests.

* WatchAction: Group all FormAction-related methods together.

* WatchAction: Make token consistent with other actions now that it is POST-only
  (no "stronger" salt containing the page title).

* Remove ununsed mediawiki.page.startup dependency from mediawiki.page.watch.ajax.

* WatchAction: If accessed over GET directly (e.g. for users without javascript)
  display a confirmation form that submits the token. Similar to PurgeAction.

Change-Id: I504f457e68a133bcfc418cff13b838080fec1008

Include additional analytics in Special:Search

* Adds query performed to javascript config, to be picked up by
  the TestSearchSatisfaction schema.

* Adds data-serp-pos attribute to serach result anchors.
  This is picked up by the TestSearchSatisfaction schema
  and appended to urls (when participating in the test)
  as the `wprov` query parameter.

Bug: T113291
Change-Id: Ia0e8a955806e225e2f5a500e9290145895737274

Fixed sanity cache clear in User::saveSettings()

* This works by adding a refresh mode to clearSharedCache()
  when we want to purge the cache in case it might stale to
  avoid further CAS errors. Because an exception will be
  thrown, the usual DB callback will not get fired, so avoid
  using commit hooks when doing these cache purges.
* Also lowered the tombstone TTL for such purges, since no
  data actually changed.

Bug: T114023
Change-Id: Iaad87b4ed24733dac40bc9607d3c97c940710087

Merge "Made HTMLCacheUpdateJob flush the trx between jobs"

Merge "Made convertNamespace() use APC"

Merge "Fixed IDE errors due to SquidUpdate::newFromTitles"

Localisation updates from https://translatewiki.net.

Change-Id: Ibae1e9fa24a63f2465f17ca0e253957f084ddc5f

Merge "Avoid CAS errors on double post to ApiOptions"

Merge "Begin 1.27 development cycle"

Merge "Add release notes for protect log changes"

Add release notes for protect log changes

Follows I57b3bd8a7dc823acdbb56520d2364f5542283373 and
I3bb85c61b857972e66c99c499d7d785c88cafb25

Change-Id: If6fe021ad73b8ab74848feb6eeb3c1d21b4187c7

Merge "resourceloader: Cleaned up makeModuleResponse() param docs"

Merge "resourceloader: Tidy up RL to simplify ResourceLoaderEditToolbarModule"

resourceloader: Cleaned up makeModuleResponse() param docs

Change-Id: Iabf53452598e13a98241793c317ceffb746aa46e

resourceloader: Tidy up RL to simplify ResourceLoaderEditToolbarModule

* Remove ResourceLoaderFileModule::getLessCompiler(). There is no reason for a
  module to need to get a compiler in a different manner than
  ResourceLoader::getLessCompiler().
* Add ResourceLoaderModule::getLessVars(). This method provides a means for
  subclasses to easily inject custom LESS variables. The default implementation
  simply returns an empty array.
* Make the $context parameter for ResourceLoaderFileModule::readStyleFiles()
  non-optional (via graceful deprecation). The only callers I found were either
  already calling it with a ResourceLoader context, or had a perfectly usable
  ResourceLoaderContext in local scope.
* Make ResourceLoaderFileModule::{readStyleFile,getLessCompiler} require a
  context. These methods are protected, so I didn't bother with a deprecation.
* Call ksort() on the LESS variables array in the only place it matters -- when
  hashing its serialized representation to construct a cache lookup key. This
  relieves getLessVars() subclasses from having to remember to re-sort the
  variables array if they modify it.
* These changes make it possible to substantially simplify
  ResourceLoaderEditToolbarModule, because the only thing it needs to do now is
  implement its own getLessVars() method.
* This also allows it to be versioned like any other ResourceLoaderFileModule,
  rather than having to use enableModuleContentVersion().

Change-Id: Ic3eab71691e502bfe19bdf4eb6f82cc679a7782f

Merge "Code cleanups and simplifications to SpecialMergeHistory"

Merge "build: Set private flag in package.json (for now)"

build: Set private flag in package.json (for now)

This fixes the warning about missing package description and license.
This matches what we do elsewhere for dev-only npm pipelines.

Change-Id: I29ee43482b90ea8875a5376409d656648e3c4b27

Remove blue border from DateInputWidget calendar

CalendarWidget will now only get the blue focus border when it is
focussed itself, not always inside a focussed DateInputWidget.

Bug: T113741
Change-Id: I0332a9bac4c5ec6abb5a836125718e6b779bf496

Give instructions on removing email address from account

Update the description for the Special:ChangeEmail form to include
instructions on how to remove email address association from the user's
account.

Bug: T98799
Change-Id: I10dca0c24faf24b117ef7d1e3c20319575805aff

Use variable documentation in Preprocessor_DOM.php

Instead of having comments behind variable declaration.

This also avoids mixed tabs and spaces at begin of line

Change-Id: Iba62430f4413fd52bac1d51f5c5df4cb6479284d

Remove border radii from CalendarWidget

Also remove rule removing border from 'today' on hover.

Bug: T113740
Change-Id: I9d71064b847433ed98ebaa8a1dce36af2cca7d27

Fix Memcached key decode

Flow had a key: flowdb:flow_ref:wiki:by-source:v3:Parser\'s_"broken"_+_(page)_&_grill:testwiki:1:4.7
the '+' in there was not being encoded (it only does /[\x00-\x20\x25\x7f]+/)
but coming back, it was decoded into ' '.
getMulti() shows a key=>value array or results. Since key was different,
we couldn't find what we had requested.

Bug: T110326
Change-Id: Ia92edd73d0eb7fe0d35e38e7e7af8174fb85cbcc

mediawiki.widgets: Cleanup TitleWidget styles and improve API

* Cleanup LESS
* Make label an <a> instead of appending $link
* Make real thumbnails get darker during over using opacity
* Increase gap between results to 2px
* Make config and config.data required
* Replace config.title with config.url
* Remove link 'click' event killer, may be reapplied in later,
  more complete, commit
* Fix documentation

Change-Id: I3cca0c2c546e1c9f840c33b7cb8281aeb75a81f1

Put InfoAction distinct user counts behind miser mode

* The random I/O due to secondary lookups causes timeouts on
  larges pages that keep showing up in the logs.

Change-Id: I9bddcd3ba9ad5ff2f26ccec4553906ecc4a8129b

Merge "In parserTests.php, fix upload directory handling"

Code cleanups and simplifications to SpecialMergeHistory

* Fixed all IDEA errors for this file

Change-Id: I5cbb660c1a138ce7a8bf023929d97f1c75f0a80a

Don't overwrite all available object caches

Just because some defined object caches are problematic in the installer, it
doesn't mean, that all others are problematic, too. Disable only the object
caches, which are known problematic ones by preserving the others.

Follow up: I1c1938b3fac395d

Bug: T113843
Change-Id: I951ef22d97b4c66d3b781a9bc4a30e7e2fca1395