Make the indexName functions more obviously laid out

Move Sqlites's indexName to Database to have no remapping by default.
This is because MSSQL and Postgresql both don't need remapping, it's
just MySQL that's the problem.

So move the one from Database into MysqlBase to only do the remappings
there.

Also fix up the comments to make them less vague, not mentioning tables
when the function has no table parameter passed, so hence, no context
as to which table said index is wanted to be used

Bug: T154872
Change-Id: I46c575771e6cb36176bcf6cd1a865b703db0275d

Merge "Revert "Remove old remapping hacks from Database::indexName()""

Merge "Update psy/psysh 0.8.1 -> 0.8.3"

Merge "Update phpunit 4.8.31 -> 4.8.35"

Update phpunit 4.8.31 -> 4.8.35

Change-Id: I909466567b9f66dc391258ca7f7bac12ce98b23c

Update psy/psysh 0.8.1 -> 0.8.3

https://github.com/bobthecow/psysh/releases/tag/v0.8.2
https://github.com/bobthecow/psysh/releases/tag/v0.8.3

https://github.com/bobthecow/psysh/compare/v0.8.1...v0.8.3

Change-Id: Ief2e0444d6e79df4433172d00d2db2b137bc2bb2

Linker: Render selflinks as href-less classed <a>s, not <strong>s

Self-links are still semantically links, and representing them as <strong>s
is inelegant and more important a real pain to work with, especially in
contexts where they may change state (like inside an editor).

Instead, render them as <a>, with no href to avoid user agent style over-
rides and with a class to style them as before, named 'mw-selflink' to go
with 'mw-redirect'. This allows much easier adjustment later. The old CSS
class 'selflink' is retained for backwards compatibility, but deprecated.

Bug: T160480
Change-Id: If058843924c3b30c116df2520aef93a004d98a5d

Merge "Update OOjs UI to v0.20.1"

Merge "EditPage: Refactor getCheckboxes() to allow changing the format"

Merge "ResourceLoaderImage: Correctly ignore 'deprecated' key"

Merge "Special:RecentChangesLinked show "no results" message"

EditPage: Refactor getCheckboxes() to allow changing the format

getCheckboxes() directly generated the HTML for the
"This is a minor edit" and "Watch this page" checkboxes,
and allowed extensions to add more HTML checkboxes (and
modify existing ones) using the 'EditPageBeforeEditChecks'
hook. This prevents us from ever changing the format of
the HTML (e.g. to use OOUI checkboxes).

Introduce new method getCheckboxesDefinition(), which
generates the checkboxes in a machine-readable format,
with a new hook 'EditPageGetCheckboxesDefinition'.
Rewrite getCheckboxes() in terms of that. The old hook
'EditPageBeforeEditChecks' is now deprecated.

Change-Id: I3dbe973dcac6cba0c3a1ac5d983cafcfb49d833c

Localisation updates from https://translatewiki.net.

Change-Id: I18df74d64336e8539b7ac85fc6a456759d283e9c

Update OOjs UI to v0.20.1

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

Depends-On: I9b50eee459085eaa00819cbabe340ac442a332bd
Change-Id: I9b50eee459085eaa00819cbabe340ac442a332db

Merge "mw.widgets.Complex*: Add setDisabled"

ResourceLoaderImage: Correctly ignore 'deprecated' key

It turns out that 'resources/lib/oojs-ui/themes/mediawiki/Moved since v0.20.1, use from the 'interactions' pack instead.'
is not usually an existing file, and doesn't have the extension '.svg'.
Not sure why this didn't break earlier.

Bonus: Add module name to exceptions to make these errors easier
to track down.

Bonus #2: Use the post-expansion, not pre-expansion, definition everywhere
to avoid confusion when debugging.

Change-Id: I0325d4dab5658fd29c3c33fd3e762834b53d1b5d

Merge "Move DBConnRef/MaintainableDBConnRef to Rdbms namespace"

Move DBConnRef/MaintainableDBConnRef to Rdbms namespace

Change-Id: I7ca6b40f973b2aae83747de92e280ded222c87e3

Special:RecentChangesLinked show "no results" message

Bug: T160482
Change-Id: I26743928b359ad052ff53bde281f0479cbe68a99

Merge "Make Title::getFirstRevision() ignore the rev_timestamp index"

Make Title::getFirstRevision() ignore the rev_timestamp index

We want the page_timestamp index to be used in this case, but sometimes
the rev_timestamp is chosen which leads to bad performance.

Also update WikiPage::getOldestRevision() which uses the exact same query.
I'll implement one in terms of the other in a follow-up commit.

