Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I41401477973247555f5be35d76620d7b9dc71510

Merge "Treat message cache loading as fatal"

Cleaned up UIDGenerator docs a bit.

Change-Id: I32ca575ca4d7b446b4a27b91d1e8aaa7173c575b

fix sort order

Change-Id: I07e29fbdae88ce5164ba4fb60dbb597dbc2c4dcf

Add UIDGenerator to AutoLoader

Change-Id: I288eef52590033002d7e36f0ca2f667f46dd6aec

Class for getting statistically unique IDs NOT a RFC4122 standard string representation

Change-Id: Ia4133b65afb63931b5f887feb247dac1b1e42360

Merge "add method to check if ORMTable exists"

New parser tests to spec wikitext behavior found on en:WP

* Added tests for:
  - table-list nesting interactions
  - templated-table tags
  - templated-list tags
  - stray *include* tags
  - autoclosed *include* tags

* These tests should help catch regressions from recent
  Parsoid fixes.

Change-Id: I54f0c13a16f4330fa258ff3718921b71bdafa6f0

Merge "Added a few tests to spec HTML-pre behavior and test Parsoid."

Merge "(bug 42483) Avoid serializing database object in update.php"

Added a few tests to spec HTML-pre behavior and test Parsoid.

* Renamed existing pre tests to use Indent-Pre to distinguish
  them from the HTML-pre tests.

Change-Id: I3500c3ba7db19b19baa1d9ab4e3bfb5ccbc491d2

add method to check if ORMTable exists

Change-Id: I7ef34545a9b4cfa6ed8bcdd302ad49faa7747974

Treat message cache loading as fatal

I suspect this is the primary cause we occasionally have
* Sidebar stuck in default values
* No gadgets at all
* Main page full of <int:custom-message> occurances

Change-Id: I811755d4c250800177c4ea9e509bf6e1d13443a0

Merge "(bug 42638) Fix API action=options&reset=1 & unit tests"

Make Language::isValidBuiltInCode() return bool

Make Language::isValidBuiltInCode() return true or false, as
documented. Previously returned 0 or 1. Hacked around in I81ba3228.

Change-Id: Iaa7515095d687d745d878faaa957aae51737abf7

Merge "(bug 42601) Make deleteOldRevisions.php no longer throw database errors on empty sets."

Localisation updates from translatewiki.net.

Change-Id: Ied9bc631fb1feac6b31ce015f9507537f4f16b7b

(bug 42483) Avoid serializing database object in update.php

Change I96b4cfd4 changed the data being stored in updatelog, trying to
store the mangled parameters (which may included an added copy of the
DatabaseUpdater object itself) instead of the input parameters. This
causes warnings because database objects cannot be effectively
serialized.

Go back to storing the original input parameters instead.

Change-Id: I96e6dffed98772ebb9b812773ff2f608e7e40cb1

Merge "(no bug) Fix per-connection db name in ORMTable."

[Upload] [Schema] Added us_props column to avoid expensive GETs.

* We already compute the props when stashing, so including that in a column
  lets us reuse it later when the stashed file is to be published.

Change-Id: I3d6adb6528f1e992d8986058806e745c4e1c0300

[JobQueue] Use SELECT for deduplication rather than more expensive DELETEs.

Change-Id: Iff5dbc0ca6dc7032596d1b4ffddb3cc1848b9e10

(no bug) Fix per-connection db name in ORMTable.

ORMTable was bypassing the mechanism used by DBAccessBase to
ensure that the correct database name is selected when getting
a database connection.

Change-Id: Ic82ce46f1852dcaa5f52b40660ba4a9f52df22f6

(bug 42601) Make deleteOldRevisions.php no longer throw database errors on empty sets.

Also changes DatabaseBase::query() calls to more specific ones.

Change-Id: I7b1eef9ac7cfc89c535a932477fcf1598d3bc0b2

(bug 27202) Add timestamp sort to list=allimages

New parameter 'sort' that defaults to 'name' so if not specified it will
act exactly as before.

Changes in the access to $params['dir'] to a boolean variable
$ascendingOrder in a try to be more meaningful.

Some existing filtering parameters are mutually exclusive with new ones.
The reason is because the table is sorted by name, as primary key, while
sorting by timestamp use the timestamp or usertext_timestamp index which
is not the primary key, thus filtering and ordering by different indices
doesn't make an efficient index usage.

Exceptions are:
* mime: disabled in Miser Mode
* size: although there's apparently no use for this when sorting by
  timestamp, there's an index on that and the EXPLAIN doesn't use filesort
  algorythm (unlike other combinations):

