Merge "Fix Postgres support"

Merge "Skin: Don't use parser cache in getCachedNotice()"

Skin: Don't use parser cache in getCachedNotice()

Just use wfGetCache( CACHE_ANYTHING ) which should be sufficient for
most cases.

Change-Id: Ic97549c9649d0cc1938773b10e26f6e8f819c7fa

DB: Add join conditions to selectField, selectFieldValues, and insertSelect

selectField() and selectFieldValues() are trivial, they just need to
pass it through to select(). In fact, selectFieldValues() was already
doing it, just no one ever updated IDatabase.

insertSelect() is a little more work. nativeInsertSelect() was
originally written as largely a copy-paste of select() and has since
gotten well out of sync. Now that we have selectSQLText(), we should be
able to just use that. DatabasePostgres's implementation can wrap the
parent implementation instead of being another copy-paste, but
DatabaseOracle seems to still need to be special.

Change-Id: I0e6a9e6daa510639d3212641606047a5db96c500

ParserOptions: Fix handling of 'editsection'

The handling of the 'editsection' option prior to I7fb9ffca9 was
unusual: it was included in the cache key, but the getter didn't ever
flag it as "used". This was overlooked in I7fb9ffca9.

This fixes the handling to restore that behavior. It's no longer
considered to be a real parser option, so changing it won't make
isSafeToCache() fail while reading it won't flag it as 'used'.

But to keep Wikibase working (see T85252), if 'editsection' is supplied
in $forOptions optionsHash() will still include it in the hash so
whatever Wikibase is doing by forcing that doesn't break. The hash when
it is included is the same as was used in I7fb9ffca9 to reuse keys.

Once optionsHashPre30() is removed, Wikibase should be changed to use
some other method to fix T85252 so we can remove that hack from
optionsHash().

Change-Id: I77b5519c5a1122a1fafbfc523b77b2268c0efeb1

Make Titles with an unknown namespace ID refer to Special:Badtitle.

Without this patch, Title::getPrefixedText() would return ":Foo"
if the namespace was unknown, potentially creating a misleading
link to the main namespace. With this change, getPrefixedText()
will return something like "Special:Badtitle/NS12345:Foo".

Note that round trip behavior is broken either way.

Bug: T165149
Change-Id: I0d491a2b58ff45f207f83ee62ca6e7e6ffbf790a

Merge "Hide <style> tags from Tidy"

Merge "RCFilters: Send form info even for empty checkboxes"

Merge "LoadMonitor: Group some logging parameters"

Merge "API: Restore ability for dieStatus() to be passed a non-fatal Status"

RCFilters: Send form info even for empty checkboxes

Make sure the url params override previous ones in case they are
empty, make sure we still send them through the fieldset form with
an empty value, rather than not send them at all (and then they are
being re-merged into the existing URL and become essentially 'sticky')

Change-Id: I294dba17c5a5b792f5bf5064a76dedefa4d12557

LoadMonitor: Group some logging parameters

Change-Id: Iefd59db547d799a02edc9f733cadb7668176d4cd

Merge "Update magic words and special page aliases for Arabic and Egyptian Arabic"

Merge "ApiParse: Fetch content with parsetree"

Localisation updates from https://translatewiki.net.

Change-Id: I3aa05ab2b7d18f6d134e4b395e2a50526cba5630

ApiParse: Fetch content with parsetree

I0079e10a assumed callers were correctly passing $getContent to
getParsedContent(). Apparently they weren't.

Bug: T167826
Change-Id: I85cbdfb5d0580252c1df23a0f27c8052f1d65dc3

Hide <style> tags from Tidy

Some versions of html-tidy (e.g. the one currently in use on WMF wikis)
will try to move all <style> tags in the body into the head, effectively
removing them for our purposes. We need to avoid that for TemplateStyles.

Bug: T167349
Change-Id: I133776d16f366cad73ed30af0e5a665fdf9f5ed9

Update magic words and special page aliases for Arabic and Egyptian Arabic

It is requested by User:Meno25 to update magic words and special page aliases for Arabic and Egyptian Arabic languages.

Bug: T143880
Change-Id: I5701ab523aea6c9a933586c86a478c4dbcc8cf7a

API: Restore ability for dieStatus() to be passed a non-fatal Status

This ability was accidentally removed in Iae0e2ce3.

Bug: T167690
Change-Id: I79cc9f2f8e75c3cf4e0942d17faafa84955aa020

Make Special:PageData accept two-part subpage

Bug: T163923
Change-Id: I9fc55fd1eb956871fc5dd3e104c6786d380ef445

Merge "Disable addHelpLink when including special pages"

Merge "ApiParse: Clean up parsing code"

