mediawiki.toc: Clean up left overs

Follows-up d7a40c8 which creates the mediawiki.toc module and
moved the tocToggle method from mediawiki.util.

The tocToggle callback used to be a public method and therefore
had two things about that that are obsolete:

* Taking the toggle as parameter.
  This is a private method now, and we already have a reference
  to the toggle.

* Asserting that the #toc has an <ul>.
  There is no need to defer this check this far in, nor to repeat
  it on each click. The old code checked it earlier was well, but
  we used to repeat the check here as the method was public.
  Now that it is a private helper method we can simply return
  early before setting it all up in the first place.

Also:

* Removed jsduck-like construction. This file is being indexed,
  as part of src/mediawiki/, but turning this into a class seems
  odd. It doesn't really have any structure to it.
  It's just UI glue. Turned it into an ignored file (no doc blocks).

Change-Id: I21d4f735032b96f00291502e06d9d35a74648143

Merge "Clarify docs for wgCookieExpiration"

Clarify docs for wgCookieExpiration

Change-Id: Ie6c77c5251e906dcee1650b3a4598da6adb30e30

API: Expose git branch in siteinfo

For browser testing, I need a reliable way to determine the branch in use
on a target wiki.  One could parse the generator name forged when
querying the siteinfo, but that yields:

 MediaWiki 1.24alpha  instead of master
 MediaWiki 1.24wmf2   instead of wmf/1.24wmf2

I am not really willing to reverse engineer the version to figure out
the branch, instead just expose it over the API.

Bug: 62509
Change-Id: If3db8f480ce0d590522741d40fb8e43b753efb41

mediawiki.toc: Hide toc without animation in initial collapsed state

Bug: 64927
Change-Id: I5b46390b3dc6e42031f636119673892507151f54

Merge "Pass $user to LocalFileDeleteBatch"

Merge "Make gallery properties publically accessible."

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

Localisation updates from https://translatewiki.net.

Change-Id: Ieca93135f7af56bc2079362cc763e7204a8024cb

Pass $user to LocalFileDeleteBatch

This avoids the use of $wgUser

Change-Id: I60cab27e0708a818c77791cc89194b6dd726da0b

Check for valid git repository in GitInfo::getHeadCommitDate

Ensure that GitInfo::getHead returns a valid value before attempting to
use the git binary to extract a commit date.

Bug: 64948
Change-Id: Ib0d245306842706dbb5d4b9522bb381a5f55526c

Merge "Fix typo for system message name"

Merge "SpecialSearch: Get title from one we already have on hand in the context"

Fix typo for system message name

Change-Id: Ic23f71ab9f6018152f89f35a944c9daae9f94970

Merge "Add version comments for 1.24 to all updaters"

Merge "Special:BookSources: Remove link to PriceSCAN"

Merge "Fix missing import in mediawiki.ui"

Merge "resourceloader: Use $.ajax instead of duplicating its "script" logic"

Merge "Move profile In/OUT call out of MWPHPUnitCommand"

Merge "mediawiki.util: Clean up mw.util.$content"

resourceloader: Use $.ajax instead of duplicating its "script" logic

Also removed isFunction check. This is an internal method and the
parameter is either a function or omitted/nulled. No need for aggressive
validation.

Change-Id: I6bbd82ca1b986b45a2f4653427396b9a6fceb6e8

Merge "Move teardownTestDB and wfLogProfilingData out of MWPHPUnitCommand"

mediawiki.util: Clean up mw.util.$content

* Document how to use it (including the tricky bit with it needing
 a dependency on mediawiki.page.startup).

* Refer to mw.hook and #mw-content-text as recommended method of
  getting just the wikipage content.

* Remove the deprecated backwards-compatibility selectors.
  The core skins (Vector, Monobook, Modern, CologneBlue) all
  have mw-body now. Third-party skins should adopt those if they
  haven't already.

Change-Id: I30bdd893e0a0a340076efa7ce32fbd4b775bc3db

