ParseMaster is designed to use big regexes, with many starting characters.
It makes sense to pcre_study() them.

A few return type hints

Per fixme on r78786, obey sortDescending in Cached mode (ie if sortDescending, force order by qc_value DESC)

(bug 27021) Session key was changed somewhere from an int to a string, which broke stashed uploads totally. Changed $wgRequest->getInt to $wgRequest->getText. The API never explicitly defined it as int, so should not be affected.
Note that this also may break API clients in certain cases for strongly, statically typed languages (e.g. int sessionKey = request->sessionKey;). I see no way around this, so I will announce a breaking change on the API mailing list shortly.

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

* (bug 22606) Follow-up r63059: don't send the "someone registred an account" message when setting email address (i.e. old one empty) in user preferences

* (bug 27016) Fix for r77638: dumpTextPass.php now consider the "output" parameter

Also added "output" parameter to the doc since nothing was mentioning it

fixing php Parse error: syntax error from r81169

Removing more, no longer needed, overrides for the white table background  (which is no longer the case since bug 26449 was solved in r79087 and r79091)
* table#mw-search-top-table (r52669 / 55748)
* table.allpageslist, table.mw-allpages-table-form, table.mw-allpages-table-chunk (r49500)
* (r32269, r55393, r78865)
Thanks to Fomafix!
See also r80495 CR

Update comments to take into account r80443 and r80614 changes, per CR.

Follow up r81100. Don't hardcode hiddencat quotes, per r80936 CR.

(bug 26938) "wikipedia compatible" should not be part of the license name. We already explain this in the help box, there's no need to repeat it here

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

(bug 26939) $wgMetaNamespace is not being generated

* Adding cursor:pointer when hovering the headers (which hide/show stuff)
* Remove unneeded parens from r81151.
* Whitespaces

(Follow-up r81151, r81152, r81126)

Made mediaWiki JS test suite table no longer sortable. This functionality broke with the introduction of hide/show clickers in r81126. But we don't need this so we might as well remove it.

Some adjustments to new functionality from r81133
* Floating result to the right (looked a little messy)
* In Safari the 0.8em looked really pixelated and unreadably. Font-size back to normal size.
* Converted variable names to camelCase
* Fixed bug from r81133 (first array item is 0, first counter is 1). Added a +1 to fix it for now (since the headings were overwritten and the first one was removed). Perhaps should be done better at a later time.

open a series of files for xml input, if supplied (used for multiple xml prefetch files)

open a series of files for xml input, if supplied (used for multiple xml prefetch files)

Random fixes in mediawiki.js: add a var, fix indentation, and get our ABC right

Restore global wrongly removed in r80892

JSMin is gone, please talk with JavaScriptDistiller
(r80656 follow up)

Remove empty statement (r80993) and unused global (r80766).

Follow-up r81132 removal of pre-8.3 support.
Still needs removal of $wgDBts2schema inside initial_setup()

call parent constructor so that include path and other things work right

goodNames isn't used anymore

Followup r81132, missed some more ts2 removal

Minor cleanup, docs

Per comment on bug 26612, we should drop the pre-Postgres 8.3 support with the TSearch2 schema.
This removes a bunch of back-compat code in the installer and search code.
Minimum postgres version is now 8.3

JS tests: fold headers, adapt style

Update the style by removing padding inherited from .wikitable and make
the font smaller (0.8em). Make easier to see the tests on a small screen.

I have folded out the tests by section (using headers as reference), added
some classes and id around to help selection.

TODO: we probably only want to hide tests which are OK

Implement prop=wikitext in action=parse, (optionally) returning the original wikitext. Requested by some Google folks wanting to get data about a revision in one request.

Add the resolved URL of each langlink to the API's action=parse output

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

(bug 26285) Extensions will be automatically generated on upload if the user specified a filename without extension.
Note that this still will throw a warning.

invalid 'var' in JS email validation
follow up r81101

Fix c&p error in r81103

(bug 26929) Introduced the edittools-upload message, which will be inserted under the upload form instead of edittools if available

Bug 26948 - hyphens incorrectly interpreted as range

This is the kind of easy to fix, hard to review bug. Email validation
make uses of strings listing characters, those strings are then
enclosed between brackets "[]". Inside brackets, the hyphen is used
to describe a range of character [a-d] being a b c d.
The string containing an unescaped hyphen, made JS/PHP validation
to match the incorrect comma ",".

* Backslash hyphen
* JS,PHP: add tests for commas and hyphens in username or domain
* JS: add var to rfc_1034_ldh_str
* JS: minor code cleanup

TESTS:

php phpunit.php -c suite.xml --filter ValidEmail
OK (13 tests, 32 assertions)

Special:BlankPage?action=mwutiltest&debug=true
Ran 66 tests. 66 passed test(s). 0 error(s). 0 partially passed test(s).

Followup r81067, use the right syntax for doing a join

Added triggerClientTestResources, which makes the use of separate test resourecs optional. Needed for testing installer functions.