Merge "Sync up with Parsoid parserTests.txt"

Merge "RELEASE-NOTES: OutputPage::enableTOC was removed"

Localisation updates from https://translatewiki.net.

Change-Id: I11b382a06772ebb9f908ba543f2276a8cf6e1a8b

Sync up with Parsoid parserTests.txt

This now aligns with Parsoid commit 108eed81b3eb78b77ade5ba5daac71fb43fff6de

Change-Id: Ib2b31f008adaf16866de16ef963bc58d6cabb088

Merge "Delete CologneBlue messages"

Delete CologneBlue messages

They are moved to the CologneBlue repo.

Bug: T159921
Depends-On: Ide5c96b418f6abfb4244efa777c57ec2ffa9f087
Depends-On: Ib46d62e2464d7292482e8ea63ce187987104c59d
Depends-On: I522a52e548b88d0904f6890ddc7d0adb5d54eba2
Change-Id: Ib262d89e4485dfff2ef7ee9bbaa1152604626c4c

Merge "Allow CategoryPageView to set $mCategoryViewerClass"

Merge "Remove X-Content-Dimensions header"

Merge "Log when transactions affect many rows in TransactionProfiler"

Merge "Add $wgMaxJobDBWriteDuration setting for avoiding replication lag"

Merge "Allow skins to control style of mediawiki.hlist"

Remove X-Content-Dimensions header

Reverts 84e4d7508893, and parts of cdfe08439c3b and 4511f6fa9fa0.

Bug: T150741
Bug: T167034
Change-Id: I39cfcf2cb231b6dfef569968fba6f473da258916

Merge "Do not try to parse empty argument in getErrorsOrWarnings in OOUI"

Merge "RCFilters: Retain extra url params when comparing url equivalency"

RELEASE-NOTES: OutputPage::enableTOC was removed

Follows-up c7e00974c7eb1 (T130632).

Also announced on Wikitech:
<https://lists.wikimedia.org/pipermail/wikitech-l/2017-June/088301.html>

Change-Id: I9dde207d2314993b9e88e6bb000b0a11b37799ad

ApiParse: Clean up parsing code

Now that ParserOptions->isSafeToCache() exists, use it where necessary.
This also moves the use inside the makeParserOptions() method so other
callers can pick it up as well.

Then pass the flag as $forceParse into WikiPage::getParserOutput()
instead of duplicating the logic in several cases, and generally clean
up the logic in the module to let WikiPage decide when to use the cache
in more cases.

Change-Id: I0079e10a40997e4a3b59ac21ef6c92246a147736

Remove unused and unnecessary imports

Change-Id: I26e623a4e4ba965c07670369a90c8a95185ea1e4

Move HttpAccept* to libs

Bug: T163923
Change-Id: I4984f6b77843669950afeff82351827cd416a80d

Use "target" instead "title" as the param name in SpecialPageData

Using "title" confuses mediawiki and causes redirect loops

Bug: T163923
Change-Id: I29e0e6a61703cc04397205a3f7f38df5906622e1

RCFilters: Retain extra url params when comparing url equivalency

The extra params should also dictate whether the URL should change,
so we update parameters that are external to RCFilters' model, like
limit and from, etc.

Bug: T167551
Change-Id: I21e35e7871abbc57634231b6330cc1fad8299860

Merge "Start a very basic version of Special:PageData"

Merge "Move HttpAcceptNegotiator and HttpAcceptParser from Wikibase to core"

Do not try to parse empty argument in getErrorsOrWarnings in OOUI

Bug: T167644
Change-Id: I0dcb9bd51ad0cecbdab868fed056b4077d1c9fa8

Merge "SpecialNewimages: Do not add the module when the special page is included"

Disable addHelpLink when including special pages

The help link does not make sence on includes special pages and is not
shown. But the style module 'mediawiki.helplink' is still loaded.
This change disables the loading of the module 'mediawiki.helplink' for
included special pages.

Change-Id: I8115dc2e46f3498441c30caede4648d387d55d5d

Merge "CryptRand: only use random_bytes on php 7 and HHVM"

SpecialNewimages: Do not add the module when the special page is included

When the special page is included with
{{Special:NewImages}}
the module 'mediawiki.special.newFiles' throws an error.

This change loads the module 'mediawiki.special.newFiles' together with the
form.

Bug: T167601
Change-Id: I5a0589f62388292851664daf1ab5e63008556a64

Localisation updates from https://translatewiki.net.

Change-Id: Ia549296daf2a571be8bab81768a656bb12570116

Move closing quotation mark before punctuation in rcfilters* messages

Change-Id: I09fb50ee5e3fa9d34535b787a1bfd8aa365d880c

Localisation updates from https://translatewiki.net.

