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

New changes:
205fa98 [BREAKING CHANGE] Window refactor, introduction of Processes

Change-Id: Ia26243f74ecad4931cf455284a037216a8393348

Removed clumsy unused IP::toUnsigned() method

* Also avoid calling ip2long() on invalid addresses in toHex()
* Also use native base_convert() in toHex() for IPv4

Change-Id: I4fe4e1ac3c91edb9812e4c5bd173e4b54e315934

Update OOjs UI to v0.1.0-pre (0f101c6f5d)

New changes:
e86dae9 [BREAKING CHANGE] Emit a single event from confirmation dialog

Change-Id: I305224cad8e7bcb7e48dfc31faafeab8ed8a376c

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

Merge "Add category output to ApiExpandTemplates"

Update OOjs UI to v0.1.0-pre (469d40c88f)

New changes:
9af9116 Ignore SelectWidget mouseup events that started outside the widget
28a7ca8 Localisation updates from https://translatewiki.net.
469d40c Localisation updates from https://translatewiki.net.

Change-Id: Ife58c324e39e2ee1b6297e5803fb8dd69445ffab

Merge "Don't use isset() to check for null"

Merge "Allow adding Deleted log entries"

Save advanced search namespace prefs on Special:Search itself

* Checkbox on own row below power search checkboxes per MatmaRex;
  avoiding a mw-search-ns* id leaves it untouched by All/None JS.
* The option searcheverything is removed: a "shortcut" which is no
  longer necessary now that options can be (un)selected at once
  with All/None buttons on search page itself.
* Require a token for saving: no accidental preferences changes.
* Keep the searchoptions/advancedsearchoptions prefs section in case
  something is using it (no known extension does though); options
  are converted to "api" type so it's empty and hidden by default.
* Add minimal documentation for saveSettings() and friends
  (@todo since 155ddf6de, 2009!).

Bug: 52817
Change-Id: I514cee835988600cc013658049e88a10b670e64a

Merge "Fixed fatal in SwiftFileBackend when a bogus HTTP status is given"

Fixed fatal in SwiftFileBackend when a bogus HTTP status is given

* Error was "Fatal error: Call to a member function getSize() on a non-object"

Change-Id: Id1a1b1add9daee4402b2ed017b95c9a9f0e08170

Localisation updates from https://translatewiki.net.

Change-Id: I2ac98863fc1326e9eae37272a19a7174d80a8822

Add category output to ApiExpandTemplates

The preprocessor sometimes adds pages to categories itself, which don't
have any representation in the output of ApiExpandTemplates. Because of
this, add a new field containing these categories.

Change-Id: I7ae2eb9daf68e032dcc7a650bd8391d445ee1037

Add PPFrame::isVolatile and PPFrame::setVolatile

Most wikitext is safe to parse once and then cache for when that same
wikitext is used again, such as for multiple transclusions of the same
template within a page. There are occasions, though, where some piece of
wikitext has side effects and so should not be cached; a prominent
example of such wikitext is the <ref> and <references> tags in Cite.php.

This change adds PPFrame::setVolatile so parser hooks such as <ref> and
<references> can indicate that they have done something that should not
be cached, and PPFrame::isVolatile so that callers of PPFrame::expand
can know when to avoid caching.

Bug: 46815
Bug: 31834
Change-Id: I95b3cf8781cf047cdb63da221cef45f3e7d1632e

Merge "Reduce some code duplication"

Release notes for Ie32cc54a

Change-Id: I93d2dce7705b8855819ba0a816b2dc39fdeb3e38

Merge "PostgreSQL: Only rollback when in a transaction"

Merge "Fix typo"

PostgreSQL: Only rollback when in a transaction

SQL errors would issue a rollback, even when it was
not actually in a transactions.  Doing so would cause
another error to be reported.  This could obscure the
original error, particularly during unit tests.

Fix this by not rolling back when not in a transaction.

Bug: 58095
Change-Id: Ib5220e37dd6c364feee6b7f8e7ecbdae2a2e0ba1

mediawiki.special.unwatchedPages: Use closest() instead of parents()

Follows-up Ie32cc54abb6f.

While it is unlikely that this item will ever end up being nested
inside another list, it is semantically incorrect and also
quite a bit slower to use parents() here.

parents() collects all ancestors between this node and the <html>
document element (each and every one of the elements in between),
and then, if a selector (e.g. "li") is supplied, it reduces that
set to only <li> elements. This can be more than one if your list
is a nested list (whether a plain sublist, or perhaps many levels
apart, e.g. part of the skin layout or whatever).

