Update.

Check for valid timestamp input, thanks to Brion for pointing this out.

Appears to be a dupe of the line above it, and ParserOptions::initialiseFromUser() [ which is called by the ParserOptions constructor, which in turn is called by ParserOptions::newFromUser() ] does not appear to have any side effects.

$fname is not defined / null in these contexts, replacing with __METHOD__

$functionname not defined in this context, replacing with hardcoded "-setup" array key.

Probable variable name typo, currently passing ref to NULL.

Variable name typo, but again I'm guessing this function isn't used if no-one has complained.

Connecting to the database may help, but I'm guessing this function isn't used if no-one has complained.

Completing code housekeeping stuff for rest of includes/ directory: removing unused local vars, removing unused globals, replacing extract() where simple to do, declaring output arrays before calling preg_match(), and so forth.

convert "::1" and other pseudo-IPv6 addresses that Apache may throw at us to their IPv4 counterparts

*Automatically pre-fill the edit-summary field when undoing an edit.*Add a message at the top of the edit page when undoing an edit. This gives the user some feedback that their undo actually worked, rather than just directed them to an edit page. (Unless you actually look, there's no evidence of a successful undo, which could confuse users)

Prevent the following strict-standards warnings - i.e. when running with error_logging(E_ALL | E_STRICT); - which seems to disable the yucky "@" operator, as well as maxing out the pedantry of warnings. Nothing major found, just nice to be as explicit and as forward-compatible as possible.

* Strict Standards: Undefined index: switch in includes/Parser.php on line 3849
* Strict Standards: Undefined index: ref in includes/Parser.php on line 3818
* Strict Standards: Non-static method OutputPage::setEncodings() should not be called statically in index.php on line 11
* Strict Standards: Only variables should be assigned by reference in includes/Skin.php on line 888
* Strict Standards: Non-static method Title::newFromURL() should not be called statically in includes/SpecialContributions.php on line 178
* Strict Standards: Only variables should be assigned by reference in includes/GlobalFunctions.php on line 2054
* Strict Standards: Undefined index:  contributions-summary in languages/Language.php on line 764
* Strict Standards: Undefined index:  trackbackhtml in skins/MonoBook.php on line 86
* Strict Standards: Undefined index:  blockip in skins/MonoBook.php on line 204
* Strict Standards: Undefined index:  tagline in skins/MonoBook.php on line 261
* Strict Standards: Undefined index: uselang in includes/SkinTemplate.php on line 1159
* Strict Standards: Non-static method CoreParserFunctions::plural() cannot be called statically in includes/Parser.php on line 2902
* Strict Standards: Undefined offset:  0 in includes/SkinTemplate.php on line 196
* Strict Standards: Undefined index:  USE INDEX in includes/Database.php on line 1015
* Strict Standards: Undefined index:  image_tests in includes/Parser.php on line 3488
* Strict Standards: Undefined offset:  0 in includes/Parser.php on line 3507
* Strict Standards: Non-static method ChangesList::newFromUser() should not be called statically in includes/SpecialWatchlist.php on line 361
* Strict Standards: Non-static method RecentChange::newFromCurRow() should not be called statically in includes/SpecialWatchlist.php on line 367
* Strict Standards: is_a(): Deprecated. Please use the instanceof operator in includes/Exception.php on line 168
* Strict Standards: Non-static method LogPage::logName() should not be called statically in includes/SpecialContributions.php on line 325
* Strict Standards: ob_end_flush(): failed to delete and flush buffer. No buffer to delete or flush. in maintenance/commandLine.inc on line 191
* Strict Standards: Undefined index:  meatball in languages/Language.php on line 234
* Strict Standards: rmdir(/tmp/mwParser-2108164586-images/thumb): Directory not empty in maintenance/parserTests.inc on line 605
* Cleaning out some new temp files left over by parserTests (there were one or two straggler dirs/files that would persist after the test run ended, due to new tests being added over time)
* Strict Standards: Non-static method CoreParserFunctions::special() cannot be called statically in includes/Parser.php on line 2902
* Strict Standards: Declaration of ListUsersPage::preprocessResults() should be compatible with that of QueryPage::preprocessResults() in includes/SpecialListusers.php on line 38
* Strict Standards: Only variables should be passed by reference in includes/SpecialBlockip.php on line 175
* Strict Standards: Skin::include_once(skins/Standard.deps.php) [<a href='function.include-once'>function.include-once</a>]: failed to open stream: No such file or directory in includes/Skin.php on line 121
* Strict Standards: Declaration of ApiMain::getResult() should be compatible with that of ApiBase::getResult() in includes/api/ApiMain.php on line 35
* Strict Standards: is_a(): Deprecated. Please use the instanceof operator in includes/WikiError.php on line 63
* Strict Standards: Non-static method WikiError::isError() should not be called statically in includes/SpecialImport.php on line 64
* Strict Standards: Non-static method ImportStreamSource::newFromInterwiki() should not be called statically in includes/SpecialImport.php on line 58<b
* Strict Standards: Only variables should be assigned by reference in includes/SpecialUndelete.php on line 501
* Strict Standards: Non-static method Image::newFromName() should not be called statically in thumb.php on line 56
* Strict Standards: Non-static method CoreParserFunctions::numberoffiles() cannot be called statically in includes/Parser.php on line 2902
* Strict Standards: Non-static method CoreParserFunctions::statisticsFunction() should not be called statically in includes/CoreParserFunctions.php on line 139
* Strict Standards: Non-static method CoreParserFunctions::isRaw() should not be called statically in includes/CoreParserFunctions.php on line 128
* Strict Standards: Non-static method CoreParserFunctions::grammar() cannot be called statically in includes/Parser.php on line 2902
* Strict Standards: Undefined offset:  1 in includes/SpecialMIMEsearch.php on line 130
* Strict Standards: Undefined index:  recentchangeslinked in skins/MonoBook.php on line 184
* Strict Standards: Declaration of DumpNotalkFilter::pass() should be compatible with that of DumpFilter::pass() in includes/Export.php on line 612
* Strict Standards: Declaration of DumpNamespaceFilter::pass() should be compatible with that of DumpFilter::pass() in includes/Export.php on line 665
* Strict Standards: Non-static method ImportStreamSource::newFromUpload() should not be called statically in includes/SpecialImport.php on line 46
* Strict Standards: Undefined offset:  5 in includes/Sanitizer.php on line 396
* Strict Standards: Undefined index:  wikidbUserName in includes/SpecialUserlogin.php on line 562
* Strict Standards: Only variables should be assigned by reference in includes/api/ApiQueryBase.php on line 95
* Strict Standards: Only variables should be assigned by reference in includes/api/ApiQueryBase.php on line 116
* Strict Standards: Only variables should be assigned by reference in includes/api/ApiQueryWatchlist.php on line 128
* Strict Standards: Undefined property:  stdClass::$rc_id in includes/api/ApiQueryBase.php on line 131
* Strict Standards: Undefined property:  stdClass::$rc_last_oldid in includes/api/ApiQueryBase.php on line 164
* Strict Standards: Undefined property:  stdClass::$rc_moved_to_ns in includes/api/ApiQueryBase.php on line 285
* Strict Standards: Undefined property:  stdClass::$rc_patrolled in includes/api/ApiQueryBase.php on line 176
* Strict Standards: Undefined index:  comment in includes/api/ApiFeedWatchlist.php on line 85
* Strict Standards: Undefined offset:  0 in includes/Skin.php on line 302
* Strict Standards: Non-static method User::SetupSession() should not be called statically in includes/SpecialUserlogin.php on line 15

... There are certain to be other things too, so this is not intended to be comprehensive, rather the above just stops most of the notifications I observed.

(bug 6016) Left-aligned images should stack vertically, like right-aligned images, not horizontally.

Fix broken contribution listings with postgres by adding a new variable
to check if we are using integers or explicit timestamps.

Slight UPDATE syntax adjustment for non-mysql databases.

* (bug 6329) Date/Time settings on Frisian wikis do not conform to Frisian usage.

* (bug 6910) Dates in Vietnamese localization broken

Revert r17964, breaks diff links for no apparent reason.

Revert special:deletedcontribs for now; it's not ready to go live without the
index change so would hold up other work for a few days, and the code's kind
of ugly and should get refactored.
Would love to see this on a branch for the moment.

New special page Special:Deletedcontribs for browsing deleted edits made by a user.

NOTENOTENOTE: This is a one-night hack; feel free to tweak it mercilessly.  It has
a lot of common code with Special:Contributions that should be factored out.  There
is no link to this new page anywhere in the UI yet; I'd like people to test and
review this a bit before officially announcing it.  The page is currently only
available to users with deletedhistory privileges; a possible additional feature
might be to allow all users to list (but not view) their _own_ deleted edits.

make undelete preview more consistent with normal article view (tidy, pagename)

Woops, left a parameter off of the AuthPlugin default interface class. *embarrassed*

* Don't mess up diff link when revision is not deleted

* Indentation, magic numbers

*I like verbs

* Xmlify a little

* Fix regression when selecting mysql 4.0 schema in installer
* Detect PHP parse errors in generated config and dump the file
  as a debugging aid, instead of silently corrupting the behavior
  of the installer

Message order compatibility.

Fixed wikia link

Fixes.

(bug 8012) Update for messagesSv.php (Swedish language)

Changing lines like this: "extract( $dbw->tableNames( 'page', 'archive' ) );" to be like this: "list ($page, $archive) = $dbw->tableNamesN( 'page', 'archive' );".

Three reasons for this:
1) It's better for analysis tools [which want explicit variable declaration]
2) It's easier for a human to read, as it's completely explicit where the variables came from [which is something you don't get with extract() ]
3) It makes it easier to find everywhere where a variable is used with search/grep [which you can't currently do with $tbl_page variables from things like: "extract($db->tableNames( 'page', 'revision'), EXTR_PREFIX_ALL, 'tbl');"].

Otherwise, from a functionality/efficiency perspective the two forms should be identical.

By doing this have been able run static analysis over the usages of these variables, thus eliminating 5 unneeded table names from calls, plus removing 3 unused calls entirely, and it just feels subjectively slightly nicer to me.

Fixed Parser::getRevisionTimestamp()

Make passing an invalid revid for undoing fail gracefully

Fix display of link in "already rolled back" message for image/category pages ($1 -> :$1)

Fixing a PHP warning about passing variables by reference from the function call (should be passed from the function decleration itself, and they are passed in that way), like the other call to wfMerge in this file; tested.

(bug 8040) Turkish internationalization contains errors

(bug 8039) Update to Indonesian localisation (id) #42

Update.

* Add new editFormPageTop placeholder for text injection
* Use that instead of editFormTextTop for undo failure, so it displays above the preview if applicable
* <strong class="error"> for undo failure
* Explicitly declare $editFormX as public

* Add AOL IPs to whitelist by default
* Add period to "undo failed" message
* Use <p>, not <br />

Add message explaining if undoing an edit failed.

* (bug 6295) Add a 'revision patching' functionality, where an edit can be undone (with a functionality similar to diff rev1 rev2 | patch -R rev3 -o rev3). This is triggered by including &undo=revid in an edit URL. A link to a URL that will undo a given edit is shown on NEW NON-CURRENT revision headers on diff pages.

(bug 8009) Title change for ancientpages in de

oops

Fixed inefficient use of array_keys() introduced by Nick in r17880. If this is setting off alarms, then fix the alarms.

Removed wfRangeStartEnd() and wfIsAddressInRange() -- avoid proliferation of global functions.

* Added temporary special-case AOL proxy detection, they're automatically counted as trusted proxies for now.
* Removed wfRangeStartEnd() and wfIsAddressInRange() -- avoid proliferation of global functions.

(bug 7859) Updates for Kazakh i18n

(bug 7859) Updates for Kazakh i18n

hooks.txt documentation for UploadForm:initial hook added in r17917

Fix.

(bug 8031) Update to Japanese localisation (ja)

Patch by Leon Weber that adds a hook 'UploadForm:initial' before the upload form is generated, and two member variable for text injection into the form, which can be filled by the hooks.

* Use INSERT IGNORE for db-based BagOStuff add/insert, for more memcache-like
  behavior when keys already exist on add (instead of dying with an error...)

* Installer support for experimental MySQL 4.1/5.0 binary-safe schema

(bug 8031) Update to Japanese localisation (ja)

(bug 8009) Automatic edit summary for redirects is not filled for edits in existing pages

Removing several ignored messages from the language files.

Removing newtalkseperator from all the language files except for English.

(bug 8026) Update to Japanese localisation (ja)

Probably a typo.

(bug 7859) Updates for Kazakh i18n

(bug 8020) Update for Cantonese language (zh-yue) #6

Setting newtalkseperator as an ignored message, and removing it from the Hebrew translation; will remove from other translations later.

(bug 8012) Update for messagesSv.php (Swedish language)

Update.

(bug 7892) Add media count to category page, à la subcats/pages.  Used a new count() method I added to ImageGallery class.

Update.

As per Rob's suggestion, remove autosummaries for short new pages, and replace them with autosummaries for all new pages. On more reflection, this makes a lot more sense

(bug 8013) Add autosummary for new pages with >500 characters. I would merge this with the original short new page autosummary, but this would break current message customisations on other wikis. en.wp comes to mind.

Would probably be wanting "$wgStyleVersion" rather than "wgStyleVersion", otherwise if metadata.js changes can bump $wgStyleVersion till the cows come home, but still won't cause it to be refreshed ;-)

Probable bug in wfIsAddressInRange() - $unsignedIP and $unsignedip are not the same thing. One is defined, and the other is undefined / null.

Code housekeeping stuff (and barring any stuff-ups on my behalf, there should be no changes in behaviour whatsoever after this) -
* removing some unused global declarations.
* removing or commenting out or adding comments for unused local vars.
* Adding one or two local var declarations.
* Declaring $matches array passed to preg_match() / preg_match_all() as array() before using [not required, just have a slight preference for the explicitness].
* remove one or two pass-by-reference function declarations where the value is not modified.
* Adding some braces to if-else blocks.
* In Parser.php, stripstrate is now an object rather than an array as per r17820, so we no longer need ask for a reference to it (as in "$x =& $this->mStripState;"), and in fact it's probably just simpler to get rid of $x altogether.
* Moving some preg regexes from "" quoting to '' quoting to stop static analyzer whinging about bad escape sequences.

... up to "LinksUpdate.php" in the includes/ directory.

wfGetIp-> in checking autoblock whitelist in doAutoblock. Kudos to Simetrical for spotting the issue.

Revert last three commits.  I totally broke autoblock and didn't notice somehow.  :|

More bugs!  Why did I only spot these after commit >:|

Argh, syntax error

* Moved Werdna's new IP functions to the IP class
* Handle bad input to new IP::isAddressInRange gracefully (return false)
* Block::doAutoblock always returns a bool now, as the docs say
* Split off Block::isWhitelistedIp from Block::doAutoblock
* Put AOL proxy IPs in whitelist, and also one from Singapore that was troublesome on enwiki (more should be added, probably?)
* Improve some docs
* Fix a bug: check if the passed IP is in the whitelist, not if the request IP is in the whitelist

Added placeholders for text injection by hooks to EditPage.php

There used to be a big warning message, I think. Putting it back because nobody reads the damn documentation.

Removing several ignored and optional messages.

(bug 8008) Update to Indonesian localisation (id) #41

revert until proper cleanup

Update; setting autoblock_whitelist to be an ignored message.

remove 'subkey' (255) for a field which is 40-chars long, fixes stuff...

fixed a special page name comparison

* (bug 7883) Added autoblock whitelisting feature, using which specific ranges can be protected from autoblocking. These ranges are specified, in list format, in the autoblock_whitelist system message.

Applying patch (slightly modified) from bug 8003 by AlefZet. The patch is clearly necessary, but I'm not sure if it is sufficient. Also fixed indenting.

Implementation of delimiterReplace() with a behaviour much closer to that of the model regex. Tested using comparitive fuzz testing. The only known difference now is where the start delimiter ends with an initial substring of the end delimiter, e.g. the previously mentioned case of C-style comments.

(bug 7554) The correct MIME type for SVG images is now displayed on the image page (image/svg+xml, not image/svg).  This is a hack because it only affects display, and doesn't correct the internal storage.  I don't know how the latter works, so I wasn't confident touching it.

Since formatComment and commentBlock can take null as well as Title objects (per r17833), corrected documentation (Title -> mixed).  Since revComment would immediately fail if passed anything but a Revision object or descendant, restore type hinting to that.

Fixes for the script rebuildLanguage.php:
* (bug 7983) Sort blocks in the correct order
* Sort the unknown messages alphabetically
* Use single apostrophe on messages which include dollar signs which are not followed by numbers

(bug 7984) Update to Indonesian localisation (id) #40

(bug 7978) New localization for Old Church Slavonic

Fix.

Reverting the addition of class names near the parameters: it doesn't seem to have a purpose, and breaks PHP 5.0.4 on my computer (PHP raises error when the parameters are set to null).

Replacing the manual user and user talk links of the blocker in Special:Ipblocklist to Linker::userLink and Linker::userToolLinks; for that, added the function Block::getBy, to get the blocker user ID.

bump style version for wikibits.js change