Follow up r79845. No need to keep $wgCustomConvertCommand there.

Document return type.

Explicit class member variable definition

initFromQueryResult doesn't use the $db parameter.

Private method, updated callers to not pass it.

Follow up r80025.
The space is actually used inside extensions, such as Maps, covertly called as 'display_map' into
Validator, then converted into 'display map' by getName().

Fixed logic error - all tests were passing... :(

Reverts part of r80036 - jquery.client is already depended on by mediaWiki.util - which is laoded initially as well.

A wfProfileOut was missing (which messed up profiling sometimes).  Thanks to Owen Davis for finding this.

Fixed logic error that caused the test to always pass.

Included client in the initial payload. Restored functionality that used to exist in this plug-in that adds classes to the document's HTML element for browser and platform information. This will help reduce CSS hacks to only be used for non-JS rendered items.

Repair quickbar of standard and cologneblue skins to work under Resourceloader.
Refs bug #26649

Factoring this out into seperate stylesheets seemed silly. Instead just moved
that last css line from skins/common/quickbar.css and skins/common/quickbar-right.css
inline as well.

Remove "Heldergeovane" from both PT and PT_BR message files, per request on TW

Renamed user already exists as "Helder.wiki"

Forbid '<', '>', ' ', '\n', '\r' in parser hook names.
Registering a tag hook with < or > on its name resulted in a UNIQ marker.
Spaces, tabs or LF do work (CR fails due to the CR->LF transformation),
but it's easier to also forbid them, just as they are not allowed in other
markup languages.

Add tests for parser tag hooks.

Fix a few doQuery() -> query()

Fix callbacks for the final time so it actually works per documentation.
Other minor postgres fixes

No space after unary minus

Spell false with small latter

Use same wording as in skintemplate

Fix copy+paste mistake from r79989

Fix callback issue noted in r79989 (broken in r78774).
$this makes a difference.
I'm an idiot.

Followup r80003: array_unique() preserves keys, so we got module arrays that looked like {"0": "ext.liquidThreads", "58": "mediawiki.legacy.wikibits", "59": "mediawiki.util",  ... } . To prevent this from happening, use array_values() to reindex the arrays

rm stray tab

Apply array_unique() to module lists in OutputPage, should prevent lots of duplicate 'ext.liquidThreads' elements in the module array reported by thedj

Seems there was a logic error in r80001 :)

Simplify a bit:
* replaced calls to Article::getLatest() by Title::getLatestRevID()
* introduced Skin::isRevisionCurrent()
* made checks on Skin::pageStats() more coherent

Follow-up r79964: Use the existing message 'parentheses' instead of introducing an identical one.

I have no clue how the callbacks work, this at least sets up a database, everything after that is fubar

Parent in DatabaseInstaller is a CoreInstaller, not just an Installer

New test database resource file with some test configuration changes for it. Follow up to r79421 and r79982

Fixup some of the ordering of API Autoloader stuff

$reedy->ocd++;

Follow up to #79421. Cleaned up some globals and some typos. Going to follow up with some actual test changes

Normalise casing of svn:keywords Id

Add missing Id to ApiFormatDump and ApiUpload

Move the () surrounding description strings of files, out of the description and into the interface.

Follows up r68324 and r68325

Explicitally add ApiFormatXmlRsd to the AutoLoader

Resolves bug #26655 - adds support for space before and after the URL inside the parenthesis of a url() statement in CSS.

Localisation updates for core and extension messages from translatewiki.net (2011-01-10 20:56:00 UTC)

Follow-up r79915: Add new section in maintenance file too

* fixed oracle code for updater (corrected not null BLOB field adding)
* added required DB vars in WebInstaller_ExistingWiki
* oracle upgrades normally 1.16=>1.17

Fix RFC 5322 'atext'

An additional character was introducted in the RFC 5322 atext fragment.
Spotted with r79924 which send the file as latin1 encoding.

* fixed Oracle code for installer and phpunit tests
* removed ORABlob class

More postgres
* Add user creation form like MySQL
* Fix canCreateAccounts() check
* Fix ts2 setup callback
* Still needs schema creation and other misc. stuff from lines ~370-530 of DatabasePostgres

More silly whitespace

rm big whitespace

Move table prefixes to constants, add doc to addDBData()

