Adding .gitreview -- first official commit to mediawiki core in git \o/

Change-Id: Ica46ab303c89679ea8575f75f51830f36bf972d3

* Removed the $method parameters from MWCryptRand. Apparently Dantman didn't know about our awesome debug traceback functions like wfGetAllCallers(). The weird optional-middle-parameter calling convention thankfully disappears as a consequence.
* Reduced the amount of debug log noise slightly, removing a few redundant messages.

Fix Bug 13896:  Rendering of devanagari numbers in automatic '#' number lists
Add -moz-devanagari as list type for mr and hi language ordered lists.

Revert r107309, r113601, r113704, r113742, r113792, r113838, r113859, r113893, r113894, r113952, r114047, r114252, r114256, r114257. This reverts the remaining 'new' revisions in core.

All of these revisions are tagged with 'gerritmigration' and will be resubmitted into Gerrit after the Gerrit switchover. See also http://lists.wikimedia.org/pipermail/wikitech-l/2012-March/059124.html

Revert r114067, r114071, r114075, r114079, r114081, r114082, r114084, r114086, r114088, r114089, r114101, r114118, r114137, r114147, r114164, r114172, r114175, r114180, r114208, r114209, r114215, r114219, r114226, r114321, r114322.

This reverts the SpecialCachedPage and formatDuration sagas, with some collateral damage here and there. All of these revisions are tagged with 'gerritmigration' and will be resubmitted into Gerrit after the Gerrit switchover. See also http://lists.wikimedia.org/pipermail/wikitech-l/2012-March/059124.html

Security paranoia, reject requests to router.php that aren't from the cli-server sapi.

added some tests using the intervals param

handle cases where an empty array is provided by using this as default, also correctly handle cases where the smallest unit is not the second and the result is 0 smallestunit

* Use local context to get messages
* Use Language::userTimeAndDate() instead of Language::timeanddate() to display the date with user preferences
* Set the form's action to $wgScript instead of direct link to the page since it makes a GET request (and the title would be displayed twice with pretty URLs since there's already a <input name="title" .../>

Followup r114270 (essentially reverts it), and r114233, use class member variables as they are already defined

Followup r114233, define the method static variables to be used

Follow-up to r114256: Removing final assert

Follow up to r114252: Removing asserts

Refactoring dumpTextPass's error handling

* (bug 34735) Updated compressOld.php documentation to mention the different usages of -s and -n parameters depending on compression type

Based on patch by christian@quelltextlich.at - http://bug-attachment.wikimedia.org/attachment.cgi?id=10105

(bug 35316) On autocreate, use $this as the "doer" in the newuser log entry, since unlike User::addNewUserLogEntry(), there is no way the action could have been triggered by a different user. Usually $wgUser is set correctly, but there's no need to rely on it here.

Commit the cryptrand project worked on in git:
- MWCryptRand: A new api for generating cryptographic randomness for security tokens. Uses whatever cryptographic source is available and if not falls back to using random state and clock drift.
- wfRandomString - A simple non-cryptographic pesudo-random string generation function to replace wfGenerateToken which was written pretending to be secure when it's really not.
- Core updates to use MWCryptRand in various places:
-- user_token generation (to do this we stop generating user_token implicitly and only generate it when needed to avoid depleting the system's entropy pool by reading random data we'll never use)
-- email confirmation token generation
-- password salt generation
-- temporary password generation
-- Generation of the automatic watchlist token
-- login and create user tokens
-- session ids when php's entropy sources are not set
-- the installer when generating wgSecretKey and the upgrade key

Better bug reference for r114231.

Fixed a few "strip tag exposed" bugs.
* Introduced Parser::killMarkers() based on the concept from StringFunctions. Used it in cases where markerStripCallback() doesn't make sense semantically, namely grammar, padleft, padright and anchorencode. Used markerStripCallback() in other cases.
* Changed headline unstrip order as suggested by P.Copp on bug 18295
* In CPF::lc() and CPF::uc(), removed the is_callable(). This was a temporary testing hack committed by me in r30109, which allowed me to do differential testing against a copy of the parser from before that revision.

[FileRepo] Added long-since missing FOR UPDATE to LocalFile::lock(). This might help with the weird mismatched metadata for files with two uploads at the same time that I keep seeing in the DB.

added enabled field so you can disable the cache thing if you need to before its been initialized and fixed copypaste error

Fixed attempt to write to the wiki's upload directory during parser tests instead of /tmp, this is not allowed. I'm not sure what setupGlobals() is meant to do nowadays, that part is untested. It seems to be duplicated in ParserTest::__construct().

follow up to r114215, fix some fails and added CachedAction implementing the same stuff as SpecialCachedPage

Follow up to r114126: Being more conservative for HipHop compiler

split general cache helper functionality to its own class, so we can also easily use it in stuff that does not derive from SpecialPage, such as Action

fix subpage linking

only init if not already done so

More occurrence typos

Localisation updates for core and extension messages from translatewiki.net

Make the first line of debug log more consitent between CLI and web call by putting the method and the URL directly on the first line

make more generic: do not assume we want to add html to the output. Also get rid of action=purge from key args

