r92054: Some IE fixes

Updated hooks docs per r91123. The UI methods are not part of the object anymore.

Reverted r92489 and added/use hideDeprecated() to MW test framework

Test for external links in the sidebar

follow up r90949
Note: this does NOT test the parser external links only externals
links inside the sidebar.  This might help rewrite r90949.

Removing redundant check from $.fn.goIn/Out code.

Update JSMin+ to the newly released 1.4
This upstream release incorporates r92560 and r92563

Fix margin of r90743 (margin is different between ul & ol)

Adding documentation for mw.loader.register

Trim trailing whitespace

Add documentation

Remove unused variables

Remove an extract from ImageFunctionsTest

More mediawiki.js cleanup (addScript AJAX)
- Update and merge in jQuery fixes:
-- Dereference script (added)
-- Order of actions
-- Regex instead of two string comparisons
-- Unit tests still pass :)

Follows up: r92933, 93012

More mediawiki.js cleanup
- Convert prototype object modifications into object literal. Saves bandwidth (less characters) and speeds up execution (no need to access 2 level deep object member repetitively). Local testing (Chrome Web Inspector) shows 14.80KB to 14.67KB (non-cached, debug=false), and execution time on cached request 32ms to 25ms.
- Un-indent 1 tab for d.setTime (Follows up r92964)
- Whitespace consistency
- Move var statements to top of loader's addScript

Follows up: r92933

Rename class to match change of method location

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

Followup r92796, r93002

Move addition of config table to 1.19 section

Follow-up r92924, set special page group for ChangeEmail

Fix for r92984: correct method name, thanks Jack Phoenix (oops!)

Some misc fixes to Skin and SkinTemplate:
* Removed Skin::getStylesheet(), not called from anywhere
* Removed Skin::setMembers() and moved the definition of the $userpage member to SkinTemplate::outputPage() since it's only used in that class
* Removed useless $out parameter of Skin::bottomScripts()
* Call Linker methods statically
* Use Linker::linkKnown() instead of Linker::link() if possible
* Escape 'mainpage' message when used for description
* Removed some variables from SkinTemplate::outputPage(): nscanonical, nsnumber, currevisionid that were used when the JavaScript global variables were defined in the template. Also changed pageclass and skinnameclass to be defined only when $useHeadElement is false
* Removed some unused members of SkinTemplate: $iscontent and $iseditable
* Refactored the part of SkinTemplate::outputPage() that sets copyright, viewcount, lastmod, credits and numberofwatchingusers to always define them to false first and set them when needed. Also modified the check to display the copyright; it will now only be displayed when $out->isArticle() is true and the page exists, since it is currently possible to display it on any page by adding the oldid parameter to the url.

Fixing messages.inc and rebuilding languages en and he.

Add lang & dir attributes around the recentchangestext message, because it is in the content language.

fix doc

fu r92933 - fix breakage

It is stupid to test that a void method returns null. Add a comment explaining that
what we are testing is actually the function existence (see r91123 summary)
Change the onArticle* to is_callable()s, so that a db is not needed.

Make wfUrlEncode(null) reset the static. Two skipped tests work now.

Changed Maintenance::shouldExecute() to allow for multiple requires() in the stack. This avoids failing over requires due to Het wrappers.

DifferenceEngine: introduce setDiffLang() to change the language in which the diff text is written.
Translate: set the the diff text according to the source language, on Special:PageTranslation and the translation messages. (Also fix direction of CSS class mw-translate-fuzzy).

Localization update for he.

Dummy-proof incLoginThrottle/clearLoginThrottle with trim()

trim $this->mUserName var so that login attempt throttle key isn't fragmented with spaces

Fix typo in r92933 that resulted in global variable leakage

Minor mw.loader fixes
- No functional changes
- Whitespace correction
- Comment fixes
- Combine var statements
- A few [[JSPERF]]ies
- Using mw instead of mediaWiki
- Putting global alias on top, to avoid a ReferenceError in case the immediately invoked "new function(){}" refers to mw during the construction
- "class" -> "constructor"
- caching "typeof dependencies" in mw.loader.using
- fix var "group" collision in mw.loader.work
- move var statements to top of functions
- fixing some occurrences of var statements in for-loops by declaring them before the loop
- Remove space before slash in html-fragement for jQuery html regex in <link /> and <br />

Follow-up r86482, r86485: removed user_name portion of user_email index per CR

Change != {} to !$.isEmptyObject(): the former always returned true because objects are compared by reference. This didn't cause any realistic problems, though: the worst that could happen is that if you have a module name longer than the query string request limit (which is extremely unusual), mediawiki.js would probably go into an infinite loop.

Updates for new autopromote system & block hook

* Moved email changing from sp:Preferences to new sp:ChangeEmail, which requires confirming the user password. This reduces the impact of session hijacking, which was increased slightly with r86482. Changing a password already required confirming the old one. This change closes the loophole of changing the email address and then doing a reset.
* Parse 'mailerror' message correctly

Followup r86622: add initial QUnit test cases for jquery.textSelection module.

* tests .textSelection()'s encapsulateText method mostly (also uses setSelection, getContents, and getSelection)
* examples from WikiEditor toolbar: sig, bold, h2 (ownline), ulist (ownline & splitlines)
* confirms that splitlines works more or less as expected, at least for basic single-line, single split-line, and multi-line cases

Doesn't test the WikiEditor iframe mode since that's in a separate extension; when it's possible to test those things too, that'll need to be run there.

