Fix backwards canCreateAccounts() check, remove setting search_path for $wgDBuser, since this is set per-session on open()

Rm unused member variable

Per bug 26612, set datestyle and timezone in open() rather than relying on $wgDBuser having specific settings.

Followup r82659

Comment out the USE INDEX, it creates a query error as the query is built wrongly.

Will report as a bug.

Will revert to previous behaviour of not forcing any index

Ununsed variables

Documentation and explicit definitions

(bug 20244) Installer does not validate SQLite database directory for stable path.

This is a must for 1.17 so adding a note to HISTORY instead of RELEASE-NOTES

Fixup SpecialListusers, $useIndex wasn't being used in the query, so make it be

Very basic JavaScriiptDistiller tests, needs serious expansion

Revert coloring from r79117: doesn't work on Windows

* Rewrote StripState to not use ReplacementArray. The memory usage of FSS was excessive when there were many (>10k) strip items. I used preg_replace_callback(), which is slower than strtr() in the simplest case, but much faster than it when the markers have different lengths, which they usually do.
* It was not necessary to preserve the $stripState->general->setPair() interface since it wasn't used by any extensions.
* Moved StripState to its own file.
* Refactored serialiseHalfParsedText() and unserialiseHalfParsedText() so that the bulk of the functionality is in the relevant modules, instead of using scary direct access to object member variables. Made it support the new StripState. It seemed like a lot of work to go to to support an "emergency optimisation" feature in Cite. Cite updates will be in a subsequent commit.
* Fixed spelling of serialiseHalfParsedText() and unserialiseHalfParsedText(), there is unavoidable interface breakage anyway, due to cache object versioning.
* Moved transparent tags to their own function, as requested in a fixme comment.
* Added documentation for markerSkipCallback().
* Removed OnlyIncludeReplacer, unused since MW 1.12.

Followup r82641, set default for 2nd param

Fix caller in SpecialLinkSearch.php

Remove now redundant comment after r82641

Followup r78786, per complaints on r82636, revert parameter ordering

Mark doQuery as deprecated/not the suggested run point

Add a couple of braces

Localisation updates for core and extension messages from translatewiki.net (2011-02-22 21:13:00 UTC)

Follow-up r82504: use JS redirection only for IE, I don't like how it looks at least in Opera (sometimes images are not loaded)

* random redirect-related fixes:
** automatically add redirect=no to all links on the redirect page in Article::viewRedirect.
** properly check if redirects are enabled if $wgMaxRedirects < 1 (moved check from Title::newFromRedirectArray to Title::newFromRedirectInternal).

* (bug 27633) Add characters to linkTrail for Potuguese (pt and pt-br)

Add RELEASE-NOTES for r82547

Remove trailing ?>

Rename startExternalParse() when used for internal usage, per r80818 CR.

Merge r67177 from 1.16wmf4

maintenance/commandLine.inc loads DefaultSettings, LocalSettings and then runs Setup.php
As Setup.php assigns variables based on the cache config, bootstrap.php was late on reseting them, as some objects were already created.
So we could end up with a SqlBagOStuff created there, which when later accessed (such as trying to invalidate the cache for a user) would
-as any non-sqlite SqlBagOStuff- open a new db connection. Which is precisely what we shall not be done when dealing with temporary tables
(and would indeed fail miserably due to not finding unittest_objectcache table).

In summary, reenabling temporary tables disabled in r79411.

remove hardcoded local path

Follow up r80205 magic, so that you can also use require.

improve namespace related methods

MWNamespace::getTalk() could give erroneus results when using it on specials
namespaces (NS_MEDIA, NS_SPECIAL). It now use MWNamespace::isMethodValidFor()
which will throw an exception if a special namespace was given.

MWNamespace::getSubject() is now returning identity for specials namespaces.

New MWNamespace::getAssociated() used to find out the subject page of a talk
page and vice versa. Special namespaces will results in an exception.

TESTS:

Added tests for almost complete code coverage. Functions relying on global
$wgCanonicalNamespaces are still incomplete though.
MWNamespace::isMovable() needs more assertions.

Tests results (ignoring incomplete tests output):

$ php phpunit.php --filter MWNamespace
PHPUnit 3.5.10 by Sebastian Bergmann.

.........IIIII..........

Time: 1 second, Memory: 31.75Mb

OK, but incomplete or skipped tests!
Tests: 24, Assertions: 99, Incomplete: 5.

Add missing  to mw.config

Localisation updates for core and extension messages from translatewiki.net (2011-02-21 20:13:00 UTC)

