Change collationUpdate batch size from 1000 to 50

It selects that many rows, then does PHP processing and an individual
update query for each one.  This is not a good idea when each batch is
done in a single transaction: 1000 MySQL updates interspersed with PHP
processing might take a second or more while locks are held.

Don't regenerate raw sortkeys in CategoryPage.php

Previously, when adding pages, we'd use $title->getCategorySortkey(
$row->cl_sortkey_prefix ) to generate the initial letter on category
pages.  This is fine if the migration script has already been run, but
until then, cl_sortkey_prefix will always be the empty string, so the
user-specified sortkey will be ignored when generating initial letters.
Plus, if getCategorySortkey() had nontrivial computational cost, this
would be a waste of resources.

So just use $row->cl_sortkey instead.  This is at least guaranteed to be
consistent with how pagination works, no matter what state the DB is in.

Already deployed by Roan to fix user bug reports in #wikimedia-tech,
from apergos, Alpha_Quadrant, and Romaine.  Tested on Wikimedia, so far
so good.

More documentation and class definitions

Few tweaks to braces

misc w/s cleanups, “svn diff -x-w” clean

* Followup r76127 and r83521 by adding parsertests and using caseFold

* Update class name: FakeMemCachedClient -> EmptyBagOStuff
* Don't show "Division by zero" warnings when there's no statistics for cache hits

Make updateCollation.php a bit less murderous for WMF databases:
* Don't run a COUNT(*) query on what's potentially the entire categorylinks table on enwiki (hundreds of millions of rows). Put it in a miser mode check
* Wait for DB replication to catch up before processing the next batch. Implemented LoadBalancer::waitAll() for this purpose, which should behave more nicely than wfWaitForSlaves()

revert r83521; this needs to be fixed better, but fixing the crash is first priority

followup r76127 — use language-aware comparison between link text and possible title attribute

LocalSettings generation tweaks:
* Mention extensions only if user chose some during installation
* Added a comment at the bottom suggesting where to put new settings

switch 'column' for 'colon'

I always thought the colon and semicolon punctuation marks were named after
their architectural equivalent: column. At least, to me, it made more sens
than using a term relating to thedigestive system.

Per Nikerabbit r83296

Set $wgInternalServer to false by default, and fall back to $wgServer in the referring code. This avoids having squid updates break when $wgServer is set in configuration but $wgInternalServer is neglected. For immediate deployment, tested locally.

Last few svn property fixes

And some more mime-types

* Increased the size of the tipsy callout, removed the shadow and made the background of tips white to try and improve the visibility of tips.
* Added ui-button-green, ui-button-blue, ui-button-red classes to the Vector jquery ui theme

Comment the var_dump() and add a comment.
Follow up r83208.

Fix r83390, need to do where ipb_deleted IS NULL as we're left joining in the table, so it can be null

Duh

removing fallback, not needed (messages are in-core and defined in Resources.php). FU: r78966

Strip cookie classes out of the middle of HttpFunctions.php into Cookie.php using svn copy

Localisation updates for core and extension messages from translatewiki.net (2011-03-07 19:38:00 UTC)

Follow-up r83292/r83310: Add new message to the maintenance file too

Merge corrected r81741 and resolve the TODO.
Keep disabled with a class constant.

Bug 27920 - Installer sets $wgUseImageMagick = true even when IM is not found

* Make Skin::formatDebugHTML() work with $wgDebugTimestamps = true
* Removed escaping of ' since the string is defined with "

Follow-up r83461, replace deleted with hidden and reverse the word, e.g. deletedfile->filehidden. Consistency with prop=revision etc. (In Soviet-Russia, file hides you!)

Remove second parameters from wfEmptyMsg() calls

The second parameter was removed in r64178 and is now useless.
There is probably no need to backport this in 1.17.

(bug 27722) list=filearchive now supports revdel
* Adds a condition fa_deleted=0 if the user does not have the suppressrevision rights. This field is unindexed. This should however not be a big problem as files with fa_deleted are rare. Unfortunately this hides files that do not have DELETED_RESTRICTED, but I don't know how bad fa_deleted & DELETED_RESTRICTED = 0 is for performance
* Added deletedfile, deletedcomment, deleteduser and deletedrestricted to the output for what I think are appropriate fa_deleted fields, but it's hard to tell what's corrent without a single line of documentation or even comment in the code. Perhaps somebody can dig up a commit message where the purpose of the constants is explained?

Keep monospace as a fallback for HTML debugging

Per CR on r83336. We also keep this as inline style since it is only
needed by developers anyway.

More documentation

Remove trailing whitespace

Documentation and explicit variable definitions

Followup r80069, swap private to protected

Rephrase 'config-using531' per suggestion of Krinkle: "Granted the bug is in PHP, but imho "PHP" isn't compatible with MediaWiki, it's the other way around."

Also removed the link to the PHP bug report. Not that relevant for the user installing MediaWiki.

Reorder so that wfProfileOut() is the last sentence before the return.

Remove unused globals

