Revert "Use display name in category page subheadings if provided"

This reverts commit 8ccde8984913896d59a3c2b529768cfe74100afd.

For the reason, see T43720#2531092

Bug: T149510
Change-Id: Id1ace9599642a36b333c63eaeebab0537466e7bd

Merge "installer: Update assets README to mention public-domain.png."

Merge "Make doAtomicSection() return the callback result"

installer: Update assets README to mention public-domain.png.

Follows-up be0b28d4. This way it'll be found when searching for the file.

Change-Id: I57fafa0b3e6a63a61f7c67e3279ee875eae2a344

Merge "ObjectFactoryTest: Add tests for 'factory' option"

Merge "phpunit: Add @covers to ObjectFactoryTest"

Make doAtomicSection() return the callback result

This makes the method a bit more convenient

Change-Id: Ic140e200cddcdf8e1a09b65c94d2da3aed62756d

Merge "Fix text extraction where we don't have proper file handler"

Merge "Remove pointless override of doStashFile in UploadFromStash"

Merge "resourceloader: Add structure unit test to confirm messages exist"

Merge "Make transaction enforcement stricter"

Remove pointless override of doStashFile in UploadFromStash

Change-Id: If6bc31d7398c68320a6633b045c33279bf6f8dbd

Fix text extraction where we don't have proper file handler

Bug: T143251
Change-Id: I611f6a001bbcea971cc9126bd3f004622e88b47d

resourceloader: Add structure unit test to confirm messages exist

Bug: T129976
Change-Id: I5333523efe3e7b9191e871d67d9eb650002c7784

Localisation updates from https://translatewiki.net.

Change-Id: I2ab6b5289ad6ef9afec11b3306a76e09ab9b5371

Merge "Run 'UploadStashFile' hook for chunked uploads too"

Merge "objectcache: Optimize changeTTL() for SqlBagOStuff"

objectcache: Optimize changeTTL() for SqlBagOStuff

Change-Id: I539dd83175a8ea922221f194e23ca47aea7539cf

Merge "Log failure reasons in ApiLogin"

Merge "ApiBase::getModuleSourceInfo(): Use $wgExtensionDirectory"

Merge "AuthManager: do not rewrite PRIMARY_REQUIRED to REQUIRED"

ObjectFactoryTest: Add tests for 'factory' option

Increase coverage of getObjectFromSpec():

* Case for 'factory' option.
* Case for 'class' option with no arguments.
* Case for missing 'factory' and 'class' options.

Change-Id: Idc9ee73de4f6e55372b4ab5b1afbaa4c7e54509a

phpunit: Add @covers to ObjectFactoryTest

* expandClosures() wasn't covered at all.
  Make the test dedicated to this feature cover that method.

* constructClassInstance() coverage was almost complete except for
  the exception case. This was covered in a separate test, but it
  was missing its @covers tag.

Change-Id: I73252849a92440fd7ae2c44f9ca58f7bf9c1f92c

Run 'UploadStashFile' hook for chunked uploads too

In UploadFromChunks, doStashFile() only stashes the current chunk,
and stashing the whole file after all chunks are uploaded is done
in concatenateChunks().

Also more custom code on top of custom code to handle ApiMessage
errors, because action=upload is special.

Follow-up to 2ee27d9a4de163dc3a2b752823a04f5338005627.

Change-Id: I968280353f15bbca9b1059631fa2cfd7c3cd2f1d

Merge "EditPage: Fix display of errors with multiple messages"

EditPage: Fix display of errors with multiple messages

If a status object returns failure with multiple messages (like in
Content::prepareSave()), then previously the first list item would have
a raw asterisk in front of it instead of being a list item because the
<div> tag was on the same line. Adding a newline before the status
object's wikitext will ensure that the first item is interpreted as part
of the list.

Change-Id: If42b3e745f1bd57c6e7964c8e297dab2bf742d4c

AuthManager: do not rewrite PRIMARY_REQUIRED to REQUIRED

