Style fixups

space to tab

follow up r101480

Cache the result of User::getDefaultOptions(), so the UserGetDefaultOptions hook (introduced in r97365) is run only once rather than on every call. Discovered this issue after seeing an insane number of memcached fetches in my RL2 code, caused by a UserGetDefaultOptions hook accessing the Gadgets storage backend (which has a memc layer) and being called a zillion times due to this bug.

Some cleanup to ProtectionForm:
* When the user doesn't have permission to protect/unprotect the page, display a page with the title set to "View protection level for ...", then the permissions errors and the current protection levels. Except for the title, this was already the case for permissions errors, but when the database was read-only this was throwing an exception due to the change in OutputPage::readOnlyPage()
* Moved check for NS_MEDIAWIKI a level up, so that is it also catched on page save, and make the output in such case much more user friendly
* Changed edit token to be salted with array( 'protect', 'page name' ) (as for action=watch)
* Allow extensions to return false in ProtectionForm::save hook for consistency with other hooks and in that case fill the error message with 'hookaborted' if the extension does not provide an customised error message.
* Changed ProtectionForm::show() to accept an array of message name and its parameters in addition of a string with raw HTML and converted calls to use the new method. This also means that ProtectionForm::save can use this
* Removed the setSubtitle() call for the 'formerror' message in case of error since it's overriden by the "back link"

(bug 32022) phpunit.php can not execute from another directory

Adds tests/phpunit to the include_path to let PHPUnit find out our files.
Also add to change the suite.xml paths from './path' to 'path' so
stream_resolve_path() correctly find out the suite by using include_path

bug 32086 Special:Permanentlink require a parameter

When Special:Permanentlink was not given any parameter it assumed oldid=0
and redirected to the Main_Page. This make it throw an error page instead.

bug 32100 installer complains about suhosin GET limit

Our logic was to warn whenever the suhosin GET limit was set.
This patch skip the warning if the limit is 1024 or more.

Also added 'qqq' message for 'config-suhosin-max-value-length'

Fix GROUP BY clause in Special:MostLinkedTemplates

This only cause a trouble for Microsoft SQL which require the column to be
referenced by their input name, not the output one (AS alias). Might trigger
on Oracle database.

This is part of bug 31913: 'GROUP BY clauses incorrect in SQL generated
for special pages'

Need backport in REL1_18

Updated the title of Special:Linksearch to "External links search" and updated qqq documentation accordingly.

checkbox could not be checked through URL parameter

On [[Special:EmailUser]], someone ought to be able to pass the checkboxes
names to overrides the default. As an example from bug 31770, the email
user page has a checkbox wpCCMe which let the user as for a copy of the
email being send. This is a user preference.

One change that checkbox state by appending ?wpCCMe=<boolean> ie:
Special:EmailUser/Hashar?wpCCMe=0
Special:EmailUser/Hashar?wpCCMe=1

The logic added in r84814 could have allowed checkboxes to be overriden
for GET form. Unfortunately, HTMLForm is mostly with the default POST.
Hence, when appending the query parameter, we would never honor it!

The fix is to still unconditionally look at the value if the form was
correctly submitted (ie has wpEditToken) and additionaly whenever
there is a checkbox name appearing in the query (wherever it is false
or true, hence the use of getVal()).

This is a regression in REL1_18.  I dont think it deserves a release notes
since it is not fixing anything compared to 1.17.

Bug fixed:
==========
* (bug 31770) Allow URL parameter wpCCMe on Special:EmailUser
* (bug 30909) URL parameters for checkboxes in Special:Block no longer work

Followup r101449, just changing forceRecache isn't actually enough

Add a method to set force to true on the options array to RebuildLocalisationCache

Bug 30120 - Interwiki links error after upgrade

Per Emufarmers, in wgDBTableOptions replace TYPE with ENGINE