Bug: T159319
Change-Id: I7c5c0a9b1af99ce2b5f4bdcc99710d8400ca8bcf

Merge "Allow filtering by username on Special:NewFiles"

Merge "HttpFunctions: Increase code coverage"

Merge "Move IDatabase/IMaintainableDatabase to Rdbms namespace"

Merge "Defer calling ChronologyProtector::initLB() until connecting"

HttpFunctions: Increase code coverage

* Complete coverage for Http::getProxy().
* Remove bogus @covers tag on data provider, and add the
  relevant MWHttpRequest::getFinalUrl to the test instead.
* Convert test to use dataProvider and add missing test cases
  to increase getFinalUrl() test coverage to 100%.
* Minor clean up in getFinalUrl to consistently use early-return
  for all cases, not just for relative 'domain' and 'isset-host'
  cases. Without this coverage actually couldn't reach 100% due
  to the remainder of the empty else branch never being reached
  (CRAP: "Redundant 'else' after 'return'")

Change-Id: I775d95965dc23a1e6c4c62ed84f9da64b6c72135

Defer calling ChronologyProtector::initLB() until connecting

Bug: T160678
Change-Id: Id51c2854f38d9e4697e52776168b16996e9152e4

Merge "Deprecate "Avoid page_touched update for HTTP GET action=purge requests""

Move IDatabase/IMaintainableDatabase to Rdbms namespace

Change-Id: If7e8a8ff574661fd827de8bcec11d2c39a687300

resourceloader: Avoid deprecated getModifiedHash() in SkinModule

Use getDefinitionSummary instead, which uses a single serialisation
pass instead of requiring every stage to be a string. This way
we don't need to call json_encode and md5() multiple times.

getModifiedHash() was deprecated in MediaWiki 1.26.

Change-Id: If9e9caa3d12976c99543ad53ab280355b70acb17

Merge "Catch errors in more cases inside MediaWiki::triggerJobs()"

Merge "ForeignStructuredUpload.BookletLayout: Use CategoryMultiselectWidget"

Merge "Rename CategorySelector to CategoryMultiselectWidget"

Localisation updates from https://translatewiki.net.

Change-Id: Ie34e238728bbf19d934f98e3a6d8e847f9be60a5

Merge "hidemyself/hidebyothers: Use rc_user_text since there is an index"

hidemyself/hidebyothers: Use rc_user_text since there is an index

hidebyothers was extremely slow (on large data sets) due to the
lack of an index on rc_user.  To fix this, changed to use rc_user_text.

hidemyself seems to be fine (assuming normal usage patterns), but
to avoid edge cases and ensure full coverage, it's been changed as
well.

I'll inquire about adding an index for this.

Bug: T161557
Change-Id: I61efe11de12e8ab6c01e8d913cdeda471132a6ee

Merge "Change Title::getPreviousRevisionID (and next) to ignore PRIMARY"

Merge "MWHttpRequest: optionally add original request data"

ForeignStructuredUpload.BookletLayout: Use CategoryMultiselectWidget

...instead of CategorySelector

Bug: T161285
Change-Id: Icd9bb162caa63c5a43f90e31818fc1432c6d66be

Rename CategorySelector to CategoryMultiselectWidget

For consistency.

Bug: T161285
Change-Id: I7c9e0089ba6b77f4bc73e14b989630487f6f3174

Merge "Add tests for CSSMin::getMimeType"

Merge "resourceloader: Optimise getMimeType() for common case"

build: Make Travis CI 'Postgres' build non-voting

This will make it easier to detect regressions by not making all
builds marked as fail, but only if one of the other three fails.

Mute the known failure from Postgres by adding it to an allow_failures
section. It'll still run every commit, but is non-voting.
To be re-enabled once T75174 is fixed.

Bug: T75176
Change-Id: I2ea415edd308f2a012ef240d562c0073f15b9118

Merge "RCFilters UI: Only show full coverage message if item isn't highlighted"

Catch errors in more cases inside MediaWiki::triggerJobs()

This catches things like "DB is read-only" when doing pop()/ack() from the
job table with sqlite.

Also spun off some code to new trigger*Jobs() methods for readability.

Bug: T88312
Change-Id: I2a09248e40867684d48e6739da5e4a90581fa6ce

Localisation updates from https://translatewiki.net.

Change-Id: I9a519432d6d8d818987818a6d7c07d1fdebe2f81

Merge "rcfilters: Avoid $.type()"

rcfilters: Avoid $.type()

Followup to I9a0c5e40b813e075ec33eea882b625dc43a15df6