This one is slower (traverses all the way up) and creates two
collections (all ancestors, then filtered down).

closest(), as the name implies, always requires a selector and
traverses only up to the first match and then returns.

This one is faster (traverses only once and not all the way).

A 1:1 (closest) vs. a 1:many (parents) relationship.

Change-Id: I1cd9a4638285aeab4b5d538072e598601eb475b6

Merge "PostgreSQL: Support table prefixes"

Merge "Restrict empty-frame cache entries to their parent"

Restrict empty-frame cache entries to their parent

Remove the parser's global $mTplExpandCache, and replace it with an
alternative that is separated by parent frame. This allows the integrity
of the empty-frame expansion cache to be maintained while also allowing
parent frame access.

A page with 3 copies of
http://ja.wikipedia.org/wiki/%E4%B8%AD%E5%A4%AE%E7%B7%9A_(%E9%9F%93%E5%9B%BD)
has the following statistics: Without this change, there are 4625 cache hits
on this page, and a sample of 3 parses took 16.6, 16.9, and 16.8 seconds.
With this change, there are 2588 cache hits, and a sample of 3 parses took
16.7, 16.7, and 17.0 seconds.

Change-Id: I621e9075e0f136ac188a4d2f53418b7cc957408d

Merge "jquery.suggestions: Handle CSS ellipsis better for IE"

PostgreSQL: Support table prefixes

PostgreSQL is usually not used with table name prefixes,
and some of the PostgreSQL-specific code took the
shortcut of not dealing with prefixes.  However, the PHPUnit
tests do force a prefix to be used, exposing this limitation.

Make PostgreSQL code add the prefix to table names being used
in FOR UPDATE OF clause.

This fixes 13 of the 17 PostgreSQL unit tests errors against
git HEAD:

EditPageTest::testAutoMerge with data set #1 ('Elmo', 'one
EditPageTest::testAutoMerge with data set #2 ('Elmo', 'Intro
EditPageTest::testAutoMerge with data set #3 ('Adam', 'one
EditPageTest::testAutoMerge with data set #4 ('Adam', 'one
EditPageTest::testAutoMerge with data set #5 ('Adam', 'Intro
EditPageTest::testAutoMerge with data set #6 ('Berta', 'one
EditPageTest::testAutoMerge with data set #7 ('Berta', 'one
EditPageTest::testAutoMerge with data set #8 ('Berta', 'Intro
WikiPageTest::testDoEdit
WikiPageTest::testDoEditContent
WikiPageTest::testDoRollback
WikiPageTest::testDoRollbackFailureSameContent
WikiPageTest::testExists

The change also applies to 1.22.6 and fixes several unit tests
errors there.

Change-Id: I054690f49e250cadd2dc2a6d9e20dea879d896b6

Localisation updates from https://translatewiki.net.

Change-Id: I4b2468b880c571df42a99f1a61bc452fc7c66567

Hide toc caption on print when collapsed

Increase specificity of the CSS selector to re-enable this function.

Bug: 482
Change-Id: I0157884c30e1a0e5624b65aa06b2c5db666dc064

Merge "SECURITY: Don't parse usernames as wikitext"

Merge "Add hash_equals() fallback and use it"

SECURITY: Don't parse usernames as wikitext

On Special:PasswordReset, don't parse the username as wikitext since
the wikitext is parsed according to the wiki's configuration (might
include wgRawHtml), and the wiki may be private.

Bug: 65501
Change-Id: Ic3e5d42e1be5acc42ba89ae853c5ecbfec04fa91

Merge "Allow media handlers to mark files as expensive"

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

Merge "Add border and background styling to the Parsoid styles for images"

Update OOjs UI to v0.1.0-pre (09b223d279)

New changes:
09b223d StackLayout: Clean up inconsistent event firing and documentation

Change-Id: I1297880400f776bcf73753ef31ae613f0fcb0ad4

Add border and background styling to the Parsoid styles for images

Without these, framed images aren't actually framed in the Parsoid HTML.

Change-Id: Ie19f1c9b4652562aef1a92de8c52c484c4407c95

Add hash_equals() fallback and use it

Two classes (User and SpecialRunJobs) currently contain string
equality checks that purport to be timing-attack resistant.

Reduce code duplication by adding and using a fallback for the
hash_equals() function from PHP 5.6 (currently in beta), in a way
addressing the comment "@todo: make a common method for this".

Change-Id: Iece006ec0216edb3fc5fbef7cc6ec00a6d182775

Merge "Reorganize commonPrint.css"

Reorganize commonPrint.css