Fix for IE7, jQuery magic doesn't help here, it needs to be valid html, shorthand doesn't work in all browsers

r82867 converted $wgCaches into a class instance. Update the parsertests.
This faking is needed to avoid Table 'unittest_objectcache' doesn't exist errors when using temporary tables.

Follow up r83140. Add a clear() method to ObjectCache so that it can be used by ForkController.

Kill filtering by hash because the query is unindexed. We need a condition on fa_storage_group, but I'm not sure how index merging will work out. I don't have time to investigate this at the moment, so killing it in its entirety seems the safest option for now.

Breaking change: remove faminsize/famaxsize because they lead to an unindexed query, which takes a very long time if there are only few rows that match the filter. Introduced in r66168, so luckily only live since 1.17.

Add $namespaceGenderAliases support to Spanish. Part of bug 27691.

Only call ParserOptions::getNumberHeadings() when needed for the benefit of cache sharing when this is not used.
Also fixed a bug in ParserOutput when no option is used; otherwise getUsedOptions() will return false and throw warnings in ParserOptions::optionsHash().

* Added UserGetLanguageObject hook to change the language used in $wgLang

Script to purge all pages of a given namespace

Followup r83390

If the user is allowed to view hidden users, put a missing property on all users.

For list=users, do the same, but we can't easily distinguish to other people the difference between "hidden", or "missing", so no reason to do it

Fix where ipb_deleted from r83390

Followup r83399, add back base conversion

(bug 17691) Support CC0 license in installer

* (bug 27588) list=filearchive&faprop=sha1 returns empty attribute

Refactored code out, to be reused in ApiQueryFilearchive

Simplified too per Bryan on IRC

(bug 26690) Value of the 'Name of wiki' field is not retain after two subsequent click on 'Continue' button. This has been a complaint since the beginning, that the installer "forgets" $wgSitename.

The original intention, as I saw it, was to encourage people from choosing 'MediaWiki' as the name of their wiki. Just hardcode it, I don't think we're changing the name anytime soon.

i18n tweaks for r83395

(bug 27903) Button text is actually continue, not next

* (bug 27717) API's exturlusage module does not respect $wgMiserMode

Implement ApiQueryCategoryMembers style miser mode stuffs

Fix leading whitespace before class ApiQueryUsers

Followup r83390, change cache modes

(bug 27901) Missing php version in config-using531 msg

* (bug 27897) list=allusers and list=users list hidden users

Refactored addition of block information, and also checked if user is allowed to view hidden user fields

PHPUnit coverage comment must not have parentheses

Those comments have the format:
 @covers Class::Method

In PHPUnit 3.5.12, the comment is split by :: and method_exist called on
the resulting string. When one use  Class::Method(), the result is a call
to method_exist( 'Method()' ) which is always false and raise an exception

Removing parentheses from r79118 solve the issue. This also kind of
revert r79164.

Fix for r78512: set the default value to "view" for the action parameter

Fixed (bug 27652) [jQuery.client] versionBase is wrong for versions higher with two or more digits
* A userAgent like the following:
> "Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10_5_8; nl-nl) AppleWebKit/533.19.4 (KHTML, like Gecko) Version/5.0.3 Safari/533.19.4"
Will now have a versionBase of "10" instead of "1".
* A few JSHint warnings fixed (missing semicolon)

(follow-up: r83371) Fix variable name typo. $this to $loginForm.

Fixed fatals in UploadFromUrlTest. It still fails though

(bug 27508) SVGMetadataExtractor takes too much resources on huge svg's. Change it so it only looks at begining of file.

Add a new config variable $wgSVGMetadataCutoff (currently set to 256kb, chosen rather arbitrarily)
and only read that much of the svg file when finding metadata. In general:
*Most (non-crazy huge map) svgs aren't that big, so there'd be no change in general
*Almost all files have any relevent metadata (well except for when we look for animation tags) is at the begining of the file
before actual image data.
*At the end of the day, even if this does miss metadata in some files (which I really doubt it would), I'd consider that a better
situation then the current situation where it can take 10 minutes or have OOM to parse the likes of [[:File:Puerto_Rico_ecosystems_map-fr.svg]]

Also has parts of/parts are based on Hartman's patch from bugzilla in it.

Also changes how it recurses into child elements looking for animation, to do so only when neccesary.

Trims the results of reading values, because i was getting extra leading spaces when testing this.

Last of all, add a comment to the MediaHandler class about how the first parameter of MediaHandler::getMetadata is kind of useless.
(it confused me when I was doing this)

Documentation updates for r83371

Fix and Enhance:
1. Fix - previously unresolved possible return value (ABORTED) added to switch
2. Allows a message to be returned when status "ABORTED" instead of just one of existing generic messages.

If $wgEnotifMinorEdits=false, the enotifminoredits pref does nothing, so put it in $wgHiddenPrefs.

Its confusing to have it as an option, if its disabled, especially since there is no indication to the
user that it is disabled.