Replace $.type() with typeof or Array.isArray

Change-Id: I4f0f717c345ab1279b626b158b0ed6ada056bbc1

MWHttpRequest: optionally add original request data

Bug: T161029
Change-Id: I16a2eec46ae9e0e242fe740be20e39a2422dc867

Merge "Add array typehint to 2 DatabaseUpdater methods"

Merge "Add a tracking category when a template loop is detected"

Merge "Add support for Atikamekw"

Merge "EditPage: Minor code style improvements"

Deprecate "Avoid page_touched update for HTTP GET action=purge requests"

This reverts most of commit c84ba4d86420d7af918e572e2cd4613d7be185b3.
The changes made there are no longer needed, and nothing in Gerrit seems
to have started using them since they were added.

The added constants in WikiPage, WikiPage::getLastPurgeTimestamp(), and
Article::getLastPurgeTimestamp() are deprecated, useless, and unused,
but not removed yet since they snuck into 1.28 so we can't revert them
without a deprecation period. Sigh.

Bug: T145649
Change-Id: I526fd4e004bee84c831a4cee71e44e92ee73480b

Add support for Atikamekw

Language code: atj

Bug: T137376
Change-Id: Ieb9b6bcd09aedbf520b384a6572f96d7db1eca18

Localisation updates from https://translatewiki.net.

Change-Id: I52bcb35ec9bae29b0adf6d745ad63706fd69e011

Add tests for CSSMin::getMimeType

Direct follow up for I1768646. The tests are copied from my patch
If985f33, with one important difference: Now the file extension always
wins. Mismatches are not fixed any more by this code. This is intended.
See the discussion at I1768646.

Change-Id: I06b873a808d58ae7e53272765b156ac0fa293c2c

EditPage: Minor code style improvements

Mostly whitespace fixes. Avoiding some temporary variables.

Change-Id: I65f31f2bdd01ab677caff5f628222993ad719c2f

Merge "build: karma now reports with mocha formatter"

Merge "Ensure we use mw.now() instead of 'new Date' for relative measurement"

Ensure we use mw.now() instead of 'new Date' for relative measurement

'new Date' is subject to clock drift etc.

mw.now() uses performance.now() when available, which will always
increase at a constant rate independent of the system clock.

Change-Id: Ib653103bf6116544f35c930fb33421f1bb362c7d

resourceloader: Optimise getMimeType() for common case

About 0.74% of load.php is spent in CSSMin::getMimeType/finfo_file.

We don't apply this to user-generated content of external urls,
only local files in version control. And in terms of correctness,
we don't support misleading file names, so it's not a problem that
a foo.png containing XML/SVG content would get the wrong mime-type
when base64 embedding it in CSS.

Change-Id: I17686467c897984117671098e94db8732446dc75

Tweak CSS for Parsoid <figure>s

* These tweaks give us zero pixel diffs for images (when the
  magnify icon is hidden) outside galleries.

Change-Id: Icd5a280a13162401f0bced93a91f9dfa45a3a481

Merge "Decode '0'-valued user options to integer 0"

Merge "Add @since tags to WikiRevision"

Merge "Add missing @group Database tags in tests"

Merge "Check for string initialization in lcfirst() for HHVM 3.18"

Add @since tags to WikiRevision

Change-Id: I02005184d2cb3fa50aa0a51221b40e7c621219d2

qunit: Remove experimental CompletenessTest feature

The CompletenessTest was my attempt at measuring a basic code coverage
using run-time inspection instead of static instrumentation.

Originally added in 540419a82e5a5 (2010; MediaWiki 1.17).

It was never finished, remained fairly buggy and disabled by default.
It is also no longer used anywhere.

Bug: T155194
Change-Id: I26e7466426dddb43596f402e31005a89060c1b96

Localisation updates from https://translatewiki.net.

Change-Id: I52ddf38b7a669842ed8157f5929cf4519ff9a57a

RCFilters UI: Only show full coverage message if item isn't highlighted

Bug: T161273
Change-Id: If62bbab3e12fc3d9e83f9452723a9b2d6b75854a

Check for string initialization in lcfirst() for HHVM 3.18

HHVM 3.18 emits a notice when attempting to access the first offset of
an empty string.  We had that fixed for ucfirst() in 3605066c96. This is
the same for lcfirst().

Bug: T161095
Change-Id: I1456611222c24290f259298e883ca89dd830c74b

Revert "Remove old remapping hacks from Database::indexName()"

This reverts commit bec6151e5fe30e780d82527c4c53e54379149a4e.

Bug: T154872
Change-Id: I1c67aa464a1f151562ef92fce03f825113847814

