mediawiki.template: Mustache should return jQuery object instead of array

This is consistent with Flow and Hogan in Mobile.

Change-Id: If5c9e08ade185ebdb42a292f87d7506a68528c88

Merge "Combine deleteArchived{Files,Revisions}.inc into the .php scripts"

resourceloader: Use mw.log instead of 'throw' for scripts with syntax errors

For scripts loaded from load.php using mw.loader.implement(), an
exception is handled fine and results in the module state progressing
from 'loading' to 'errror'.

However, for wikis modules loaded with only=scripts from the HTML directly,
there is no wrapper. As such, the state() call appended to the script is
never reached and the module indefinitely stays "loading".

VisualEditor (and others) already explicitly ignore the error state
of the wiki modules (with inline mw.loader.using and casting the Promise to resolved).
However in this case the Promise never completed to neither success nor failure.

As such, this was causing users with syntax errors in their user script to
encounter a stuck VisualEditor loading bar that never goes beyond 70%.

Change-Id: I091a020bdd3392b965e0e25d03b216037fadc48f

Merge "cache: Add in-process caching to MessageBlobStore"

cache: Add in-process caching to MessageBlobStore

This allows us to continue to use MessageBlobStore::get() as a way
to pre-fetch messages in batches for multiple modules while also
allowing individual modules to retrieve  their messages in a convenient
way.

This is in preparation for implementing ResourceLoaderModule::buildContents
and using it in ResourceLoaderModule::getVersionHash.

ResourceLoader::respond() already fetches message blobs in batches for the
actual response, but it also needs getVersionHash(), which would otherwise
fetch the same messages a second time.

Change-Id: I7e4c8b65765b54807123e85cfbb7eb2e5b2f39bd

Merge "Check result of preg_match_all in Parser.php"

Use less fuzzy User::getDBTouched() in ApiStashEdit::getStashKey()

* The getTouched() value can bump more often or vary in DCs

Change-Id: Idffa22c9dde7c7950943b104ed61f5779636b7e7

Localisation updates from https://translatewiki.net.

Change-Id: I860b228e2f23cdb32f066943d377b583bc6436eb

Fixed $flags bit operation precedence fail in User::loadFromDatabase()

* The bug made READ_LATEST also act as READ_LOCKING

Bug: T100577
Bug: T100869
Bug: T98706
Change-Id: I85092970c236fc4482371216b9f6592c49a27942

Move xml_free_parser out of destructor.

Attempt to reduce error log spam. I'm not sure about the exact
cause of this error, nor have I really reproduced locally, so
I don't know if this will help, but getting it out of destructor
seems like a good idea.

I also do not know if calling xml_free_parser is really needed
(Would php automatically destroy parser during garbage collection?)

Bug: T89532
Change-Id: I5709b6cf26cf4a9032a79d2b5f48ed3b4238eaee

Merge "Use a fixed comment placeholder string in CSSMin"

Merge "OOUI theme support"

Use a fixed comment placeholder string in CSSMin

Both the problem and solution are nearly identical to the ones presented in
I31d4556bb. Use a constant string as a comment placeholder, and differentiate
it from possible user input using \x7f, which is not valid CSS anyway[1].

  [1]: http://www.w3.org/TR/CSS21/grammar.html

Change-Id: I3d3c2578d5e9c1fe52c02e87901cb08d03847ea7

Merge "Fixing definition and use of $attrs parameter in Html::buttonAttributes"

Merge "Improve doc of maxage and s-maxage API parameters"

Merge "Made LoadBalancer debug log entries use "localhost" placeholder"

Merge "Rewrite Language::hebrewNumeral()"

Merge "Reduce page deletion lock contention"

Rewrite Language::hebrewNumeral()

Use arrays instead of strings, to avoid using
string functions with Unicode.

Handle thousands according to how years like 1000, 2000, etc.
are named in the Hebrew Wikipedia.

Bug: T97444
Change-Id: I5334e86793d28dfcf8939a249b03a5ea85fa4e69

Cleanup the code of Language::hebrewNumeral()

Change == to === and add empty lines for readability.

