Merge "Change delimiter for multiple namespaces and tags"

Change delimiter for multiple namespaces and tags

It's ; for namespaces (since that is generally what we're using,
e.g. for STRING_OPTIONS), and | for tags.  I would have
preferred them to all be consistent, but there is one revision
that has a ;, and this seems better than modifying the DB.

Comma is already used as a delimiter between separate parameters
(e.g. hideanons and namepsace).

Also, fix multiple namespaces for parseParameters and add test
cases.

Bug: T164132
Bug: T164133
Change-Id: Iad061e0bc17e3522a3f5d330ac2c8bf9cf0e614f

Merge "MessageCache: Avoid 'contributions' replicas"

Merge "Prettify MessageCache::loadFromDB()-small query"

Merge "ApiQueryRevisions: Restore use of rvstartid/rvendid as a tiebreaker"

Follow-up I5cede8: fix JS error when user has no filters saved

Change-Id: Ie7de070925874d2c29e185a90d65a0b0a72f07cd

Followup I5cede8: Make the SavedQueries popup larger

The labels and popup are now slightly wider.

Bug: T151994
Bug: T164128
Change-Id: I64bd496ade241c800fcb0bb12023486e7d601a23

Followup I5cede8: Fixup SavedQueries styling and event

Includes small styling fixes to saved queries and the ability
to update the label of an existing matched query if we rename
it.

Bug: T151994
Bug: T164128
Change-Id: Ic46b082cbfc3462f1d61895bcfc7c545902b2d1a

Merge "RCFilters UI: Add a 'saved queries' quick filters feature"

RCFilters UI: Add a 'saved queries' quick filters feature

Bug: T151994
Bug: T164128
Change-Id: I5cede87633147736d3b4ee5b8ea178ae21bd441f

Localisation updates from https://translatewiki.net.

Change-Id: I001225f92a48f46a6a9418958bd8603c2d3f403d

ApiQueryRevisions: Restore use of rvstartid/rvendid as a tiebreaker

If rvstartid/rvendid are used with revisions that have the same
timestamp, we should omit any revisions with the same timestamp but
out-of-range IDs.

Bug: T163532
Change-Id: I6ba5e0724945668dad55eec0d09cb7d3fe08e347

MessageCache: Avoid 'contributions' replicas

On WMF wikis, the partitioning of the revision table on the
'contributions' replicas makes the query here perform really poorly.
Specify 'api' as a hack for now to avoid those replicas.

This query happens once per day per wiki, plus when someone edits the
MediaWiki namespace, so it shouldn't be much additional load.

Bug: T164666
Change-Id: I5ae74d1702144f6475e9cfb13effc43389d66233

Prettify MessageCache::loadFromDB()-small query

Adjust the call so it uses JOIN...ON syntax rather than comma joins.

Bug: T164666
Change-Id: I93c8569620d543e9507d0adf1283443915f50d4b

Merge "RC Filters: Single boolean filters are not contradictory"

Enable extensions to allow software-specific tags to be added by users

Bug: T121880
Bug: T121872
Change-Id: I6e054cfc065c8d8ae7d20a518b1963cd0551c5c2

Merge "Fix a couple of issues with cleanupInvalidDbKeys.php maint script"

Merge "RC Filters: Rephrase learner level message"

Merge "Remove "(WMDE)" from my name in .mailmap"

Merge "SpecialRecentchanges: Add class="mw-recentchanges-toplinks" to the "top links""

RC Filters: Rephrase learner level message

Bug: T164611
Change-Id: I5cd7b392fc89402e358084c8153bf8c2b9f1abd2

RC Filters: Single boolean filters are not contradictory

When all boolean filters of a group are selected
(for example: hideanons && hideliu), we consider the selection
unproductive and set them all to false.

Some legacy filters are alone in their group
(for example: ORES hideDamaging). They should not
be considered.