Change-Id: I1f0cffc9ef9fb378fd2bb024e406f84606ade576

Merge "objectcache: Use faster getArrayCopy() in WAN cache methods"

Merge "Make DeferredUpdates detect LBFactory transaction rounds"

objectcache: Use faster getArrayCopy() in WAN cache methods

Change-Id: I5f0543a59bdc59656c54fab4d7c4ae33fd233f9a

Make DeferredUpdates detect LBFactory transaction rounds

Previously, tryOpportunisticExecute() tried to nest transaction rounds,
which would fail. Added LBFactory::hasTransactionRound() as needed.

Also cleaned up some unqualified class names in callbacks and set the
PRESEND flag for the JobQueueDB AutoCommitUpdate callback. Use the
proper getMasterDB() method while at it. These follow up 24842cfac.

Bug: T154425
Change-Id: Ib1d38f68bd217903d1a7d46fb15b7d7d9620daa6

Merge "Remove id selector for toctitle"

Merge "Fix SqlBagOStuff exptime uniqueness assumption"

Fix SqlBagOStuff exptime uniqueness assumption

The WMF parser cache tables have some 30000 rows with an identical, old
expiry time. So each time deleteObjectsExpiringBefore() is run, the
first batch on each table is correct, but the keys for the second batch
are selected with exptime > X, where X is shared exptime of the many old
rows.

So, use exptime >= X instead.

Change-Id: I6853f64f88f65d4529be2a42c9ed70cfa62cf653

Start a very basic version of Special:PageData

Bug: T163923
Change-Id: I2a1a12f20a38d8d3c50a8f6c9a363be1cb656c70

Move HttpAcceptNegotiator and HttpAcceptParser from Wikibase to core

This will be needed for implementing Special:PageData

Bug: T163923
Change-Id: I2315d7dcdfa5973998917af311ebecc855b37f73

Localisation updates from https://translatewiki.net.

Change-Id: I1c435df7c4d1a6e081cbc708c20e55a62933c567

Merge "Remove unused key in WANObjectCache::newEmpty()"

Merge "ChangesList: Expose basic properties of lines as data attributes"

Merge "Add skipping to nth page option/ability for dump importing process"

ChangesList: Expose basic properties of lines as data attributes

We have several types of change lists (old RC/watchlist/related
changes, enhanced RC/watchlist, history) with slightly different
HTML, each with their own idiosyncracies. JavaScript code trying
to identify lines by log ID / revision ID has to jump through all
kinds of hoops to work with that.

To simplify the lives of frontend / gadget maintainers and provide
something approaching an API for these pages, we now expose the basic
attributes of each change line (revision ID for edits, log type/action
and ID for log events) as data attributes.

The OldChangesListRecentChangesLine, EnhancedChangesListModifyLineData,
EnhancedChangesListModifyBlockLineData, PageHistoryLine,
ContributionsLineEnding and DeletedContributionsLineEnding hooks
are updated accordingly. New hooks (LogEventsListLineEnding and
NewPagesLineEnding) are added for the change list pages which did
not yet have them.

Change-Id: I6dd006d0b1b0fd35c0020f0f9eea9113eca30b35

Add skipping to nth page option/ability for dump importing process

Usage: php importDump.php --skip-to 271500 /path_to/dumpfile.xml.gz

When importing a database dump and the import process crashes
(for random reasons) after a certain number of pages, the
"--skip-to" parameter allows restarting the import process at
a certain page instead of starting the import from scratch.

Change-Id: Ib36063b69d6846fc197800bba44287493b0632c0

Allow CategoryPageView to set $mCategoryViewerClass

Bug: T166483
Change-Id: I9937148dd8114e7167d6a581e64bfed5fdcc568e

Localisation updates from https://translatewiki.net.

Change-Id: I1c0504dc1347fd79bd0c24912bc9e42bb6b7e892

Merge "EditPage: Fix 'id' attributes on buttons in non-OOUI mode"

EditPage: Fix 'id' attributes on buttons in non-OOUI mode

Follow-up to ce2290e4817d78bbb400f1fec28cf4747eab8e1d.
I really did not think that change through.

Bug: T167390
Change-Id: I6d683cafe02137800925507d5e00663756002296

Merge "Document return string as JavaScript code"

Merge "Do not disable password reset for blocks meant to force login"

Merge "Fixing display issue with interwiki search sidebar"

Remove unused key in WANObjectCache::newEmpty()

Change-Id: I5d11736b95fbba1debcf80f80cbdb4753caf15f7

Document return string as JavaScript code

Change-Id: I47ce855cab444e139898f7269d63437b44825966

Merge "Properly check that the port key is present"

Do not disable password reset for blocks meant to force login