Change-Id: Ifeabbf0d6adca25d8a797bd423f1bbdc0f8223ee

Add tests for Language::hebrewNumeral()

Some failing tests are commented out and will be properly fixed
in subsequent commits.

Bug: T97444
Change-Id: I19721b5dc3dc6bbe923d9bf401fcf5d765fb7a7c

Improve doc of maxage and s-maxage API parameters

They set HTTP cache control headers.  i18-only change.

Change-Id: I9eefdd51d6937388326ceb7b6baa72288221c2f7

Merge "Add List-Unsubscribe header to emails"

Merge "mediawiki.ui: checkbox: Render "on top of" the label"

Fixing definition and use of $attrs parameter in Html::buttonAttributes

Also clarifying definition of $modifiers parameter.

Also simplifying code for case where $attrs['class'] is not set.

Change-Id: I425211681ba75cb71c1ccc3b3c038c075ea9acb9

registration: Remove "additionalitems" property from array (authors)

It doesn't make sense for arrays which can't have items to begin with.

Follows up bfe4ddd8108efe

Change-Id: I851b70782fa8210802fe576fee771ddf1bdd7fec

Merge "composer.json: Downgrade PHPUnit to 3.7.37 (temporarily)"

Merge "Follow-up 727454f: Add mediawiki.widgets to jsduck"

Merge "Add help link for 8 more special pages"

Follow-up 727454f: Add mediawiki.widgets to jsduck

Change-Id: I1715c5bbca8735de8c391dee4e2cec8269414acf

Merge "Commit all connections after each job"

Commit all connections after each job

If you don't commit the slave connections then they keep their old snapshots.
This clears the snapshots so they don't get out of date views of the world.

Bug: T100838
Change-Id: I1f6f910d88324beb589b2ad9466d8786376eda55

Localisation updates from https://translatewiki.net.

Change-Id: I491182176e3357ed36de4856e20af71d735dcd27

Merge "Moved addAutopromoteOnceGroups() call to a deferred update"

Merge "Add tests for HttpError"

Add tests for HttpError

Still missing tests for HttpError::report, but would
have caught the issue that Ie216d19 fixed.

Change-Id: Ic35d70af52c3a2d2a25fc3b9952383198db31fb1

Remove legacy vertical-align from select fields

They shouldn't be aligned at the top by default.

Bug: T98493
Change-Id: I17b0825079ff28dfc9ea2c1ec24018073aec3794

Fix undefined variable $header in HttpError

Change-Id: Ie216d1959f2ea903fdbc124b718b4988e08229ae
Follows-up: I8b61d7e9ea4101e3a9ef5f9a59a97db45aeef68c

Merge "Add 'mediawiki.skinning.logo' module"

HttpError: Consistently escape document title

Wasn't actually a vulnerability because HttpStatus::getMessage
can only return one of a fixed set of values which are all plain
text without any special characters. However the return value
there is meant to plain text and not html, so just like
Html::element and other interfaces, things should be consitently
escaped.

Also renamed variables for clarity.

Change-Id: I8b61d7e9ea4101e3a9ef5f9a59a97db45aeef68c

Merge "Add Vivek Ghaisas (polybuildr) to the CREDITS file"

Merge "Made isAllowed(), showHistory() and formatRevisionRow() protected to allow for easier subclassing"

Merge "SpecialMediaStatistics: Protect against invalid indexes"

Merge "Fix some RecentChange phpdoc return types"

Merge "Tweak 'position' of 'mediawiki.skinning.*' modules"

Fix some RecentChange phpdoc return types

Change-Id: I3e75a97f7fc76e2dc052463511e32f330cfca9ce

Merge "Use a fixed marker prefix string in the Parser and MWTidy"

Merge "Move MWTimestamp::getHumanTimestamp() to Language"

Use a fixed marker prefix string in the Parser and MWTidy

Generating one-time, unique strip markers hurts us in multiple ways:

* The strip marker regexes don't benefit from JIT compilation, so they are
  slower to execute than they could be.
* Although the regexes don't benefit from JIT compilation, they are still
  compiled, because HHVM bets on regexes getting reused. This extra work is
  fairly costly (1-2% of CPU usage on the app servers) and doesn't pay off.