EXPLAIN
select img_name, img_user_text, img_timestamp
from image
where img_size>=1000 and img_size<=20000
and img_timestamp > '20120916153705'
ORDER BY img_timestamp\G

* Same for sha1 (see comment on Patchset 1)

In function run:

* Added USE INDEX query options when sort=timestamp to ensure the proper
  index is being used.
* Table selection and result fields moved at top, then filter parameters,
  and finally order by and options.

Parameter description and definition sorted by purpose: sort and dir
first, then pagination, result fields, filters and limit.

Change-Id: Ide2ff3dbc3e617e749685d558444854617535bab

test: raise timeout for DatabaseSqlite

DatabaseSqliteTest does file access and hence needs a slightly higher
timeout in PHPUnit.

Change-Id: I4b5eb7d7c5a3b9d4790f11cdb4efa9f578763183

test: raise timeout for ExifRotation

ExifRotationTest does file access and hence needs a slightly higher
timeout in PHPUnit.

Change-Id: I871c4a4fde5e71708e55b3f28e7267e1023c2df5

(bug 42638) Fix API action=options&reset=1 & unit tests

Change I98df55f2 broke action=options&reset=1, causing it to return an
error "No changes were requested" rather than resetting the options as
it should. Unfortunately, that change also broke the unit test that
would have caught this regression.

This changeset fixes the bug and the unit tests.

Change-Id: I7fe63640d54efab4572538e9d08f5b75c61243a4

Localisation updates from translatewiki.net.

Change-Id: Ib22bb939459977f160056d83697434b5a456474b

Merge "Fixed documentation"

Merge "(Bug 40367) MediaWiki:Contributions should support gender"

Merge "Show comment not as block in tables"

Merge "(bug 37755) Set robot meta tags for 'view source' pages"

Merge "(bug 42598) Database: fname was not logged for BEGIN/COMMIT."

(bug 37755) Set robot meta tags for 'view source' pages

Apply the same robot behavior of the edit & history pages (noindex,nofollow), on view source pages
(case of ?action=edit when having no edit permissios).

Change-Id: Ifc162332798a3dea93db4096d26e928545fbf4b6

Merge "Add some __METHOD__ to begin/commit/rollback calls"

(bug 42598) Database: fname was not logged for BEGIN/COMMIT.

Relied on presence of whitespace. From git-blame looks like $commentedSql has been this way for a long time. Which is odd
since I've seen many commits since then specifically improving
callers of commit() and begin() to pass the __METHOD__, which
doesn't appear to have been logged anywhere up until now.

Change-Id: I48dc8b3f68a6764327b4b1a8f7c06c59a25a31bb

Show comment not as block in tables

Special:BlockList and Special:ListFiles showing comments inside a table
cell. In that case the parentheses not needed, because the table cell
splits it from the rest. Using the Linker method, which does not produce
the parentheses.

The file history of an file pages does this the same way.

Change-Id: I7fbb068bd2acd90f8117719706c32ca05b27b09f

Merge "mediawiki.Uri: Update doc comments."

Merge "Fixing typo"

Fixing typo

Change-Id: I930c37ad7550c7dd0c4856c22921155dec6ccb3b

Add some __METHOD__ to begin/commit/rollback calls

Change-Id: I20dad8d6bb7a523e8a6f50bc0af5cdba57d7160f

mediawiki.Uri: Update doc comments.

Change-Id: I2c49ae07587b23fc522f8c2dda9be955fa4602be

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: Iad76ee37cb4dfd50f8bc9e7b61b5a7cf31238d4c

Merge "(Bug 40145) Changed the wording of the protectedpagetext message to include moving and other actions."

Merge "Implement static public Parser::getExternalLinkRel"

(Bug 40145) Changed the wording of the protectedpagetext message to include moving and other actions.

Change-Id: Ia8a478586128c647a6957a125fdf5e6ae85ab150

Implement static public Parser::getExternalLinkRel

I've implemented the function Parser::getExternalLinkRel which
gives the 'rel' attribute for a given link in a given NS. Per Tim's
suggestion, as it's currently impossible to invoke the logic in
Parser::getExternalLinkAttribs externally.

Change-Id: Id0bfed81e2afd6730d820b6c9a4a09155a557f37

Refactor edit notice rendering so that it's not baked into EditPage

* Move edit notice rendering to Title class
* Use new getEditNotices method in EditPage

This opens the door for alternative editors (such as the VisualEditor) to use the same notice systems already in use.

Change-Id: Ib0e40714f5433f4d75c54a3c3d60b1590fded7f1

