Type hinting

Braces

Remove extra whitespace

Followup r86898, fix $wgProxyScriptPath

proxy_check.php is probably useful to keep around, but it's not really an includes script

Move it to maintenance/proxy_check.php

* (bug 28627) External link normalization now handles file: URL cases without throwing notice warnings.

Added some test cases for wfMakeUrlIndex() to GlobalTests (tweaks $wgUrlProtocols to toss in file:// support so it can test them).
Needs more cases for other URL styles probably; some of the more pathological file: URL cases still won't normalize really cleanly but will go through the function without exploding. The most-needed variants will be the Windows/IE-compatible ones I think -- so file:///c:/foo or file://server/foo.

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

Fix for r86670: properly convert uppercase latin to syllabics

Revert r86872: Breaks LiquidThreads page moves with the below failure. Threads are lost and nowhere to be found any more.

[25-Apr-2011 18:12:45] /wiki/Special:MoveThread/Thread:User_talk:Siebrand/test/One_new_message: Exception: MWNamespace::getTalk does not make any sense for given namespace -1
#0 /www/w/includes/Namespace.php(81): MWNamespace::isMethodValidFor(-1, 'MWNamespace::ge...')
#1 /www/w/includes/WatchedItem.php(73): MWNamespace::getTalk(-1)
#2 /www/w/includes/User.php(2304): WatchedItem->addWatch()
#3 /www/w/includes/actions/WatchAction.php(53): User->addWatch(Object(Title))
#4 /www/w/includes/Action.php(376): WatchAction->onView()
#5 /www/w/extensions/LiquidThreads/classes/Thread.php(115): FormlessAction->execute()
#6 /www/w/extensions/LiquidThreads/classes/Thread.php(435): Thread::create(Object(Article), Object(Article), NULL, 1, 'One new message')
#7 /www/w/extensions/LiquidThreads/classes/Thread.php(414): Thread->leaveTrace('move test', Object(Title), Object(Title))
#8 /www/w/extensions/LiquidThreads/pages/SpecialMoveThread.php(107): Thread->moveToPage(Object(Title), 'move test', true)
#9 [internal function]: SpecialMoveThread->trySubmit(Array)
#10 /www/w/includes/HTMLForm.php(279): call_user_func(Array, Array)
#11 /www/w/includes/HTMLForm.php(228): HTMLForm->trySubmit()
#12 /www/w/includes/HTMLForm.php(242): HTMLForm->tryAuthorizedSubmit()
#13 /www/w/extensions/LiquidThreads/pages/ThreadActionPage.php(37): HTMLForm->show()
#14 /www/w/includes/SpecialPageFactory.php(459): ThreadActionPage->execute('Thread:User_tal...')
#15 /www/w/includes/Wiki.php(252): SpecialPageFactory::executePath(Object(Title), Object(RequestContext))
#16 /www/w/includes/Wiki.php(98): MediaWiki->handleSpecialCases()
#17 /www/w/index.php(145): MediaWiki->performRequestForTitle(NULL)
#18 {main}

Implement an interface and abstract class to hold the widely-reused get(Request|User|Title|Lang|Skin|Output) accessors for objects acting as a context source.  Article is rather messier because both getTitle() and getUser() are in use for other things, and Article::$mTitle is in extremely wide use both within Article.php and outside.

Type hinting and documentation tweaks.  This file is actually pretty good...

* (bug 27249) "Installed software" table in Special:Version should always be left-to-right

Patch by Jan Paul Posma

(bug 24298) Make dropdown menus in Vector more accessible by expanding and collapsing them when the hidden link behind the dropdown arrow receives a click event (triggered by an enter or space bar key press in the typical use case) and changing the appearance of the arrow when the hidden link behind it is focused. Patch by Jan Paul Posma
--This line, and those behind, will be ignored--

M    phase3/skins/Vector.php
M    phase3/skins/vector/images/arrow-down-icon.png
M    phase3/skins/vector/screen.css
AM   phase3/skins/vector/vector.js
M    phase3/resources/Resources.php

Rm all mentions of "this module" in API action descriptions. They are supposed to be read by API users, not MW coders, thus internal terminology shouldn't be used here.

Add |alt= option for galleries ( Bug 18682 ). Recommit of r86749, with nested |alt= now fixed. Patch by Jan Paul Posma

Fix typo in description

Followup to r86854: fix $ object

Followup to r86108: jQuery tries to be too smart and converts the string to a number, which breaks the value parser

Documentation, type-hinting and deprecated functions in Article.php.

(bug 16921) Add JavaScript-based enforcing of byte limits on move and protection forms. Patch by Jan Paul Posma

(bug 21180) Move the old toolbar below the summary input to match the new toolbar

Followup r86625: per CR, use loose comparison against null instead of typeof foo != 'undefined' because jQuery 1.4 (which MW uses) actually returns null for unset .data() keys, but 1.5 returns undefined.

Remove hints since $this->isNew was not possible anywhere else.

clean up eol w/s

In DBConnectionError:
* Reimplement msg() instead of forcing useMessageCache() to false
* Use that version of msg() instead of always checking for $wgLang
* Simplify a bit fileCachedPage()

Localisation updates for core and extension messages from translatewiki.net (2011-04-24 21:19:26 UTC)

Fix comment, language is handled correctly with parse and parseinline since r79642

Followup r86764: don't parse the comment. Spotted by SPQRobin@Translatewiki and fixed the help of Ialex

* Use wfMsgNoTrans() instead of wfMsgReal()
* Simplify getPageTitle()
* Remove a useless call to getPageTitle(), <img> tag does not have content

Follow-up r86775: restub $wgLang.  Not because it's a good idea, but because I can't see how to avoid the circular dependencies on it *without* stubbing it.  Probably easier to just continue the drive to deprecate the global variable altogether.

Localisation updates for core and extension messages from translatewiki.net (2011-04-23 21:50:00 UTC)

Tidy up some unused variables and such

Move patch contributors to the right section

Remove Jan Paul and Derk-Jan Hartman from patch contributors as they're developers

Move diebuche and bawolff

Some type-hinting and documentation in HTMLForm.php

Follow-up r86347: the $mFile written to in SpecialUndelete is not actually the 'same' variable as is used in SpecialPage (SpecialPage->mFile is the php file where wfEvilSpecialPageFunction() can be found, while SpecialUndelete->mFile is a deleted file), but it still complains about private access unless it's redeclared.

Add missing semicolon from r86750

* Add preprocessor tests that Bergi supplied for Bug #28642
* Add Bergi to CREDITS

Use assertStringEqualsFile in place of assertEquals and reading the file.

missed some w/s

w/s cleanup

Fix code accidentally committed in r86735

follow up r86752 with some content for the result of “Gallery with wikitext inside caption”

Change getUrl to getFullUrl for {{filepath:Filename.jpg}} of non-thumbd local files. This way it matches the paths for thumbnails and for foreign files. (bug 28618)

random w/s cleanup

* Fixes Bug #27860 - Minor edit after clicking 'new section' tab

Patch from Jan Paul Posma of which he writes:

    This patch hides the minor edit checkbox when editing a new page or new
    section, and makes sure it cannot be enabled by injecting the form value in
    HTML.

* Add Jan Paul Posma to CREDITs re Reedy's earlier comments.
* Add note about isNew to other possible places that it can be used.

wfMsgReal() -> wfMsg() or wfMsgNoTrans()

* (bug 28560) list=deletedrevs should die, if combination of param is invalid

No checking for NS or unique in the mode 1/2 checks, as they have defaults of 0 and false, which is impossible to determine if they had been set as that, or just using the defaults. They are ignored in the code still on the incorrect path

Update usage of dir

Don't double escape html entities for messages obtained through Message::escaped().
Since we require PHP 5.2.3+ and htmlspecialchars() has a fourth parameter to prevent double escaping since... PHP 5.2.3 (per manual), let's use it

Remove unused messages 'recentchangesall' (removed in r36682) and 'imagelistall' (removed in r14091). Not found in current source code.

* (bug 27344) add drprefix param to list=deletedrevs

Call $wgRequest->getLimitOffset() directly instead of wfCheckLimits()

Stop stubbing $wgLang and $wgContLang.  There are no major code paths which do not call either $wgLang or $wgContLang at least once.  All index.php calls unstub $wgContLang from MediaWiki::parseTitle() except in the edgecase of viewing pages referenced only by "curid=123", and since those will end up calling OutputPage::output() they will eventually be unstubbed at some point as well.  All calls through load.php unstub $wgLang in ResourceLoaderContext::getLanguage() from ResouceLoader::respond() --> ResourceLoader::preloadModuleInfo().  All calls through api.php unstub $wgContLang in ApiResult::cleanUpUTF8() from ApiMain::printResult().

Followup r86763, and also fix ancient revision from SpecialProtectedpages (Aaron at fault), use $db->getInfinity()

Document SpecialPageFactory::getAliasList()

Documentation

Fix call to undefined self::getTitleFor

Documentation and type hinting.

Simplify a bit to make nicer links: Special:ListFiles/Username

Add some more spacing due to long parameter names

Use str_repeat for ease rather than "     " etc

Fix MiserMode to MiserMode

Embrace comment with () only when really needed. In Special:ListFiles and ImagePage the comment is shown alone in a table cell -> no () needed

* (bug 26882) Allow listing of indefinite protections with the api

Swap some !empty for count

Followup r86670: Define section header for new variant

* (bug 28558) Add iw_api and iw_wikiid to meta=siteinfo&siprop=interwikimap

Waiting on bug 28673 to be merged to core and be used, before this is of any use, but it's done anyway

* (bug 28672) give information about misermode on api

Reverting r86749: The alt stuff is far too simplistic. One way to fix it would be to generalize makeImage() in the parser to process gallery params as well, but I don't have time to that now

SQLite compatibility for patch-profiling.sql

Add ability to use add alt texts for images in galleries ( Bug 18682 ). Patch by Jan Paul Posma. Also cleaned up some comments and var names

Allow rows missing iw_trans or iw_local to be considered valid.

Move the Interwiki class to includes/interwiki/ to make way for a more flexible Interwiki system. Do this first since svn doesn't support editing and moving a file in the same commit.

Breaking the 'math' table setup out to Math extension. Should be the last main step in extension-ification of the math options!

The 'math' table will no longer be created on a default install unless you've explicitly enabled the Math plugin at install time; the usual update.php procedure will add it in.

Postgres, Oracle, MSSQL, and DB2 variants are included -- broken out from the core files -- but have not been tested.

I know there has been some code duplication in parser test infrastructure but could only find one instance of the parser test temporary table setup to remove the 'math' table from (the extension adds it back via the hook). If the phpunit-based runner breaks, please track it down and fix it there too.

Use wfMsgExt() instead of wfMsgReal()

Implemented param tracking for hook users, feels a bit hackish

Followup to r86312

<ialex> Reedy: that rev is breaking usage of {{PAGENAME}} in messages, such as in MediaWiki:Noarticles

Allowing optional passing in of a Title object (like it may be set in Message), but if it's not set, or not a title object, fall back and use $wgTitle (I'm sorry!)

