Add srcset for images in <gallery>

Bug: T64709
Change-Id: I476beb6cf072c5a38570143d1f56baa3f49d23d4

Don't alias tt special pages to titles with double underscore

Such titles are invalid, as far as I can see. I get "no such special
page" at https://tt.wikipedia.org/wiki/Special:Unusedimages etc.

Change-Id: Id9869a9af5c67c18af78a925c8c4fdbaef062d32

Merge "Start moving objectcache into libs/"

Merge "Avoid calling rollback() in handleWriteError() if there is no trx"

Merge "Fix parameter order for block logs"

Start moving objectcache into libs/

The base BagOStuff class and some of the implementations are
not dependent on MediaWiki and can be in the libs folder.

Change-Id: I299f9abb778abc65a70461a2aff3015c02a1f04d

Allow subscribing to module loading exceptions

Provide access via mw.track to module loading errors which are
otherwise swallowed by mw.loader.

Bug: T513
Bug: T85263
Change-Id: Ifc13c207af36dc56be931a3b140063354513a4ec

mediawiki.sectionAnchor: Indent with tabs

Change-Id: I3105e96c7ef95ddaf457e40c120f489427db7b86

Localisation updates from https://translatewiki.net.

Change-Id: I88964ba360569a63d7d3d0d7e78e70001bad5806

Avoid calling rollback() in handleWriteError() if there is no trx

Change-Id: Ic3be89151160ad57acda5d95e3995a77f3835717

Fix parameter order for block logs

The new block log params does not known the message number 4 (which is
index 3), therefore LogFormatter::getMessageParameters adds empty index
to keep the sequence in strong order.
But the loop was starting at index 4, not 3, which skips the needed
empty index 3 for the order.

Due to the missing $4 the legacy log params returning index 3 and 4,
therefore move them one up to match the new numbers.

Also fixed undefined index warnings for api's list=logevents

Follow-Up: Ibc7fcaa5a952ff90d42a6477da4baa429f3de64b
Change-Id: Ie23be129ee2bd1d2bf753c3b5cba293d64b8e0e8

Remove getSecondaryDataUpdates and friends from ParserOutput.

This is a hard deprecation, with getSecondaryDataUpdates returning an
empty array and addSecondaryDataUpdate throwing an exception. This seems
prudent since there are no known users of these methods, and they
interfere with the parser cache:

DataUpdates are basically jobs, they need access to services to
function. That makes them inherently non-serializable. This interferes
with the function of the parser cache, which serializes ParserOutput
objects in order to persist them.

This could be solved by splitting DataUpdates into DataUpdateDefinitions
and DataUpdateHandlers, similar to how JobSpecification works with
wgJobClasses. That however seems pointless and overkill, since
ParserOutput already has a mechanism for storing arbitrary data,
including any info needed by an UpdateJob: the setExtensionData method.

After this change, the preferred method to introduce custom data updates
is to store any relevant data using setExtensionData and
implement Content::getSecondaryDataUpdates() if possible. If not,
use the 'SecondaryDataUpdates' hook to construct the necessary update
objects from the info stored using setExtensionData.

Change-Id: I0f6f49e61fa3d8904e55f42c99f342a3dc357495

Merge "Require ext-iconv and suggest ext-intl in composer.json"

Merge "Add i18n for the infamous "Fatal exception of type MWException" errorbox"

Merge "More debug diagnostics for upload by URL"

Add i18n for the infamous "Fatal exception of type MWException" errorbox

These errors usually need to be reported, so it's better for the user
to understand them as much as possible.

Bug: T40095
Change-Id: Ibd2f984b66acb9150400126f21acf7cc5fc85acd

Merge "DatabaseBase constructor cleanups"

More debug diagnostics for upload by URL

Increase debug log verbosity to
troubleshoot file upload by URL issues.

Bug: 73200
Change-Id: I530bd38b85911e0c153280b7d9a9da023f799b02

DatabaseBase constructor cleanups

* The array based format is now required
* Pass any additional parameters through factory() to __construct() even
  if the base clase does not recognize them. This makes DB specific
  options simpler.
* Also added missing bits to Sqlite constructors

Bug: T90288
Change-Id: I29b79a3c1bb2b5e51c1c8f5e04cd08c71e0662a3

Move API release notes to correct places

Change-Id: I301088e4ce4287bd8c9252704ebb190bfcb4aea8