* The size of the PCRE JIT cache is finite, and the caching of one-off regexes
  displaces from the cache regexes which are in fact reused.

Tim's preferred solution (per his review comment on
https://gerrit.wikimedia.org/r/167530/) is to use fixed strip markers.
So:

* Replace usage of $parser->mUniqPrefix with Parser::MARKER_PREFIX, which
  complements the existing Parser::MARKER_SUFFIX.
* Deprecate Parser::mUniqPrefix and its accessor, Parser::uniqPrefix().
* Deprecate Parser::getRandomString(), since it is no longer useful.
* In Preprocessor_*:preprocessToObj() and Parser::fetchTemplateAndTitle,
  replace any occurences of \x7f with '?', to prevent strip marker forgery.
  \x7f is not valid input anyway.
* Deprecate the $prefix parameter for StripState::__construct, since a custom
  prefix may no longer be specified.

Change-Id: I31d4556bbb07acb72c33fda335fa5a230379a03f

Remove deprecated MWFunction::newObj()

Change-Id: I180e9e1e0bcf17c9e72b607c69cae00f47de6579

OOUI theme support

* Split off OOjs UI's module definitions to a separate file from
  Resources.php.
* Extend ResourceLoaderImageModule to support per-skin images and
  variants.
* Allow skins to specify their preferred OOjs UI theme using new
  skin registry attribute 'SkinOOUIThemes'. The default remains the
  'mediawiki' theme, 'apex' can also be chosen now.
* Implement custom ResourceLoaderOOUIImageModule which implements some
  special sauce required to shoehorn OOjs UI image set definitions into
  ResourceLoaderImageModule with skin-specific styles support.

Bug: T100895
Change-Id: I3fbacbce95126a4cc29221352f3ef846f41e7b1b

Whitespace fix for RELEASE-NOTES-1.26

Change-Id: I23a02f465b20117a1855e2a0881fc3bdf1e9ab14

Merge "Remove mw-ui-icon*:after"

Merge "Create mediawiki.widgets and mw.widgets.TitleInputWidget in it"

Merge "Implement MalformedTitleException for JS and use in constructor"

mailmap: Add an additional e-mail for Bartosz per request

Change-Id: Ia3c9f1c83a8d15cb5758288d435b31c3bf1e8b19

Implement MalformedTitleException for JS and use in constructor

Implements MalformedTitleExceptions to provide information on
  invalid titles.  Throw these exceptions from Title constructor
  instead of generic Errors.

Bug: T89648
Change-Id: I9ed16a5a30b01df785ad736df4b84111c5939bbf

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

Localisation updates from https://translatewiki.net.

Change-Id: I42c49984b15a57196744b95c393615fe7f4f797f

resourceloader: Fix broken 'version' param in mw.loader requests

All requests were using version=da39a3ee5e6b, where da39a3ee5e6b
is the sha1() of the empty string.

Follows-up f37cee996e which incorrectly assummed the local
'modules' variable to hold a list of objects, when in fact it
holds a list of strings.

Change-Id: I717b8657300263606832ff1ec67a34a7163de842

Create mediawiki.widgets and mw.widgets.TitleInputWidget in it

Change-Id: I43fb412ff65271fd411b14352784cb92f1f5487b

Merge "list=categorymembers: Do not log feature usage when using default value"

Remove mw-ui-icon*:after

Not being used anywhere from what I can see.

Change-Id: I83cff1444a12fdd390f78dd1c09b5b09b0c84704

Fix to digits grouping rule

Avoid grouping in negative numbers too.
Following the example of LanguagePl.php

Bug: T94300
Change-Id: Ib96616497507c9347cc72f894243b583619c2b11

Move MWTimestamp::getHumanTimestamp() to Language

We're trying to remove MediaWiki dependencies from MWTimestamp so it can
be split out into a separate library. In addition to getting rid of a
dependency on RequestContext, Language, and User, it makes more logical
sense there anyways.

Bug: T100924
Change-Id: If46eaea42d8a5a808c10f0dc353e181714295a44

Localisation updates from https://translatewiki.net.

