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

W/S cleanup

(bug 27825) Make the raw watchlist edit messages use formatNum.

Hopefully i added to the right section of RELEASE-NOTES. Wasn't sure.

* Bug 27823 - Re-enable functionality of --uploads option of importDump.php; introduce --namespaces filter option for importDump.php

Patch from Dmitriy Sintsov of which he writes:

    I needed to selectively import files from URL's defined in
    previousely made XML dump. This worked fine in 1.15, but to my
    surprise wasn't working with 1.17, producing an error. I've
    partially fixed upload code (including supplying of original User
    instance for local upload) and introduced new option to
    selectively import only the listed namespaces in importDump.php
    command line.

Underscore are not valid in language code

* Reject underscore in validation
* Still case unsensitive
* Corrected tests using underscore

Follow up r83160 which was a follow up of r82927 (language code validation)

(bug 26995) File size is now checked before uploading in HTML5 browsers
* Exposed $wgMaxUploadSize to JS
* Exposed message largefileserver to JS

(bug 27781) Pt 2: Check for minimum PHP version in installer (which is seperate from the PHP4 check)

Cleanup r83022: just use IGNORE

Localisation updates from translatewiki.net.
* Adding support for Liv language with fallback to Estonian.

Also remove trailing whitespace in RELEASE-NOTES

* Added an Ehcache client.
* Fixed encoding of spaces in memcached keys, in r83140 they would have been encoded as "+".

Adding isIp-tests to MW JS Test Suite (based on IPTest.php)
* fixing a typo in variable statement (semicolon->comma, thx JSHint, browsers tend to ignore)
* Clean up comments

Moving IP-check to mw.util & some enhancements to and applying code conventions to mw.special.block.js
* This commit addresses the @TODOs that were in mediawiki.legacy.block before it was migrated to /resources in r83183 ("@TODO: find some better JS file for [is IP functions]")
* JSHint warning fixed: Using ==== when comparing to '' (safer and faster than ==)
* Instead of checking the initial onload state by faking a onkeyup/onchange event, seperate the two functions and use those
** --> This fixes issue where it falsely triggers other bindings on-load that are globally bound to input elements via $('input').live() or $('[type="text"]').delegate() because of the .keyup()/.change() call )
* Caching selectors (var $wpBlockExpiry = $(..) etc.) instead of re-creating a dozen jQuery objects again and getting the document element by id – on *every* onkeyup/onchange
** --> Notable speed improvement in Safari 3 and IE6 with the caching, these had a little bit of a lag when typing fast, mw.legacy.block.js had the lag as well. Fixed now.
* Adding instantToggle-argument to switch between fade or hiding/showing right away. When setting the initial state they're called with instant=true so that there's no fading, the page loads and stuff hides that shouldn't be visible. Any later interaction will still have the fade.
** Calling a function like foo(true) sucks, adding a local DO_INSTANT = true.
* Making behaviour the same as the legacy.block was (if isEmpty, do bring the element back to view ( show() / fadeIn) instead of doing nothing)

Call to two undefined functions. fixed, Follow-up r83183

PHP Warning:  filemtime() [function.filemtime]: stat failed for /www/w/skins/common/block.js in /w/includes/resourceloader/ResourceLoaderFileModule.php on line 365 ; FIXED (r83183)

JavaScript clean-up (Code conventions, cross-browser fixes and JSHint validation)

; history.js
* Applied whitespace conventions + removed trailing whitespace
* Combining var statements

; mediawiki.action.history.js
* Calling in document.ready instead of directly (jQuery(fn) is short for jQuery(document).ready(fn))
** Originally it was a hookEvent('load'), during rewrite in r74397 it was dropped. Restoring now.
* Wrapping in jQuery-to-$
* Combining document.ready and wrapper into one (jQuery internally calls the ready-fn with jQuery as first argument) ($(function(){ $(function(){ /* domready */ })(jQuery); --> jQuery( function( $ ) { /* domready */ } );
* (http://jqbug.com/8229) Attributes must be quoted, just like CSS requires that.

; mediawiki.action.view.rightClickEdit.js
* Wrapping in jQuery-to-$

; IEFixes.js
* Moving function calls within the if statement since the passed arguments are undefined otherwise

Convert Special:BlockIP JS to jQuery/ResourceLoader.  Change the order of the checkboxes on the block form so that options which appear/disappear when the type of user is changed are grouped together.

Followup r82060

Calling intval() on an array of integers isn't going to get people very far

simplify logic, and then only do validation if $min or $max are not null

Localisation updates for core and extension messages from translatewiki.net (2011-03-03 21:42:00 UTC)

Optionally disable including mediawiki.legacy (wikibits.js) module.

Fix language code validation (from r82927)

A language code may contains the underscore character (be_tarask)
and might as well be upper case (FR).

Add tests for Language::isValidBuiltInCode() against some language codes

Merge r83152 from 1.17wmf1 to trunk

rm unused member var

* When CACHE_ANYTHING is requested, return the cached instance, don't construct a new object for each use.
* In MemcachedSessions.php, register a shutdown function to shut down the session early, before $wgMemc is destroyed. I'm not sure why my recent changes caused this problem to show up now.
* Use EmptyBagOStuff instead of FakeMemCachedClient in DefaultSettings.php for CACHE_NONE.

Followup r83140: FakeMemCachedClient -> EmptyBagOStuff in tests

Tweak debug() to  use name of class in debug output

Followup r83141. Forgot to commit the hooks.txt update.

Reduce usage of $wgOut inside Skin.

* Rewrote ObjectCache.php to conform to the modern coding style, and to be less convoluted about how CACHE_ANYTHING and CACHE_ACCEL are resolved. Moved most functionality to static members of a new ObjectCache class.
* Moved the global functions to GlobalFunctions.php, where they are now just convenience wrappers. Made them return non-references. Updated callers (none found in extensions).
* Added an advanced configuration method, $wgObjectCaches, which allows a lot more detail in the object cache configuration than $wgMainCacheType.
* Made all object cache classes derive from BagOStuff.
* Split the MWMemcached class into a generic client class and a MediaWiki-specific wrapper class. The wrapper class presents a simple BagOStuff interface to calling code, hiding memcached client internals, and will simplify the task of supporting the PECL extension.
* Added some extra constructor parameters to MWMemcached, configurable via $wgObjectCaches.
* Removed the *_multi() methods from BagOStuff, my grepping indicates that they are not used.
* Rewrote FakeMemCachedClient as a BagOStuff subclass, called EmptyBagOStuff.
* Added an optional "server" parameter to SQLBagOStuff. This allows the server holding the objectcache table to be different from the server holding the core DB.
* Added MultiWriteBagOStuff: a cache class which writes to multiple locations, and reads from them in a defined fallback sequence. This can be used to extend the cache space by adding disk-backed storage to existing in-memory caches.
* Made MWMemcached::get() return false on failure instead of null, to match the BagOStuff documentation and the other BagOStuff subclasses. Anything that was relying on it returning null would have already been broken with SqlBagOStuff.
* Fixed a bug in the memcached client causing keys with spaces or line breaks in them to break the memcached protocol, injecting arbitrary commands or parameters. Since the PECL client apparently also has this flaw, I implemented the fix in the wrapper class.
* Renamed BagOStuff::set_debug() to setDebug(), since we aren't emulating the memcached client anymore
* Fixed spelling error in MWMemcached: persistant -> persistent

Fixed patch from r33520