Merge "Allow first letter data to be invalidated"

Allow first letter data to be invalidated

Just a class constant for now, but that should suffice to deal with the
current emergency. Proper dependency tracking via the CacheDependency
hierarchy would be pretty cool in the long term.

Change-Id: Ibbe7fa2814434d4869aba20f628bd43269e611fa

Merge "Adding output parameter to PageHistoryBeforeList hook"

Merge "Adding hooks for thank you notification links"

Adding output parameter to PageHistoryBeforeList hook

This will allow us to easily add other RL modules to the page

Change-Id: I2ec00d44e37298ef91ab428759bda037072b0120

Restore SET cl_timestamp=cl_timestamp

Apparently cl_timestamp=cl_timestamp is a workaround for obscure
behaviour of the timestamp type in MySQL

Change-Id: I803f20bcf4e28e8e2833a07bcf00e7edc00ad84b

Reduce disruption during updateCollation.php

Have updateCollation.php order by cl_to, so that each category is
updated all at once. This minimises the time during which a category
will appear to be incorrectly sorted, while the maintenance script is in
progress.

Mark the cl_collation index as needing deletion, it was always pretty
pointless. You can't do much better than a full table scan when you're
changing the collation value on a wiki.

Increase the batch size since the lack of a cl_to,cl_from index means
that it will have to filesort each category. A larger batch size means
less sorts. As noted by Liangent on bug 45970, you can't order by
cl_sortkey since that will change during execution.

Also fix an inappropriate use of $wgMiserMode and remove a no-op from
the SET clause of the UPDATE.

Very lightly tested.

Change-Id: I19bc8d6701f5f78040aa9c521427ac98ef488d89

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: Iad3494719f752a78d7b414fedaf9a36ae45f9d6e

[FileBackend] Fixed "ignoreMissingSource" bug in multiwrite backend.

* Beefed up unit tests for this case and other "ignoreMissingSource" cases
  such as when the source container or parent directory do not exist.

Change-Id: Iea6dae2424edfd0c4367e5cff606c09a4e8a865b

Adding hooks for thank you notification links

These will allow Echo to add a 'thank' link next to the 'undo' links

Change-Id: Ifb93e49b30bb325ecd17ece9f36bcd83c63a2563

(bug 44719) Remove mention of non-existing script in DefaultSettings.php

maintenance/migrateCurStubs.php does not exist; and I was unable to find
any script that would achieve that functionnality.

Bug: 44719
Change-Id: I78d8a8aa95a5406bf5a979867bc12a9bb34907f6

Merge "Normalise documentation in tests/selenium"

Merge "Correct case of Title::getPrefixedDBkey() calls"

Merge "Documentation improvements in includes/actions"

Merge "Don't use isset() to check whether an existing variable is null"

Merge "Changed formatting of "anontalkpagetext""

Merge "Use User::groupHasPermission"

Merge "jquery.makeCollapsible: Clean up issues caused by wrong nesting"

Merge "jquery.makeCollapsible: Move functions out of the var statement"

jquery.makeCollapsible: Clean up issues caused by wrong nesting

The way it was done - switching first on action (expand/collapse), then on
elements - caused the logic to be split all over the file.

This caused:
* code duplication (e.g. computing the elements to be acted upon in
  the same way for expanding and collapsing, repeated same comments)
* regressions when the logic was changed for one but not for the other
  (this was the case e.g. with table expanding/collapsing).

As for the second, I fixed all spotted inconsistencies; as for the
first, I'll let the diffstat speak for itself.

Change-Id: I2d2592b4d00424f0c23c493f6de6c824d0714dfc

jquery.makeCollapsible: Move functions out of the var statement

Converts function expressions into function declarations.
A 230-line var statement is too much for me to handle.

Change-Id: I22dc787229f2150dbad8de4e19a7fa034dcdca3d

Merge "Fix ORMRow::insert() on PostgreSQL."

Merge "[JobQueue] Try to cut down on waitForBackups() calls in runJobs.php."

[FileBackend] Suppress missing container errors with "ignoreMissingSource".

Change-Id: I2b9f4764c4938dcee61f17c31ed4d4b84fbfbcb2

[LockManager] Various fixes to lock managers.

* Improved handling of corrupt values in cache for MemcLockManager.
  Also improved the use of Status warnings a bit.
* Removed broken special-case handling for SH->EX lock escalation.
  Updated MysqLockManager to compensate.