Test / spec handling of HTML entities in link tails

Change-Id: I97ed5e17efb1d783797c46161be56e83b94307a7

Merge "Added assertException method to MediaWikiTestCase"

Fixed documentation

Change-Id: I799304e44af405b58207b0f4f1ec4996f17a9c71

Added assertException method to MediaWikiTestCase

Change-Id: I3ce667b4405241c66c5a979863d1cea2cf39956b

(bug 42202) Validate preference values in action=options

Previously, there was no validation whatsoever and the module would
happily write any preference you asked it to. This, combined with the
fact that the code using the 'editfont' preference didn't perform any
validation or escaping, led to a CSS injection vulnerability.

Using Preferences::getPreferences breaks some existing test cases
because a MockUser doesn't have groups for preferences.

Change-Id: I98df55f2b16ac1b6fce578798b6f58b5dad96775

(Bug 40367) MediaWiki:Contributions should support gender

Change-Id: I80fea312e6f84b2aefaca23c751e6d44dd2ee787

(bug 42202) Validate editfont before embedding it in CSS

If the editfont preference somehow had a value like "foo; color: blue",
we have a CSS injection problem. Normally preference validation should
protect against that, but the API module for setting preferences doesn't
perform any validation.

Change-Id: I5c12aa9a48bf4f6ea4a8fb44554d13189e7757fb

Remove the throw from Message::extractParam()

Per Tim Starling's comment on bug 41400, I replaced the throw (when a
message parameter is invalid) with a call to trigger_error().

The string [INVALID] is used as a placeholder to make it clear from
the output that an error has occurred.

Change-Id: I974d55d44d392c956e7de6d243da9d8dc07d8833

Add comments to Linker::formatLinksInComment()

Per Tim Starling's suggestion, added comments to the regex in
Linker::formatLinksInComment().

Change-Id: If85567e5a2e57dd56f5f263f3512e9ed2f9c940c

Optimized regex in Linker.php

Optimized the regex in Linker::formatLinksInComment().

Change-Id: Idea5554bcb8d7e78ddf972ce4c9827fa738ff837

(bug 40995) Refresh SessionId on login

SpecialUserlogin updated to refresh the user's session_id on each
successful login.

Change-Id: I1bd76f2c199b515f570e18669ca2138668bf847e

Merge "Change msg check sp-contributions-explain from exists to isBlank"

Localisation updates from translatewiki.net.

Change-Id: I554504cd3d622f5eccc27feaf55057c1af443212

Merge "Add a hook to Special:Version so we can customize the version URL"

Add a hook to Special:Version so we can customize the version URL

Change-Id: I02fe38c7b13dcfe570e78d28783098fc18756204

Test handling of templates with invalid template target

Those templates are not expanded, and their contents are treated regular
wikitext.

Change-Id: I3cbc0c8bed8d44435b83c2523f26db09b27215fa

Merge "(bug 42215) "Welcome, X" account creation title #2"

Merge "[FileBackend] Actually unset the "conn" field in closeConnection()."

Skin: Make portlet rules backwards compatible with html cache.

Follows-up I9a2ebd50.
Caused regression with 1.21wmf5.

Change-Id: Iabda81555d2af6d91447a51ff3d8b6ea1f1f00c2

Change msg check sp-contributions-explain from exists to isBlank

This avoids adding the empty message.
Also add a missing text() call.

Change-Id: Idb85abecdf2fa83e89bf6ebe1161e482d1ab2ce7

(bug 42215) "Welcome, X" account creation title #2

Follow-up to 8842727 https://gerrit.wikimedia.org/r/#/c/34063/ .
Renamed welcomecreation-agora to welcomecreation-msg since it's not
agora-specific; actually use this message in SpecialUserLogin.php;
improve Qqq docs.

---- Patch 2: fix alignment nit

---- Patch-3: rebased, and remove 'welcomecreation'
'welcomecreation-msg' obsoletes it

Change-Id: I3f5552b0e60ea89461744c823c646d35891ecdb6

Localisation updates from translatewiki.net.

Change-Id: I838c080f6e060e42daf9c5e889a213dff9f6e83b

Merge "Improve type hint"

Merge "Additional table tests to spec parser behavior."

Merge "API: Fix description for YAML format"

Genderize be-tarask namespaces, fix spaces in definitions

Previously, gender-related namespace aliases existed,
but $wgExtraGenderNamespaces wasn't used at all.

Also, spaces replaced with underscores in alias definitions
where needed.

Change-Id: Idc5e9e6928a21c8e5fb20aba9aae0a93acee71d3