Merge "Remove includepath stuff from MediaWikiPHPUnitCommand"

Fix missing import in mediawiki.ui

The .box-sizing() LESS function was moved in:

https://gerrit.wikimedia.org/r/#/c/127843/

And the usage in forms.less wasn't finding it anymore.

Change-Id: I7ae97bd5d4aa113af039182ff39817926bbbf3ed

Vector: Don't supress focus outline on watch star

This is an old remnant that was intended to remove outline from
images inside anchor links. However the watch star is no longer
implemented this way and this is doing nothing. Except when
the element is focussed (using keyboard tab index), then this
rule is overriding the default :focus and causing the user to
potentially get confused since it is no longer clear where the
focus has gone to.

Bug: 64934
Change-Id: I57ee089c1ae18bb7a6c55ae17eab3a06f9e1f7d2

Support precomputed data in GitInfo

Support reading git repository information from a JSON file in the cache
directory. When present, this file serves to provide information needed
by getHead, getHeadSHA1, getHeadCommitDate, getCurrentBranch and a new
getRemoteUrl method. A GitInfo::precomputeValues method is also provided
which can generate the cache file for a given GitInfo instance.

This support can be combined with a deployment step to reduce the need
to repeatedly gather information from the .git files/git binary. It also
allows computing information that can be lost when directory structures
differ between deployment staging hosts and hosts running MediaWiki.

This change also adds memoization of computed values for a given GitInfo
instance which may provide a small performance boost even for
deployments which are not using precomputed cache files.

Bug: 53972
Change-Id: I66e058acc5a71e5d82644f85d819f49d6ee9d1e6

Merge "mediawiki.mixins: Move .box-sizing mixin from mediawiki.ui utilities"

Merge "Don't call legalSearchChars() statically so it can properly inherit"

SpecialSearch: Get title from one we already have on hand in the context

Change-Id: Id7e761ad4882ec5c5f9324644eca143b727b0ad9

PostgreSQL: decode image.img_metadata

The img_metadata field was not being decoded from bytea into text.

This change to core fixes the reported problem with PdfHandler against
1.22.6, origin/REL1_23 and git head.

Bug: 59147
Change-Id: I285a317a57a29713af1f7f630b340278bd5400da

Merge "PostgreSQL: Make l10n_cache.lc_value binary"

Merge "Update OOjs UI to v0.1.0-pre (7d2507b267)"

Update OOjs UI to v0.1.0-pre (7d2507b267)

New changes:
9ccab2d Use .isDisabled() instead of .disabled to support inheriting
8f2a439 Localisation updates from https://translatewiki.net.
5f49c04 Remove unnecessary space in footless dialog
11dde77 Make autoFocus default to true

Change-Id: I9ae775a7c2a167d66542436ebd21064d8d254cc4

mediawiki.api: Fix API postWithToken method

Referred to "this" instead of "api" ("this" here refers to the Deferred object).
Fixes "Uncaught TypeError: Cannot read property 'ajax' of undefined".

Change-Id: I78ba47a614512f6218e23d03e7c688e2c9efbe45

Don't call legalSearchChars() statically so it can properly inherit

Change-Id: Iaabc10ce2905eefe9de8bceeec3290082ba9eb2e

Set Redis::OPT_READ_TIMEOUT by default

* The value is also configurable just like connectTimeout
* This deals with problems like https://github.com/nicolasff/phpredis/issues/99
  and https://github.com/nicolasff/phpredis/issues/70

Change-Id: I05e91e05764020854d04673b7decae30359f57e9

Localisation updates from https://translatewiki.net.

Change-Id: I72e85ae5bf1ff819f8ee6e1a7d89c7828b940379

Merge "Avoid raw sql in LocalFileDeleteBatch::doDBInserts"

Merge "Fix doc for Revision::newFromPageId"

Fix doc for Revision::newFromPageId

Change-Id: Ice5b7f42facd30f856f06c1bc1f4408d61a4121d