* (bug 26650) Remove $wgAPICacheHelp in favour of $wgAPICacheHelpTimeout

Be sure to use the latest rev id and not whatever getRevIdFetched() could be

Removing unneeded wraps and using $ and mw instead of jQuery or mediaWiki since r79246 enabled the private scope by default. No need for two wraps.
(the wraps had an extra indention, hence the diff will probably look huge)

Adding comma's at end of arrays where this hasn't been done already for consistency

traversal indention. Follow-up r79924

Documentation addition/tweaks

* Consistent single quotes
* The problem with onKeyUp was that is shows errors even when initially typing a (possibly valid) address. onBlur has the downside of not giving feedback while correcting the mistake + since e-mail is the last input element at the moment, it doesn't blur on submit and required a seperate event handler.
** Solution: Bind onBlur once, and then listen for onKeyUp.
* Adding missing semicolon(s)
* Compare with strict (===) to empty string / null
* Moved non-global wfFunction to mw.util
** The "FIXME:" note was kept
* Comment "apostrophe" fixed to "at". Comments don't run :)
* Caching selector in wfUpdateMailValidityLabel
* Renaming "wfUpdateMailValidityLabel" to "updateMailValidityLabel"
( Ping r79910, r75670, r75627  )

Followup r68325, with comment describing intent of code.

Correct {,in}valid js messages in UserPreferences
Add them to messages.inc

Thanks Roan and ^demon for the translation.

FU r79904

Add a .tld to emails when testing for white space before/after.

This might avoid confusion when testing. Currently, an email without
a tld is invalid (ex user@host).  The assertions could have been
invalid just by the lack of tld, not the spaces!

js email validation improvment (onBlur)