Additional table tests to spec parser behavior.

Change-Id: I4f4852744d463d77fc622c1bcce27e3d9751bc6b

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I104647cc6b24e8f6b3717c7230b4c53799670ab7

Merge "Test table / list interaction"

Merge "MWTimestamp::getHumanTimestamp uses 'just-now' message, which did not exist"

Improve type hint

Eclipse and phpstorm where showing 'User' as return type before, which causes me to not check for false somewhere and thus fatals happening :)

Change-Id: Ibd5b5598f05e6b08481ad65060c7cae18762dc4e

Merge "Add 'dimensions' and 'thumbmine' to resultproperties in prop=imageinfo"

Test table / list interaction

Add a test for indented tables with lists in table contents in start-of-line
context.

Change-Id: I6e81e4148873b3ae97287f72630a8ef353cd6e25

API: Fix description for YAML format

The description for format=yaml in the auto-generated docs is coming out
as "Output data in YAML formatOutput data in JSON format". That needs
fixing.

Change-Id: I20185fba6b70cc995a41eb6f42565da4cc7c338d

MWTimestamp::getHumanTimestamp uses 'just-now' message, which did not exist

Change-Id: Ie91572948238b4ebcc3b942f74693c4687263a0c

Merge "Addition of SearchAfterNoDirectMatch hook"

Addition of SearchAfterNoDirectMatch hook

Needed for mediawiki/extensions/SearchExtraNS

Change-Id: If0a7da5c8691c418bc388e50dd11eea3b57ed45d

[JobQueue] Improved refreshLinks/htmlCacheUpdate job de-duplication.

* Added JobQueue::deduplicateRootJob() function which uses cache
  records of the last time a "root" job was initiated for a task
  in order to invalidate prior jobs for that task. For refreshLinks,
  the "task" is basically "enqueueing the refresh jobs for title X".
* (bug 27914) Also added new Job::getDeduplicationFields() function
  and made use of it with refreshLinks to exclude things like 'masterPos'
  from duplicate job check comparisons for refreshLinks.
* (bug 27914) Always resolve refreshLinks2 jobs down to refreshLinks jobs.
  For each affected pages, one of them will get their job popped
  first, which will remove the duplicates for that page unless
  one page is in a refreshLinks2 jobs and the other in refreshLinks.
* (bug 37731) Made LinksUpdate/HTMLCacheUpdate defer the large
  backlinks query by doing it in an outer job.
* (bug 42065) HTMLCacheUpdate will no longer purge pages that were
  already purged since the job was added.

Change-Id: I71b743e0a38e60a874ca856e80cb761bea06b689

Store "bad metadata" placeholder metadata for SVGs.

* This prevents excessive download (getLocalReference()) and metadata
  extraction operations to keep reloading metadata again and again.
* This also avoids DB load and lock timeouts due to constants UPDATEs.

Change-Id: I9d35467d3a5e23a07e880dbdf59fccda4b598faf

Added a script to populate file HTTP headers for existing files.

Change-Id: I17a0e3d0e84fea1638afe18f2ee3d36ea6be831a

Merge "Fix typo (sentances => sentences)"

Fix typo (sentances => sentences)

Change-Id: Ifaec0d0bd24703dbe80ab1869366b404aa001cce

Changed Status class to allow passing of Message objects.

Adjusted internal Status functions so that a Message
object can be passed as an error message instead of just
a string. Also removed deprecated message functions in
the process.

Change-Id: Icb4b5edeb25190f0aba4f3096a9ea94f22c27bda
Signed-off-by: Tyler Romeo <tylerromeo@gmail.com>

Merge "Fix hash validation in api"

[FileBackend] Actually unset the "conn" field in closeConnection().

* This means that handleException() will actually trigger a new auth
  attempt for the next operation if there was an unexpected error.

Change-Id: I8c651cc57d0967549ec67257078c8e126674235d

RefreshLinksJob parsed with incorrect revision variables

It is necessary to pass the revision id to the parser output to get the
revision variables like REVISIONTIMESTAMP correct. Doing this self
in the job prevents from calling Title::getLatestRevId in
Content::getSecondaryDataUpdates or other places.

This behaviour was lost in I551a90b0

Change-Id: I4bf0e8199edacaa703699f45ec8e65ca9e31e765

Localisation updates from translatewiki.net.

Change-Id: I1832224e316757869dc2071d0f52b9a376ad0a1c

Rebuild MessagesEn.php

Change-Id: I83b0a325c703d19a76e84cb160dc849785d140af

Merge "Fix doc/hooks.txt for EditPage::showStandardInputs:options"