Cleanup next/prev links on special:search. Removed unused mPrefix.

Follow-up r86732: really mark it as deprecated ;)

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

Doc comment updates for CoreTagHooks callback functions + Parser::setHook() & Parser::setTransparentTagHook().

Added notes about the 'extended' return values from some of the hooks, which seems to be undocumented but is vaguely similar to what is documented for parser function callbacks. It doesn't look like a very safe interface, and could stomp on internal variables if one isn't careful. Needs to be better defined and doc'd.

setTransparentTagHook() seems to have been largely undocumented and is only used by the 'geoserver' extension currently. Is this considered obsolete? Should it be simply replaced by use of the frame & recursiveTagParse goodies, or is it still needed for something?

Added DBError::getContentMessage() to factorise common code; DBUnexpectedError now displays shows like a DB error for consistency with CLI mode

Apply patch from Karsten Düsterloh in Bug #28103.

Karsten writes:

    If the $table_prefix is not empty, several table renamings will happen,
    including for 'pagecontent' and 'mwuser'. This breaks the import into postgres,
    because
    - 'pagecontent' doesn't exist (temporarily renamed to 'text')
    - 'mwuser' won't exist later (temporarily renamed to 'mw_mwuser')
    Also, some sequence names are wrong.
    Furthermore, some of the generated ALTER statements lack the final semicolon

    The attached patch contains the changes I had to do to make this run.