Change-Id: I4d6988a77f0414b4906f123d888f930b9cdfe253

list=categorymembers: Do not log feature usage when using default value

list=categorymembers&cmstartsortkey was logged even the default value
was set

Bug: T100878
Change-Id: Idc52fe0c208216bc3e475d0e4b2e398cb73a6b80

DeferredUpdates doc comment tweak

Change-Id: Ib58bcb065ef7fac58c3c579eb0ca45e51ccdae11

Add 'mediawiki.skinning.logo' module

Provides ability to display wiki logo with ResourceLoader without loading
the big 'mediawiki.skinning.interface' module.

Used by GreyStuff skin: Id79688a4eb4bf7f813e0c5af1d89556c488b0bde

Change-Id: I7cf51214d9fa378a321c90fc85b0c6799e3b3931

Tweak 'position' of 'mediawiki.skinning.*' modules

mediawiki.skinning.elements:
mediawiki.skinning.content:
  Add 'position' => 'top', like mediawiki.skinning.interface.
  Follow-up to d6b4d3c537fa5944c2f234213b20037fe3192d62.

mediawiki.skinning.content.externallinks:
  Set 'position' => 'bottom', as this module only provides icons
  (and provides a ton of them), which can be loaded after initial load.

Also tweaked some comments.

Change-Id: Ib26afd6cbb2b5d4a7557c5c8243d2f7482a6f064

QA: Upgrade mediawiki_selenium to 1.2 for Raita logging

Change-Id: I4c6bddca1a7c964f3e8543ff7853565505485c51

Fixed Job constructor IDE notices about variable types

Change-Id: I4b4e4e38e8d416c3445c52ced311f5fbfcde868a

Moved addAutopromoteOnceGroups() call to a deferred update

Change-Id: If7941465889c1848ccf1be4ff5505c5eb53a14f1

registration: Don't hardcode list of extension types in schema

Extension types can be added via a hook, so don't try and validate
against a hardcoded list.

Change-Id: I0bc6fbcedc0b582b81c7037724ad01441ae415f6

registration: Improve ResourceModules schema definition

* "-" is a valid character in module names
* debugScripts, loaderScripts, dependencies, and messages can all be
strings in addition to arrays.

Change-Id: I76570ebf1cb02ca51c5185d4b0c48a553a0e7d2f

Cleaned up various revisiondelete IDE warnings

Change-Id: I4750eabf9985785520aaac8eca12d488ca746966

Refactored entry points to have uniform shutdown handling

* Added doPreOutputCommit() and doPostOutputShutdown(),
  which most entry points just using the later
* Also fixed problem where text profiling did not show up
* Avoid calling triggerJobs() in the file streaming
  entry points

Bug: T100127
Bug: T100085
Change-Id: Ibc7e768fd483389a01847f08cdeba4058c853d3f

Merge "Make ResourceLoaderWikiModule support custom position"

Make ResourceLoaderWikiModule support custom position

Bug: T97410
Change-Id: Ib1180c5b2126f4ba8ac5b54578fcce18a3f35d85

ApiOpenSearch: Ignore any other unexpected keys in json output

Iad295d2f fixed warnings to not break the opensearch json output. But
other stuff like $wgDebugToolbar can do the same thing, so let's strip
all unrecognized keys to avoid that problem.

Bug: T100635
Change-Id: I20b6502482af537ccb703f400508960038213a91

Add Vivek Ghaisas (polybuildr) to the CREDITS file

Change-Id: I5bc31ba9f727123a374f0d14c8da22e0c38a0f3f

Localisation updates from https://translatewiki.net.

Change-Id: I202e3eff5b911eaf11f960c66e3d2d5bc95e1d0f

Reduce page deletion lock contention

Bug: T98706
Change-Id: Ib37e74c4737de9d705af28ac2eac06e716dd5f10

API: Add more parameter types and improve info

New types 'text' and 'password' for where a <textarea> or
<input type="password"> would be preferred over <input type="text">.

Some timestamp parameters get actually tagged as 'timestamp'.