- Group related rulesets together
- Harmonize comment headers for group rulesets
  and make them more visible for improved navigation

Change-Id: Ie9928e8fe566c3907fcc2b2790f2e0a15ffd0436

Merge "Add SelfLinkBeginHook"

jquery.suggestions: Handle CSS ellipsis better for IE

IE is not impressed by our puny hacks and still reports the width
"in context of" the position in the document, limited by the width
of ancestor elements.

Let's temporarily apply position: absolute; to the involved elements.
This pulls them out of normal document flow and lets us figure out the
real width at last.

Also wrote a proper comment on why we need this stuff.

Verified that this fixes:
* IE 8
* IE 11

Verified that it doesn't break:
* Firefox 3.6
* Firefox 29
* Opera 12
* Opera 22

It *does not* fix IE 6. I don't think that investigating why is a good
use of my time, so I didn't. I84fbae5a made the functionality usable
on IE 6, which feels good enough for me.

Bug: 65224
Change-Id: I4a7357543ca244585ade2061b92f5a6d1e439278

Merge "Skin: Remove dried up remains of Standard/Classic skin"

Merge "mediawiki.searchSuggest: Show full article title as a tooltip for each suggestion"

Allow media handlers to mark files as expensive

Allows media handlers to signal that thumbnail generation
for this file is expensive and should be throttled more agressively.
For now this is only done for large TIFF files.

Bug: 65691
Change-Id: I01b34a1d46745649f179fdee435a8cfb19c5474e

Merge "Adding header font change to release notes for 1.24"

Skin: Remove dried up remains of Standard/Classic skin

Change-Id: I70a35cbe0d17bf8d35bbe04b1254ee0d6ce0accd

Localisation updates from https://translatewiki.net.

Change-Id: I4d2d181fa0ccda0dd781008d4ed0e68103de451a

Merge "Correctly parse 'redirect' XML tag during Special:Import."

vector: Add tabindex back to search box

Another regression, this time in Iaea60834. This brings back the
implementation as it was originally before I220057c799bef7e removed it as
part of the collapsibleNav module.

Bug: 29199
Change-Id: I3f29c3d80e43b9df16b397fff3dc983dc60652a7

Added BeforeHttpsRedirect hook allowing extensions to cancel forceHTTPS

For ZERO, we need to be able to avoid forceHTTPS from taking place in
case user logged in while on wifi, and later switched to
zero.wikipedia.org for the carrier who doesn't support HTTPS.
In that case, the extension will need to be able to cancel redirect
and force regular HTTP handling.

Open Question: should this code log off user in case hook returns false?

Bug 65567

Change-Id: If04c83066c5d47b3c04ad7674e3c4e95a4cd464b

Add release notes for Ie3f7273c

Change-Id: Id39c4d779d138493a2a7a1db2cc4388b125e5bb5

Allow filtering log entries by namespace (API)

Add parameter lenamespace to the API, allowing filtering of log entries by
namespace.

Change-Id: I53c4c6411e0b9e6383969afced0e4c193f1b64a1

Correctly parse 'redirect' XML tag during Special:Import.

Bug: 65481
Change-Id: Id9b3b7878b2e7b6fc7a06b163e5bac60e700490e

Tweaked HTMLCacheUpdate immediate run treshold

Change-Id: I123c27862821e01fe232f55d984905fcda2a2498

Reduce Title::invalidateCache contention a bit

* Lots of deadlocks seems to happen on page deletion,
  where the ID is likely 0 (this also has the worst locks)

bug: 37519
Change-Id: I028783a998946615b93e1425c6f69e00afd7b1bb

Add Composer managed libraries

This is the initial step towards implementing the Structured logging RFC
[0].

The Psr\Log library is declared as a requirement in the composer.json
file and Monolog is suggested as a package that can enhance MediaWiki.

To use this in a local checkout:
* Install composer [1]
* `composer update`
* `composer require monolog/monolog 1.9.*`
* `composer dump-autoload --optimize`
* `git update-index --assume-unchanged composer.json`

Additional work will be needed to provide both the Psr\Log and Monolog
libraries available for deployment to the Wikimedia Foundation servers.
The Psr\Log library will also need to be packaged in the release
tarballs.

[0]: https://www.mediawiki.org/wiki/Requests_for_comment/Structured_logging
[1]: https://getcomposer.org/doc/00-intro.md#installation-nix

See also: I1431b24 (Monolithic implementation)

Change-Id: Ie667944416187cfd2ae6016c9e2fa28f4204bcd7

Merge "Add missing possible errors to ApiRollback.php"

