Quick refactor of the http logic in ForeignAPIRepo

Automatic caching, will enable slightly smaller patches elsewhere.

Resetting after failing at a rebase.

Change-Id: I6849107a3af773687a31155b23a2d7b4d7a60645

Merge "Fix revision table cleanup on delete"

mediawiki.action.edit.preview: Fix for LiquidThreads hack

LiquidThreads is loading this core module when rendering an
inline edit form. Because the form doesn't have the same ID
as the one in core, this broke after becd284.

It previously worked because the click event on the button (of
which LQT did duplicate the ID from core) bubbles up to the
document body and thus trigger this private handler of the
core edit preview module.

Change-Id: I9e35d0c48b86e1c9073b68313a8a330c6ea85b86

Merge "Localisation updates from translatewiki.net."

Merge "Add file-link "parser" to mw.Title from commons"

Localisation updates from translatewiki.net.

Change-Id: I331db0fe25638bdab35fbf54d00dd77f93488a7a

Add file-link "parser" to mw.Title from commons

Rillke has been amazing and provided us with a first pass at a file URL
parser for MediaWiki:

https://commons.wikimedia.org/wiki/MediaWiki:Gadget-libUtil.js

Hoping to use this for Extension:MultimediaViewer and others in the near
future - potentially replacing patchset Ib69d1ffdfd3b58c2e8e3314940a2cad311b85533

Also tests.

Change-Id: Ie31e6df376bda087e92ec889d54df3374ba96b5e

Merge "Allow $context->setTitle( null )"

Add doc comment to Status explaining use

Tim posted a nice explanation of the recommended pattern for using the
Status object to wikitech-l. This change copies that explanation into
the phpdoc comment block of the Status class in an attempt to propagate
the meme.

Change-Id: Ia98543caaa829cad443abf0f0f5038b3de943ef8

Merge "Fixup parameter type hints"

Merge "Add method parameter type documentation"

Merge "Move forceHTTPS check until after wgTitle is setup"

Added $wgExtensionEntryPointListFiles for use with mergeMessageFileList.php

Going to be used to change target file for labs vs production

Change-Id: Id4b16083435ef7f4fce31861c72889e664d07236

Add list=allfileusages

This allows the client to enumerate through entries of the imagelinks
table.

This can be useful to find images from foreign repos used on the own
wiki.

Change-Id: I2ad0579e3521868d39f7d495ea9c83a2cd9731d6

Fix help string to avoid PHP notice

Escape a $ symbol in help message and rephrase slightly.  Without this,
`php maintenance/language/checkDupeMessages.php --help` prints
PHP Notice:  Undefined variable: dupeMessages in
        language/checkDupeMessages.php on line 56

Change-Id: Ie7c0f07e68cee1f7cd2758f71b09d34e339d4b7b

Merge "Localisation updates from translatewiki.net."

Removed RC query UNION

* MySQL is smart enough that this is no longer needed

Change-Id: If173cdc53599cd02068ce2480ab667c5f9e69b76

Merge "mediawiki.action.edit.preview: Clean up and optimisation"

Merge "Add script to purge changed pages from cache"

Make .transition mixin take only 1 value and fix usage

Follows-up 99c92a87fc which turned
  transition: left 250ms;
  transition: margin-left 250ms, padding 250ms
into:
  transition: 'left 250ms';
  transition: margin-left 250ms padding 250ms
  > Invalid CSS property value: 'left 250ms'
  > Invalid CSS property value: margin-left 250ms padding 250ms

Removed quotes from 'left 250ms' which was producing invalid CSS.

The @arguments operator in Less joins a variadic list of
arguments by space. In the case of .transition this is a problem
because its values are supposed to be separated by a comma
(like background-image and font-family).

Changing the .transition definition to take only 1 argument
instead of multiple (...). There is no point in supporting
multiple arguments. All it does is it take away our ability to
spot these errors at parse time (Less would be able to throw an
error for passing a 2 arguments to a mixin that only takes 1).