(bug 27854) Http::isValidURI is way to lax. This is a much simplified regex that accepts a subset of the previous
regex, but also accepts ftps because both cURL and php support it. It no longer accepts thing like 'foo http://bar bax'
which was my main concern

Note the previous regex kind of looks more restrictive, but is not since saying "anything not containing a space
optionally followed by anything not containing a bunch of characters including a space" is the same as saying anything
with no spaces. See also r83296. This obviously doesn't catch all cases, but I personally think its sufficient.
At the very least it is a very significant improvement over the previous version that caught almost nothing.

Use 'Courier new' for $wgShowDebug HTML output

'Courier new' is rendered with a constant size in both monobook
and vector skins.  monospace is rendered too small under vector
with firefox/Linux.
Thus, this patch makes the font size consistent.

* NS_SPECIAL should not have any applicable restriction type
* Removed some trailing whitespaces

Follow-up r83218: Added var declarations. Html escape the largefileserver message. Introduced mediaWiki.Message.escaped() to bring the interface up to date with the PHP Message interface.

Follow-up r83218: Fix bug# per mediawiki.org/wiki/Special:Code/MediaWiki/83218#c14635

keep checkbox state when adding cat/ns pages

Follow-up r83292: Messages of the type foo.css are optional to translate.

Fix bug in makeCollapsible.
* The instantHide implementation (added in r82471) didn't cover the case where a table is collapsed by default, it needs to pass the toggle to the function so that the row it is in can be excluded.
* The bug was found by user Helder.wiki, also reproducable on the demonstration page (I guess it was cached for everyone, since nobody noticed?)
* Demonstration page has been refreshed and all tests are passed now.

Fixup to schema help, there's only one now

Follow-up r83240: seems people prefer returning false than the current version when no revision with given ID is found

tests for IP class, mostly incompletes

* IP::sanitizeIP()
* IP::canonicalize()
* IP::isInRange()

(bug 27018) Added action=filerevert to revert files to an old version. Copied procedure from FileRevertForm, as most of the verification procedure is simple enough to not warrant the effort of writing a dedicated backend.

Further changes:
* Added Status::getErrorsByType() which returns the internal error array untouched
* Added ApiResult::convertStatusToArray() which converts a Status object to something useful for the Api

Normalise table column ordering against ImagePage

Follow-up r83280; looks like I accidentally deprecated a piece of legacy JS :D

Explicit variables, documentation, newlines, braces

HTMLFormField class to build a select-dropdown-with-additional-text-field structure, with the select options maybe parsed from a system message as is done for the protect, block, delete, fileupload, etc, dialogues.  Serves as a replacement vector for Xml::listDropDown().

Fix E_STRICT

Tests for bug 27854 - Http::isValidURI is to lax

Those tests are made to help someone fix bug 27854. You might want to add
some more.  Please note we want this function to only validate curl supported
URI, hence the hardcoded protocols http,https and ftp.

Some of those tests currently fail because of bug 27854.

Don't dupe/drop parts of searchindex on SQLite

Include MediaWiki:Noscript.css in <noscript></noscript> tags in the header, to allow wikis to load styles for users with JS disabled.

MediaWikiTestCase: factored prefix generation to a separate function

Follow-up r83286: detabify description for mediatype

(bug 27587) list=filearchive now outputs full title info

Follow-up r83284. Fixing JSLint warnings: No need to escape a pipe (|) but a caret (^) should be escaped

(bug 27590) prop=imageinfo now allows querying the media type

Follow-up r83282: Re-add the lost $this->getNamespace() == NS_FILE, forgot to test the code for normal pages.

Fixed (bug 27848) Escape '-' in $.escapeRE()

Move the file related parts of Title::isValidMoveOperation to its own function, Title::validateFileMoveOperation

Follow-up r83183, r83202:
* Update SpecialCheckUser.php to new location of IP functions
* Spin out the 'hide-other-field-if-select-box-not-on-other' function as one which should apply to all such fields, especially those created via HTMLForm.  SpecialBlockip and SpecialGlobalBlock should ultimately be converted to use HTMLForm anyway.

(bug 27862; follow-up r77714) Make emailuser api module not freak out when SpecialEmailUser returns a status object instead of true.

If the unknown element is empty, there won't be a matching END_ELEMENT. There's nothing to peek inside.

trim the upload_by_url field from upload form, since if you have an extra space at the beggining like I
did when copying and pasting a url, gives a really misleading error message.

Fix and make file moving better resilient against errors by first copying the files, then doing the db updates and finally cleaning up the source.
* FsRepo::cleanupBatch now accepts either pairs, virtual urls and real paths
* Add missing begin() and commit() to LocalFileMoveBatch
* Actually set an error after incrementing failCount because otherwise the status is still good
* Changed some isOk() to isGood() and set ->ok explicitly to false after rollback

* (bug 27763) Article::getParserOutput() no longer throws a fatal given when an incorrect revision ID is passed

Localisation updates for core and extension messages from translatewiki.net (2011-03-04 19:51:00 UTC)

followup r83227 ... missed bits of patch