Allow extensions to customize the search forms. This required some cleanup and refactoring to special:search and search engine.
Should be fully backwards compatible. Lightly tested, but only with MySQL search backend.
Introduces concept of search profiles, which replace long list of namespaces in the url.

Followup r86705 fix typo in comment

Update StartProfiler.sample after r86175

Updated patch from Bergi re: r86676 with tested code to fix bug #28642

* insert page 0 (for FK) if updating

* Remove manual query building in search mysql
* Remove reference to mysql3, no longer supported
* Changed method signatures, I found no class extending this class in extensions
* Tested lightly and returns same results

* begin, commit, rollback functions updated
* using ->execFlags instead of hardcoded OCI_DEFAULT
* added dummy page (id=0) to satisfy FK checks

Add docs

(bug 28650) Refactor dynamic byte-based maxlength of edit summary into a jQuery plugin. Patch by Jan Paul Posma

(bug 26603) Followup r82232: fix double-escaping of returnto and returntoquery. Was caused by using two sources ($this->thisurl and $wgRequest) where one was already escaped and the other wasn't, then unconditionally escaping the result.

Fix bug #28642 - wrong part->commentEnd set to abbutting comments

    To be able to search for headline-closing equal sign even when
    there are comments between the last = and the linebreak, there are
    some position values appended to the recent domParts. This works
    well with the first comment, but as soon as a second one comes,
    the commentEnd value is set to $wsEnd instead of $endPos.