Add SelfLinkBeginHook

Called before a link to the current article is displayed to allow the display
of the link to be customized.

Bug: 63592
Change-Id: I343e1b1b08ec841d22a1b22bcb8af43bb65a5a50

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

New changes:
71d9671 Add more Agora dialogs styling
457a0ef Localisation updates from https://translatewiki.net.
c92ba96 Compute directionality for frame correctly
f3414b8 Localisation updates from https://translatewiki.net.
90fbb6f Localisation updates from https://translatewiki.net.
1d4d52a Localisation updates from https://translatewiki.net.
08d1c7b Add ability to put a header at the top of a toolGroup pop-up

Change-Id: I6f4adbca80abeec0931380233e0e45a9f45af330

Localisation updates from https://translatewiki.net.

Change-Id: I1ac7593bbc66a78e74a62ea03bab4fd105728a28

Merge "Remove Vector from the default LESS import path"

Remove Vector from the default LESS import path

When Ia193571a added Vector to core for VectorBeta's benefit it introduced
a regression that broke all skins that used imports like 'variables.less'
which happen to match a filename used by Vector.

These paths aren't supposed to be exposed anyway, and VectorBeta has been
fixed to no longer rely on this.

Bug: 64595
Change-Id: Iee47bdc23630e02ccfcbd28496ec5268892eb629

Merge "Enable mediawiki.skinning.content.parsoid on Mobile target too"

Merge "Remove "Help/Project" profile from Special:Search"

Merge "Remove silly 'm' prefix from StubObject/DeprecatedGlobal members"

Merge "Move Modern and CologneBlue out of core"

Enable mediawiki.skinning.content.parsoid on Mobile target too

Bug: 65823
Change-Id: Iee2e3d00576556418c666d393e8935f75801326c

Merge "Introduce ContentHandler::exportTransform()"

Move Modern and CologneBlue out of core

Import in the new repositories:

- Modern: I5348fedddf14affcee2688833f4e759e5af6499c
- CologneBlue: I9cc051f48074a033c684e5efa56a7b4dde51d366

Change-Id: I65768b4c29faf46895a7005a3090f43cf6e86258

Merge "Introduce ContentHandler::importTransform."

Merge "Cleaned up database reconnection logic"

Merge "Installer: output css correctly when session errors occur"

Merge "Use language from context in Skin::subPageSubtitle"

Merge "Add ar_text to the list from Revision::selectArchiveFields(). It is checked later."

Merge "Fix docs on File::getImageSize() they were very misleading"

Merge "Revert more visibility changes in HistoryBlob.php"

Merge "Don't use isset() to check for null"

Add ar_text to the list from Revision::selectArchiveFields(). It is checked later.

See Revision::newFromArchiveRow().  This is called from
populateRevisionLength.php's upgradeRow() which ultimately populates
$row from the fields returned by Revision::selectArchiveFields().

Revision::newFromArchiveRow() has the following condition:

