Merge "Allow search suggestions in skins operating in mobile mode"

Merge "resourceloader: Allow mobile target by default on SkinModule"

Allow search suggestions in skins operating in mobile mode

MobileFrontend silently removes this skin from the page. This is
now made explicit and the module can be safely loaded in a mobile
environment.

This is used by both Timeless which although does not work
perfectly on a mobile device it should be easy to fix with additional
work.

Change-Id: Iedea2872d14430db452cec7e758f20d854778414
Depends-On: Ic36e9792f9217f3fd37bbd1f5c66d894301363f0

resourceloader: Allow mobile target by default on SkinModule

If a skin is using this class, it's likely to be pretty new.
The targets system was mostly created for older code.

Let's make this the default so skins don't need to do anything
additional to work on mobile.

This simple change makes the Timeless skin work on mobile
when MobileFrontend is installed: ?useformat=mobile&useskin=timeless
It looks beautiful :)

Change-Id: I2ab8a1a634bdc0b5b2084d227c7388b5382e93e8

Merge "Make file purging also purge old versions"

Move wgBreakFrames client code to mediawiki.page.startup

Follows-up f7c324685195, which migrated this from legacy wikibits
to the 'mediawiki.page.ready', however it's better suited in
'mediawiki.page.startup' because that one loaded on all pages blindly
(used to be hardcoded in OutputPage, now part of 'core' group
in Skin::getDefaultModules).

mediawiki.page.ready on the other hand is primarily for enhancing
the page content, loaded in Skin::getDefaultModules in the 'content'
group, which extensions like MobileFrontend may override with an
alternate implementation. This means frame breaking is bypassed!

Change-Id: Ia7206fac5c4ec6ace87304cfaeef375916b94fcf

Localisation updates from https://translatewiki.net.

Change-Id: Ic9c8bdac869b2a4f218933be4803887a623c51fa

Merge "TidyDriverBase::validate throws an exception"

Merge "Update phan issues & estimated counts"

Merge "API: Split non-English description messages into summary + additional text"

Merge "RCFilters: Fix popup head with long message and icon"

RCFilters: Trim spaces in saved query names

Make sure to trim the spaces before saving, and also to verify that
names that contain spaces-only are considered empty.

Also make sure that the same behavior is working when saved queries
are edited - but since the save query button may be disabled, make
sure that any blur event on the input gets us out of the edit mode
either after save or without saving (if the string is invalid)

Bug: T169273
Change-Id: I16da9fcde0bf6be2b854243d7facc80d2860e458

API: Split non-English description messages into summary + additional text

Per request, automatically split non-English messages to avoid a lot of
work for translatewiki.

Change-Id: Ifb9928dfbc59028d0df65ff07e067aa17bcf0c2f

Merge "Add missing class propertys in QuickTemplate"

TidyDriverBase::validate throws an exception

Change-Id: I05e31c757ed92323ff905d993ac4d030b8aba1da

Update phan issues & estimated counts

Since the last update there is a total of 550 less issues.
744 removed and 194 added in various places.
(roughly)

Change-Id: I0431773973c146e1492de72d869f6d33de4084e8

Improve error messages for (known unknown) network errors

Bug: T132611
Change-Id: I22f26e56e089edad7c423ac8f0007b5439de3e58

Added chunked upload support to upload.js

Meanwhile also made a uploadWithFormData abortable.

Bug: T103400
Change-Id: Idb4afbbf24c84100630e12869a0a30326a30736f

Merge "Make mw.widgets.SearchInputWidget extend OO.ui.SearchInputWidget"

Expose LinkRenderer for LogFormatter instances

Pass a LinkRenderer instance from SpecialLog, through LogEventsList,
onto LogFormatter instances, which tend to make many links.

Bug: T168924
Change-Id: I264da6f63678917c7335adee5a0b64cbcded66f3

Add missing class propertys in QuickTemplate

Change-Id: I327ac1207af8fcf9db792b3835b8a35378330eb6

Merge "DidYouMeanWidget: Don't use deprecated Linker::linkKnown()"