Localisation updates from https://translatewiki.net.

Change-Id: I2d870140da7fd8f4a2c68562ab1d2cb112ce47d4

Merge "Warn when user trying to block nonexistent user"

Merge "Removed cas() from ReplicatedBagOStuff as it is unused"

Merge "Remove "transwiki import" jargon term"

Warn when user trying to block nonexistent user

Now, Special:Block/Some_User and Special:Block/No_Such_User act the
same. So show warning when user opened 'Special:Block/No_such_user'
page.

Bug: T37687
Change-Id: I60292e1016b64224a6146add58106c9028e7914c

Merge "New hooks ApiMain::moduleManager and ApiQuery::moduleManager"

Merge "Adding sample code to TemplateParser::processTemplate() docs"

Require ext-iconv and suggest ext-intl in composer.json

Since iconv is required, per running install.php, and
intl is recommended.

Bug: T90439
Change-Id: I70f6ce9d8d71f8abd3800bbe00aa0f754ce297a2

New hooks ApiMain::moduleManager and ApiQuery::moduleManager

Needed to conditionally register API modules. The previous way, messing with globals
from extension functions, is getting problematic because Config class will make this
impossible.

Change-Id: I86b40aeec555dd6b3cd82cab31d96e85fdf0a665

ComposerJson: Ignore dependencies that start with "ext-"

Any requirements of MediaWiki core to depend upon PHP extensions should
be handled in the installer rather than checkComposerLockUpToDate.php, which
doesn't work anyways since PHP extensions are not included in the lock file.

Change-Id: Ib7af7ccf2f10bc941222d1ce534c7c20690b4c6f

Merge "CategoryView modified to use css columns"

CategoryView modified to use css columns

The categories on Category page were listed using table layout. This layout has
been changed to the responsive css column structure, which is 3 column wide
beyond 768px, otherwise shrinks to a single column.

break-inside is used which needs some consideration, see here:
http://css-tricks.com/almanac/properties/b/break-inside/

Bug: T55130
Change-Id: I437e6079b20e110047f93fb5c09aae40189f1ec0

Special:WantedPages only lists page which having at least one red link pointing to it.

Bug: T19665
Change-Id: If2f0f09ad20ded630ad497a9c84a921c8233cfb1

Remove "transwiki import" jargon term

import-logentry-interwiki will be dealt with in a followup patch,
to avoid breaking changes before the new logging system is adopted.

https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Import-logentry-interwiki/pt

Change-Id: I3ca8b21bce49b41cac7109efb8056ca4469b88d7

Merge "Make HashBagOStuff::lock() compatible with BagOStuff::lock"

Merge "Add default type param for recentchanges and watchlist query api modules"

Merge "Add hook for custom difference engine (WikEdDiff)"

Make HashBagOStuff::lock() compatible with BagOStuff::lock

Fixes php strict standard error.

Bug: T90427
Change-Id: Ica2bddead9b5ad367510e898eeea1257a53b5b19

Merge "Small optimization to ResourceLoader's canExpandStylesheetWith()"

Removed cas() from ReplicatedBagOStuff as it is unused

Change-Id: I2ff33ba806856ec4cc140a52590a922e13191bf2

Merge "Made BagOStuff::cas properly optional"

Merge "Removed FORCE INDEX from API category queries"

Made BagOStuff::cas properly optional

* merge() using the locking version by default. The callers that implement cas()
  override merge() to use the cas-based version. Those that do not no longer
  need to define the unused dummy method.
* Removed some redundant doc blocks.

Change-Id: I49f7dd5432efa8d76f4758c273f0859376ddafb7

Removed FORCE INDEX from API category queries

Change-Id: I7333256b90e18b9ccb6cc0852d021a5fad2c7b45

Add a ReplicatedBagOStuff class

Bug: T88634
Change-Id: I1f7cccd6ac376055ff2b84641ff38a85571c65b0

Merge "Removed doCascadeProtectionUpdates method to avoid DB writes on page views"

Small optimization to ResourceLoader's canExpandStylesheetWith()

The import CSS at-rule can only go in the very top of a style sheet.
(Technically, the charset at-rule may precede it, but that rule is not valid
for style sheets that are embedded in <style> tags.) So we don't need to scan
the entire CSS text for '@import' -- we just have to make sure that the CSS
text doesn't start with it.