mw.widgets.Complex*: Add setDisabled

These widgets inherit from OO.ui.Widget and so need to implement
setDisabled. Calling this method right now would set the right
CSS classes but wont actually disable the widget.

Change-Id: I6747f4e64dd81197bb3e37c09876399f1cb3be07

RC Filters docs: Directly mention group classes

Directly mention ChangesListBooleanFilterGroup and
ChangesListStringOptionsFilterGroup so people know what to construct.

Change-Id: I3c105d072889c1da26f94402326ba48635081833

Merge "SpecialAllMessages: Remove unused variable langcode"

Merge "Tweak figure styles based on visual diff testing"

Localisation updates from https://translatewiki.net.

Change-Id: Id631b382683ce4ac3ef648bdb546821c74a9985c

Merge "Follow-up Improve sorting on SpecialWanted*-Pages"

Merge "ResourceLoaderImage: Point to the right skin when generating URL"

Merge "ResourceLoaderOOUIImageModule: Actually load non-default themes' images"

Follow-up Improve sorting on SpecialWanted*-Pages

Change the SpecialWanted*-Pages so that they do sort
1. by the number of links to a site (as is now) and
2. alphabetically for entries which have the same number of links (new)

This is a follow-up for commit 5b15728.
The previous patch was reverted because using getOrderFields and
prefixing each order field with qc_ in fetchFromCache threw DB errors in
production. This follow-up differs in introducing a new function
getCacheOrderFields that defines order fields only for fetching from the
cache and doesn't reuse getOrderFields which is for the raw query
(without cache). It returns ['value'] by default and thus keeps the
known behaviour.

Bug: T4335
Change-Id: Ic01a3c28892c78ba9a9f0a7d665937efe87b44bd

Merge "RCFilters: Only show filter capsule as conflicted if it's selected"

RCFilters: Only show filter capsule as conflicted if it's selected

Bug: T161172
Change-Id: Ib15ffd58290c616de216178cee6ae9b86178d871

Decode '0'-valued user options to integer 0

I13d0e402f fixed a MySQL strict-mode bug by having boolean false be
sent to the database as 0 rather than "", since so many of our
logically-boolean fields are typed as tinyints. That happened to also
cause logically-false user preferences to be stored in the
user_properties table as "0" rather than "", which works fine in PHP but
confuses JavaScript since it considers string-0 as truthy rather than
falsey.

To avoid this situation, convert "0" to 0 when loading the user
options. Completely solving T54542 is left for another time, since
identifying which type to normalize each option to seems nontrivial.

Change-Id: Ia3280b7ce923641eac077141b47cba10d3fb88db

Merge "RCFilters UI: Pass in userId to ChangesListHighlights schema"

Merge "RemexCompatMunger: fix a couple of memory leaks"

Merge "Update RemexHtml to 1.0.1"

Add MusikAnimal to CREDITS

Ran maintenance/updateCredits.php which determines new contributors
using git log. Found 1.

Change-Id: I0d3a2626051ca058ea391c39550d4c1d19f4f133

RemexCompatMunger: fix a couple of memory leaks

Change-Id: I47578b3f73320e84a157417c288de97b5d26e18f

Update RemexHtml to 1.0.1

For a bug fix

Depends-On: I4421afbfa52d7711d9369398891e91ae09db50c9
Change-Id: I5285c0b1c6f42891c002637cd3e4bd2f54615940

RCFilters UI: Pass in userId to ChangesListHighlights schema

Bug: T160928
Depends-On: Ia2bc679e37deb34b215ad16dfc3b8cfefdc8365c
Change-Id: I9d3f9281c2681ab1c89676234ddd544f76f3dc7f

Merge "Allow extensions to add jQueryMsg magic words"

Localisation updates from https://translatewiki.net.

Change-Id: I83074d7d6edd5cf2e3b69d0bb2382012a5b4e438

Merge "RCFilters UI: Make the 'no results' message bold"

Merge "RCFilters: Normalize overlay font-size for Monobook"

Allow extensions to add jQueryMsg magic words

Change-Id: Ie82a147ff32ccda3f757108474f5cbab71d45ace

Merge "Stop declaring ImageHistoryPseudoPager props dynamically"

Stop declaring ImageHistoryPseudoPager props dynamically

The 3 properties declared here were introduced in:
0ca92d0bf5dd4e71b678d373a1beb3089d16a866

Change-Id: Ibbd0a86f02ab010720446db1aa2dc9ea0eb05aef

Merge "Fix log msg param in UploadFromUrl::saveTempFileChunk"