Make file purging also purge old versions

Also fixes purging for repos with sha1 thumb URLs.

Bug: T169198
Change-Id: Ibb98ecce83d690cc46769644038b54e37aea0b0d

Merge "Keep the same DBLoadBalancerFactory in tests"

Merge "filecache: Use current action instead of "view" only in outage mode"

Merge "SearchInputWidget: Fix documented default for 'pushPending' option"

Merge "Clean up release notes"

Merge "UsersMultiSelectWidget: Port from CapsuleMultiselectWidget to MenuTagMultiselectWidget"

Merge "Fix \n handling for HTMLUsersMultiselectField"

UsersMultiSelectWidget: Port from CapsuleMultiselectWidget to MenuTagMultiselectWidget

Bonus:
* Remove puzzling code that claimed to fix a bug that I couldn't reproduce
  but instead made single-character searches never display suggestions
* Clear the input after choosing a menu item

Change-Id: I44e72205880d152639ee823238dc5ab84d34402b

Fix \n handling for HTMLUsersMultiselectField

Bug: T166836
Change-Id: I51b772946f1e50a21fb86cab969defb4647b820b

Merge "resourceloader: Restrict addModuleStyles() to type=styles modules"

Merge "resourceloader: Improve ResourceLoaderClientHtmlTest tests"

Merge "jsminplus: Fix JSNode memory leak from 'continue' statements"

Merge "SpecialPageFactory::getPage can return null"

Merge "API: Don't handle non-preflight OPTIONS as CORS"

Make mw.widgets.SearchInputWidget extend OO.ui.SearchInputWidget

Mixin mw.widgets.TitleWidget instead of extending mw.widgets.TitleInputWidget.

* Remove code that reimplemented pieces of OO.ui.SearchInputWidget.
* Remove code that overrode pieces of mw.widgets.TitleInputWidget.
* Copy the code from mw.widgets.TitleInputWidget that we actually want.

This should result in no functional changes, other than losing the
TitleInputWidget API (some methods and config options) that no one
relied on, as far as I can tell.

Bug: T169194
Change-Id: Ic1482b4c7cfde7d4cf0b8900654bd3a454776010

SearchInputWidget: Fix documented default for 'pushPending' option

Change-Id: Iadee48af255498c6608cfd00fca29eed9b14f1be

Localisation updates from https://translatewiki.net.

Change-Id: I420c049cd13179f6780c6a7f85857e2bf1389f99

RCFilters: Followup I5cede87633147: Set actual value rather than object

Bug: T169042
Change-Id: Ib99981c212dbe1522defc1c1cc34fa01fff91dba

Keep the same DBLoadBalancerFactory in tests

Bug: T168242
Change-Id: I9b57b8bcfdcb823adb89213518abfdfb834ef223

Clean up release notes

Change-Id: Ief5b9a69a5ace43efe8a221e3118c09059361d36

Update OOjs UI to v0.22.2

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.22.2

Depends-On: If6a7aa1d924b416fc95831e1a1b26313e6482cbf
Change-Id: I36a5329fe7d6e134ec1f7b5583017ddff64ca03a

resourceloader: Restrict addModuleStyles() to type=styles modules

If a type=general module is enqueued, don't try to load it as a
stylesheet.

* Per a464d1d41d69f12fe, state tracking is already disabled for
  these loads (as otherwise we wrongly claim state=ready, when in
  fact only the styles and not the scripts were loaded).

* The warning was added in a464d1d41d69f12fe.

* Default install (tested in Vagrant), Wikimedia Beta cluster, and
  Wikimedia production have seen zero violations of this warning
  in the past 7 days.

Raise severity to ERROR and add the 'continue' statement so that
these are now not loaded at all.

Bug: T92459
Change-Id: I211d56ac2df479ebf5b98667c613ecf81489539b

resourceloader: Improve ResourceLoaderClientHtmlTest tests

The example for a mixed user module is outdated. Current user modules
all separate styles and scripts.

To make the effective change by the next commit easier to review,
update these tests first seperately.