Follow-up r73270. Moving the attribute check down to the point where it's for sure not a string or undefined.  (caused 'cant call getAttribute member function on non-object el [undefined]'. Also add a check to verify that el is a dom element and not just not indefined

* (bug 27616) Add userid of blocked user and blocker to list=blocks

Also tweak comment about "blocking admin". Non admins can block if they have the right... Sooo,  "user" is more appropriate

* (bug 27611) list=blocks: Use ipb_by_text instead of join with user table

Save a table join when looking up blocks (yay!)

Minor issue, for blocks where users have been renamed, we get their old name (Not a big deal IMHO)

De-escaping $ at end of linkTrail regexes dating back from r15823 and beyond

Per Hashar on r82547, u modifier needed on regexes

Fixup and add another return type

Some more member variable type hints

Rewrote LanguageConverter::autoConvert() to make it use preg_match() with an offset instead of preg_split(). Reduces memory usage for my test case ([[台灣演員列表]]) to a negligible amount. This should eliminate the most common cause of OOMs on Wikimedia. Produces the exact same output for that test case for the zh -> zh-tw, parser tests pass, seems to work.

Add basic linktrail to SH and SL

Gallery's perrow was overwriting the original style= attribute.

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

(bug 27355) Guard against context being null, this happens when wikiEditor disables itself in unsupported browsers and was causing a JS error

spaces, added some braces to DairikiDiff.php

Spaces to tabs

Force wgDebugTimestamps false

