Merge "Purge videos from squid on action=purge"

Merge "(bug 42466) Allow djvutxt to use more memory"

Purge videos from squid on action=purge

While we don't want to re-encode videos on purge, they should still
be removed from squid; compute the list of all urls before extensions
can filter it to remove files that should not be deleted from the
file backend.

Change-Id: Id837843b5a203e49a8594439ce3620da9b3c7b6a

Merge "[Upload] Added async upload concatenation support."

Merge "LinkerTest needs to set the wiki language to en."

LinkerTest needs to set the wiki language to en.

The localisation of Special:Contributions break the
testUserLink otherwise.

Change-Id: I25a40356a96e03975d8ceca975449f7b064bca2b

Merge "While testing, turn off DataBase::ignoreErrors."

Merge "mediawiki.Uri: Don't raw encode the 'title' query parameter."

Merge "test: detects parent setUp not being called"

Localisation updates from translatewiki.net.

Change-Id: Ib48025b8cdc6296b429f398e37f8bca54b8703ee

Merge "Update misleading variable names in docs"

Merge "Update docs for DifferenceEngine::getDiff()"

Revert UIDGenerator commits.

Changes seemed to be merged without anyone really looking at it
and the authorship in git is wrong as well. All related changes
can later be batched as one commit.

Following commit / change are reverted:

 - 7ac3edd https://gerrit.wikimedia.org/r/36803
 - 036d03b https://gerrit.wikimedia.org/r/36813
 - 2b168e3 https://gerrit.wikimedia.org/r/36812
 - 6030d92 https://gerrit.wikimedia.org/r/36801

See wikitech-l thread
http://lists.wikimedia.org/pipermail/wikitech-l/2012-December/064801.html

Change-Id: Ie39fde2ee4121372b6d790943f306e56fbacc2b5

[Upload] Added async upload concatenation support.

* Clients can send the last chunk with 'async' to get an immediate
  response and then check the status of the upload by polling the API
  using the 'checkstatus' parameter.
* Pass the User object along to stash functions within UploadFromChunks.

Change-Id: Ie2ad4c7e94862a728e8a687c3195306e16a5059e

While testing, turn off DataBase::ignoreErrors.

Database errors should cause an exception, not return false,
unless the test case, or the code under test, explicitely calls
ignoreErrors(true). The DB object should be reset to fail fast
and safe for every test.

Change-Id: Ifc2a720ec46a1843d9ffb1488c54743a0099ef9d

test: detects parent setUp not being called

Our test classes often overload MediaWikiTestCase::setUp() but forget to
call their parent. This patch makes MediaWikiTestCase to flag whenever
its own setUp() is called and then simply assert it got called. Any
class failing the assertion is missing a call to its parent setup which
is easily fixed by adding: parent::setUp().

It would be nice to find a similar trick for tearDown().

Change-Id: Ia2afed6052eb3863d6c8e68c551cf03b33bb4be9

Update docs for DifferenceEngine::getDiff()

Previous docs were incorrect and misleading.

Change-Id: I6403c7c7c3b9803cc7e10cc45999244ff78dc14d

Update misleading variable names in docs

$article for a WikiPage object isn't a great choice. Let's use $wikiPage.

Change-Id: I98832355b14fc4a281d3f0e2dbee85e142d83418

Merge "Add wfDeprecated to wfMsg* methods from 1.21 on"

Merge "Add 1.20 RELEASE-NOTES to HISTORY"

Merge "Add type hint"

Merge "(bug 37263) better handling of minor/bot in enhanced changelist"

Merge "(bug 36211) add cursor:pointer to .mw-enhancedchanges-arrow"

(bug 37263) better handling of minor/bot in enhanced changelist

It makes more sense, to have the whole group as minor/bot, when all of
the group are minor or bot edits, not when only one of the group is
minor/bot.

Change-Id: I62bf1099f2e3fb73e9b50374cbb7ba8bfd8a009e

(bug 36211) add cursor:pointer to .mw-enhancedchanges-arrow

With this, the arrow is shown with a pointer and looks like clicking on
a link. Without this, the user has no feedback, that he can click the
arrow.

Change-Id: I8bee6e51b1786785128d56931dc1ff9a09ba1d78

(bug 42619) Adjust comment for log_params database field

Change-Id: Ib78fd5ad7bef78b07460200d5d8b143e5a9a8802

Merge "Improve documentation for "pipe trick""

[JobQueue] Removed redundant DBO_TRX handling in JobQueueDB.

* The onTransactionIdle() function already handles this.

Change-Id: I4851e75a63a270984399cef288f370dcc3138e30

Add type hint

Change-Id: If529cfbf568a10e8cea184fa229959701268ebcb

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

Add wfDeprecated to wfMsg* methods from 1.21 on

These were marked soft deprecated in 1.18.

Change-Id: Ifa82da981c69b2ea9d5b298e9dfb04fc29483206

Add 1.20 RELEASE-NOTES to HISTORY

Change-Id: I2f5062c518017bdb26688df3278132befeb80826

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

(bug 42466) Allow djvutxt to use more memory

The djvutxt utility of DjVuLibre 3.5.24 packed with Ubuntu 12.04 use a lot
of memory more that the old version packed with Ubuntu 10.04. S, to work,
the memory limit must be increased.

User:Phe suggest that this increase of memory consumption is maybe caused
by the loading of localisations file at the start of the tool.

Change-Id: Ia735e1b7d71f5aa512fb14e444989854c440074e

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

mediawiki.Uri: Don't raw encode the 'title' query parameter.

Previously:

Input:
 http://alpha.wiki.krinkle.dev/w/index.php?title=Sandbox/2&oldid=611&veaction=edit
Action:
 remove query.veaction
Output:
 http://alpha.wiki.krinkle.dev/w/index.php?title=Sandbox%2F2&oldid=611

Now the title parameter stays the same.

Change-Id: Ieb9eee450ea8ae3cc97bab12de935c6e7b103842

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"