Patch from Bergi

Conversion script between Syllabics and Latin for the Inuktitut language

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

Added jquery.json library to core.

Removed false comment introduced in r86633

* Added Profiler::isStub() to check if we are using a stub profiler, instead of checking whether $wgProfiler is set
* Replaced wfProfileClose() and wfGetProfilingOutput() by direct calls to the Profiler instance and removed them, no uses in extensions
* Also removed useless params from Profiler::getOutput() call in wfLogProfilingData()
* Only generate profiling output if it'll be used; introduced Profiler::logData() that saves profiling data (database, udp, ...) to separate it from output generation
* Removed unused Profiler::getCaller(), not used at all, and we have wfGetCaller() that does the same thing

Use Profiler::instance() instead of $wgProfiler

(bug 27833) CollapsibleTabs doesn't collapse tabs due to JS errors
* Change remove() to detach() so .data() isn't lost
* Be more careful using the parent's .data() in addData()
* Change ele (which mysteriously isn't available in the closure) to this, they're the same anyway

(bug 28643) Merge Serbian language variant conversion improvements to trunk (r85224, r85239, r85308) from Nikola's branch

(bug 22209, bug 22574) Make indent, bullet list and numbered list buttons in the WikiEditor toolbar act on each line individually

* Reintroduce support for the splitlines argument in encapsulateSelection(). Was implemented by Adam Miller in r61493 but got lost somehow
* Add 'splitlines': true for the bullet list and numbered list buttons. The indent button already had it, so that part of r61493 did survive
* Copy options.pre and options.post to local pre and post variables instead of using (and modifying!) them directly all over the place

Followup to 86603 per CR: Fix var declaration of i

Move edit.js stuff to mediawiki.action.edit.js, and remove wikibits dependency. Wrap publicly accessible functions inside mw.toolbar & provide legacy shortcuts to those. Make the traditional toolbar use $.fn.textSelection, to reduce code duplication. This fixes Bug 27116, Bug 15705, Bug 11011

Fix Bug 25794: Disallow multiple table captions & only keep the first

Followup to r86202: Pass jQuery as argument in .ready(). Thanks Krinkle\!