* Fixed Oracle new installer support, broken by r80957. This is a minimal patch and doesn't address the architectural issues.
** Moved the responsibility for calling setupSchemaVars() on install to the DatabaseInstaller subclass. This allows it to be called after setupDatabase() has completed, as required by Oracle and PostgreSQL.
** Fixed OracleInstaller::getConnection() so that it respects $this->useSysDBA correctly.
** In OracleInstaller, added some more variables to the list of schema vars, which are needed by user.sql and tables.sql
** In SearchOracle, specify the database name when calling ctx_ddl.sync_index(). This fixes a fatal error in the createMainpage step, caused by the schema name not being equal to the current user.

* In oracle/tables.sql, fixed a couple of indexes with missing table prefixes.
* Improved debugging output in DatabaseInstaller::getConnection() and Installer::createMainpage().
* In DatabaseBase::selectDB(), set $this->mDBname correctly, as in DatabaseMysql.

Removed getConnectionOrDie(), accidentally added in r80957

Fixup query fail from r80936

Localisation updates for core and extension messages from translatewiki.net (2011-01-26 21:38:00 UTC)

Followup fixme on r80861,

Do cl_from <> 0 check not in SQL, remove order by.

Fixup casing error noticed also

Escape the output of $wgLang->timeanddate() in SpecialContributions. Was escaped in one code path but not in the other

* Whitespace fixups
* followup r66675 - Robla caught a problem with the parameter munging

MessageCache.php: fixed a typo, tweaked spacing, added braces where needed, etc.

Last $wgArticle junk in Skin(Template). Just pass the article on hand to lastModified(). Nothing outside of here calls it anyway

syntax error, stupid me

Followup r81034, remove the global statements

First round of $wgArticle removals

Hooks.php: trim trailing spaces, tweak docs, etc.

* Replaced $wgMessageCache by MessageCache::singleton(); since we only use one instance of this class (as for ParserCache, LinkCache)
* MessageCache::singleton() calls wfGetMessageCacheStorage() directly instead of using $messageMemc, just in case this would be called before that variable is set
* Per TimStarling: also removed deprecated methods in MessageCache class: addMessages() and related, [get|set|enable|disable]Transform(), loadAllMessages(), loadMessageFile() and some others. Same for the legacyData stuff in LocalisationCache that was only used by MessageCache::addMessages() and related.
* Converted remaining extensions

Create index only if it doesn't already exist

Provisionally reverting r80430 (bug 529, 12974)

As noted in CR there are some issues where we're trading one set of weird behavior for a different, but previously unknown, set of weird behavior which breaks existing markup that works around the old behavior.
I'd recommend keeping this in store for after the 1.17 stuff calms down so unexpected parser changes aren't cropping up in the middle of things for people working with trunk.

If these are the right changes to make then great -- but they should be done after the consequences are better understood and folks can prepare for changes.

Transforming a message uses automatically $wgTitle, which may not be set if running from eg. a script, usecase broken by r80819.
Creating a new title in transformMsg() in such case. Reverting r80987 (its revert), and r80898 (workaround).

Confirmed that now both the installer and extensions/Translate/scripts/sync-group.php work.

generateUpgradeKey() could produce NULL return, which means a fatal error when
Call to a member function isOk() in includes/installer/WebInstallerPage.php on line 1039

The function wfShorthandToInteger() is precisely for that.
The previous way only worked for megabytes, which made it
'raise' the memory limit from several gigabytes to 50M

Make the UploadStash repo specific by creating FileRepo::getUploadStash(). In practice this will probably not be used and makes getting an UploadStash object slightly more type work, but I think it is cleaner to have an upload stash explicitly bound to a repo.

Test uploading a file of size $wgMaxUploadSize

TODO: method testMaxUploadSize() shoud be able to use differents
settings.  The assertion could then get moved elsewhere in a nice
helper.

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

Uploads that throw a warning (UploadFromStash) a thumbnail is now shown.

HTMLForm: All section headers and footers. Added an optional second $section parameter to addHeaderText and addFooterText.

revert r80819 for now so we can use phpunit

function foo (Title $title = null ) {} does not work

Fix r80982, re-add accidental removed AND during debugging

Followup r80856, don't unconditionally add the "AND" as if '' is returned by a function, we can get some silly errors (Yay for crappily built SQL strings)

Remove trailing whitespace