Change-Id: I56c2ebd55d4a7e90dde8e89e76b6d1b52ed004f8

Revert "Make eval.php try to handle newlines in code"

Completely broke the script for me.

<TimStarling> MaxSem: that was a pretty crazy change
<TimStarling> fancy trying to detect the end of a statement by shelling out to detect parse errors
<TimStarling> maybe just revert that?
<TimStarling> we can probably do a better job with a few regexes
<TimStarling> I wonder how php -a does it

This reverts commit f0ae292da7ef0673b30c68b4a6f3789b2eea8464.

Change-Id: I16c9c200dc3f4db55cecc23aaa0d9ff55504c19d

Removed doCascadeProtectionUpdates method to avoid DB writes on page views

* Use special prioritized refreshLinksJobs instead, which triggers when
  transcluded pages are changed
* Also added a triggerOpportunisticLinksUpdate() method to handle
  dynamic transcludes

bug: T89389
Change-Id: Iea952d4d2e660b7957eafb5f73fc87fab347dbe7

Merge "Wait for slaves in RecentChangesUpdateJob"

Merge "Avoid @backupGlobals in ExtensionRegistryTest"

Wait for slaves in RecentChangesUpdateJob

Change-Id: Iba8dca2f92d1c9ba2e5ab064aae828e4d88779bb

Localisation updates from https://translatewiki.net.

Change-Id: I9ae7340714bcaf5d5c8d95c9325b011a1988a9ce

Merge "Give objects to DatabaseLogEntry::newFromRow"

Export: Preloading the category name through a URL parameter

Bug: T88683
Change-Id: Iff7d4c1e7138c5160dbaa7b2455695837026d6c1

Give objects to DatabaseLogEntry::newFromRow

No need to cast to array, because the called function will recast to
object.

Change-Id: I0eaa62450a0ebcc9ab8458b225dfadb71197a2ad

Merge "DatabaseBase: Document behavior of nulls in array conditions"

Avoid @backupGlobals in ExtensionRegistryTest

Change-Id: Ia2c764b1c1ee8cff64e1ccdb863e5ef7534a286a

DatabaseBase: Document behavior of nulls in array conditions

Follows-up 26235c73376b.

Change-Id: Id008557e53a8c27f49e87cb2120298ce6478b6df

Merge "Migrate block log to new log system"

Merge "Use new log system when create log entry for revision delete"

Merge "Call LogFormatter::getPreloadTitles on ChangesListSpecialPage"

Merge "DatabaseInstaller::getGlobalDefaults: Return all needed globals"

Merge "Use context when parsing message 'redirectedfrom'"

Merge "Escape message 'redirectto' in Article"

DatabaseInstaller::getGlobalDefaults: Return all needed globals

Since Id364306d883e0d494b948854e05f3f79ba7dd6d2 the text boxes on the
gui installer were not preloaded with the default values from
DefaultSettings.php.

Changed this by return all needed globals (defined by getGlobalNames)
from DatabaseInstaller::getGlobalDefaults(). This injects the as default
value in function getVar and than gets used, when no value is set.

Bug: T71281
Change-Id: I8217b25e903e40ec82be3d700381ff7aea3b481f

Pass user to File::delete from Special:MovePage

When doing a deleteAndMove in the file namespace the user was not
provided to the delete function which result in use of global $wgUser

Change-Id: I6953b83108d9f53b81534a8a12f86050f7e33bac

Skip missed dependencies in ResourcesTest::testUnsatisfiableDependencies

When a dependency is missing the test
ResourcesTest::testUnsatisfiableDependencies gives a error because of
Undefined index and stops further checks. Skip this situation because
the existing test ResourcesTest::testMissingDependencies already gives a
failure with a better explanation in this situation.

Change-Id: I71a88551df67d52ea96bf7283b5de892692f9336

Merge "Fix @backupGlobals annotation in ExtensionRegistryTest"

Localisation updates from https://translatewiki.net.

Change-Id: I81e65e9ea6d9574a81dbe3bb764a76f4441247e8

Fix @backupGlobals annotation in ExtensionRegistryTest

Change-Id: I9a38354030eb2638d7988c0f903fb2181ec17cc6

Merge "Escape message editsection on action=view"

Upgrade composer-merge-plugin to 1.0.0

* Smaller install size
* Included files are processed recursively, with an option to disable