commit log more or less follow the diff order:
* adds an id to the UserPreferences form (#mw-prefs-form)
* Add an handler on the form submission to validate the email. This also
  update the label and, when email is invalid, get focus on the field.
* use onBlur javascript event instead of onKeyUp (per r75670 CR)
* Label update is now a function ... (wfUpdateMailValidityLabel)
* ... let us update the label on form submission
* Clear the label message when email is empty (null)
* Made the js regexp to stop capturing
* Fix js email validation function to return a boolean
  (it still return 'null' when given an empty email address)

Partial revert of r79903, unstubbing needs localisation cache which might need database

Fix for r79874: only set $mRevIdFetched from fetchContent(), it was overriden by the loadLastEdit() call in getRevIdFetched() for old versions which caused it to be always the latest rev id

js email validation fixed + i18n support

The Javascript email validation in User:Preference got broken by an id change,
this patch fix the issue.

A long standing issue was a lack of proper i18n support, message helper
was hard coded in the js file.  It now use the resourceloader to get the
i18nized message.

Follow up r75670

Follow-up r79880: check if $wgLang can be used.
Changed the checks in Database.php to use is_object because of stub objects

Fix bug in ResourceLoader causing LTR->RTL flipping to only be applied to the first CSS file of every module. Thanks to thedj for reporting and investigating this bug and to Nikerabbit for finding the culprit (bad use of array_map)

* (bug 26208) Mark directionality of some interlanguage links

Bug 25010 - Bashkir-language interwikis: linktext change from Башҡорт to Башҡортса

Avoid dot bloat in paths

* (bug 25506) Exception is thrown if OutputPage::parse is called inside a tag hook, which would reset parser state

Followup r78924: keep track of exception/warning comments separately, to prevent them from being eaten by the minifier. This also prevents filter cache pollution. Additionally, uncomment some wrongfully commented-out code introduced in r78924, and add newlines after exception/warning comments

Fix syntax terror from r79884

* (bug 25708) Update case mappings and normalization to Unicode 6.0.0

Kill Article::$mCurID, not set anywhere, not used anywhere (including extensions)

Fix typo from r79874

Changed to calls to Article::getContent() to Article::getRawText():
- in Article::insertRedirect(): if this will be saved in the redirect table, it should only be checked with the current text (in fact this is never called when viewing an old revision, but just in case someone breaks this)
- in Article::prepareTextForEdit(): as for Article::doEdit() this should be actual page's contents

Add a hook for registering variables in ResourceLoaderStartUpModule::getConfig() to complement MakeGlobalVariablesScript

* (bug 26187) Confirmrecreate no longer parses the edit summary

Horrible tabsoup converted to spaces

Fix encoding

The actual error is always in English, so wrap it in ltr element

* (bug 26638) Database error pages display correctly in RTL languages

Followup to r79856: s/0.4/0.5/ and added "bytes" attribute to "text" element

Creating export-0.5.xsd as part of followup to r79856

Made it so that our existing 0.4 exports validate

Localisation updates for core and extension messages from translatewiki.net (2011-01-09 20:27:00 UTC)

Only set $mUser, $mUserText, $mComment and $mTimestamp fields in fetchContent() if we have loaded the latests revision

Added QuickTemplate::getSkin(), useful for extensions when executing hooks passing only the template as parameter

Fix for r33432: isArticle() is to check if the content displayed is the one of the article, isArticleRelated() is more appropriate here

Follow-up r79868

Clean up mediawiki.special.upload.js
* (Whitespace) conventions [[Manual:Coding conventions]]
* Single quotes where possible
* Removing useless parens in some cases
* Missing semicolons (JSLint)
(Kindafollowupon r79867)

Follow-up r79845: Add rotation support to the upload preview using the <canvas> element. The actual rotation angle still needs to be extracted from the file using a library such as jsjpegmeta <benno.id.au/jpegmetaexample/jpegmeta.js>

(bug 26620) JavaScript error in insertTags() when jQuery is present but currentFocused is not set (e.g. on Special:Upload)

Followup r74298: add ORDER BY wl_namespace, wl_title to the query in WatchlistEditor.php , rather than conditionally adding an ORDER BY that only seems to play nice on Postgres

Per r79581 CR, add @embed comments to jQuery UI's default theme's CSS too, and combine a few background rules to prevent duplicate embedding.

Attempt at fixing bug 26370, which seems to be an infinite loop caused by ob_get_level() never going all the way down to 0 no matter how many output buffers we destroy. Use a for loop instead

No longer correct

Removes $wgSecureLoginStickHTTPS which was never really used

Follow up r75585

Follow-up r79845: add function documentation; only call wfMakeDirsParent if we have a dest path.

On r79845 I submitted a wrong commit summary, here is it for reference:

(bug 6672) Images are now autorotated according to their EXIF orientation. Following the Flickr example, this only affects thumbnails, with the source unrotated.

* Introduced BitmapHandler::canRotate() and BitmapHandler::getRotation() to check if rotation is supported and what the amount of rotation is
* Factored out scaler determination into BitmapHandler::getScalerType()
* ImageMagick uses the -auto-orientation option
* GD uses imagerotate()
* Unconditionally show the thumb size on the image page, don't know why this wasn't shown for SVGs etc.

Bug 26563: Add bytes changed per revision for stub and full article dumps

Fix & additions for strtr vs str_replace benchmarks (for bug 26605 discussion)

The strtr function in the benchmark was slightly artificially accelerated by replacing '_' with '' (empty) instead of ' ' (space), making the output string shorter and thus processing faster. This is now fixed, which makes the results slightly closer.
I also added 'indirect' functions to the benchmark, which instead of calling strtr/str_replace directly, call a global function which then calls them (equivalent to what we'd do if we turned the replaces into wf* global functions)

strtr() does appear to be slightly faster than str_replace() for this workload, but it's fairly modest and approximately the same as the overhead of a function call.

Results on my test box (MacBook Pro, 2.4GHz Core 2 Duo, Mac OS X 10.6.6, 64-bit PHP 5.3.4 built via MacPorts) with 10,000 reps:

$ php bench_strtr_str_replace.php --count=10000
10000 times: function bench_strtr_str_replace->benchstrtr() :
    19.67ms (  0.00ms each)
10000 times: function bench_strtr_str_replace->benchstr_replace() :
    22.05ms (  0.00ms each)
10000 times: function bench_strtr_str_replace->benchstrtr_indirect() :
    22.53ms (  0.00ms each)
10000 times: function bench_strtr_str_replace->benchstr_replace_indirect() :
    26.29ms (  0.00ms each)

Follow up r79848. Fix the syntax error.

* (bug 26625) fix regression in Special:PrefixIndex from r75314; pages weren't listed if namespace was set but no prefix given

Rm code that has been commented out for almost 6 years

Add DatabaseBase::classFromType() to reduce the 'Database' . $type duplication

Follow-up r79713: add authors to README as well