Small fixes
* Missing space in SpecialPage.php
* Raising z-index of .suggestions (when used in a jQuery UI modal box the suggestion list appeared behind the modal instead of on top (ui modal has z-index: 1000; )

Resolves bug 26931, where comments were not only not being properly stripped out, but were being transformed into syntax errors.

LanguageConverter::captionConvert(): remove HTML tags and escape HTML special chars to prevent disrupting the layout.

whitespace fix (tab too many)

Follow r80973, fix a tiny error.

Update Chinese conversion tables.

Reset the cache used in User::idFromName(), otherwise tests will try to add the user '127.0.0.1' multiple times, resulting in a database error (duplicate key for `user_name` field)

Allow the $result parameter of the isValidPassword hook to be an array if the message requires parameters

* Fixed a bug causing the installer to ignore the "engine" and "charset" settings when installing a MySQL database.
* Fixed a bug causing the engine and charset settings to not be properly preserved when adding new tables on upgrade.
* Fixed total breakage of SQLite upgrade, by reusing the administrative connection to the SQLite database instead of creating a new one when wfGetDB() is called. Added LBFactory_Single to support this.
* Introduced a "schema variable" concept to DatabaseBase to avoid the use of globals for communication between the installer and the Database. Removed a lot of old global variable names from Database::replaceVars(), most were only added on a whim and were never used.
* Introduced DatabaseInstaller::getSchemaVars(), to allow schema variables to be supplied by the DatabaseInstaller child classes.
* Removed messages config-mysql-egine-mismatch [sic] and config-mysql-charset-mismatch. In the old installer it was possible for users to request a certain character set for an upgrade, but in the new installer the question is never asked. So these warnings were shown whenever a non-default character set or engine was used in the old database.
* In MysqlInstaller::preUpgrade(), fixed the incorrect strings used to identify the MySQL character sets: mysql5 instead of utf8 and mysql5-binary instead of binary.
* On install, initialise the site_stats table, using code copied from the old installer. Unlike the old installer, use SiteStats to increment the user count when the initial user is added.
* Fixed several instances of inappropriate call-by-reference.
* Replaced call_user_func_array() with call_user_func() where possible, it is shorter and simpler.
* Moved the caching boilerplate for DatabaseInstaller::getConnection() to the base class, and have the derived classes override an uncached function openConnection() instead. Updates r80892.
* In MysqlInstaller::getLocalSettings(), escape PHP strings correctly with LocalSettingsGenerator::escapePhpString().
* Reduce timeout for checks in dirIsExecutable() to 3 seconds, so that it doesn't take 30s to run when apache is in single-threaded mode for debugging.
* MySQL and SQLite have been tested and they appear to work. PostgreSQL upgrade is totally broken, apparently it was like that before I started. The Oracle code is untested.

fixed typo from r80924

Couple more raw SQL to query arrays

Missed the case where there is an equals.
It's ugly having to add that literal node there.

port rtl specific parts of modern RTL css to main.css

In the new auto flipped CSS, the "html > body div#mw_contentholder ul li" specific hacks were breaking layout on other browsers, so removing those.

Follow up to r80776

Integrate what was functionally left of common_rtl.css into oldshared.css
Commit the Resourceloader conversion of oldshared.css that I had forgotten to commit

Follow up to r80785

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

update colors (I could not see them)

moving tests to the mediawiki.util section + note about order

Tweak comments on User::isValidEmailAddr to replace the old @todo for RFC 2822 validation with a brief explanation of why we're using this instead. (followup r80913)

Email validation for @localhost address + tests

See r80694 commit note

(bug 26907) MediaWiki Default -> MediaWiki default

User::isValidEmailAddr comment update

Follow up r75682

Explicitely enable language and title conversion otherwise this breaks tests when disabled in local config

Resolves issue in r80656 where escaped quotes within strings are not handled properly, resulting in the string being truncated.

Silly bug with value nodes.

Workaround null Title error in parser during main page creation. Used $wgTitle. I feel dirty

Fix for r80864: the $dbname variable was not there only to make pretty debug output, but also to let the connection choose the correct database. This was breaking connections to other databases since they were always using wiki's database.

* Cleanup massive duplication across Database constructors. Default implementation fairly sane. Now they all share the same if( $server ) logic to allow constructing the class without forcing open a connection (MySQL has done this since at least r15094)
* Get rid of intermediate installTables() callback
* Actually cache the result of DbInstaller::getConnection() like the documentation says

Followup r80864: DatabaseSqlite exploded on installer page because it tried to open an invalid $wgSqliteDataDir.

Even though it doesn't use $server, only call open() if it's set, like in the other constructors.

The amount of code duplication here is stupid.

Followup to r79848 (and really, make it useful...)

Turn DatabaseBase::classFromType() into newFromType() factory function for constructing a new object based on a given type and (optional) params. Documented it fairly clearly.

I think it looks nicer :)

Fixup usage of raw sql in Title

Followup r80856, add missing __METHOD__

Start another attack on raw sql queries

(bug 26827) Fix JS error on IE caused by the rename of {save,restore}StuffForIE() not updating callers. Changed callers to use {save,restore}Selection() or {save,restore}CursorAndScrollTop() as seemed appropriate

(bug 26895) in /include/db/LoadBalancer.php function "closeConnecton" should be called "closeConnection"

Seemingly no callers in extensions OR phase3 (weird...), searching for "closeConnecton" or the correctly spelt "closeConnection"

The back-compat typo thing probably wants removing.. Seems to have been as such since it was added in r32578...