* Made FSLockManager only use one handle per file, which is more
  efficient and avoids errors when escalating locks (SH->EX).
* Made lock unit tests have more useful output on failure.

Change-Id: Ib304712fa2b6b3fd02bfc1b08b6f238c771960c2

Merge "Update docs"

Update docs

Change-Id: I65fb974f159a7b317843b44e90f8225ab1bf722d

[JobQueue] Try to cut down on waitForBackups() calls in runJobs.php.

Change-Id: I2fc97ef8dbc02d1184959ba962dcafdae9fae808

Rewrite user_touched comment, since apparently it is not clear enough

Change-Id: Ib993e5a6dc56963e9afbbdc2ada77762e1594f0b

Merge "IcuCollation::$tailoringFirstLetters: implement letter removal"

Fixed redis auth error spam in logs.

Change-Id: I52e074b530d4dba2a0a8ec9b7d32f96d4527c6f4

Fixed connection comparison in RedisConnectionPool::handleException().

* This was comparing a Redis with a RedisConnRef.

Change-Id: I6353736c8ad42a18d90e8474d38398d305950346

IcuCollation::$tailoringFirstLetters: implement letter removal

This is necessary for Swedish, where 'Þ' ("thorn") - considered a
separate letter by default in the first-letters-root.ser file - is
sorted as 'th', causing unexpected output on category pages - words
starting with 'th'..'u' were placed under a heading with the thorn.

There were three obvious ways to do this:
* somehow include information that this letter is to be removed in the
  string itself, as in 'sv' => array( "Å", "Ä", "Ö", "-Þ" ) - could
  potentially clash with valid uses
* create a separate array other than $tailoringFirstLetters to store
  this information - would cause the data to be fragmented all over
  the file
* include information about letters to be removed in a separate key
  "linked" to the regular one, as in '-sv' => array( "Þ" ) - I see no
  obvious downsides, so this is what I ended up doing

Bug: 45446
Change-Id: I57e07a2027c391c5baa767a68f4409b9de7b4618

Merge "Localisation updates from translatewiki.net."

Merge "Made backend sync script update the pos file periorically."

Localisation updates from translatewiki.net.

Change-Id: I285aebdacc930b2eeb97f56987c958ae3f4811cd

Merge "Reformatted some array parameter docs."

Merge "numRows on MySQL no longer propagates unrelated errors"

Reformatted some array parameter docs.

Change-Id: Ie9eefd3693e82f483a9ef0da3f972452ae7bf49e

Fixed @param tags to conform with Doxygen format.

Doxygen expects parameter types to come before the
parameter name in @param tags. Used a quick regex
to switch everything around where possible. This
only fixes cases where a primitve variable (or a
primitive followed by other types) is the variable
type. Other cases will need to be fixed manually.

Change-Id: Ic59fd20856eb0489d70f3469a56ebce0efb3db13

Merge "Make sure Parsoid doesn't get snobbish and treat non-html5 tags badly."

Merge "Fixes for e288e4036"

Merge "Fixed many small spelling mistakes and php docs, var decl."

(bug 45978) Correct case of Special:Userrights' canonical name

Fix for I50796216 (17e17ba).

The error is PHP Notice:  Found alias defined for Userrights when
searching for special page aliases for UserRights. Case mismatch?

Bug: 45978
Change-Id: Ib82bd38d0229af7571f2ab3c0470fa0f0384b635

IcuCollation::$tailoringFirstLetters: 'en', 'it', 'hu', 'pt', 'uk' verified

* en: obviously
* it: per Nemo_bis in comments on change I97273c52
* hu: per Tisza Gergő in comments on bug 45596
* pt: 'uca-default' collation is deployed on pt.wiki, 'uca-pt' is the same thing
* uk: per Dmytro Dziuma in comments on bug 45444

Change-Id: Ia7568a9ad40ef991b73059b5269e6236f52681f1

lowercase second character in digraph letters in IcuCollation tailorings

This is *the* valid way for Hungarian (per bug 45596 comment 10), and
it's likely more appropriate for other languages as well.

I should have done it this way in the first place; the original data
source includes these forms along with the all-uppercase ones (I
checked them all), so they're certainly at least not wrong. Just an
overlooking on my part.

Change-Id: Ie0ca297a082ddba8d757beb85655f86b3ee70b02

