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

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)"

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

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

Localisation updates from https://translatewiki.net.

Change-Id: Icdfeedc8ec01bee22e8de12e53b360a2440901cc

HTMLForm: Use ObjectFactory instead of Reflection

ObjectFactory has a hack that constructs an object directly if there are
less than 10 parameters, which there typically is for HTMLForm classes.
This is faster than using ReflectionClass, and whenever ObjectFactory is
updated to take advantage of the splat operator, this will automatically
use it as well.

And use ::class while we're at it.

Change-Id: I7a696c127c237713448b165b9b4faee13f4ff88e

Localisation updates from https://translatewiki.net.

Change-Id: Id0fb9fda29209c1418d8bbae4f649750f5bf21dd

Merge "content: Use "::class" when overriding TextContent::getContentClass()"

content: Use "::class" when overriding TextContent::getContentClass()

Change-Id: Iea03d2cd24fdb90253145a8abfefe9f8a09e46cd

Use waitForAll() for slow JobRunner commits

Using waitForOne() barely goes beyond semi-sync replication
already in place on serious DB clusters.

Change-Id: Idb719deaa5993bc2f818cd110d49d09567e0afb3

Clean up array() syntax in docs, part III

Also fixing some typos here and there

Change-Id: I29c29acf87f84ba9993ba75ebf2ad8091d981574

Merge "DefaultSettings: Clarify enotif documentation"

DefaultSettings: Clarify enotif documentation

Change-Id: I4cc5ff94c7000b4c2ed584e49a590065f8d5724c

Merge "Use cache TTL constant in InfoAction"

Localisation updates from https://translatewiki.net.

Change-Id: I8b959312b1d6bbf49bb795ca40108dfaa298877f

Use cache TTL constant in InfoAction

Change-Id: If9924863500af612eaaab1c8cd5c655466efdc97

Merge "Better handle already-used rev_id when restoring"

Add GENDER to messages on Special:(Deleted)Contributions

Bug: T113346
Bug: T130960
Change-Id: I3452a6e1fc7b8604442127f94b68a613e6df33ab

Better handle already-used rev_id when restoring

Leave it in the archive table, rather than the data being gone
from both the archive and revision table.

Bug: T135852
Change-Id: I202cea7910be82f1073b4365209e3b320e85d62e

Merge "objectcache: Add changeTTL() method"

objectcache: Add changeTTL() method

This can change the TTL without fetching the object
so that keys can be renewed or set to expire soon.

Change-Id: Id1c2c9f89b3445cfa34263057dc5029cbe170833

Merge "Deprecate various ResourceLoader modules"

Merge "AuthManager: Always whitelist some special pages"

Merge "Revert "Move CSS for patrol from mediawiki.legacy to new module mediawiki.page.patrol""

Revert "Move CSS for patrol from mediawiki.legacy to new module mediawiki.page.patrol"

This reverts commit e9f896d6a5c09838b92bcb871c306b25526c3f96.

Bug: T142750
Change-Id: Idb815d64d795d790ea2c07f2b5188ea5a46bfefb

SkinTemplateTest typofix

There's no "t" in "make"

Change-Id: I41dc25162262bddec94f3af35db86a246ccd7499

Localisation updates from https://translatewiki.net.

Change-Id: I6bf5f6c5b9030b6af77ff7ab879baaba3e2d5832

Merge "Rename 'authmanager' log channel to 'authevents'"

AuthManager: Always whitelist some special pages

Special:UserLogout should be whitelisted, if Special:UserLogin is whitelisted,
as otherwise the user is able to login, but not able to logout anymore (given,
that the sys-admin hasn't change the $wgWhitelistRead config variable).

Special:ChangePassword redirects to Special:ChangeCredentials. Instead of
adding this special page, it should be done explicitly in the
wiki configuration (LocalSettings.php). For this reason, the whitelist is
removed, too.

Change-Id: I271efb517bf0ec0022e6fe29673570ca08e8fe1a

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

Change-Id: I791e7133d49ed9cd6dd40bb3fa35ea38d1ceba10

This patch adds Localisation to the links, and changes wording around.

Change-Id: I3c34e7491da81ac034d0b7b3f0517f1a8153b249

Merge "resourceloader: Move queue formatting out of OutputPage"

Merge "Fix link to Help:Contents in UPGRADE file"

Merge "Add SpecialPage::getTitleValueFor()"

Make Database disconnect and error suppression more robust

* Disallow $ignoreErrors in query() on deadlocks, since that would otherwise
  silently rollback all changes from any other callers.
* Move recoverability checks for disconnects to canRecoverFromDisconnect().
* The first write of a DBO_TRX transaction is now considered recoverable.
* Run onTransactionResolution() callbacks on disconnect/deadlock rollback.
  Some DeferrableUpdate need this to know to abort.
* Disallow $ignoreErrors on disconnects considered unrecoverable. This
  makes it so that query() callers cannot cause writes from other callers
  to be silently lost, which is hard to reason about.
* Moved ping() logic to simple reconnect() method and ping() simply do
  a dummy SELECT, which triggeres reconnection if safe. Previously,
  ping() might cause subtle partial transaction loss.
* Remove ping() from strencode(), which would cause partial transaction
  loss where it actually reached.
* Remove mysqlPing() per https://bugs.php.net/bug.php?id=52561.

Bug: T142079
Change-Id: Ifb7f772ae849d67c0d92240a115c3f392e252937

Upgrade unmatched explicit commit() of DBO_TRX to an exception

Follows-up f8d5075656 and 820f5d6ce5.

This mostly for sanity, as normally the exception would
happen on begin().

Change-Id: I2c2fb20ed5184d3c00d3050e929a11e3e37f50f4

Merge "Fix comment about how to use AutoloadGenerator"

Fix link to Help:Contents in UPGRADE file

Change-Id: I4c8c44f8c75200f33985f2e769e88f171162cc83

Merge "Replace extract() with explicit variable definitions in DjVuImage"