Also remove resetpassword right (killed in I3ab5962d) from tests.

Bug: T161860
Change-Id: Ic7e7e9b4ff7fe94001578a895962ef732b690384

Remove id selector for toctitle

In 1bf5a652 the id selector was changed to a class selector for toctitle.
The cached HTML has been expired now and the id selector is not necessary
anymore.

Also remove the id selector #toc.tochidden for print style. This is not
necessary because the tochidden gets only added to .toc and not to #toc.

Change-Id: I43cfffdb0807e8ed8f6b7b8732ba857b709bee80

Merge "Adding release note about counting page moves as edits"

Merge "Split off styles for PHP DateInputWidget to separate module"

Adding release note about counting page moves as edits

Bug: T163966
Change-Id: I084a1acb5af0b5226fb24c56434b36ecd0008eab

Localisation updates from https://translatewiki.net.

Change-Id: Idda25d2f73ce81e48f6763a8f4a7d5d4a7fb0540

Split off styles for PHP DateInputWidget to separate module

This lets us load them with addModuleStyles() (without generating a warning).

Bug: T167343
Change-Id: I88f0716c6a5ac702cc07e8015cfe25d81538a536

Merge "Improve test coverage in includes/password"

Properly check that the port key is present

Bug: T167301
Change-Id: I009c3d8e5805ddd2b75640ba582a648663c72b47

Fixing display issue with interwiki search sidebar

Bug: T167301
Change-Id: I9a24eca4df79b8d9e1ae3c440be2124724148c20

Improve test coverage in includes/password

From 21% to 82%.

* Added missing @covers, broadened @covers where appropriate.
* Added tests for some code that lacked them.
* Added a parameter to control the use of hash_pbkdf2() so that the pure
  PHP fallback could be tested. In the non-fallback test, force the use
  of the extension, and mark it skipped if it is not installed.

Bug: T167003
Change-Id: I987e1a89ec343907f4ead7f6192b2d4deb58ac16

Merge "Use AutoCommitUpdate instead of Database->onTransactionIdle"

Localisation updates from https://translatewiki.net.

Change-Id: Id393acd9be01ab3342d202931bc284e8437f5d9d

Merge "Fix action=feedcontributions date filtering parameters"

Merge "Rename all files used in ResourceLoaderImageModule tests to fake names"

Merge "Add appropriate OOjs UI icon pack dependencies for OOjs UI itself"

Fix action=feedcontributions date filtering parameters

* Convert year/month date filter parameters to use start/end so that
  ApiFeedContributions still works as expected after b668887
* Move SpecialContributions::processDateFilter (used to convert
  year/month parameters to start/end parameters) to ContribsPager
  since ApiFeedContributions also uses it now

Bug: T166859
Change-Id: I34fc8388a29e4cd36474934e6266127d0e3253cd

Rename all files used in ResourceLoaderImageModule tests to fake names

Using names that match real OOjs UI icons is confusing when trying to
find usages of the real icons and when debugging the tests.

Bug: T166730
Change-Id: Ibb97c0347476efc95f1a50c97822d7dab19737f2

Use AutoCommitUpdate instead of Database->onTransactionIdle

This is needed for deferred updates LinksDeletionUpdate and LinksUpdate, else
callbacks registered with onTransactionIdle prevent other transactions from
being executed, at least in this case.

Bug: T154425
Bug: T154438
Bug: T157679
Change-Id: Iecd396d584a62ac936cd963915339159467b44cd

Merge "Restore the ability to set $wgResourceModuleSkinStyles for 'oojs-ui-core.styles'"

Merge "Try harder to avoid parser cache pollution"

EditPage: Give edit summary field's 'tabindex' as a number

There's a bug in OOjs UI that causes the string value to be ignored
after the input is infused (e.g. by the mediawiki.action.edit script).

Bug: T167070
Change-Id: Ibeefc8327b462552c67459aac2c75cca9b1f4739

Restore the ability to set $wgResourceModuleSkinStyles for 'oojs-ui-core.styles'

In c8ad83310f425433791102e8a958397f2d701b65, 'oojs-ui-core.styles'
was changed to use ResourceLoaderOOUIFileModule instead of plain
ResourceLoaderFileModule. This had the unintended consequence that
ResourceLoader::isFileModule() no longer returned true for it, and
this $wgResourceModuleSkinStyles no longer worked, breaking styling
in Vector.

Additionally, the new ResourceLoaderOOUIFileModule did not respect
the skinScripts/skinStyles options passed in the module definition
(therefore also those from $wgResourceModuleSkinStyles). Merging
them turns out to be a major pain, but it can be done.

Bug: T167042
Change-Id: I7547bbe996467745c1f0b168b40f27eb95c6238d