Merge "Maintain remember me state on password change"

Avoid raw sql in LocalFileDeleteBatch::doDBInserts

Use Database::conditional instead of raw sql.

Change-Id: Ic00146e9ec582a263b9dec117b12532fdc24789e

Merge "TextContent: Document parameters to convert()"

Add version comments for 1.24 to all updaters

pp_sortkey was added with I217c42656fb877ff35a36eb446a22bdaf119faac,
which is part of 1.24, so adding the comment before that.

Change-Id: I7b970705ad7de75cb9542f8a5a963697386940a7

Merge "Removed unused getZoneHandlerUrl() method"

PostgreSQL: Make l10n_cache.lc_value binary

Change-Id I427c6de5a0a29b43cff755db0eb8a750db620173 increases the
probability that a null byte will attempt to be stored in the
lc_value column.  PostgreSQL does not allow that byte in a text
column, so convert the column to bytea.

If the column already contains corrupted data, the upgrade routine
might fail.  To prevent this, delete the contents of the table before
changing the type.

Bug: 62098
Change-Id: Ie8368bde398b2ae4d3cfc9ee7bf35874bd2ded68

Merge "PostgreSQL: Fix syntax error in page_props index"

PostgreSQL: Fix syntax error in page_props index

Follows up I217c42656fb877ff35a36eb446a22bdaf119faac

That change introduced a syntax error during update.  I have verified
that this fixes the syntax problem so that update.php runs, but have
not verified that the new functionality behaves as intended.

Also, the PostgreSQL schema changes introduced during the update
process were not present at all in a fresh install to PostgreSQL.  Add
analogous changes to tables.sql.

Bug: 64807
Change-Id: I736344839ec478216402fdfe260968afbd663498

Merge "API: Check return value from $gitInfo->getHeadCommitDate()"

TextContent: Document parameters to convert()

Mostly extracted from I979e2438.

Change-Id: I764aae970a2ad90a54aa9816d81070d0899fd10a

Merge "Documentation cleanup in Content-related files"

Documentation cleanup in Content-related files

Mostly extracted from I979e2438.

Change-Id: I6562d111c15425957eaaad31ded175811d7abf1c

API: Check return value from $gitInfo->getHeadCommitDate()

action=query&meta=siteinfo&siprop=extensions calls
$gitInfo->getHeadCommitDate() to attempt to find the commit date, but
does not check whether that function returned false before using the
value. This resulted in displaying the current date for "vcs-date" when
the function failed.

Bug: 64821
Change-Id: Ic39a74abe5160b3b7fbfb7c15323328d6b317560

Move profile In/OUT call out of MWPHPUnitCommand

Change-Id: I710fddc20e88745c9091432dff87b5e361c1f71a

Move teardownTestDB and wfLogProfilingData out of MWPHPUnitCommand

Change-Id: Ic175c1457e7f76a2b5232a2b5d3563c5982cfee2

Merge "Remove color opt setting from MediaWikiPHPUnitCommand"

Remove includepath stuff from MediaWikiPHPUnitCommand

Change-Id: I949c1ca350a28c606fba7406e9904e836e596b49

Remove color opt setting from MediaWikiPHPUnitCommand

Change-Id: I08d3b53f80520452aef3fcf2017747406a38664d

Merge "Vector: Restore collapsibleNav design improvements"

Merge "Fix typos in WikitextContent.php"

Merge "Return $wgSitename as displayname for local repo in filerepoinfo API"

Fix typos in WikitextContent.php

Fix typos in documentation for
/core/includes/content/WikitextContent.php

Change-Id: Idc0d4d9101309cea858473c86a4b12cf6e5f5ee4

Vector: Restore collapsibleNav design improvements

Follows-up db13c65 and c3d544412, which removed the
collapsibleNav module. However it also undid various design
changes that really should not have been part of the
collapsibleNav module but were significant, I've restored those.

Basically, this merges the old collapsibleNav.less into
navigation.less. I won't list all chnages, but here's a few:

* Increase spacing between heading and side of portal (via margin and
  padding on the portal, not on the heading itself).
  The headings were almost against the browser edge.

* Change color of heading slightly (#4d4d4d instead of #444).

* Move separator line from below each heading to between sidebar
  sections.

* Change spacing between list items from 0.5 on the bottom to
  0.25em on top and bottom (effectively the same, except aligns
  the first and last item better in relation to the heading and
  the separator line).

* (js-only enhancement) Mark first section with class "first",
  and remove the separator line, heading and offset from that one.

Also:

* Clean up 'h3' padding/margin situation.
  They were inheriting 0.3em of bottom margin from the global
  'h3' selector for content. This was a bit of a fragile
  dependency and was just a coincidence. Reset the margin against
  the global selector and added it to the local padding instead.
  No visual change.

* Remove unused less variables.

Change-Id: I979179a82c1de9b1b7d66310ebd5daf8e6aa1802

Localisation updates from https://translatewiki.net.

Change-Id: I3cfce4dbbfe6c17713ad4d824d2514feb9e2fb67

Vector: Remove unused collapsibleNav.less

Follows-up db13c65.

Change-Id: I5fbfe4df5a7b7b3df8a295957eb8db1f2363014e

mediawiki.api: Don't apply file extension stripping to category names

`mw.Title#getNameText()` strips anything following a dot (.), assuming
it's a file name (which doesn't make sense for category names).

C.f. https://doc.wikimedia.org/mediawiki-core/master/js/#!/api/mw.Title-method-getMainText

This causes some trouble in UploadWizard's category-suggestion system.

This bug has been there since the very first version (cb0cf72eba9ff)
which, ironically, was specifically written for UploadWizard.

Bug: 64764
Change-Id: I2925e1050215272cdf03c8c6b1de6ba79348f6a5