One of the h2 tests fails in IE6, returning selected text that's offset by one character from what's expected. I'm not sure whether it's actually selecting the wrong text or whether it's returning the wrong text -- needs further investigation.

Also note that there's no setContents submethod in textSelection, despite there being some notes about one.

in "marked version X of [[Page]] patrolled" on IRC, the "version" displays according to the user language. If this is the correct fix (didn't test), it can perhaps be merged with 1.18 and 1.17wmf1.

Follow-up r92887: clear throttle count once the password is accepted as normal

Removed Special{$this->getName()}ModifyFormFields from r86482. Seems kind of weird, was undocumented, and is unused by extensions.

Improved r92884 comments a bit on second thought

Fix for r86482: throttle password attempts for SpecialChangePassword (uses r92884, r92886 code)

Follow-up r92884: mark these functions static

Refactored code out into incLoginThrottle/clearLoginThrottle functions (for use by SpecialChangePassword et al)

Document parameter in loadInputFromParameters

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

Fix nukeNS maintenance script to correctly reference nukePage. Add in all option to nuke everything regardless of revision count, defaults to false.

And I forgot to rename that variable in r92855

Reduce a bit the coupling between Skin and OutputPage:
* Removed Skin::setupUserCss() and merged it in OutputPage::buildCssLinks() and OutputPage::buildCssLinksArray(), also removed its call from ApiParse.php
* Moved Skin::userCanPreview() to OutputPage since two of the three calls to that function are made from there and the last one in SkinTemplate.php is in a somewhat deprecated function
* Removed the Skin parameter from OutputPage::buildCssLinks(), OutputPage::getHeadScripts(), OutputPage::getBottomScripts() and OutputPage::makeResourceLoaderLink() since we now have a context

Also made ApiParse.php call createContext() instead of create a new RequestContext manually and set its Title  so so that it does not rely on $wgRequest and $wgTitle to get that member objects

Avoid redefining TESTWIKI constant (Het wrappers already do this)

follow-up to r91499 - set padding for the elements inside the button rather than for the button itself - this is how normal jquery.UI skins handle it, as it prevents the button padding from screwing up the positioning of the icons.

Remove $rtl = true; from translation file in Latin script.

Actually alias sessionkey to filekey, fixes bug in r92459

Fixes for r92538 & r92528:
* use default null for $local
* use foreach for database result

self rv r90483. Still needs investigating though

Use proper nosuchuser msg (fix for r86482)

Fix for r92702: there's no such method in Article class

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

Address fixme on r86697: simplify the code a lot by not escaping thisquery and killing thisurl in favor of thispage. Per CR comments these vars aren't used anywhere else. I grepped phase3/includes , phase3/skins and extensions/ to confirm.

Fix r78824 after some misunderstandings in the CR comments. The querypage API module was checking for disabled status and setting the 'disabled' attribute in the output, but would then happily proceed to call doQuery() on a disabled query page.

Followup r92329: Fixes the localize test cases in IE 6 and 7.

Looks like we do in fact need to select on 'msg,html\\:msg' in order to find the <html:msg> bits on both IE 6/7 (search for 'msg') and everything-else (search for 'html:msg').

decodeOptions() used by GlobalAuth - made it public

Remove mw.util.inArray
- redundant and makes it even more confusing than jQuery's "inArray" already was.
- Reverts r92261
- Fixed usage in /trunk/*
-- http://toolserver.org/~krinkle/wikimedia-svn-search/view.php?id=245&hash=4ef50cd2f7017954e562155ccceba878

While at it, also took take of the r92297 fixme. Item argument must be before array argument.

Fix for r92740: removed debugging code

Whitespace update.

Remove messages that shouldn't be here.

Removed SkinTemplateSetupPageCss hook and SkinTemplate::setupPageCss(), no extension use this anymore. This can only add raw CSS to page; it is much better to use the ResourceLoader (or to link to a file). But if it is really needed, extensions can use the BeforePageDisplay to achieve the same functionnality.

* Made Skin::userCanPreview() use Title::getUserPermissionsErrors() instead of Title::userCanEditCssSubpage() and Title::userCanEditJsSubpage() so that the checks are the same as EditPage's ones
* Marked Title::userCanEditCssSubpage() and Title::userCanEditJsSubpage() as deprecated since these were the lasts calls to that functions (core and extensions)
* Get the action parameter from Skin::userCanPreview() instead of requesting it from the callers

Revert non-functional parts of r82343 -- 'select' parameter that's never actually been exported or used, and was always being forced to true before being discarded anyway.

RELEASE-NOTES for r79383

Revert DatabaseBase::tablePrefix() part of r79272: doesn't seem to be really necessary (tests are all still passing, Mysql && Sqlite) and as Tim points out this breaks anything trying to use it as an accessor rather than mutator.

Revert r85034, r81186, r77638: per CR on r77638: XMLReader and XMLWriter are memory-hungry beasts and this script OOMs constantly.

This was already reverted in 1.17wmf1 (r82930) by Ariel. Doing the same in trunk now. I'm not opposed to using the new pretty XML* classes, but somebody needs to debug why they eat memory for breakfast, lunch and dinner.

(bug 25355) Parser generates edit section links for special pages

Per Tim on r58362: disable edit section links on all text parsed via $wgOut. Article already handles setting this to true for page content, so this shouldn't really hurt anything. Could use some tests to confirm the behavior, however. Should resolve the last problems with r70498 and friends.

Use $this->getLang() per Nikerabbit on r91648.

* $wgDnsBlacklistUrls now accepts an array with url and key as the elements to work with DNSBLs that require keys, such as Project Honeypot.

* 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