'submodule' types change the 'submodules' output property from a boolean
to an object indicating the mapping from values to module paths. And
they get an indication of the submodule parameter prefix (e.g.
generator's "g"), if applicable. "generator" actually gets reported as a
submodule type, using this new mechanism.

action=paraminfo will now indicate ApiBase::PARAM_RANGE_ENFORCE status,
and return better-formatted defaults for timestamps and booleans.

Change-Id: Ic862d6f8fe13f7eb6b4298683514d33af5823e47

Merge "Cleaned up JobQueueRedis exceptions"

SpecialMediaStatistics: Protect against invalid indexes

Avoids Undefined index: \d in
includes/specials/SpecialMediaStatistics.php

Bug: T89534
Follow-Up: Icd06d2b22b1fcb57a8849ef6dc8659b424f27fdc
Change-Id: I21b0bd4329c9b6fa06526326342edddba3f250f6

Merge "registration: Remove unused variable"

Merge "Update context stats in SiteStatsUpdate"

Check result of preg_match_all in Parser.php

preg_match_all can return false on failure, which than results in
undefined index access.
Check the result and just keep it as nothing found by processing an
empty array

Change-Id: I1f11894240dc6869506d68d3513715abdc3abb5d

Update context stats in SiteStatsUpdate

In addition to updating the database / memcached, make SiteStatsUpdate also
update the context stats object with the deltas it is processing.

Change-Id: Icc12c07ab4233e1105a402b80d290bdb64df6ea7

composer.json: Downgrade PHPUnit to 3.7.37 (temporarily)

In preparation of fetching development dependencies (such as PHPUnit!)
for unit tests via composer in jenkins runs, we should initially use the
same version of PHPUnit to avoid any issues that might come during a
version bump. Once that is stable, we can upgrade PHPUnit again.

Bug: T90303
Change-Id: I907e6d061cab72d1647fcbb2eea35ff144febd7e

registration: Fix types of ResourceModules properties

styles and scripts can both be strings, and that is treated as if that was
the first value in the array.

style can be used as an object, the filename is the key, and value can be an
object that states what media type the stylesheet is for. An example of this
can be seen in the MonoBook skin.

Change-Id: If7d3a220cdccbb1ce096d776592535fe5782a954

registration: Version the extension.json schema

Versioning the extension.json schema will allow us to make breaking changes
to the schema in a non-breaking manner.

Extensiosn and skins will set a 'manifest_version' value, stating which
version of the schema the file is written for. Processor::extractInfo() will
be  passed the version number, and can switch behavior depending upon it.

For backwards-compatability, a version number of 1 is assumed if none is set.
The validateRegistrationFile.php script will emit a warning if this is the
case.

Bug: T99344
Change-Id: I2086a1465ceaeedd1ccc6804fda2c304ad16ffa0

registration: Remove unused variable

It is immediately overwritten. Follows up c573fc5a47204d29314.

Change-Id: I901cfebe088ca0020323db03378fe23117bb3985

Localisation updates from https://translatewiki.net.

Change-Id: I9c9efe2b5b83d4971ef8ebddb7c4ce57006365a2

Merge "Don't run phpcs on node_modules folder"

Warn when duplicate arguments are found

Currently, duplicate arguments result in a categorization but not a
warning, and it's often difficult to find where in the template hierarchy
the problem lies. This causes a warning to be provided containing the
calling page's name, the called template's name, and the parameter's name.

Bug: T85352
Change-Id: I26b9a7ed5a2f246d00a49a5f6effe40b4443a9d0

Don't run phpcs on node_modules folder

This is an artifact created by npm in certain modules. These modules
may contain PHP e.g. grunt-phpdocumentor

Change-Id: Ie46bceb5acbe6141d1e0dc4d3397d04f41d01485

OOUIHTMLForm: Access context after parent constructor is called

This avoids "ContextSource::getContext (OOUIHTMLForm): called and $context
is null. Using RequestContext::getMain() for sanity"

Change-Id: I20c2bb58b3a0e5e259241a3715ee8fdf4b57e148

HTMLForm::factory() doesn't throw MWExceptions

Change-Id: I0104bf1dd9ebb813aa37f92b21515a964ca2a709