if ( isset( $row->ar_text ) && !$row->ar_text_id ) {

And from the bug report:

> for old archived revisions, the rows in the archive table do _not_
> contain a pointer to the text in the text table, but that the text
> is stored in the archive table directly. Obviously the above code
> cannot handle this situation properly.

Credit goes to Joerg who found this and figured out the problem.

Bug: 65765
Change-Id: Iadcaae5e531d439780f5b6095286df78926ccaa7

Merge "Change one Linker::formatSize call to Message::sizeParams"

Merge "DBConnectionError: Expand {{SITENAME}} in pagetitle with Message::text()"

Merge "Restructure Media related tests to avoid duplicated code"

Restructure Media related tests to avoid duplicated code

Change-Id: I4428d38e0199b12255f3746e79f8093259a03da1

Merge "Fix doc comments on get[General](Short|Long)Desc"

Merge "SpecialRevisiondelete: Simplify if/elseif logic"

SpecialRevisiondelete: Simplify if/elseif logic

This avoids checking $comment twice
by Krinkle on I3b21f66f84d68777a18e66e4e5f59886e55d7936

Change-Id: I39df27d3edb097024d136455145505429ccdbe39

Merge "DerivativeContext::setConfig should take a Config object"

Merge "Allow more flexibility in RC feeds"

DerivativeContext::setConfig should take a Config object

Change-Id: Icc16a9c206c57fac2d26547cb874e02ef09c0485
Follows-Up: I13baec0b6d4ea7badf20b9c5f9b40846348838e4

Allow more flexibility in RC feeds

The user can now specify which feeds to send to in
RecentChange::notifyFeeds if they don't want to use
$wgRCFeeds.

Additionally, the 'formatter' in $wgRCFeeds can now be an object
rather than just a class name.

Change-Id: Ibfdffc17a934e35223887c123331795563102752

Merge "Make abstract Config class truly implementation-agnostic"

Localisation updates from https://translatewiki.net.

Change-Id: Idab23e6c2f532b1aae67dc15e9138f6602c15a0f

Use language from context in Skin::subPageSubtitle

Change-Id: Ia6cff8aa9e0b12215839f8b9c5a64a09150d4002

Change one Linker::formatSize call to Message::sizeParams

Change-Id: I2a91f0ef1cf82360da2d17c7dc42633ae3e79100

Reduce some code duplication

MediaHandler::getLongDesc and getGeneralLongDesc have same
implementation, so forward one method to the other. Same for
short.

Also get rid of one unnecessary $wgLang global usage.

Change-Id: I688e27ce2b9cf950defa16ec6bce2e52c5959b03

DBConnectionError: Expand {{SITENAME}} in pagetitle with Message::text()

Follows-up 485ff9ead94b by making the same change in DBConnectionError,
which overrides MWException::msg() to not use database messages.

Bug: 58447
Change-Id: Iafa9dd741f1f099a4a6cfb8cf655b7d5f504011a

Make abstract Config class truly implementation-agnostic

Follow up to I13baec0b6 ("Config: Add Config and GlobalConfig classes"):

Config:
* Rather than returning Status objects, Config::set will now throw an exception
  if an error is encountered
* Config::factory was moved into it's own ConfigFactory class.
* Since there are no more functions in it, Config was turned into an interface.

GlobalConfig:
* Remove $prefix args from Config::set and ::get. The idea of having an
  abstract Config class is to abstract some notion of configuration data from
  the particular way in which it is currently implemented (global variables).
  So the abstract base class has no business dealing with variable name
  prefixes.
** Instead GlobalVarConfig's implementations of get and set call getWithPrefix
   and setWithPrefix internally, which are now protected
* Rename GlobalConfig to GlobalVarConfig, which makes it clearer that it isn't
  referring to the scope of the configuration value, but to the scope of the
  variable name which provides it.

ConfigFactory:
* ConfigFactory is where Config objects are registered, and later constructed.
* Config objects are registered with a given name, and a callback factory function.
  This allows for implementations to construct the object with the parameters they want,
  and avoids the overhead of needing an entire class.
** The name 'main' is the default object returned by RequestContext::getConfig(),
   and is intended to be used by core.
* This is a singleton class, the main instance can be obtained with:
  ConfigFactory::getDefaultInstance()

In addition to the above:
* $wgConfigClass was removed, and $wgConfigRegistry was introduced, which
  stores a name => callback. The name is to be what the Config instance is
  registered with, and the callback should return an implementation of Config.
* Tests were written for the new ConfigFactory, and GlobalVarConfig's tests
  were improved.

Co-Authored-By: Ori Livneh <ori@wikimedia.org>
Co-Authored-By: Chad Horohoe <chadh@wikimedia.org>
Co-Authored-By: Mattflaschen <mflaschen@wikimedia.org>
Co-Authored-By: Parent5446 <tylerromeo@gmail.com>
Co-Authored-By: Reedy <reedy@wikimedia.org>
Co-Authored-By: Daniel Kinzler <daniel.kinzler@wikimedia.de>
Change-Id: I5a5857fcfa07598ba4ce9ae5bbb4ce54a567d31e

Merge "Return value directly when using getter"

Merge "Revert "Pass phpcs-strict on includes/revisiondelete/""

Revert more visibility changes in HistoryBlob.php

This follows-up bebcf2c58381, which only fixed DiffHistoryBlob.

MediaWiki will fail to load some revisions if the following properties
do not unserialize correctly:

* HistoryBlobStub::$mOldId
* HistoryBlobStub::$mHash
* HistoryBlobCurStub::$mCurId

Visibility of HistoryBlobStub::$mRef is also restored; though the
property's value does not appear to be used after unserialization,
the property is nevertheless a serialized property.

Bug: 65665
Change-Id: I2724d15f9206d66d615d020117e60bb772048c19

Localisation updates from https://translatewiki.net.

Change-Id: I5fcddad181c862b8f825611aa2aa3a17864e1b28

HTMLForm: fix double escaping of checkbox labels in vform

mLabel has already been escaped if required. It isn't run through htmlspecialchars()
in other display formats, so doesn't need to be done in vform.

Change-Id: I0b0feec408687cdadf01da5f2078980015538bb7