Bug: T164625
Change-Id: Ibb6721ccdfb226b3baac7775c30af230c68309e7

Remove "(WMDE)" from my name in .mailmap

No other WMDE employee does have this. I think this is supposed to be
the "canonical name", which means it should not contain such extra
information, right?

Change-Id: Ic5bbb53d650843dfb627baa4c7508d88eba03729

API: Convert rvstartid/rvendid to timestamps for query

We tried something like this once before, but reverted it because it was
an unintended behavior change (see T98467). This time it's intended, we
need it for query optimization.

The behavior changes here are:
* rvstartid/rvendid is exactly equivalent to specifying rvstart/rvend
  with the corresponding revisions' timestamps.
* If the revision for rvstartid/rvendid is not found in the database, an
  error will be thrown.

This will pull timestamps from deleted revisions, i.e. the `archive`
table. While this is technically an information leak (that some revision
ID exists as a deleted revision and the time the revision was made),
it's minor and in line with the information revealed in Tool Labs thanks
to T51088.

Bug: T163532
Change-Id: Ida64a377c38b3553aa82ac754d80e8f898caf6c5

Merge "Have Title::get(Next|Previous)RevisionID sort by timestamp"

Have Title::get(Next|Previous)RevisionID sort by timestamp

Revision IDs are usually increasing as timestamp increases, but not
always. Callers almost certainly want next/previous timestamp when the
two differ.

This also takes care of a minor bug in the nearby getFirstRevision()
where it'll choose an arbitrary tied revision ID if there were multiple
revisions made in the same second.

Bug: T4930
Bug: T163532
Bug: T159319
Change-Id: Iab2060a0ad5e45edbaa0ff36e863cb014b8e876f

Wrap parser output in <div class="mw-parser-output">

This will allow CSS to target just the parser output, without also
accidentally targeting the edit form, diff tables, and so on.

Bug: T37247
Change-Id: If4eb5bf71f94fa366ec4eddb6964e8f4df6b824a
Depends-On: I330c6aa4aaee045614b1801ed34bc9e03be69650
Depends-On: I52a518fa44e017841fe78474012cd69823e0a41d

Merge "Add DatabaseUpdater::modifyTable"

Merge "CREDITS: remove duplicate entry for Nik Everett"

Localisation updates from https://translatewiki.net.

Change-Id: I2b3b0cb604caa001245786be38730195e242e621

Merge "Add single angle quotation marks and  to special characters"

CREDITS: remove duplicate entry for Nik Everett

Change-Id: I8bc4bcecd12aa4dc41e1826896dbe89104958125

Merge "Update CREDITS"

Update CREDITS

Manual checks have shown that these people did really just start
contributing to core and didn't only switch mail adresses.
Change-Id: I8b09ae04a9d50a677d8491eb3fe6ff5af8dcaf53

Merge "Upstream the transform mixin from MobileFrontend"

Upstream the transform mixin from MobileFrontend

This is useful and used by various extension including
CentralAuth, CodeEditor, Collection, Cards.

Change-Id: Iad3f568c4e7dbd7bd5a8c2058fe4b467750a9f07

Merge "phpcs: Run MediaWiki.Commenting.FunctionComment.WrongStyle sniff"

Add DatabaseUpdater::modifyTable

This introduces the function modifyTable to the DatabaseUpdater class
which will allow to alter multiple columns on the same table in a single
sql patch file. In favor of this new function, usage of
MssqlUpdater::updateSchema for mssql patches is discouraged for the
future.

Also fix T70786, which is about update rows being inserted regardless
whether applying the patch was successful or not (no retry on failure).
Now update rows only get inserted if applying the patch was successful.

Bug: T70786
Change-Id: Id2a1583cd9efa5ef271ae67238119bc24b76a3c6

Merge "Remove $wgDisableUserGroupExpiry feature flag"

Localisation updates from https://translatewiki.net.

Change-Id: I58d5e6eb92b935c8337e7ab730e59b23292b9201