Should at least stop the errors. Do need to do something in the updaters (can we have some "delayed messages" or something?

Needed more for the CLI updater/update.php, as putting it in the middle of the updaters is just pointless, as it won't be seen. Needs to be at the end...

Doing it in the webupdater is saner/easier, as it'll be displayed

Certainly, this does suffice for the moment

Fine on MySQL 4, as it accepts {TYPE|ENGINE}

Followup r101449, bit better message output

Bug 30934 - Warning: Invalid argument supplied for foreach() in /languages/Language.php on line 423

Doing a run of doRebuildLocalisationCache on each run of update.php

Follow-up r96261: remove the !important again, and then do the script defaults differently (so that those css rules are only for the default and don't affect when the user selected "never" or "always" underline)

FR r101442: forgot to replace another getBlocker()->getName() with getByName()

(bug 31739) Made Block code support ipb_by = 0 convention with for foreign users again, as it did pre 1.18. The byText param has been restored in the Block() constructor as CentralAuth still uses it. Some callers have been updated to reflect the fact that getBlocker() may be a local User or a string username.

Ideally, we might have a ForeignUser class and a generic User interface...but this will do for now.

Made some callers of getBlocker()->getName() just use getByName()

Another old, uncommitted, test (passing on NtPP)

Template as parameter name.
I had this test pending longtime, as it is failing in NativePreprocessor.

jquery.tablesorter: More selector fixes
* Limit to rowSpans (td or th) within tableRows that are a direct child of this table's tBody
* Follows-up r99092, r101417

jquery.tablesorter: Selector fixes
* Various fixes to limit the selector to the current table and avoid selecting similar elements from other, nested, tables.
* Should fix bug 32049
* Follows-up r97150 and likely some other refactoring commits

Localisation updates for core and extension messages from translatewiki.net

Followup to r101021, add back to Pg schema so we can install mediawiki

* Made mRedirectTarget, mLastRevision, mTimestamp, mTouched of WikiPage protected; no outside users.
* Added TODO comment to doRollback() and fixed mLatest comment.

Prevent notice in OutputPage by casting to an array. This already exists on the WMF cluster as a live hack

Renamed annoying --iknowwhatimdoing param to --force (bug 32073)

Simplify r101365, this cruft is not really needed

Fixed annoying intermittent BlockTest failures. No surprise we had 'em since blocks set to expire RIGHT NOW tend to expire at that very time:)

* (bug 31818) 'usercreated' message now supports GENDER

On the preferences form, make preferences and other items at the top level of a section (e.g. with 'section' => 'foo') appear above rather than below subsections (e.g. stuff with 'section' => 'foo/bar'). This seems like natural behavior to me, but for some reason HTMLForm had it the other way around. I left the default behavior unchanged in HTMLForm and changed it only in PreferencesForm.

This change is needed for my work on the Gadgets extension in the RL2 branch (one example is adding text on top of a preferences section using a dummy type=>'info' preference, you want that to show up on top, not after all the subsections). The targeted use case (sections that contain both subsections and form elements) does not occur for core preferences, and is very uncommon in extensions. I managed to find such uses in only 3 of them: CreateAPage (in unreachable code), Tasks and EditSimilar add preferences to core sections that also contain subsections.

Followup r101354: pass $this to the PreferencesGetLegend hook so the callee will have a ContextSource to work with

Introduce the PreferencesGetLegend hook. I need this to customize the labels of <fieldset>s in the preferences for the Gadgets extension in the RL2 branch and I can't think of a cleaner way to do this.

Add namespace gender aliases for 'stq'
http://de.wikipedia.org/w/index.php?title=Benutzer_Diskussion%3ARaymond&action=historysubmit&diff=95232734&oldid=95208259

(bug 32051) Fix description for wlprop=sizes

Fix r101294: newNullRevision isn't using selectField() or the like...it needs to specify sha1

Localisation updates for core and extension messages from translatewiki.net

Introduce cmstartsortkeyprefix and cmendsortkeyprefix parameters so categorized pages can be looked up using a category+sortkey combination. Based on a feature request from Danny B on IRC

Hmph, yet another bug caused by not having @since

FU r101021: made newNullRevision() re-use the sha1 of the reference revision

rc_cur_time is obsolete; added comment

Bug 31098: Template loop through MediaWiki: messages (int: function) not being detected.
The trick is not to replaceVariables() ourselves, but to leave that to the preprocessor,
which is then correctly detecting the loop.

Suggest RewriteEngine in sample config instead of ErrorDocument to get correct status headers and short-circuit some cases (e.g. apache not bothering to hit thumb_handler for bogus urls).

The test "{{Foo|1=bar|2=baz}}" got removed in r96887.
Readding without the linestart="1", which is all the difference removed on this file in r96887.

Localisation updates for core and extension messages from translatewiki.net

Followup r100927 and r100905, when setting a FauxRequest use the NEW context

* (bug 19186) Special:UserLogin's title on Special:SpecialPages now says "create account" when the user cannot create an account
* Also added a missing call to setHeaders()

Consistency fixes:
* punctuation for error messages.
* capital letter after colon

Updated rc_type documentation

Remove more assignments in conditionals (while each to foreach)

Followup r100995, missed the darn @private tag

More documentation

Die @private

Kill various unused variables

Comment some out also

Add some bits of documentation

Fix private member variable access, use getName(), rather than relying on __get()

Disable proxy for local URLs instead of using a local proxy (which might not always exist). In r61357 a comment "Not sure this makes any sense." was removed. Indeed, that comment seems to have been right :)

Fix title case.

FU r100716:
* Pass the File object in LocalFilePurgeThumbnails so handlers can use the getRel() functions and such
* Also added the hook to hooks.txt, as this could be useful for other caches or things in thumb_handler.php

Localisation updates for core and extension messages from translatewiki.net

FU r101117:
* Moved 'checkCache' callback to cURL function as the 'fillCache' function is only ever called from that function

Followup r101161 having array() doesn't work so well with isset, changing to null

More documentation updates

Update documentation

Fix some context method calls which were considered deprecated

Fix Bug #30383 for Pg

* use outline instead of border to not change the dimensions of the element

* Factorise the code used to prepare the OutputPage object to display an error page in OutputPage::prepareErrorPage()
* Changed default value of OutputPage::getPageTitle() to match the value of the 'internalerror' message

An alternative to using DOMDocument for normalizing when using Preprocessor_Hash.
Follow-up r99909.

Follow-up r99909. If there is a preprocessToXml method, use that.
Some preprocessor could have preprocessToObj as a native type.

PHP Fatal error:  Call to a member function __toString() on a non-object in phase3/tests/phpunit/includes/parser/PreprocessorTest.php on line 119

* Use local context to get messages
* Also override QuickTemplate::msg(), QuickTemplate::msgHtml() and QuickTemplate::msgWiki() in BaseTemplate to use the context

svn:eol-style native

Pass the (correct) session doApiRequest()

Followup to r101120, replaced hardcoded parentheses with the 'parentheses' message.

Adding dirmark after changed bytes to prevent mixing with LTR usernames and summaries.

FU r100535:
* Thumb handler can now also work without cURL
* Combined related config vars into array config vars
* Folded $thgThumb404File into $thgThumbCallbacks
* Avoided some global pollution

Rm unused parameter

Break long lines (and prove svn is working)

Removed $wgUseNormalUser var; not used by any scripts anymore

Fixes Bug #6793 — “New Variable __NEWSECTIONS__ like __TOC__” by
adding Nx.devnull's patch that adds a hook needed by
Extension:PlaceNewSection.

Bug 32004 - [regression?] style class .mw-specialpagecached for Special:SpecialPages is not defined

Patch by MrBlueSky

Reverted r84395,r98710: thumbnails must be purged on file deletion

Add dropTable method

FU r100535: renamed thumb-handler.php to match the other files here

Added pfunc profiling to braceSubstitution()

Convert the string to single quoted and manually escape it.
http://us.php.net/manual/en/language.types.string.php

addslashes() is convenient, but this way we have no legitimate
usage, and it's easier to have it banned everywhere.

Followup r100905, disabling testGetRollbackToken because it reports as failed on Jenkins but no one can reproduce it.

FU r100915: split out GlobalWithDBTest (tests which need the DB)

The early return added in r98698 was skipping the closing wfProfileOut().

Removal of unused globals

Per Nikerabbit's comment on r100621:
* Removed OutputPage::setPageTitleMsg() and OutputPage::setHTMLTitleMsg() and make OutputPage::setPageTitle() and OutputPage::setHTMLTitle() accept a Message object
* Updated core calls (including some that I missed last time because of non-matching case)
* Added Message::setContext() and use it in RequestContext so that I don't need to duplicate the call in OutputPage
* Yes, I'm calling $this->msg() on places and then setting the context one more time in OutputPage::setPageTitle() or OutputPage::setHTMLTitle(), but at least I won't be confused about which objects $

Localisation updates for core and extension messages from translatewiki.net

* Added pageJoinCond() and userJoinCond() to Revision and exposed them publicly
* Removed commented-out code

Fix for r100905:
* Make tests work again
* Added ApiTestCase::doApiRequest() and related to pass it to the context and removed override of $wgUser where possible
* Fix ApiLogin to get the correct User object (i.e. the logged in one)
* Fix ApiBase to feed a RequestContext to setContext() so that ApiLogin can call setUser()

Reinstated r94289 et all - rev_sha1/ar_sha1 field for bug 21860

Added default updateSkippedMessage() message

Set batch size in PopulateRevisionLength

* Pass in $user to Revision::getText()
* Minor style cleanups

Fixes Bug #28277 by allowing redirects on the stream reader.  Fixes a
regression where pages specifying a w:en:Sprint would result in a url
like http://pt.wikipedia.org/wiki/en:Special:Export/Sprint which, at
least in 1.16, would mean the reader follow the redirect to
http://en.wikipedia.org/wiki/Special:Export/Sprint.

parsertest now destroy virtual table under SQLite

Fix:
Bug 29912 - Unit tests break if parsertest tables are still present

The parserTest suite duplicate available tables. When using temporary table
it assumed every table are temporary. Under SQLite searchindex is a virtual
table and thus remained after a run of parserTests.php
As a side effect, when running the PHPUnit with that remaining table, it
caused a nice error:

The last attempted database query was:
"DELETE FROM unittest_parsertest_searchindex"
from within function "MediaWikiTestCase::resetDB".
Database returned error "1: SQL logic error or missing database"

This patch is not needed for REL1_18 continuous integration since we
never run the parserTests.php script.

Remove todo, it's public now

Use canonical case