* Use local context instead of global variables
* Call Linker methods statically
* Throw exceptions directly instead of calling OutputPage's methods that will throw them
* Use Title::getUserPermissionsErrors() instead of Title::userCan() so that the User object can be passed to them

Localisation updates for core and extension messages from translatewiki.net (2011-07-20 19:00:00 UTC)

Documenting new parser hook, re r92506.
This refers to an extension that's not checked in quite yet, FYI

Make Title::isNamespaceProtected() require the $user parameter, the only call to it is in the file itself

Commit a list of unimplemented tests.

Follow up r92635

Fix for r91885.
These tests were added in r91885.

Follow up r92635.

I like better checking for the actual extension, loading if needed (and able).

Fixup r89029. You don't include the brackets in the method name!

--ta línea y las que están debajo serán ignoradas--

M    includes/media/SVGMetadataExtractor.php
M    tests/phpunit/includes/media/SVGMetadataExtractorTest.php

Removed todos, not useful
Moved separators to separatorTransformTable

Remove @return void documentation

Add missing error to getPossibleErrors

Allow blocked sysops to view Special:Unblock

Previously, trying to visit [[Special:Unblock]] while blocked would give
an error, even if the user has the unblockself permission.  I moved the
permission check from execute() to right before the code that does the
actual unblocking.  This should probably be examined closely for
security, since I'm not familiar with this code, although the impact of
a bug would be small.  I tested some simple cases manually and they all
worked as expected:

* Unblocking self works (as before)
* Submitting the form fails unless you're trying to unblock yourself (as
  before)
* GETting the page with any parameters works (previously failed unless
  the target was your own username)

Fix inaccurate return type in comment

added missing null checks in callers of Revision::newNullRevision

$wgUploadNavigationUrl should be used for file redlinks if $wgUploadMissingFileUrl is not set. The first was used for this until the second was introduced in r69997 (1.17). According to the comments there it was broken in 1.16 but as far as I tested it worked in 1.16. In any case, the old behavior should be restored as most WMF wikis have set wgUploadNavigationUrl but not wgUploadMissingFileUrl.

Follow-up r92588. Make it work regardless of $wgHtml5

Since r92364 UserTest.php needs a database