fix for r114164

update docs

some refactoring to allow for nicer usage in deriving classes

Reverted r113688, r113691 per CR and filed #61440 upstream.

* Use local context to get message
* Use $this->getTitle() instead of SpecialPage::getTitleFor( 'Preferences' )

Follow up to r114081; address krinkles comment

Fix switch fall through

Remove code for getting svn rev from SVN less than 1.4 - Ubuntu 8.04 LTS has 1.4.6 so should be pretty safe

[Html::namespaceSelector clean up] fix broken html from r114131

Follow up to r114081;

Removed XmlTest::testNamespaceSelector(); it just breaks when $wgDevelopmentWarnings is set to true due to the fact that Xml::namespaceSelector() calls wfDeprecated().

Change calls from Xml::namespaceSelector() to Html::namespaceSelector() since the former now throws deprecation warnings...

Sanitizing already existing use of PHP's assert

follow-up r114116: move release-notes

Bug 35309 - formatDuration message missing/inconsistent

Patch by Dan Collins

* (bug 35303) Make proxy and DNS blacklist blocking work again

[jquery.byteLimit] Set vars in return this.each loop
* Set vars in return this.each loop. This is the defacto standard plugin structure
  but somehow it slipped through this one (it's a 2 line wrapper, easy to miss).
* Added unit test (which failed before this commit)
* Fixes:
-- (bug 35294) jquery.byteLimit shouldn't set element specific variables outside the "return this.each" loop.

[jquery.byteLimit.test] clean up
* js conventions
* whitespace indention

Localisation updates for core and extension messages from translatewiki.net

Disable callgraphs for the time being

Per Yuvi, most people don't need them, and are a PITA on a slow connection

I'd be happy to re-enable them if we can get them on hyperlinks or subpages or some such

{{BeBold}}

follow up to r114081: qqq, and to r114082: match rename

follow up to r114084, add tests for stuff w/ multiple units

Trim trailing whitespace

Add/improve documentation

Explicitly define a few variables

Noralise a few of the magic numbers used in r114084

Unit tests for r114067

Followup r114067 rename to formatDuration to match other methods

adding special page with scaffolding for caching chunks of HTML

Followup r114067

Add centuries because the messages were already there but not used

Add decade and millennia for the luls/completeness

follow up to r114067, use correct language

Follow-up r113939: Add divs to addWikiText().

Follow up to r114067;

adding a duration function to language that converts seconds to text

Localisation updates for core and extension messages from translatewiki.net

follow up r113862 . changed public function name extensionTableExists to tableExists

fix for bug33214 - catch all exceptions in api execute and provides necessary parameters required by the custom printer

Localisation updates for core and extension messages from translatewiki.net

partial fix for bug 35057 - backporting a patch from jQuery 1.7.2pre

Remove incorrect "the" in 'filereadonlyerror'.
Make use of reasons more consistent in a few messages.

Make UsageException extend MWException

In the $clearState=true case, remove strip marker prefixes from the Parser::parse() input. It's only possible for strip markers to be present in this case if the caller manages to guess the output of Parser::getRandomString() in advance. Even if that is possible, it's not recommended as a robust calling procedure.

Also cleaned up some old commented-out code in this area.

docs++

Localisation updates for core and extension messages from translatewiki.net

[FileRepo] Tweaked transformErrorOutput() to allow more useful error output (like backend errors) for problems moving thumbnails into storage.

* r113886: use <div> instead of span to wrap element that has p tags.
* Also fixed an older instance of the same problem.

Move RELEASE-NOTES after merging r111647 into 1.19 in r113933

Follow-up r113922: tests

Bug 35051 - [Regression] If heading has a trailing space after == then its name is not preloaded into edit summary on section edit. Regression from r105380.

[jquery.client] clean up
* Move translate() out of the if-block, no need to re-define
* `else` after return in `if`
* update js coding style
* remove bogus '=' in comment
* line-wrapping of long arrays

[mediawiki.action.watch] clean up
* Fix implied global variable 'i' in mwUriGetAction()
* Indention nearly everything an extra level (svn di -x-uw)
* other minor js coding style

[mediawiki.action.watch] Move re-used logic into local function
* Follows-up r112440, r107969, r107350

Fixed 'Illegal string offset 'LIMIT' in <b>C:\wamp\www\MediaWiki\includes\db\Database.php</b> on line <b>1462</b>'

Followup-To: r113888 Remove extra wfDebug()

Unbreak maintenance/deleteDefaultMessages.php for PostgreSQL

deleteDefaultMessages.php was failing during upgrade
from MediaWiki 1.7.3 with a databaser error.

A stub user:

   $user = User::newFromName( 'MediaWiki default' );

has user ID 0, so that $user->isAnon() is true.

Unfortunately, ManualLogEntry::publish() from r96441
tries to insert $user->getName() ("MediaWiki default")
into rc_ip.

PostgreSQL won't allow this, because rc_ip is of
Postgres-specific CIDR type.

Traceback:

Checking existence of old default messages...
...deleting old default messages (this may take a long time!)...A database query syntax error has occurred.
The last attempted database query was:
"INSERT INTO "recentchanges" (rc_timestamp,rc_cur_time,rc_namespace,rc_title,rc_type,rc_minor,rc_user,rc_user_text,rc_comment,rc_this_oldid,rc_last_oldid,rc_bot,rc_moved_to_ns,rc_moved_to_title,rc_ip,rc_patrolled,rc_new,rc_old_len,rc_new_len,rc_deleted,rc_logid,rc_log_type,rc_log_action,rc_params,rc_id) VALUES ('2012-03-14 21:51:05 GMT','2012-03-14 21:51:05 GMT','8','1movedto2','3','0','0','MediaWiki default','No longer required','0','0',1,'0','','MediaWiki default','1','0',NULL,NULL,'0','1','delete','delete','a:0:{}','1')"
from within function "RecentChange::save".
MySQL returned error "1: ERROR:  invalid input syntax for type cidr: "MediaWiki default"
LINE 1: ...ki default','No longer required','0','0',1,'0','','MediaWiki...
                                                             ^"
Backtrace:
#0 /usr/home/saper/public_html/pg/w/includes/db/DatabasePostgres.php(332): DatabaseBase->reportQueryError('ERROR:  invalid...', 1, 'INSERT INTO "re...', 'RecentChange::s...', '')
#1 /usr/home/saper/public_html/pg/w/includes/db/Database.php(904): DatabasePostgres->reportQueryError('ERROR:  invalid...', 1, 'INSERT INTO "re...', 'RecentChange::s...', '')
#2 /usr/home/saper/public_html/pg/w/includes/db/DatabasePostgres.php(604): DatabaseBase->query('INSERT INTO "re...', 'RecentChange::s...', '')
#3 /usr/home/saper/public_html/pg/w/includes/RecentChange.php(199): DatabasePostgres->insert('recentchanges', Array, 'RecentChange::s...')
#4 /usr/home/saper/public_html/pg/w/includes/logging/LogEntry.php(479): RecentChange->save('pleasedontudp')
#5 /usr/home/saper/public_html/pg/w/includes/WikiPage.php(2042): ManualLogEntry->publish('1')
#6 /usr/home/saper/public_html/pg/w/includes/WikiPage.php(1937): WikiPage->doDeleteArticleReal('No longer requi...', false, 0, false, '', Object(User))
#7 /usr/home/saper/public_html/pg/w/maintenance/deleteDefaultMessages.php(73): WikiPage->doDeleteArticle('No longer requi...', false, 0, false, '', Object(User))
#8 /usr/home/saper/public_html/pg/w/maintenance/update.php(128): DeleteDefaultMessages->execute()
#9 /usr/home/saper/public_html/pg/w/maintenance/doMaintenance.php(105): UpdateMediaWiki->execute()
#10 /usr/home/saper/public_html/pg/w/maintenance/update.php(151): require_once('/usr/home/saper...')
#11 {main}

Add error css to errors like the other file operation pages and actions

follow up r113836  typo correction in function header dropExtensionTable

follow up r113836 added public function extensionTableExists.

[FileRepo]
* Added getReadOnlyReason() function to FileRepo to check read-only status. Added such checks to the File object functions that mutate files in storage. This should make read-only mode more tolerable (which is needed at least briefly when switching backends).
* Added lock()/unlock() calls to File restore() function.
* Use proper isOK() accessor for Status objects.

Fix encoding issue in r113197

Fixed comment typo

Localisation updates for core and extension messages from translatewiki.net

Follow-up to r15791: Rename "user" and "text" when upgrading on PostgreSQL

You can lie to me, but not to your installer.

Make DatabasePostgres::tableExists to check
for real table names, not faked ones.

DatabasePostgres is currently lying to the rest
of the MediaWiki that "mwuser" table is actually
called "user" and that "pagecontents" is called
"text". While MediaWiki does not care, the
installer (and updater do).

This allows us to overcome first hurdle
in getting MediaWiki 1.7.3 to update to trunk
on PostgreSQL and uncover further bugs.

For this commit to actually do something,
we rename those tables when upgrading to match
what we have in maintenance/postgres/tables.sql

And by the way, tell installer not to check
for "user" table, since most PostgreSQL users
will have "mwuser" instead. Picking "archive"
instead.

adding two public convenience function wrappers: dropExtensionField and dropExtensionTable

Reverted r113803: makes already messy class messier (a bit to much)

* (bug 34889) User name should be normalized on Special:Contributions. Fixes regression from r103751.

We have DatabaseBase::selectRow() to select a single row

* (bug 34841) Fix for r103502: don't show edit links when display old page versions

Be consistent in the interface and use Title::quickUserCan() instead of Title::userCan() to show items in the interface so that when the user is blocked or the page is cascade protected there is not a part of the interface saying you can edit the page and another part not.

Don't create new Revision and Title object if we can use the ones that we already have

(bug 32827) "[Regression] Block log for IP ranges not shown on Special:Block"

Cosmetic improvements to PostreSQL updater output

* Don't WARN on sequences already existing
* Align dots nicely to the rest