Change-Id: I76a8a96eed8a8c39863c535362b1d0144ef2f866

jsminplus: Fix JSNode memory leak from 'continue' statements

During parsing of scripts containing a 'continue' statement,
the JSNode for that statements, and all containing blocks (e.g. conditional
blocks, for-loop/white-loop statements , etc.) were being held
in memory indefinitely.

Bug: T31784
Change-Id: Id7006c264a470ba665434f6f275e49a1516b73ae

Merge "Remove space after cast"

Merge "clone keyword must not be used as a function"

Merge "RCFilters: Fix typo in removing a saved query"

Merge "RCFilters: Correct display of save filter popup"

RCFilters: Fix typo in removing a saved query

Bug: T169107
Change-Id: I7c1535f6ae75159a069ebde7db781f6a875fcb3c

RCFilters: Fix popup head with long message and icon

If the message is long, it should remain on the same line as the
icon, and the padding should come from the 'head' element rather
than the individual icon/label elements.

Bug: T169110
Change-Id: I726649f9fbd7e095aea62693aff269108c872661

RCFilters: Followup I200720ffde8ad: Hide extended footer if gater is disabled

Bug: T169163
Change-Id: I62f63cbf06a7e42864faca1a196903eb34ea077b

Merge "resourceloader: Use "\n" instead of ";" as separator for scripts"

Merge "Add SearchIndexField::getEngineHints()"

Localisation updates from https://translatewiki.net.

Change-Id: I889cf1f5e93554486b452922edd474ef865bec7f

Add canonical id to header on action=info

Have to use the h2, because the span can have only one id attribute

Bug: T44629
Change-Id: Iaa1c46611c56210b1177a665a5edfa082683977f

Paramaterize TransactionProfiler log entries

Change-Id: I4e468d818af48dba21cd2c567f4e6a58beb0924b

SpecialPageFactory::getPage can return null

Change-Id: I12b4969ec0c6b8d5a1d8feeda27152ba6075c4fd

Record PostSend master queries via TransactionProfiler

Bug: T166199
Change-Id: Ib2f858f09aae65af6aa8324edb72d8a16094f2b3

RCFilters: Correct display of save filter popup

- Correct language in the 'apply' button
- Add a placeholder to the input
- Make the 'apply' button disabled if the input is empty
- Remove the use of the OOUI-built-in validation, since all
  we do is "validate" that the input isn't empty, and there's
  no need to show error mode (red border) for that, especially
  since the 'apply' button is disabled in that case.

Bug: T169042
Change-Id: I5e3600b1ac8e63d8a25c0540468fe42febfc3a70

clone keyword must not be used as a function

Also changed a require_once

Change-Id: Ib8030ea5320a62ee4f114d8d506382c5665e3e02

Merge "Use Devanagari digits for list items on mai.wikipedia"

Add tracking for resolved conflicts

Bug: T167863
Change-Id: I48bbfc7216d1f726dbcec3ae337466bbf8b2a486

Merge "User: Avoid deprecated Linker::link()"

Use visibleByteLimit on Special:MovePage

Change-Id: Ie9e2bef8397851ca5899bfb7a3849853d07590ff

Merge "Factor out byte limit label logic from edit page to utility function"

Use Devanagari digits for list items on mai.wikipedia

Bug: T168781
Change-Id: I0f715daaacbe2cee236d28866b23893c342922a2

Merge "Remove exception and drop cloned tables in PostgresSQL too"

RCFilters: Followup  I436f00bc87db: Remove onPopupToggle listener

Bug: T168828
Change-Id: Ie6d73f38335ff1c5bc88602119ddf3e2138eba60

resourceloader: Use "\n" instead of ";" as separator for scripts

This fixes two bugs:

* 1) When two modules are requested, and the first one ends with ";"
     inside a comment, the second module might become part of
     that comment.