Follow up r91419. A boolean attribute can be both the empty string and its name [whatwg.org/specs/web-apps/current-work/multipage/common-microsyntaxes.html#boolean-attributes]
We use the later since that's what xhtml1 required [http://www.w3.org/TR/xhtml1/#h-4.5].

* Use local context instead of global variables
* Call Linker methods statically

Spacing, removed unneded global.

Fixes for URL expanding in CSSMin: adjust the offset correctly (this could've theoretically resulted in very strange bugs) and only call wfExpandUrl() if available (the file is in includes/libs so it should work outside of MediaWiki)

Remove the globals. r91570 was wrong saying that parent::setUp()
would initialise those vars (this class parent is MediaWikiTestCase,
not MediaWikiLangTestCase) but the language doesn't seem to matter.

Follow-up r91518

Follow up r92036. The global is not needed now.

Follow-up r91246. The global is no longer needed.

r91784 forgot to change one instance of $wgUser to $user

Localisation updates for core and extension messages from translatewiki.net (2011-07-19 19:45:00 UTC)

Follow up r91561. Use canonical class names.

Expand the defines from JSTokenizer::__construct() placing them at the top of the file.
This fixes the Notices that would be produced if several JSTokenizer were instantiated.
jsminplus was added in r91591.

Defines code was generated with:
$js = new JSTokenizer();
foreach ($js->opTypeNames as $operand => $name) echo "define('OP_$name', '$operand');\n";
foreach ($js->keywords as $keyword) echo "define('KEYWORD_" . strtoupper($keyword) . "', '$keyword');\n";

Remove double ; and final ?> from jsminplus (imported in r91591).
This should be uncontroversial.

Profiling errors. r92106 missed a wfProfileOut
r92118 added a wfProfileIn but not all returns were covered. Also changed its spaces to tabs.

Follow up to r91561: Use the canonical class names.

Follow-up r92528:
* remove @static
* return database result directly (no need for while())
* getAllPrefixesDb() -> getAllPrefixesDB()

Follow-up r92528: probably better to put it in the API & new features sections

(bug 19838) API does not use interwiki cache.

Patch by Beau; modified to work with current code.

mark a non-API function private; rename it to reflect what it does; call getPath() rather than accessing object variable directly

style fix

Refactored section assembly code to be more readable
Added a hook that's called for each section.  This allows sections to be addressed separately at the document level, for example by wrapping each in a div (as in the reverted r50769).  This in-turn enables richer section-specific UI, like highlighting or in-line editing.

Revert 2 from r92498

Update code comments that point to 1.18 to point to 1.19

Add RELEASE-NOTES-1.18.old into RELEASE-NOTES-1.18

Needs svn copy in REL1_18

Removed testBug29116LoadWithEmptyIp. Fails due to wfDeprecated() call in load() and nothing uses that so it's not worth it.

Move current RELEASE-NOTES-1.18 to RELEASE-NOTES-1.18.old

Moved RELEASE-NOTES-1.19 to RELEASE-NOTES-1.18

Wipe RELEASE-NOTES-1.19

Updating text where necessary

Keeping RELEASE-NOTES-1.18.old till we've established there's nothing that just went into that changelog

Add comment

Remove unused wfGetDb()

Fix autoblocks in r92480 r92482 -- modified the wrong half

It seems people don't either pay attention to code comments, or those comments lie

r92461

Fix autoblocks in r92480

(bug 29959) Installer fatal when cURL and allow_url_fopen is disabled and user tries to subsribe to mediawiki-announce

Really, I'm not sure about the usefulness of exploding with a MWException anytime we can't do an external request, but at least we can stop the installer from exploding :)

Block.php fixes: Always store mExpiry as a TS_MW or "infinity". Convert as appropriate.

* (bug 29938) list=users&usprop=rights shows rights the user doesn't have

Let Skin extend ContextSource instead of reinventing the wheel

Refactored new getAutomaticGroups() function out of getEffectiveGroups()

Documentation updates

Add a couple of missing explicit class variables

Localisation updates for core and extension messages from translatewiki.net (2011-07-18 19:50:00 UTC)

Set explicit public modifier

Swap @private to private

changed sessionkey to filekey to keep consistent with the rest of UploadStash (while retaining sessionkey for backwards compatibility)

removing xdebug call

Bug #29874 - Image extension strict whitelist handling is inconsistent
Patch from Dan Collins

Updated doMaintenance since wmf-config/ was moved up a dir

Remove call to User::getImplicitGroups() from ApiQueryAllUsers

Per the below, as User::getGroupsWithPermission() will return */autoconfirmed, etc if applicable

> var_dump( User::getGroupsWithPermission( 'createtalk' ) );
array(2) {
  [0]=>
  string(1) "*"
  [1]=>
  string(4) "user"
}

> var_dump( User::getGroupsWithPermission( 'autoconfirmed' ) );
array(5) {
  [0]=>
  string(13) "autoconfirmed"
  [1]=>
  string(3) "bot"
  [2]=>
  string(5) "sysop"
  [3]=>
  string(8) "reviewer"
  [4]=>
  string(9) "confirmed"
}

* Use local context instead of global variables
* Call Linker methods statically
* Use Linker::linkKnown() instead of Linker::link() where possible

* (bug 29935) Improve formatting of examples in ApiParamInfo

Old example string with newlines left for the moment, pending discussion on bug...

Kill unused globals

Remove unreachable return;

-rakkaus/#mediawiki-i18n- (1 lines skipped) [18-Jul-2011 12:54:37] PHP Fatal error:  Call to a member function addHTML() on a non-object in /www/w/includes/specials/SpecialWatchlist.php on line 65

Swap $out for $output

Disallow given options twice (instead of letting the newest one take precedence). This property may be useful for a Het Deploy wrapper.

Fix r92418: 'fr', of course

Add fallback 'frc' -> 'fr'. Main translator agreed ( translatewiki.net/wiki/Thread:User_talk:Ebe123/Fallback_for_'frc' ).

Localisation updates for core and extension messages from translatewiki.net (2011-07-17 19:25:00 UTC)

Add a couple more useful URL's to the API help banner

Add help url for ApiQueryDeletedrevs

Followup r92396

Using action=paraminfo to give help urls (and examples for that matter!!) look awful as a string with an encoded \n in the middle of it

* (bug 28817) Add reference help page link to API Modules

Last followup of r92396

Should be all modules with documentation pages in core now documented...

Followup r92396

More modules documented, pretty much only action=query&list= to be done (ie links added where documentation pages exist already)

Followup r92396, add help urls for most of the core (non query) modules

Followup r92396 rename method, fix boolean comparison that was supposed to be inverted when de-indenting

* (bug 28817) Add reference help page link to API Modules

Start of this bug, fleshed out base code implemented, plus a few key modules (will finish rest up soon)

Tweaked/improved/updated some documentation as I went through also

Flesh out some TODO documentation from onwiki pages

Fix for r92388: that was the wrong method

* Changed Special:RecentChanges and Special:RecentChangesLinked to use the local context instead of global variables
* Added context to ChangesList
* Added new factory for ChangesList and subclasses in ChangesList::newFromContext() and modified ChangesList::newFromUser() to call it with the main context since anyway User::getSkin() return the Skin from the main context, so this won't change anything but the User::getOption() call
* Updates Special:RecentChanges and Special:Watchlist to use that new function
* Call Linker methods statically
* Use Linker::linkKnown() instead of Linker::link() where possible
* In Special:Watchlist: Use wfMessage() instead of OutputPage::addWikiMsgArray() with the third parameter

* Expanded documentation, added GPL header and changed the @author tag to use my real name
* Made RequestContext implement IContextSource
* Added ContextSource::msg()
* Made ContextSource fall back to RequestContext::getMain() when there's no context set

Localisation updates for core and extension messages from translatewiki.net (2011-07-16 19:52:00 UTC)
Ping r92057: Full export of core and ext-installer

* Use standard method to check user permissions
* Directly throw an ReadOnlyError instead of calling OutputPage::readOnlyPage()
* In Special:Block: don't let user think the error is temporary if he doesn't have right and the database is locked, instead check permissions first and then the database lock

* Use wfMessage() instead of OutputPage::addWikiMsgArray() with the third parameter
* In EditPage.php: call Linker methods statically

First steps for bug 14801: add backend support for per-namespace permissions to core. This extends $wgGroupPermissions syntax from $wgGroupPermissions[$group][$right] = bool to $wgGroupPermissions[$group][$right] = array( NS_X => bool ). This is safely backwards compatible; the booleans are still fully supported, and any unset namespace will default to false.

* User::getRights(), User::isAllowed() and User::getGroupPermissions now optionally accept a namespace parameter. If not set, it will check whether the user has the right for all namespaces.
* Anything that uses Title::getUserPermissionsErrorsInternal() automatically supports per-namespace permissions. This includes Title::getUserPermissionsErrors and Title::(quick)UserCan.
* Fix tests that set User::mRights

The next step would be to change all User::isAllowed() to Title::quickUserCan or pass the namespace to User::isAllowed().

* Use local context instead of global variables
* Call Linker methods statically
* Changed some messages to replace parameters before parsing where possible
* Changed usage of OutputPage::addWikiMsgArray() with third parameter to wfMessage()

Array indexes start from zero

Remove 'dk' from dummy language codes, it was removed in r71693.

Split MessagesKs into ks-arab and ks-deva, as Kashmiri is written in two scripts. The MessagesKs file contained rtl = true, which is only for the Arab script, and the digit transform table which was for the Devanagari script.
Added one lonely message from kswiki.

Also specified comment of Quechua to Southern Quechua, as it is the variety used in MessagesQu and on quwiki.

sync up RELEASE-NOTES

html:key should be self-closing in jquery.localize unit tests.

More IE6, 7 and 8 fixes for jquery.localize unit tests.

html:key should be self-closing. This was in most tests, but accidentally forgotten in a few. Other browsers are looser on this, but IE is very strict about it.

Removing deprecated <msg /> support in jquery.localize. Maintaing it is a PITA and it doesn't work properly in IE6, 7 and 8.

<html:msg /> has been the new format for a while.

Increasing timeout duration. Some TestSwarm participants appear to have slower virtual machines and/or may have network slowness. 1.5 second was a bit too short anyway. This will reduce false positives.

* Use local context instead of global variables
* Call Linker methods satically
* Use wfMessage() instead of OutputPage::addWikiMsgArray() with a third parameter

Removed unused function

r92288 : Missed second arg to inArray

ajaxCategories fixes:
* Partial revert of r92264. mw.ajaxCategories has nothing to do with mediawiki.util.init, moved into separate initiation module, only loaded when ajax categories is enabled.
* Removing instantiation from the script. mw.ajaxCategories is now a public accessible constructor
* Fix Uncaught ReferenceError: wgUserGroups is not defined
($wgLegacyJavaScriptGlobals = false;)
* Renamimng stripIllegals() to clean(), adding '#' to the regex as those are not allowed either.
* Combining var statements by separating them with comma's ( var foo, bar, baz; ) instead of "var foo; var bar; var baz;"svn up
* Using dot.notation for objects instead of array-like['string'] keys
* Whitespace conventions
* Using inArray utility instead of indexOf (cross-browser support, mostly IE6)
** $.inArray uses Array.prototype.indexOf if available, otherwise fallback to a loop. mw.util.inArray is a wrapper around $.inArray to return a boolean value (since inArray/indexOf may return 0)
* Don't use $( '#bodyContent' ) (skin specific) but mw.util.$content
* Using mw.Title
* Re-using helper functions from the outer-closure instead of re-declaring them privately for every instance again (performance!)

Follows-up: r92112, r92151, r92264

RELEASE-NOTES for r92279

(bug 28762) Resizing to specified height broken for very thin images

Restructure and comment of ImageHandler::normaliseParams(). ImageHandler::normaliseParams() will now output both width/height as physicalWidth/Height, where width/height are the dimensions used by the browser, and physicalWidth/Height are the dimensions of the thumbnail that will be created. This allows keeping the use of unique dimensions for a specified thumbnail width, by forcing client side resizing of 1px-wide images with non corresponding height.
Also fixed a bug where the check for mustRender() was in an if statement where it should not have been.

Fix r92253

No longer using the content hash as a file key.  This solves a concurrency problem when multiple people use the same account to upload from the same set of files.  Each stashed file is now a unique and beautiful snowflake, and the content-based deduping that comes at the end of the process can sort it out.