AuthManager::getAuthenticationRequests() changes
AuthenticationRequest::$required from REQUIRED to PRIMARY_REQUIRED
if the request is from a primary; it made an exception when
all primary providers returned a given request. That exception is
not particularly useful (AuthenticationRequest::mergeFieldInfo()
used to rely on it to determine which fields are required, but
since I9d33bd2 that's not really needed), and knowing which request
is from a primary is useful for other means.

This changes required field semantics in a corner case: when a
primary provider returns two required requests, the previous
behavior was to assume that they are both required; the new one
is to treat them as alternatives (as if they were returned by
two different providers). So when all primary providers return
request X, and one of them returns Y in addition, the fields of X
will not be marked required, while previously that would have been
the case.

Instead of overcomplicating the interface for something that is
unlikely to come up in any real use case, add a new requirement
to PrimaryAuthenticationProvider that it should not return
multiple required requests.

Bug: T141471
Change-Id: I1c1f44d4d6b66f77c876e3459fb97f03483db744

Merge "MovePage: Fix old, old bug with moving over redirects"

Merge "Actually enable the DBPerformance log in the API"

Let cleanupCaps.php clean up when $wgCapitalLinks is set to true

This enables wikis to go from having $wgCapitalLinks = false to
$wgCapitalLinks = true (default).

Also:
* Declare $namespace member variable
* Fix regression introduced in ac53e45035e where non-dry run page move
was always considered a failure.
* Use MWNamespace::isCapitalized() instead of $wgCapitalLinks

Change-Id: I49caf8c1b49df1519d40b50d4aa26ea721db065d

ApiBase::getModuleSourceInfo(): Use $wgExtensionDirectory

This function takes the class file path, and iterates up through its
parent directories until it hits either a registered extension, or $IP
(for core) or $IP/extensions (for an unknown extension). This change
adds $wgExtensionDirectory as another iteration stopper, so that
unregistered extensions in $wgExtensionDirectory will not cause
iteration back up to the filesystem root, or to $IP if
$wgExtensionDirectory is a child of $IP other than $IP/extensions.

Change-Id: I06553591b0b6156e945aad5176479eb71517e77d

Document that wfParseUrl() can also return false

Change-Id: I6927ee13c35ecccc45c392b77d426380cc7a5c53

Merge "installer: Remove "public domain" option in favor of CC-0"

installer: Remove "public domain" option in favor of CC-0

Some people don't consider "public domain" to be a valid license
statement, so for new wikis it's better to avoid it entirely. CC-0 works
around all of those issues, so we should be encouraging that instead.

Also it's confusing to have two "public domain" options right next to
each other.

The asset needs to stay so existing wikis that picked "public domain"
don't break.

Bug: T122916
Change-Id: I3c380f79a4f27d4793b84602002f3db5c399ac32

ParserTest: Remove warning about gd extension not being installed

The tests actually pass just fine if gd isn't installed.

Change-Id: Ia37255fd642a8f741f1e1a547702495b3a83b584

StubObject: Allow using a factory function to construct the object

If $class is a callable, then assume it is a factory function, and pass
that onto ObjectFactory.

Change-Id: I20c3650843c2803e0f9521d67bb037abfc28e1d9

Make transaction enforcement stricter

* Nested begin()/commit() will no-op when inside active
  DBO_TRX transactions, even if no writes happened yet.
  This avoids snapshot loss from REPEATABLE-READ or SSI
  SERIALABLE (postgres). An error will be logged.
* Also no-op when begin() happens, no transaction started,
  but DBO_TRX is set. That should not happen as it can make
  multi-DB commits less robust and can clear snapshots early.
  An error will be logged.
* Throw an error when explicit rollback() is used and DBO_TRX
  is set. This will cause peer DBs to rollback too via the
  logic in MWExceptionHander.
* Callers should use LBFactory methods for doing commit/rollback
  instead.

Change-Id: I31b8b1b112cf9110b805a023732bce4dcff0604c

Merge "Do not call the 'UploadStashFile' hook for partially uploaded files"

Merge "Browser tests: update to mw-selenium 1.7.2 and https"

Merge "Hide marked empty elements by default (stage 2)"

Log failure reasons in ApiLogin

Bug: T142600
Change-Id: Id5af923ef10c31fc15e5d4da43ce220e1258b723

Do not call the 'UploadStashFile' hook for partially uploaded files

Consumers of this hook don't expect partial files, and even if they did,
they have no way to tell if they've been given a partial file.

Also document a pre-existing restriction that verifyUpload() must be
called first (for non-partial files). ApiUpload previously called this
function for partial files too, causing T143161.

Follow-up to 2ee27d9a4de163dc3a2b752823a04f5338005627.

Bug: T143161
Change-Id: I107cb957e046545ea72834d72233cc1ed2867e42

Merge "SkinTemplate: Move bottomScripts() back sightly"

Merge "OutputPage: Apply target and origin filter to exempt modules"

OutputPage: Apply target and origin filter to exempt modules

Follows-up 80e5b160e, which moved filter logic to getRlClient()
so that ResourceLoaderClientHtml doesn't have to deal with this.

While that worked fine for most modules (including 'site'),
it no longer applied the filter for exempt/hardcoded modules
(such as 'site.styles' and 'startup').

Bug: T143066
Change-Id: Iec924003873bc47484a0dc2f1a215f87aa4afdfb

Merge "Note that you shouldn't use a custom $salt for 'edit' or 'csrf'"

Merge "Add convenience commitAndWaitForReplication() method"

Merge "Always set DBO_DEFAULT by default for LBFactory classes for consistency"

Note that you shouldn't use a custom $salt for 'edit' or 'csrf'

Change-Id: I795b3a17791a9e16382a7c6379b0e7a4a498d32e

Add convenience commitAndWaitForReplication() method

This also does sanity checks to avoid breaking transactions

Change-Id: I7453c245eee25a26243e606970ef5f79b21a8141

Merge "Database transaction flushing cleanups"

Update OOjs UI to v0.17.8

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

Change-Id: I03793872885c70cabd1abd550b12d89e6e509e94

SkinTemplate: Move bottomScripts() back sightly

Follows-up 80e5b160e, 95fc8a96db.

In 80e5b160e, the module queue formatting was changed and was now fixed at
the first call to headElement() or bottomScripts(), instead of being kept
dynamic in between. Doing so fixed various bugs but also broke a few cases
where things still add to the bottom queue after the headElement() is
already created (such as sidebar).

In 95fc8a96db, the bottomScripts() method was moved to the very bottom of
prepareQuickTemplate(). This was imho too far and not needed because any line
after 'headElement()' element is effectively the same since that will commit
the queue.

It also changed the order of hook execution, from:
- SkinAfterBottomScripts, SkinTemplateOutputPageBeforeExec, SkinAfterContent
to:
- SkinTemplateOutputPageBeforeExec, SkinAfterContent, SkinAfterBottomScripts

With the latter no longer being able to add things to the module queue since
it now runs after headElement() commits the queue.

Move it back up, to right above headElement(). This way the order is the
same as before 95fc8a96db (first bottomScripts, then headElement). But still
after all other properties, thus still keeping the regression from 80e5b160e
fixed.

Bug: T87871
Change-Id: I8f774880e60e31c0e996b20acdc5ce5e42a84b01

Actually enable the DBPerformance log in the API

The logger injection call was missing

Change-Id: Ieffc81b1b8437e9765a3a208de7aa430b62806eb

Database transaction flushing cleanups

* Do not commit() inside masterPosWait(). This could happen
  inside JobRunner::commitMasterChanges, resulting in
  one DB committing while the others may or may not later commit.
* Migrate some commit() callers to commitMasterChanges().
* Removed unsafe upload class commit() which could break
  outer transactions.
* Also cleaned up the "flush" flag to make it harder to misuse.

Change-Id: I75193baaed979100c5338abe0c0899abba3444cb

Sqlite: DBError expects a database object as its first parameter

That constructor is dumb. It's also dumb to need a DB object
considering the only subclass using it is DBConnectionError
to getServerName()

Change-Id: I8e09989fd6635ccdffdba1102f55a9d3b7ba2f6e

Merge "Replace blunt uses of resetExpectations() with setSilenced() for TransactionProfiler"

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

Merge "Don't use SearchEngineConfig::searchableNamespaces in User::getDefaultOptions."

Localisation updates from https://translatewiki.net.

Change-Id: I0c460c30205eabb7ce58660dbad57cd1c40e27e7

Merge "Add a script to build an HHVM RepoAuthoritative bytecode file"

Merge "watch.js: Preload module 'mediawiki.notification' after exit paths"

Merge "Call parent::getFieldsForSearchIndex in ContentHandlers"

Merge "TextContent: Normalize newlines in preSaveTransform()"

Merge "GallerySlideshow: getImageInfo: Reject promise if there is no thumbnail"

TextContent: Normalize newlines in preSaveTransform()

This does the same normalization of newlines that
Parser::preSaveTransform() does. This should be appropriate for any text
content type, especially considering that EditPage uses
WebRequest::getText() which does a less-strict version of this same
transformation.

This also cleans up the code for doing that newline replacement
to be a bit less verbose.

Bug: T142805
Change-Id: I462afcda502f031a8b0360d982ce2398a0383a96

Move thumbnail rendering to a more appropriate spot

At the moment the job might start before the transaction
that creates the file's row in the DB has had a chance
to run.

Bug: T106740
Change-Id: If5b94e83d8bbcc6aebfe7193f7b580f03cbd627d

Merge "Make autoCreateUser() transaction profiler suppression limited in scope"

Merge "Fix DBLockManager comments"

Merge "Defer maybeUpgradeRow() post-send since they can trigger on non-POST"

watch.js: Preload module 'mediawiki.notification' after exit paths

mw.notify is not called on the exit paths. A preload is not necessary.

Change-Id: I2b14a269afd78649eda86a40b33e22412e645e31

GallerySlideshow: getImageInfo: Reject promise if there is no thumbnail

Bug: T142799
Change-Id: Iacca208fa39afd1e330db1d1fa9f3bfbc6b76b81

Add a script to build an HHVM RepoAuthoritative bytecode file

This script compiles a list of known MW sources, including core and
extensions, and generates a RepoAuthoritative bytecode file. Some ideas
were taken from Bryan Davis's scap-hhvm-compile shell script.

Also, renamed the old maintenance/hiphop directory to maintenance/hhvm,
respecting the rename of the upstream project.

Change-Id: I55798729d0553d2840e8099e81d604a814e8664c

Merge "Make various MWCallableUpdate usages rollback-safe"

Merge "Use waitForAll() for slow JobRunner commits"

Merge "Remove all assert() calls with string parameters"

Call parent::getFieldsForSearchIndex in ContentHandlers

ContentHandler implementations were not including fields
defined by their parent ContentHandler classes.

merge method is added to the SearchIndexFieldDefinition
mock in SearchEngineTest, to allow merges of fields
in the way that SearchIndexFieldDefition implementation does.

Change-Id: Id04a51528f566da2666bad0394a2f61c949c69b4

Remove all assert() calls with string parameters

These fail when HHVM is in RepoAuthoritative mode

Change-Id: Ifb1628f8269b2b651154b740b95cc14163a1b186

Upgrade PhpParser to 2.1 and update findDeprecated.php

The old PhpParser had dynamic class_alias() calls in its autoloader
which caused HHVM RepoAuthoritative compilation to fail with a fatal
error. These b/c aliases were removed in 2.0.

Change-Id: I6e2e3412204249a50a5806c33f48f426ab8c4511

Merge "This changes grammar, spelling and flow of the README"

Fix DBLockManager comments

Change-Id: I5431caf4b80995c71d4bc5ab739954d70081111a

Merge "MWUI: Vertically align the icon pseudo elements"

Make autoCreateUser() transaction profiler suppression limited in scope

Change-Id: If7b2330aa8cdf49ae25e5c0af5cb3ffc3f40b257

Merge "Clone WikiPage before delete and pass the cloned copy to ArticleDeleteComplete"

Merge "Fix display of action table cells on Special:Tags"

Clone WikiPage before delete and pass the cloned copy to ArticleDeleteComplete

This allows the hook to examine the state of the WikiPage instance before
the data is actually deleted from the page table.

E.g. The hook can now do $wikiPage->isRedirect(); and get correct information.

Bug: T134502
Change-Id: I715046dc8157047aff4d5bd03ea6b5a47aee58bb

Merge "Add grant for access to private information"

Localisation updates from https://translatewiki.net.

Change-Id: I52e0ec9021f8fcc0e7304ec42c88a9b59e60e5dc

Browser tests: update to mw-selenium 1.7.2 and https

Bug: T129483
Change-Id: I4526fb9033c0ea0b23249bea507d9cf8f7345afc

MWUI: Vertically align the icon pseudo elements

The icon element's image was not always in the vertical center depending
on the line height of the icon.

Bug: T142391
Change-Id: I4d73c27286a623cb82e2f8801c919ae6f8007e59

Replace blunt uses of resetExpectations() with setSilenced() for TransactionProfiler

Change-Id: Ica42f56f61cd720a16754e93dc9c4792d66c7a0f

HTMLFormFieldCloner: Pass enpty array, not null, to avoid warning

Bug: T142912
Change-Id: I18edfcb62f7b21f2c1990c73944ee6a956fd4901

Always set DBO_DEFAULT by default for LBFactory classes for consistency

Before, only the non $wgDBservers case did this in LBFactorySimple

Change-Id: If1a962663d5e23b2087873f86efe40a1166f96a7

Merge "Set Czech (cs) and Slovak (sk) as reciprocal fallbacks"

Set Czech (cs) and Slovak (sk) as reciprocal fallbacks

The two languages are mutually intelligible at a very high degree and
most speakers can be considered bilinguals or passive bilinguals.

Consulted resources:
* Eurostat special eurobarometer 386 (2012).
  http://ec.europa.eu/public_opinion/archives/ebs/ebs_386_en.pdf
* Mira Nábělková
** http://doai.io/10.1515/IJSL.2007.004: «These differences do not prevent the users of Slovak and
   Czech from understanding discourse in the other language [...]. While contact with Czech texts
   in the Slovak environment is obvious, it does not appear to be so in the opposite case.»
** High degree of passive bilingualism even in recent generations who don't feel it much.
   https://repozytorium.amu.edu.pl/bitstream/10593/9168/1/02_Viera_K0VA%C4%860VA_Czech_Slovak_intercultural_265_280.pdf
* Discussions with several translators.
** https://translatewiki.net/wiki/Thread:Portal_talk:Cs/Fallback_to_sk
** https://translatewiki.net/wiki/Thread:Portal_talk:Sk/Fallback_to_cs

Change-Id: Id85b281a3ef2f2ae00ee0aba827e94e4f5a1f924

Merge "objectcache: expand getWithSetCallback() docs a bit"

objectcache: expand getWithSetCallback() docs a bit

Change-Id: Ifc8304848fd5e4b8f7ea5555271de09609cdbe10

Make various MWCallableUpdate usages rollback-safe

Support for this added to MWCallableUpdate and its helper method
in DeferredUpdates.

Bug: T141988
Change-Id: Ief150ea655036e2708557dd8ae847379edf21933

Merge "Move bottomScripts() call in SkinTemplate"

Move bottomScripts() call in SkinTemplate

SkinTemplateNavigation::SpecialPage in buildContentNavigationUrls() and
possibly other hooks are susceptible to adding bottom scripts to
the skin's OutputPage.

It's safer to add the bottom scripts to the template
as late as we can, as doing so will instantiate the rlClient and
populate it with the scripts collected at that point.

Bug: T87871
Change-Id: I000365a7542167d8531d6984d3a1ef6c362034ab