Test GlobalTest::testDebugFunctionTest() does not take into account the setting
$wgDebugTimestamps = true.  Setting it false makes sure the tests pass though
not testing the settings :(

(bug 27468) Move private styles out to their own section, so they override everything except user styles. Moved from one makeResourceLoaderLink() call with array_merge() to multiple calls, because makeResourceLoaderLink() orders the modules, then groups them, so the input order is not preserved. Also fixed an issue in makeResourceLoaderLink() where the $uery variable was reused in each iteration but not cleared, allowing pollution to occur in theory (doesn't seem to have happened in practice, though).

Revert errorneous line removal in r82439

wfMsgReal() -> wfMsgExt()

Per Tim Starling, follow-up r76252: move WikiDiff.php to DairikiDiff.php to not confuse with the wikidiff extension

Per Platonides; follow-up r78201: add "stubthreshold" to ParserOptions; also provides a proper fix to r81765

Followup r82509, prefix the code for urlwidth too

Remove error duplication in ApiQueryStashImageInfo by using non hardcoded prefix in ApiQueryImageInfo

Per comment on r82477, update output for prop=statimageinfo

More explicit variable definitions, function documentation

Bug 26683: Unable to download index.php from localhost message displayed at complete page in IE7 & IE8

Fix ArticleTablesTest after r81528. It needs to set both $wgLanguageCode and $wgContLang.
Added setUp() and teardown()

Break the self-reference between parser and preprocessor, so they can be counting collected.
Saved a couple of Mb in the destructive make (126.25Mb -> 124.25Mb).

Save and restore $wgHooks in NewParserTest.
ParserGetVariableValueTs hook still being set to ParserTest::getFakeTimestamp was the reason for the crazy failures of MagicVariableTest inside phpunit that I was seeing.

bug 27146

mw.loader.implement now calls script without setting mw to mediaWiki since this is already globally avaiable
* Also shortens a few wrapper functions the same way
* Partial revert of r82496. Apparantly mw-alias is not available in startUp module. TODO: Find out why ?

(See also r82496 commit message)

Using the mw alias for mediaWiki in core. Saves bandwidth and makes sense (there's no point in a wrapper function for ($,mw) if we use jQuery and mediaWiki inside of it).

TODO: Since the alias is globally available from the start, the 'mw' argument in the wrapper is redundant, so that should be removed at some point as well.

(bug 3112) Adding colors for active and visited external links.
This is in-core for internal links and red links, but there's no visual clue of active and visited links for external links.

Normal external: #36b;
/* Added: */
Active external: #b63;
Visited external: #636;

These two colors (orange and purple -ish) are in the same saturation/brightness as the blue for normal (inactive/unvisited) external links.
Since these visual clues were also lacking in external links to interwikis I've replaced that override with similar colors as well.

And some more function level documentation

More function documentation

Localization update for he.

More function documentation

Explicitally define variables

Explicitally define some variables

Bracify file

Improve method documentation

Few bits of fixup

Don't use getCheck, because the opts array is reused to generate the query for the RSS/Atom feed.
getCheck checks for presence, not for true/false, so that was why rss feeds behaved as if
deletedOnly was set.

Follow up to r65651
Fixes bug #27546

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

All functions of the importer have $title, origTitle, but processTitle() returned $title, origTitle.
This caused confusion and a misusage of the processTitle() result.

Follow up to r66267.
Fixes bug 27486

(Bug 27539) Allow attributes beginning with a digit in wiktext tag parameters.
Its removal in r70849 breaks ProofreadPage extension.
Restricted r82475 relaxation to just numbers.
Added tests.

This only affects wikitext (tag hooks).
MW_ATTRIBS_REGEX is only used through decodeTagAttributes() calls.
fixTagAttributes() calls decodeTagAttributes(), and would be nastier to
fix, since it is called with HTML parameters (eg. by removeHTMLtags)
but such incorrect parameters grabbed would be removed by validateTagAttributes()

* (bug 27560) Search queries no longer fail in walloon language

* (bug 27554) Update api information text to reflect change in bug 26125

Relaxed the allowed format of parser tag attributes as per bug 27539. One side-effect is that this will fix the widely-used pagelist tag which has "1to10" as an attribute name.

Revert r74387: it may not fix the IE8 jumping bug in all cases, but it definitely fixes it in most cases. Reverting a partial fix for not being perfect, then never coming up with a full fix is not acceptable.

(Bug #27467) preload can leave UNIQ

Patch to fix from P.Copp

Follow-up r82452: need to transform entries sinces some wiki like to use variables or parser functions in the sidebar

Improving jquery.makeCollapsible & small fixed mw.util
* Making makeCollapsible more DRY.
* Small optimalizations and fixes in mw.util

(bug 27302) Avoid unnecessary requests for user and site modules if the relevant wiki pages don't exist.

Done by adding isKnownEmpty() to ResourceLoaderModule and overriding it to check for page existence in ResourceLoaderWikiModule. Needed to rearrange some code in OutputPage::makeResourceLoaderLink() to have the emptiness check and dropping of modules work properly. Also factored the page_touched check in ResourceLoaderWikiModule::getModifiedTime() out to a separate method (getTitleMtimes()) and moved in-object caching there as well, so getModifiedTime() and isKnownEmpty() share code and caching for their timestamp/existence checks.

This does not account for the case where e.g. a user has user CSS but no user JS: I had implemented this by checking for $context->getOnly() in getTitleMtimes(), but then realized it's not safe to do this in a function called by getModifiedTime(): it causes the timestamp list in the startup module to only take scripts in account for wiki modules, because the startup module has &only=scripts set

Remove unused global.

Follow up r82091, which uses the local file constant SERVICES_JSON_LOOSE_TYPE.

Fix logic error in r73204 (!) causing ResourceLoaderContext::getHash() to always return null

Partial revert my accidental revert of r82366 in r82445.

Close the xmlreader when we are done with it.

(bug 27328) Redo r82218 properly, expanding URLs with absolute paths

Revert r82218 , doesn't fix absolute path URLs but breaks them. Will fix properly in next commit

* Use OutputPage::getSyndicationLinks() to build feed links
* Ported r82395 to legacy skins (possible double "printable" parameter in URL and hidden dependency of $wgTitle)

* Use $this->mTitle instead of $wgTitle
* Put back the wfMsgForContentNoTrans() added in r66835 but reverted in r68707 since wfMsgForContent() is breaking the {{...}} syntax

Merged retrieval of request URL and header from WebRequest and Setup.php to avoid duplicate code and reduced the number of wfDebug() calls in Setup.php.
The only side effect is that headers name are in uppercase in the debug log.

We have a OutputPage object; so use it to get the Title object instead of $wgTitle

Remove parentheses for Xmf file-info-size message

Now being done in the code. Should help fixing bug 27472

Bug 27518: Updater calls PopulateLogSearch before adding logging.log_user_text; populateLogUsertext not called

Bug 27053 - sourceFile() didn't expand all the required variables. Apparently, broken by r81084

Follow-up r76456: more thorough check for UTF-8 support. Now it checks whether PCRE can actually convert from uppercase to lowercase and vice versa.

Bug #26059 — Add support for KML/KMZ filetype

Patch from Derk-Jan Hartman of which he writes:

    I figured adding kml support would be a breeze, but I had not
    counted on the brain dead browser that is IE6.

    Unfortunately, kml contains the element <heading, which triggers
    the protection in detectScript() that protects from uploads that
    IE6 might mistake for HTML.  It triggers on "<head" not sure if we
    can work around this, but Tim will know.

get rid of unnecessary double quotes

add braces + some other coding style tweaks

More function and variable documentation

More function level documentation

Fixup some indenting

More parameter documentation

More function level documentation

Follow up r63584. The change of resetArticleID() could have all kinds of
side effects, given that it stops lazy cache fillings.