Change-Id: I65bf1e0d0e972846ed9e30761738aaf3a2127c2f

Adding sample code to TemplateParser::processTemplate() docs

Also reverting punctuation that was accidently changed in
change I28cd13d4d.

Change-Id: I1283fbfbaf8cefbbc951599ee286d923b9dedf65

Fixing links on NoLocalSettings page

This fixes a regression caused by change I28cd13d4d.

Change-Id: Iff30f08688d17da2d690c5e798c43489b1cafc7e

Merge "Use the request object provided in User::setCookies"

More descriptive message for autoloader failures

Previously, it just failed with "No such file or directory" without explaining
what's wrong where why.

Change-Id: I776e8fe61f74fd21295da48a45c86ffed9489e23

Localisation updates from https://translatewiki.net.

Change-Id: I2212217c92f8b76aa6682d082508d7d822e92a63

Correct misleading documentation

Change-Id: Ib020467488616eeaa9b53672e5cc45c72f240a54

Merge "Some clean-up of TemplateParser"

Merge "Convert UtfNormalTest into a PHPUnit test"

Some clean-up of TemplateParser

* Replacing global var with Config
* Fixing spacing
* Adding @since tag

Change-Id: I8b0a35116eef6ecead16e03a3c408081ee500aa6

Merge "Remove maintenance script dumpSisterSites.php"

API: Don't allow '#' in title-parts

As-is, this causes loss of a character from the passed-in title leading
to wrong output. But there should never be a '#' in a title-part anyway,
the best thing is to just disallow it completely.

Bug: T38358
Change-Id: Id90bfb9c63a5960e7d2d49350d3e0da0327d0e24

Remove maintenance script dumpSisterSites.php

This script would perhaps better belong in the SisterSites extension, as noted
on the extension's MW.org description page. However, that extension was never
actually developed to a usable state (first commit was r16390, no substantial
changes since) or imported into Git, so apparently the feature (or at least
its implementation) is of only limited, possibly historical interest.

Change-Id: Ia4c3b0316815f06b7e82f013edfd1798ed07d9a3

Escape message editsection on action=view

Bug: T85864
Change-Id: Id55295a0406659242bbc12a59af7e23fb742237c

Merge "mediawiki.special.preferences.js: Simplify modulo"

Merge "mediawiki.special.preferences.js: Simplify creating elements"

Revert "Removed doCascadeProtectionUpdates method to avoid DB writes on page views"

due to breakage at least in phpunit tests for mysql:

https://travis-ci.org/wikimedia/mediawiki-extensions-Wikibase/jobs/51490784

This reverts commit 132f7bb89f38780f1433caff8ec643e91a850a8a.

Change-Id: I85d19ab5ad30e8d13a956d7b7467a94c9e73219d

Allow override of page disply within CategoryViewer

Patch adds two hook which are described in hooks.txt.  This
is being used to allow Flow to offer two links instead of just
one that are relevant to the page that was categorized.

The default output without these hooks is:

   <a href="...">Topic:Soiasdf90f09</a>

This patch allows flow to provide context as to where this topic came
from, by replacing that with:

   <a href="...">Topic:Soiasdf90f09</a> on <a href="...">Talk:Help</a>

(Note that the names of pages within the Topic namespace will also
become more friendly soonish, but outside the scope of this patch).

Bug: T87793
Related-Flow-Change: Ia4f2953bcd807ba3990e762a2efcaab428c40147
Change-Id: I182e6e35fcc3a2a298e928e088579bdb22e145ff

mediawiki.special.preferences.js: Simplify creating elements

See:
https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript#Creating_elements

Change-Id: Ied43829eb19a4270030af4d3f41882f6c0df7935

Merge "Explicitly allow lines up to 100 chars in Rubocop"

Merge "Fix the Rubocop offense TrailingBlankLines"

Merge "Fix the Rubocop offense SpaceInsideHashLiteralBraces"

Merge "Remove outdated Rubocop offenses from .rubocop_todo.yml"

Merge "Fix the Rubocop offense SpaceAroundOperators"

Merge "Fix the Rubocop offense LeadingCommentSpace"

Merge "Fix the Rubocop offense EmptyLinesAroundBody"

Merge "Fix the Rubocop offense EmptyLines"

Merge "Fix the Rubocop offense AmbiguousRegexpLiteral"