* 2) A request with script=only where the requested module content
     ends in a statement without ";", but has a comment after it
     that does ends with a semicolon, then in debug=false, mw.loader.state()
     would be appended directly after the semicolon-less statement because
     the check is performed before minification.
     Previously:
     script> foo()
     script> // bar();
     states> mw.loader.state( {} );
     Became (minified separately):
     script> foo()
     states> mw.loader.state({});
     Became (concatenated)
     > foo()mw.loader.state();
     Which is invalid code.

Both of these are now fixed.

Bug: T162719
Change-Id: Ic8114c46ce232f5869400eaa40d3027003550533

filecache: Use current action instead of "view" only in outage mode

Regression from 3ad9e41be9. Prior, $action was also passed to HTMLFileCache.

Change-Id: Ie0f1c8b73d306bc710c346ad97868c53fb59809d

resourceloader: Add more concat test cases for makeModuleResponse

Bug: T162719
Change-Id: Ic8d4a022dbe8c7998924fb20d727936e68cb11d2

Merge "Record TransactionProfiler violations beyond the first-by-type"

Merge "Allow SVGs using an older proposed recommendation DTD"

Remove ParserOptions::optionsHashPre30()

The pre-1.30 version of ParserOptions::optionsHash() was kept
temporarily as ParserOptions::optionsHashPre30() to prevent a cache
stampede on WMF sites when the hash format was changed in I7fb9ffca9.

Now that the cache has been rebuilt, it's no longer needed and we should
clean it up instead of leaving it forever to bitrot.

Change-Id: I037d8dfdefe72a295547bd331bc1454e69cb418d

Merge "Add constants for schema migration feature flags"

Merge "benchmarks: Add benchmark for JSMinPlus"

Record TransactionProfiler violations beyond the first-by-type

Beyond giving a more complete picture, this can be used to adjust
the PostSend criteria to log master queries under the restInPeace()
entry point.

Bug: T166199
Change-Id: If5cf704ba10291d0eba63e66b388666fde3ca649

Merge "Fix SearchEngineTest test descriptions"

Merge "Fix highlighting for phrase queries"

Merge "Remove "@author Aaron Schulz" annotations"

Merge "resourceloader: Add basic tests for getScript() and buildContent()"

Remove "@author Aaron Schulz" annotations

Bug: T139301
Change-Id: Ib5248e8e27d60611c7373bce4b29dd5e85aa3489

Merge "benchmarks: Add reporting of resource usage"

benchmarks: Add benchmark for JSMinPlus

Example usage

> mwscript maintenance/benchmarks/benchmarkJSMinPlus.php --file resources/lib/jquery/jquery.js -v

Bug: T31784
Change-Id: I3c11574adb864a4bfbfceba137e285fdb8dc66d2

Merge "Fix typo in comment."

Merge "tests: Use TestingAccessWrapper to reload LanguageConverter tables"

Merge "tests: Reset LanguageConverter conversion tables between test cases"

Merge "RCFilters: Move legend into results (both js and nojs output)"

Fix typo in comment.

Change-Id: Ia656c55ca22fa30b5a911b8cb3a6218c916d3316

tests: Use TestingAccessWrapper to reload LanguageConverter tables

Make the LanguageConverter::reloadTables method actually private,
and use the TestingAccessWrapper to call it when running parser tests.

Follow-up to I65736520cd04bfe8949b29ade07338a6e1b88a4d.

Change-Id: I43b81b8fef6441ad50b858ff7757732ecb5eef91

Merge "Fix border-radii on FilterTagMultiselectWidget"

Merge "RangeChronologicalPager: Don't clear $offset when $this->rangeConds is set"

Localisation updates from https://translatewiki.net.

Change-Id: I27707dbaae9edf04ef400300801c339a22df3eb3

Merge "RCFilters: Always use the filled bookmark icon (unClip)"

Merge "RCFilters: Simplify "Saved filter settings" to "Saved filters""

Merge "Downgrade 'found writes/callbacks pending.' from error to info"

RCFilters: Move legend into results (both js and nojs output)

Move the legend from the fieldset area to the results area.

Bug: T168043
Change-Id: I40454b194fe3975d8d4ac13d96774bfa5b5d5a46

Merge "RCFilters: Make extra filters footer non-sticky"