phpcs: Run MediaWiki.Commenting.FunctionComment.WrongStyle sniff

All files in the repository currently pass.

Change-Id: I5549abfe8374ae4b6f8904c2d7870ea77b8ee17c

Merge "Update mediawiki/mediawiki-codesniffer to 0.8.0"

Update mediawiki/mediawiki-codesniffer to 0.8.0

Change-Id: I11c4e0e519589c2d005896bf9d3758ef017346fb

Add single angle quotation marks and  to special characters

These marks are used quite frequently in articles about
linguistics, characters, alphabets, etc., to mark graphemes.

Change-Id: I63e890d68d4bf43f3d1886ce45891d0f3edafd14

Fix a couple of issues with cleanupInvalidDbKeys.php maint script

- The output was pretty screwed. Now it works properly if you redirect
  STDOUT to a file like you're supposed to.
- \r, \n, and \t characters were escaped wrongly (for example, \r was
  escaped to \\r, which matches a literal sequence of '\' and 'r'). Now
  they are correctly escaped to \r, \n and \t in the SQL query.

Change-Id: I750b4fb3e7072ae3cdd8c40733b066a1e76de366

Remove $wgDisableUserGroupExpiry feature flag

This feature flag was added to comply with WMF's schema change policy.
It is no longer needed now that the feature is enabled on WMF sites.
External MW installations will run update.php or manually do DB updates
as required before upgrading MW.

Bug: T162517
Change-Id: I5b532d79fd1e8b178490cf2617499ae62967de2c

config: Add @covers to complete GlobalVarConfig and MultiConfig

Change-Id: Ifff51c8b078ba3a3026720f13263c7af0d7d1145

Merge "jquery.ui: Avoid deprecated jQuery.expr[":"]"

Clear postEdit cookie on server-side

* Load module 'mediawiki.action.view.postEdit' only when needed.
* Transfer message key via JavaScript config variable wgPostEdit.
* The response is maked as not-cachable to prevent that other users get the
  post edit message.

This change redefines the global JavaScript variable wgPostEdit from true
to a string and set it on server-side.

Bug: T164148
Change-Id: Id780bc280ce4a2fa4606141419932b7dcd45157b

SpecialRecentchanges: Add class="mw-recentchanges-toplinks" to the "top links"

Meaning the contents of [[MediaWiki:Recentchangestext]], which is a series
of quick links on most wikis.

Bug: T164617
Change-Id: I2934156af5aafa3162924ecf78cfd82d3a41370d

jquery.ui: Avoid deprecated jQuery.expr[":"]

Compatible with jQuery 1.11 and 3.x, as jQuery.expr.pseudos was
introduced in jQuery 1.8.

Note that this is not blocking jQuery 3 (T124742) since it is
covered by jQuery Migrate. But fixing this makes the edit page less
noisy by default.

Change-Id: I7ffbfd2e55aa7dcc31860eb0117b203e63ca7283

Merge "Properly detect if CACHE_ACCEL is available in the installer"

phpunit: Fix broken @covers for swapICCProfile in JpegTest

Follows-up a428e3f067.

Change-Id: I5755dc560fbb3314578775839b02653a4e2113d5

Localisation updates from https://translatewiki.net.

Change-Id: I498e214dec5f3939847016861b27ebd2a45e7fe1

Merge "CSSMin: Support parenthesis and quotes in url references"

Merge "resourceloader: Add CSSMin benchmarks"

CSSMin: Support parenthesis and quotes in url references

Previously they were often being cut short due to the url pattern
ending at the first single quote, double quote or closing parenthesis
regardless of which of those started the url match.

Running benchmarkCSSMin.php before and after the change doesn't seem
produce consistent improvement or regression. Repeated runs with count=100
with and without this change both have a median between 2.6ms and 2.9ms
using PHP 5.6, and between 2.6ms and 2.8ms using HHVM 3.12.

Bug: T60473
Change-Id: I6d6a077ad76588f3ed81b1901a26b7e56d2157ee

Merge "RCFilter UI: allow getParametersFromFilters to accept filter list"

Merge "ApiQueryTags: Use cached statistics instead of querying hit counts ourselves"

Merge "Fix php code style"

Fix php code style

Preparation change for updating mediawiki code sniffer to 0.8.0

Change-Id: Ib0b3fe4afea9096ffa3a1347b4f7e07d3398b0b2

Writing a useful description for the FakeConverter class

Change-Id: I6575ea13790c224c1b25c7e9373b4bf40d035a30

Merge "Fix registerTempTableOperation() return value for non-temporary table DROPs"

ApiQueryTags: Use cached statistics instead of querying hit counts ourselves

The hit count query was quite slow. Unfortunately, it seems that we do need
tagUsageStatistics() even when hitcounts are not requested, because it
might list additional tags that aren't listed by the list*Tags() functions.
I don't know if this can happen in practice, but all the code around tags
seems to operate as if it might.

Bug: T164552
Change-Id: Ifccf7f5ac7a1220ff67a68589398cbf30aefd3ad

Merge "EtcdConfig: Fix infinite timeout bug, and reduce timeout"

Merge "build: Update eslint to 0.4.0 and make pass"

build: Update eslint to 0.4.0 and make pass

Change-Id: Ib230392f332268ac801cf668f399fcefb1cb1cc5

Fix registerTempTableOperation() return value for non-temporary table DROPs

Change-Id: I338dfbbf5479d5157de8d9abc2012cc0e7d10c93

Localisation updates from https://translatewiki.net.

Change-Id: I6e3920bd06e5ca5462d392a569c817e46e041fcc

Merge "Fix contradictory RC filters and add back-compat"

Merge "mw.special.apisandbox: Don't pass $label to OO.ui.FieldLayout"

Merge "Fix typo in DBConnReff class phpdoc"

Merge "Fix inconsistent spec of InterwikiLookup::getAllPrefixes."

Merge "Remove unused var assign in Parser::getTemplateDom"

Merge "Follow-up bac92e67ca0: make order of filter groups less misleading"

Fix contradictory RC filters and add back-compat

Some combinations of RC filters should never appear
together because they guarantee to return no data
and cannot be visually represented in the new RC
filters UI (ERI project).

Examples include:
* 'hidemyself' and 'hidebyothers'
* 'hideminor' and 'hidemajor'
* All of the filters in the changeType group (which
  is extended by extensions)

This also handles an old special case, but it now redirects
instead of doing it silently:

hideanons=1 & hideliu=1 & hidebots=1 -> hideliu=1 & hidebots=1
hideanons=1 & hideliu=1 & hidebots=0 -> hidehumans=1

Bug: T151873
Change-Id: Id08dccd07b262ce61c9d38563f19a0ab181e2341

Merge "Fix all the Doxygen for the RCFilters backend"

Fix inconsistent spec of InterwikiLookup::getAllPrefixes.

The documented return type was not what existing callers expected,
and not what the default implementation actually returned.

This patch fixes the interface documentation and the behavior of
an alternative implementation.

Change-Id: Ib09bffeba3ddc5b43da1c7c299f1fa946be4e2e2

Fix typo in DBConnReff class phpdoc

Change-Id: Iba61678fb0b6bd63a1c15a074adc9d9523384cb7

Merge "Embed TinyRGB color profile when JPG EXIF Color Space = sRGB but no profile embedded"

Change default $wgShellLocale to C.UTF-8, and use it to set LC_ALL

It's less likely to cause surprises than language-specific defaults.

Bug: T107128
Change-Id: Ife7673255798f3a3d72028a26607c56b9b7fb224

EtcdConfig: Fix infinite timeout bug, and reduce timeout

removeServer() returns the modified array, rather than passing by
reference, so you have to use the return value to avoid an infinite loop
when a server is down.

Tune the timeout downwards, to 2s. With three servers in the SRV pool,
if they are all unreachable, this will mean an overall request time of
6s, which is conveniently less than the APC lock time and the cache
TTL (9-10s). If the APC lock time is significantly shorter than the time
it takes to do the HTTP requests, then additional threads join in
waiting for the server. This could have stability consequences if the
maximum HHVM worker count is exceeded.

Change-Id: I3176aa41b8833c0ba0b668859e59911cd4392250

resourceloader: Add CSSMin benchmarks

Usage:

 # Run default benchmark
 $ php maintenance/benchmarks/benchmarkCSSMin.php

 # Use custom file
 $ php maintenance/benchmarks/benchmarkCSSMin.php --file resources/lib/qunitjs/qunit.css

 # Debug the output for inspection (no benchmark)
 $ php maintenance/benchmarks/benchmarkCSSMin.php --out

Change-Id: I70d118131d0ff16d1a811b2de1328ea622b7ca69

Revert "Convert mwdoc-filter.php to Maintenance", rm wfShellWikiCmd

Revert a58948d64 and instead remove wfShellWikiCmd and escape
shell arguments directly.

This should be fine since mwdoc-filter.php does not depend on per-wiki
state.

Change-Id: Id9c6ca84bab827675b71ca16bf688fd3f5c993a1

Fix all the Doxygen for the RCFilters backend

All the lists and sub-lists now render correctly,
and missing parameters have been added.

Bug: T163069
Change-Id: I7a8c95efaff7c844e32e4375dfe6af8c2e91923f

Merge "RCFilters: Only register watchlist filter group when user is logged in"

Merge "RC Filters: always join with 'page'"

Follow-up bac92e67ca0: make order of filter groups less misleading

The significance group comes before the lastRevision group in the UI
(because it has priority -6 vs -7), but we listed lastRevision first.
Swap them so the order in the definition matches the order in the UI.

Change-Id: I5d498bd8c7505f5cab3c89564650e0e8aaa4e117

RCFilters: Only register watchlist filter group when user is logged in

Follow-up to 9a97cb6.

Bug: T164314
Change-Id: If1af0816b5b37857060b87db8e4e798b4824b5a5

RC Filters: always join with 'page'

The 'last revision' filter group relies on
the 'page' table to filter and highlight.

Bug: T163561
Change-Id: I29be21596ce02270859936203c921777c952e3a4

Fix typo in comment

Change-Id: I3c5f3e66e2bca5b1629ee2fc54992a81e36d71cb

Localisation updates from https://translatewiki.net.

Change-Id: Ie59fc60d2680bc9ed62230834e1930f16c32fb14

Merge "Use IGNORE INDEX(ls_log_id) instead of FORCE INDEX(ls_field_val)"

Merge "Upstream transition-transform mixin from MobileFrontend"

Upstream transition-transform mixin from MobileFrontend

This specialised usage of transition allows hardware accelerated
transitions on mobile devices and is commonly used in MobileFrontend.

Change-Id: Iaa622043fdbb5be965716c88b48f30f61d259c8b

Merge "mw.special.apisandbox: Specify auto width on PopupButtonWidgets' popups"

Merge "ApiSandbox: Specify a $overlay for menu-using widgets"

Add quick shorthand for SVG generation

Change-Id: Ic2a68d2fa62dc079743de641ada4c7f4dd294f2f

Use IGNORE INDEX(ls_log_id) instead of FORCE INDEX(ls_field_val)

ls_field_val was renamed to PRIMARY, but this is only partially
complete in WMF production. There is only one other index on the
log_search table, so ignoring that one is equivalent to forcing
the other one.

Bug: T17441
Change-Id: I63182a9f94eabb4cc47414d86b02c82bde1e58b0

Merge "Add mw-content-ltr|rtl to file redirect pages"