JSDuck-ify /resources/mediawiki.special/*

- Adds mw.special.recentchanges to the index.
- Adds mw.special.upload to the index.
- Register File as valid value type. Relates to the
  "File" interface (from HTML5 File APIs; http://dev.w3.org/2006/webapi/FileAPI/#dfn-file)
- Sort directories in jsduck/config.json alphabetically.

Change-Id: I282cbeeec60b9f527879aedd90ba824a833438b7

Merge "mwjsduck-gen: Abort early and report exit code"

Merge "Revert "jquery.textSelection: Remove hardcoded checks for removed WikiEditor iframe mode""

Revert "jquery.textSelection: Remove hardcoded checks for removed WikiEditor iframe mode"

Appears to have broken CodeEditor

This reverts commit c40b6fa9b0a0e17729a75661f80e7882cb0598e1.

Change-Id: I6d04cb91527defe1fdb0d7fb2a324fe1417a89f7

Remove collapsibleNav: performance cost too high

Appears to be unused by any extension in gerrit.

Bug: 39035
Change-Id: I220057c799bef7e8c4d964f47d56f61ec120d8c2

Merge "Add action-history"

Add action-history

Bug: 48619
Change-Id: Iac95e401e6d9b3e67512c512221557d3b9b7a5bf

Merge "Improve in-line documentation of $wgUrlProtocols; add 'redis://'"

Improve in-line documentation of $wgUrlProtocols; add 'redis://'

* RedisPubSubFeedEngine uses 'redis://' URLs. So does redis-py and many other
  client libraries. It should be recognized by MediaWiki. So add it.
* Remove in-line comments justifying each individual URL scheme.
  DefaultSettings.php is long enough; it doesn't need to moonlight as
  the annals of the history of MediaWiki development.
* Make the documentation of $wgUrlProtocols a bit more useful.

Change-Id: I4c2de67cc4e243659b0bed674e0ccd48221c57f3

mwjsduck-gen: Abort early and report exit code

This bash script is supposed to abort early with a non-zero
exit code if the jsduck test found any errors or warnings.

Change-Id: I2ae0976be866519dd9f41f4c02d78d300a494328

Localisation updates from https://translatewiki.net.

Change-Id: I5b35bd80d643c943aeb9a608b3299fd7d45cbf00

Return $wgSitename as displayname for local repo in filerepoinfo API

displayname is used in user interface texts to tell the user what
is the source of a file so returning null does not make sense
here.

Bug: 64800
Change-Id: Iafbc1a8e63b512d1515215fa1295e5b7434f1861

JSON RC format: cast 'patrolled' and 'minor' to boolean

This makes them consistent with 'bot'.

Change-Id: Id4342eedee8176b0b76458bdaf8ca69c89857d47

Make gallery properties publically accessible.

They were public in the previous release of MediaWiki (1.22), although
the visibility was not set explicitly.

See I779d5a683982dc25bc7cdd30addd5870f2a7efd0 and
https://github.com/SemanticMediaWiki/SemanticResultFormats/issues/12

Change-Id: I309cf46891223c5f1520e97b04d4ed315a300e7a

Merge "Use an actual URL for helppage in enotif_body"

Use an actual URL for helppage in enotif_body

The helppage can be a full URL, which canonicalurl doesn't like.

This means that if a wiki
* has a custom enotif_body which uses canonicalurl + helppage, but
* doesn't have helppage defined locally (as local/interwiki title),
then it will still have to fix either of the two locally.

Bug: 63269
Change-Id: I82d26b648ef23b69ae4fbd0a74ffa23eb7a93dae

Merge "Check for the namespace correctly in Special:Allmessages"

Added missing File::RENDER_NOW flag to thumb.php

Change-Id: I5f05b09272ebcad5cf8409af7ad61d9e9004a2f0

Removed unused getZoneHandlerUrl() method

* This was not terribly well conceived and is not used by anything.

Change-Id: Icb03477e2b33621949fab7c39fd06e28ac0777fc

Maintain remember me state on password change

Fix regression from https://gerrit.wikimedia.org/r/#/c/94614/

Ensure that when users login with "remember me" selected, they get a
new remember me token after changing their password.

Change-Id: Ibbf0d823bd5ef5304bf6ad1b67966d18dfb3580c

Merge "Update OOjs UI to v0.1.0-pre (9a6c625f5f)"

Merge "Update OOjs UI to v0.1.0-pre (e6edd86d37)"

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

Localisation updates from https://translatewiki.net.

Change-Id: I65fccf05eac621d55e6404631359d5add9499b1f

Restore mediawiki.toc dependency on mediawiki.util

Follows-up If0438b7b6f4649, which broken the toc toggle module
since it was no longer part of a global module (mediawiki.util)
but only added to individual pages by the parser.

While that is a superior load mechanism, it broke existing pages
due to parser cache. Not the general html cache for logged-out
users but the main parser cache, which affects logged-in users
as well.

Bug: 64765
Change-Id: Ia12998fb11c686e1cc965578f7beb31c79b1b12c

Update OOjs UI to v0.1.0-pre (9a6c625f5f)

New changes:
9a6c625 Fix regression in frame style scoping

Change-Id: Ia8d4a7525d951a1b59c0c983262d868ed6d379ed

Merge "Rename basic skinning modules: skins.common.* → mediawiki.skinning.*"

Merge "jquery.makeCollapsible: Use the pointer cursor for all toggles"

Merge "jquery.textSelection: Remove hardcoded checks for removed WikiEditor iframe mode"

jquery.makeCollapsible: Use the pointer cursor for all toggles

Change-Id: I82030a5879782eb2475a3a4bc2e0bb2988add06a

Merge "HtmlFormatter: fix check for empty ruleset"

Merge "Made wfThumbIsStandard() aware of the width/height handler scaling choice"

Update OOjs UI to v0.1.0-pre (e6edd86d37)

New changes:
e1c2ae8 [BREAKING CHANGE] Cleanup widget focus/select/clear methods

Change-Id: I7214257bcd33b400df4f7b1d8c2be317936f0cff