Merge "Lots of spelling mistakes and phpdoc attributes"

Fixed many small spelling mistakes and php docs, var decl.

Change-Id: I1508ed7eb77e5e4f700fb63955d626c4f5915840

Lots of spelling mistakes and phpdoc attributes

@throw->@throws
@returns->@return
@seealso->@see
@cover->@covers
etc

Change-Id: I9ae6bc3034e9790e2d66cd96473b923fe9ee7953

Merge "Add "User rights management" link on user page toolbox"

Add "User rights management" link on user page toolbox

Bug: 34040
Change-Id: I507962161581c3d9ae928594d8d280be9a4ffbdd

Merge "(bug 25325) fix wlexcludeuser for ApiFeedWatchlist"

More job queue doc tweaks.

Change-Id: I06bae67f67da23190c0576902767dc38a0d8ca5b

(bug 25325) fix wlexcludeuser for ApiFeedWatchlist

The effect of setting this option was later overwritten in the code; I
just moved an if a few lines down.

Also some null comparison cleanup.

Change-Id: I2a85673330681c6ff7facc25a7bbfb6972fe8fd5

Localisation updates from translatewiki.net.

Change-Id: I5efb53366137b0e6dceccf76a9cdaa3f7d77194f

Don't use isset() to check whether an existing variable is null

Change-Id: Ie63f060fb7928fd51a8ad71293955f923f6a18c3

Merge "Kill off numerous unused variables"

numRows on MySQL no longer propagates unrelated errors

Bug: 42430
Change-Id: Ie2277c97177be34f4a48dfa0eac736a2b5716f22

Made backend sync script update the pos file periorically.

Change-Id: I5b4028c8d69a6bf78b372f0d1cdb258207465f30

Changed formatting of "anontalkpagetext"

Change-Id: Ie551f08c4409b7515e4b10b598789051b44c8c42

Merge "API param validation: Add wrong value to error message on user params"

Use User::groupHasPermission

Instead of getting the rights of group '*' and searching for 'read',
there is a method to do a direct lookup for group '*' and right 'read',
makes the code easier to read.

Change-Id: I1d5d6e1f4a94ded2b3f87144e031d0c2aac62020

Merge "createaccount api action didn't handle no token and no cookie"

API param validation: Add wrong value to error message on user params

It is always nice to get the wrong value back to know it.
Refactor a bit to avoid the array wrapping and unwrapping for non-multi
params.

Adjust another variable, add doc comments, removed a empty line and
reorder a condition for easy reading.

Change-Id: Ia91aa5908b82ad1209dc7da1139f91e1f2b45fac

Correct case of Title::getPrefixedDBkey() calls

Change-Id: Ic9ef0cc6dbb0a8eb0ee93432299ef59759223e2c

Remove the page from the LinkCache before adding it again in WikiPage::loadFromRow()

This makes sure the LinkCache doesn't list a page both as good and bad link at the
same time, e.g. when a page gets created or deleted in the request.

Change-Id: I6774132ba8e4d28deed1908150c02fbaa42111a5

Merge "Return user ID as userid in watchlist API module"

Use ContextSource::getLanguage instead of $wgLang

Found two places, where $wgLang is used inside a ContextSource.

Change-Id: I1881a20d3cd9b50eb4af0cd588451ed941f936fa

Localisation updates from translatewiki.net.

Change-Id: I60a4f38e6b3459cbc2fb758615a311d67800d54d

Merge "Use TargetLanguage of the Parser in ImageGallery"

Merge "(Bug 34798) Synced the FeedUtils.php page with mediawiki.action.history.diff.css"

Use TargetLanguage of the Parser in ImageGallery

If there is a parser set on ImageGallery, the target language of the
parser should be used, because this parsed piece of html can get into
the parser cache.

Change-Id: I2e47b316e3dcc6a3db4fb5d13f5065e35fc369dd

Sync messages.inc with MessagesEn.php

rotate: Followup to Id15a92d19cda8256917e7e1e5ee4241012214102
pageswith props: sometimes earlier...

Change-Id: I8da23c89d5b1ebe669b4e4ef1e0d67ce8064a860

createaccount api action didn't handle no token and no cookie

If you made a request to action=createaccount and this was
the first time (so no account creation token had yet been
generated for you) a "nocookiesfornew" error was returned.

This is incorrect, what is supposed to happen in that
case is the api returns what token to use. This is
how users are supposed to request tokens, so the
issue is very confusing.