Fixed the invocation with 2 arguments to instead pass 1 argument
that has a comma in it. Both "~'a,b'" and "a, b;" would work here
(doesn't matter) choosing ; here because for consistency with calls
that do use mutiple argments:
 foo(~'foo, bar', ~'baz, quux')
 foo(foo, bar; baz, quux)

Change-Id: I80786c726a412d6b9e04c384112849e9587990b4

Add script to purge changed pages from cache

This maintenance script could be used to recover from large HTCP
listener outages and/or network partitions. It searches the database for
pages having revisions occurring within a specified time period,
converts them to full URLs and sends squid/varnish purge messages using
SquidUpdate. A command line option allows specifying a specific
IP:port combination for HTCP purge datagrams which can be used to
isolate the purge announcement to a particular varnish server or data
center.

Co-Authored by: Aaron Schulz <aschulz@wikimedia.org>
Bug: 54647
Change-Id: Id0e95290e83333545e6bb0378cc620c35c653bf9

Localisation updates from translatewiki.net.

Change-Id: Ib6cdd80506e78608d9d8af15088020528fdec256

Merge "mediawiki.Title: Fix weird <h1> in constructor description"

resourceloader: Support hashes as cache invalidation trigger

For now ResourceLoader will still internally deal with timestamps,
especially because they are easier to max() on the client side
(on the server we could make a hash of different hashes, on the
client that's a bit annoying).

However ResourceLoaderModule#getHashMtime will abstract the logic
we already use in ResourceLoaderLanguageDataModule and have
encouraged others to use, which is to put the timestamp and
hash in Memcached, and use that to observer the hash change and
update the timestamp when it does.

Updated ResourceLoaderLanguageDataModule to make use of this.

Change-Id: Ib051ef41ba239084671c30fd275b8d94099d5d52

Move forceHTTPS check until after wgTitle is setup

Since the forceHTTPS logic checks if the user is logged in, which calls
all sorts of hooks, some that assume that $wgTitle is setup (like
AbuseFilter for preventing autocreation of usernames), move the
logic down a few lines.

Bug: 55435
Bug: 53498
Change-Id: I53c82921595a9014b555e4ec468e5ba10454cd3a

mediawiki.Title: Fix weird <h1> in constructor description

The leading '#' was triggering a Markdown heading instead of a
jsduck reference. Working around by moving the line break.

Change-Id: I8bf2719938c3c4a092ce5499bae0c7b88f4544c1

Allow $context->setTitle( null )

If something is messing around with $wgTitle before MediaWiki::main()
sets $wgTitle in the first place, it needs to be able to reset it back
to null to avoid a problem like bug 53498. And to do that properly, it
also needs to be able to clear the title in the RequestContext.

Bug: 55435
Change-Id: I6c763b70bbffa0762ca52e61acf7dc200da4075f

Fix "Unsupported operand types at thumb.php on line 429"

Return type of $handler->parseParamString() wasn't evaluated,
instead $handler value was compared again

Bug: 55437
Change-Id: Iedb940d9f1ad4d137b9732e7e624e55d05bc2c96

Merge "Accessibility: Don't remove checkbox outline on focus"

Accessibility: Don't remove checkbox outline on focus

Native checkboxes and radiobuttons on Mac OS X are not fully
styleable. Don't remove the focus ring for those elements because it
makes them less accessible.

Also the outline was removed outside of the focus selector, which
isn't really needed.

Change-Id: I716c84b34b5adb26691f0d4ecd13739de7b8bdbf

Move release note for meta=filerepoinfo into api section

Follow-Up: I1aaed0895d2a0bc224c82e93975ecf6afd8cb6b8
Change-Id: I83e393aca85ae215f515024322942acafa20462b

mediawiki.action.edit.preview: Clean up and optimisation

* Remove redundant inline 'position: absolute' on the spinner.
* Remove trailing white space in jquery.spinner.css.
* Cache #editform query result.
* Remove redundant removeSelectors.join(',') logic.
* White space.
* Move delegate click event listener down from body to #editform
  (the common ancestor of the #wpDiff and #wpPreview buttons).
* Remove minHeight (was needed because the spinner was positioned
  absolute), which gets rid of the weird white gap for #wikiPreview if
  you do live diff instead of live preview.

Change-Id: Icdf975f201947ecadb886057131f2531c68791fb

Merge "Remove placeholder tests"

Set 'tooltip-iwiki' as optional

Spotted by Beta16 in
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Tooltip-iwiki/en

Change-Id: Ib60dd6072728238684112865a5ee0b015d1f2f0c

Remove redundant parentheses

Change-Id: I848e6a055df748ac758082372fe3b86f6f323cc9

Follow-up d94d29a920: Fix typo in release notes

Change-Id: Ic2cb19df45193b260d785fda5acf2ce956b51dde

postEdit: Add documentation for hook so it shows up in the index

Follows-up 011ee683a.

Change-Id: I3ce89a91f6a299516dd8dfb93dd5cc5b784fa471

Special:UserLogin UI improvements when user is logged in

* Show a warning about being already logged in above login form
* Show a button pointing to signup form with suitable explanation

Also changed some internal template stuff (s/createAnother/loggedin/).

Bug: 53306
Change-Id: I1672ee09330c231110c4db310a645d1382cc0b59

mediawiki.special.vforms: Information boxes' styles

Only styles appropriate for errorbox were defined, but they were
applied to all boxes - contrary to the comment above, Vector doesn't
have styles for them (they are shared for all skins, and they are
awful), and they were anyway overriden by these due to selector
specificity.

Added styles for successbox and warningbox with composition on the
color wheel very similar to errorbox one, simply rotated and scaled.
Removed misleading comment.

Change-Id: I68d2c9b98a9166f5781fcded0318cca52ed6f2e6

Merge "(bug 54980) Remove toolbox UserRights if anon user"

(bug 54980) Remove toolbox UserRights if anon user

Change-Id: Ie489ce327a56cf314c9dc525d6bd3eaa39739260

Make the format of UDP-logged stats configurable

The format of datagrams generated by StatCounter and ProfilerSimpleUDP is not
described by any standard and appears to have been designed for compatibility
with the logging setup of the Wikimedia Foundation, which limits reusability.

This patch adds two configuration variables, $wgUDPProfilerFormatString and
$wgStatsFormatString, that can be used to specify the precise format that
MediaWiki will use. The default values for these variables causes MediaWiki to
generate the same output as before.

The secret evil plan is to change the defaults so that MediaWiki emits metrics
that are compatible with the popular StatsD format (see
<https://github.com/b/statsd_spec>). This would allow us to replace udpprofiler
with a generic StatsD instance instead.

Change-Id: Iaf00811d3f8d6b89c1c68e84fc1f2c93425d5a2d

Localisation updates from translatewiki.net.

Change-Id: Ic112ae52452378c970d37c0b3061433131f5f55d

Merge "API: Add output of wgLogo to meta=siteinfo"

Remove placeholder tests

They give a false sense of test coverage and nobody's shown any
interest in implementing them.

Change-Id: Id15ddb555e059bb6a76c69ca57d09c91e96e91ce

Remove use of deprecated LoadBalancer pickRandom

Change-Id: If28b6dfe7d70827d1c7f7b3604cea2409b6ec483

Introduce .transition mixin

Cleanup existing transition rules for readability

Change-Id: I2be09a7a8e7eeb42d8bf80cb033c81867da108a5

Turn screen.css into screen.less

* Separate external links into own less file to showcase how
many unnecessary css rules we have
* Introduce new mixins to allow embedding for list-style-image
* Replace use of /* embed */ with mediawiki mixins

Change-Id: If79d2d1f72579627597b291eec759441d6763c34

Merge "Rewrite collapsibleNav as LESS file"

Merge "Strip excess newlines from formatting test"

Rewrite collapsibleNav as LESS file

* Rewrote css file as a LESS file for better maintainability
* Make use of core embed mixin
** Add .background-image-svg mixin for rendering svgs as background images

Change-Id: I72a66553d6ecee5018d4efaabfd0fa0df6c4135f

Merge "LESS embeddable(): Use lessc::toBool"

Merge "Mouseover explanations for interlanguage links in native language"

Strip excess newlines from formatting test

The result's basically the same either way, only difference is
things like

 "emptyObject": {

 },

Are now

 "emptyObject": {
 },

Results are still pretty, now passes in both Zend and HHVM

Change-Id: I6afa0fd49a7aef3f6b7ca1b4574f3de555521348

mw.Title: Add missing dependency on jquery.byteLength

Follows-up 4894793ab.

Bug: 54977
Change-Id: If884324674198e26636785c0e95e5d61fd5c543e

Merge "Fixed silly unlock bug in LocalFile"

Fixed silly unlock bug in LocalFile

* Removed the lock instead of just purging some cache data

bug: 54978
Change-Id: Ida007c28e07b0fe1803ab8ed027bc2ec4f55af19

Merge "Allow appending text to redirects from moved pages"

Mouseover explanations for interlanguage links in native language

Adds a language name in User's language from CLDR
to an interwiki 'title' attribute. Believed to be
RTL-compilant.

New message added: tooltip-iwiki, added to $preloadedMessages

Bug: 5231
Change-Id: I3dcb00cdc28a10711b6703fa509b8b8d06e325a2

Allow appending text to redirects from moved pages

Add message move-redirect-text, which is appended to redirects created by
page moves (empty by default). This could be used on enwiki to automatically
add {{R from move}} while still allowing non-admins to revert the move, for
example.

Change-Id: Ic2158897ce46177b695a7fb90c8ed3349df18bde

Restrict <abbr> formatting to where a title attribute is present

Reasoning:

* It does not make sense to draw attention to an <abbr> element that
  doesn't have a title attribute because there is nothing special to
  see there.
* It matches Gecko's default stylesheet [1] and the HTML5 style [2].
* It does not negatively impact compatibility, as far as I know
  (IE 6 does not support the [title] selector, but does not support
  the <abbr> element either).
* <acronym> elements are currently not allowed and won't be because
  they are non-conforming in HTML5 (cf. bug 671, comment 37), so this
  part of the rule can be dropped.

[1] http://mxr.mozilla.org/mozilla-central/source/layout/style/html.css
[2] http://www.whatwg.org/specs/web-apps/current-work/multipage/rendering.html#phrasing-content-1

Bug: 54729
Change-Id: I1b00ddb7996911e9b477365e7a84458cefd5da90

Support GENDER for "contribsub2"

Also remove BC code that was added in MediaWiki 1.9. Spotted by Protnet at
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Blocklogentry/el/reply

Change-Id: I37eefd1de95d6ef3429403cd351db315bc3ab2bf

Fix revision table cleanup on delete

The ContentHandler branch inappropriately moved this essential
query to content-type-dependent code, despite the fact that revision
table cleanup needs to be done independently of content type. Before
r15707, the revision query was done immediately before the page query,
but it was moved down to allow it to be brought into the same
if (!$dbw->cascadingDeletes() ) block as the link updates. I have moved
it to immediately after the page delete, so that the rollback which has
since been added will not affect the expensive revision table query.

This should fix the database corruption we are seeing on some page
deletions (13 affected pages on enwiki, 14 on nlwiki, etc.)

Bug: 53687
Change-Id: I7a54c4416be07fb8cc7a5698e77927ccb24cbd6e

Merge "mw.Title: Rewrite from scratch (porting logic from Title.php)"

LESS embeddable(): Use lessc::toBool

With Ib6bc76736 in place, we have access to lessphp's custom function helpers,
which provide some nice encapsulation for interpreter internals like boolean
nodes. This patch makes embeddable() call $less->toBool() rather than construct
its own AST nodes.

Change-Id: I0a015564dff19f5ced764af182558eb351e3d816

Merge "Update lessphp to 261f1bd28"

Update lessphp to 261f1bd28

From https://github.com/leafo/lessphp/blob/261f1bd28f/lessc.inc.php

Changes, 6e8e724fc7..261f1bd28f:
* 261f1bd Merge pull request #480 from atdt/master
* 0f85200 Miscellaneous small touch-ups
* 5504066 Improve handling of /* comments */ when 'preserveComments' is set
* 667702c Make function helpers public

Bug: 54673
Change-Id: Ib6bc76736d460081dc8f48aee09ca32066dae296

Merge "Added wiki ID to getCoalesceLocationInternal()"

Added wiki ID to getCoalesceLocationInternal()

Change-Id: Ia1ab1883808b669e3c3b442d860c1c562e984564

Merge "Avoid extra HEADs for stash clearing"

Merge "Maintenance: detect missing parent::__construct()"

Merge "Add missing default true value for 'vector-simplesearch' preference"

Localisation updates from translatewiki.net.

Change-Id: I7f8298c727621e51c73f689ea5287da44dea3af4

Avoid extra HEADs for stash clearing

Change-Id: I96e78b219fe12b43daf4a313eb297b98577ef955

Merge "Use HPHP_VERSION constant to detect HipHop"

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 3d6d0723f93e2a838df717b5036e5cd6b0345eca

Change-Id: Iadc98cc4cc33c7725e5fbf4a16df7e1a898aee80

Maintenance: detect missing parent::__construct()

The first time I forgot to include that line in a maintenance script,
I spent a few minutes debugging "Notice: Undefined variable: IP".
Hopefully, this change will save someone else a few minutes of his
or her own time.

Change-Id: Icd26a5dc59e25d2bcc6da9bf0a2f6e95d494e5a6

Merge "Removed more FORCE INDEX clauses for logging"

Use HPHP_VERSION constant to detect HipHop

Follows-up Ic3e769f1fbad4f7ad26dd819406796fee48c6b45.

Change-Id: I0210002c05efd79242b00592639b7be50a79020b

Merge "Allow AbortLogin hook to override error messages"

Remove wfSpecialList()

Has been deprecated for several releases, nothing in Git or SVN
uses it.

Change-Id: I899a0a76a5cb23686caaf6fd244f926a6938fe77

Merge "Show whether tags are active on Special:Tags"

Merge "Correct attribution for Wikimedia project visitors and restore grammar"

ParserTests: allow test of precedence between interwiki links and namespaces

Bug: 51680
Change-Id: I8ad82c596139824fb97d513a62513edf2941b958

Allow AbortLogin hook to override error messages

The AbortLogin hook has a $msg argument that allows for specifying a
MediaWiki message to display to the user. But this message is only used
when the hook is aborted with LoginForm::ABORTED. There's no reason not
to use this same parameter for the other possible failure constants.

Note this is a slight change in behavior for the hook, as previously the
$msg parameter would be coming in as "login-abort-generic" unless an
earlier hook altered it, while now it will be null.

Change-Id: Ib03a1aed9dfb566a027c31c015aa30619174a6c2

Removed more FORCE INDEX clauses for logging

* Same reasons as 7003a7c5e825f391c1a83f8e46cdf332a228d874

bug: 54876
Change-Id: I743bd21e81395c2866b4241730caa8c5990cc73f

Merge "Installer: remove dependency on JSON extension"

Merge "Remove convertPlural methods already served by CLDR plural system"

Localisation updates from translatewiki.net.

Change-Id: Ia5773eca5bdbe62ab8d4123bb417c2f71a6aa9f5

Merge "Standardise wording of verbs relating to revision deletion"

Make sure LocalFile::lock() works for brand new uploads

bug: 54736
Change-Id: I10f950939d34ff5917d168ef1d905c86e2cfc075

Add missing default true value for 'vector-simplesearch' preference

It was off and only enabled (unconditionally) by the Vector extension
which is being killed off per bug 45051. The preference was added in
r68177, moved to core in r68224.

Change-Id: Id5e96f5ed9dd532b33bc011104388a39f57277dd

Vector: Add navigation collapsing feature

* Moved from the Vector extension (removed there in I47950b5375cae38e2).
* Removed bucket testing
* Removed "new" version of collapsible nav
  (the language portal splitting feature) as it didn't seem to be active
  on any wikis. Can be revived later if considered useful still.
* Killed the vector-collapsiblenav preference
** Usage on Wikimedia wikis:
  - 0.008% on en.wikipedia
  - 0.02% on de.wikipedia

Bug: 46512
Change-Id: I87149d7e15931f02ab700164e9e1d3d707b5e6a5

mw.Title: Rewrite from scratch (porting logic from Title.php)

Changes:

* Add support for fragments.

* Use wgLegalTitleChars instead of the old clean() method
  that stripped out characters instead of throwing an exception.

* Implemented various other parts of Title.php to make it more
  strict like Title.php. It is still slightly looser, but it
  now takes care of the following that Title.php did already:
  - Directory patterns ("../" etc.)
  - Extra initial colons
  - Titles in NS_TALK that don't round-trip to NS_MAIN
  - 3 or more consecutive tildes
  - Limited title size (255 bytes)

* Extracted parsing logic into a private static #parse method
  and introduced mw.Title.newFromText (a constructor that returns
  null|Title instead of throwing an exception).

* Extended test suite to cover the added features and fixed bugs.

* Since the PHP test suite was lacking these, added them there
  as well.

Bug fixes:

* Fragments are now excluded from the title instead of causing
  the input to be invalid or malformed (e.g. "Foo#bar" was being
  normalised to "Foo_bar").

* ".com" now parses and round-trips properly. The extension and
  rest of title are still separated, but only at the very end
  after all other processing, so though title cannot be empty,
  since we only do a lazy split afterwards, it will split into
  title="", ext="com" internally and join back together when
  needed (bug 38081).

* "Example.js " (trailing space after extension) was previously
  incorrectly parsed as title=Example.js,ext=null.

* "Foo    bar" (multiple consecutive spaces) was transformed
  into 1 space correctly, but "Foo___bar" was not. This has been
  fixed to match the PHP implementation (it merges underscores
  and whitespace of any kind).

Clean up:

* Removed various redundant private helper methods.

* Removed fixNsId as getNsIdByName uses wgNamespaceIds which
  always yields a valid value. The fixNsId was verifying
  something that was already valid.

* Yoda conditional in Title.php, got rid of.

* Use newFromText in jquery.byteLimit.test. It was previously
  using a very basic invalid test (=== '') and no try-catch.
  Since we're getting more strict, typing 'User:' results in
  an invalid title, which should result in the same behaviour
  as it previously did for the lazy === '' check.

Bug: 38081
Change-Id: Ief1c11dabadc2f822065c40be91e04d655933e4f

Merge "Added comment about not using ignoreErrors()"

Added comment about not using ignoreErrors()

* Using this makes it easy to forget to restore the old value, which then
  complicates anything that inserts to the DB.
* Also removed mention of the flag from some places.

Change-Id: I838183222518e0d008dee47ec8df6b3f3dc578e9

Merge "Fix comment typos from 5c51cb96eaea76d22b9c48928044a942c4be2feb"

Fix comment typos from 5c51cb96eaea76d22b9c48928044a942c4be2feb

Change-Id: I56f37010bce8a325af65eab62e7add232df108f4

Localisation updates from translatewiki.net.

Change-Id: I901fdf6051c34c1b35cc024b6f8b36dd92f76aaf

Merge "SearchUpdate should read from master."