No release notes since this module was introduced in
this version.

Also removed sessionfailure from possible errors,
as it isn't one (that status triggers the needtoken
result).

Change-Id: Ibfc3879fa89b1e11303aef65feb45a91afc215e6

Updated job queue README.

Change-Id: I8f884823cc5396e38bc5cd2f803f062a2dbdfac8

(bug 45922) Fix arbitrary inconsistency in page title and headings for message accmailtitle

bug: 459222
Change-Id: If582242f0473bce81a80f05d77421bdf6e5a3f29

(Bug 34798) Synced the FeedUtils.php page with mediawiki.action.history.diff.css

->Added the background,border in diff-context,diff-addedline,diff-deleted line in $styles array in the FeedUtils.php

Change-Id: Icbfb6cfc659652c146e565b760d0815640fdad64

Merge "(Bug 45854) Unset the cgroup if we don't have a memory limit"

Merge "Fix more UnitTests for databases that do not use integer timestamps"

Localisation updates from translatewiki.net.

Change-Id: Icec7f2891935766ba5406d7932198b0f07a8a6d9

Merge "Add the global correctly"

Fixes for e288e4036

Changed static calls to instance functions.

Make BitmapHandler::rotate() non-static.

We don't have a Bitmap class. It should have
been falling back to BitmapHandler class.
(JpegHandler ← ExifBitmapHandler ← BitmapHandler)

Change-Id: I17be410456b00cef2ded8d6e2282ae0de4785695

Fix more UnitTests for databases that do not use integer timestamps

Change-Id: I6f8b19aa6e6a9efc3320bb18fa8c7ec8b72a0c17

Merge "Fixed $vCallback comment and removed unused return value."

[JobQueue] Made supportedOrders() protected.

Change-Id: I7237472d8281c06aff11c4644e9a59848c325263

Fixed $vCallback comment and removed unused return value.

Change-Id: If438d83e1b43c110b3c23e8d0a7705fa0795049f

fix MessageCacheTest for non-english wikis

MessageCacheTest::addDBData is called before setUp, so wgLanguageCode
and wgContLang are not set to 'en', which result in mediawiki pages
created as $key/en. When the test runs, the language is set correctly
and mediawiki is searching under $key, and does not find anything.

Set wgLanguageCode and wgContLang at the begin of addDBData

1) MessageCacheTest::testMessageFallbacks with data set #0
('FallbackLanguageTest-Full', 'ab', 'ab')
Message fallback failed.
Failed asserting that false matches expected 'ab'.

Change-Id: I8e70b7b424df6a9ad98bfb6c228809f4b4437d1e

Merge "Use output/error maintenance functions"

Add the global correctly

Change-Id: I03be0cf1e82fc9f8f00727bed5aca9d9d2758c2e

(Bug 45854) Unset the cgroup if we don't have a memory limit

We only create the cgroup if there's a memory limit, but it
tried to always cleanup it, regardless if it was used or not,
resulting in a «/sys/fs/cgroup/memory/mediawiki/job/32466/tasks:
No such file or directory» error, when run with MW_CGROUP='mediawiki';
MW_MEM_LIMIT=0

Change-Id: I7fdb545b336ead62aa743b1bc3afdbfba2c3113f

Merge "Add SpecialPage::getGroupName and use it"

[JobQueue] Cleaned up handling of orders a bit.

Change-Id: I83d060e0db97d5bebf042041b0bd20563b093418

Removed "poolSize" option from RedisConnectionPool.

* The size is now automatically based on the number of servers.

Change-Id: I6ea53378c9c8254e8556c0a1e4efe64fd71ba356

[JobQueue] Added JobQueue::getAllQueuedJobs() function.

* Also added a small MappedIterator class for convenience.

Change-Id: I7dac1001a8e048cb524f7fbfee50a20a32e598ba

Use output/error maintenance functions

Change-Id: I9f9ebee9e577ab3cc7a042e5280161d2b2bad57a

Kill off numerous unused variables

Change-Id: I7039f1328f37ee669b694f73ee282602186bffd1

Merge "mw.loader: Buffer cssText in addEmbeddedCSS."

Merge "mw.loader: Create new style tags instead of appending text"

Adding validation for checkmatrix (same as for multiselect)

Change-Id: I825ec8a6094405cf7d74597a0eeda3898981763b