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.

Followup r49655

I'd guess there's never been a Title::newFromTitle method, swapping for Title::newFromText

Parameter documentation

Explicitally define some variables

Function documentation

Fixup assignment in loop

comment out undefined variable used in function call

Mark a fixme comment in Oracle DB file for undefined variable

Add documentation

Fixup whitespace, brackets etc

Remove usage of @

re bug #25523: add mime.info and mime.types for *.dwg files so that when people add dwg to $wgFileExtensions, they can move files.

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

Fix invalid HTML introduced in r68096: Xml::buildForm() adds <table></table> too

* Added a third parameter to ChangeTags::buildTagFilterSelector() to pass the Title object instead of using $wgTitle, only used when the second parameter is true (no such call both in core and extensions)
* Expanded documentation

Follow-up r82391 per Nikerabbit's CR: Fix it in a better way

Remove comment that makes no sense post r82399

Fix comments and whitespace for \\\\ .

JavaScriptDistiller fixes:
* Removed some repeated subexpressions, /(.)*/ etc. These cause the stack space to be rapidly exhausted, leading to a segfault for malicious input. I replaced the ([\r\n]|.) subexpressions with a dot with a /s modifier. Added a pcre.recursion_limit hack to take care of the rest.
* Supported assertions and non-capturing subpatterns in ParseMaster::add() by recognising that parentheses that aren't followed by "?" are capturing. Used another assertion to do this.
* Fixed a bug whereby if a single-line comment had a slash in it, it was recognised as a regex instead of a comment. This occurred because the leading whitespace caused the regex regex to match at an earlier string position than the comment regex, giving it undue precedence. This was the subject of several reports on IRC and the main reason for me starting work on JavaScriptDistiller. Used a lookbehind assertion.
* Give comments precedence over regexes and strings where there is ambiguity at the same string location. Not sure if this does anything, but it seemed like a good idea at the time.
* Removed unused variable ParseMaster::$TRIM.
* To test it, I ran the old and new jquery.js output through Google Closure Compiler. The result was the same, proving that there are no functional differences.

* Use appendQueryValue() instead of appendQuery() so that if e.g. printable=no is already present in the URL, it doesn't become printable=no&printable=yes
* Call getLocalURL() on the local Title object instead of $wgTitle in WebRequest

We have already have a printable flag in the OutputPage object, so use it.
This was causing to have the printable version for every non-empty version of the printable paramter, instead of only for "yes".

(bug 27526) B/C in Special:EmailUser

Don't clutter the HTML source with useless class=""

Remove useless elseif compare. If the else if is doing the inverse (!) of the original if, it's redundant

Comment out 2 unused lines in JavaScriptDistiller

* For readability, use /x in regexes. Removed the start and end slashes from the input to ParseMaster::add(), to avoid confusion over the lack of a /x modifier in that input.
* Broke up several regexes and added comments, so that humans might be able to understand them.
* Tested md5(minify(jquery.js)) as before. Checked speed, it's the same.

Made the regexes slightly more readable by undoubling the backslashes where possible. Tested by ensuring that the minifier output is the same before and after, with jquery.js as input.

Add the cache key to the filter output, like what we do with the parser cache, to make it easier to debug filter issues.

Remove unused variables

Remove unreachable line in DifferenceEngine

Documentation for ObjectCache.php

Remove unused variables from CompareParsers and PreprocessDump

Fix wrong static calls

Fixup assignment in conditional

rem not needed die() and made vars actually private

2 subclasses of ResourceLoaderWikiModule implement a duplicate version of getFlip. Move it up into ResourceLoaderWikiModule

(Almost looks like it could all go into ResourceLoaderModule... But that uses a different version, seemingly, the only one. 3 other subclasses of ResourceLoaderModule implement the same version of getFlip as is moved into a parent class here... Seems daft to have a different version in the base abstract class... Minor oversight?)

Some documentation

Fix a bug in string recognition in JSDistiller. This *really* fixes bug 27481

Version the minifier cache in ResourceLoader

Fix for r82344, which causes JSDistiller to stop minifying comments because it thought they were regexes. Make the regex regex a little bit stricter again, forbidding a regex from starting with * and other invalid things that were disallowed prior to r82344 (but still dropping / ) and forbidding multiline regexes

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

Pass the url to ChangesFeed::getFeedObject() instead of grabbing it from $wgTitle
(no use of that function in extensions)

Revert r82340, caused something linke bug 27481 where spaces were being wrongly collapsed. Fixed by changing regex to be more flexible in protecting regexes; nothing wrong with being overinclusive in what we're protecting

Followup bug #27447 and r82308 and r82328 with patch supplied by
Carsten Nielsen to take care of the warning and supply some
documentation.  Also fixed up language in the comments.

(bug 27492) Fix regexes for matching JS regexes which could get messed up by the minifier in extreme circumstances

* Don't use assignment in a condition check
* Pass the Title object so that it doesn't rely on $wgTitle

Fix broken Special:Newpages Atom feed.

Per r82285: Add new messages for the default user groups. This way they can be found in Special:AllMessages at least.
Messages fot WMF specific user groups can be added to WikimediaMessages.i18n.php when the function is live.

Rv part of r82308 that causes warnings

Reverting r76017 per discussion on Bugzilla. Lack of community consensus for change, and the whole purpose of the "new messages" box is *NOT* to fit into the skin, so people will see it immediately.

more capitalization and less abbrevative ('T', 'err')

* (bug 27479) API error when using both prop=pageprops and prop=info&inprop=displaytitle

Followup r75282

Patch by Brad Jorsch

Follow-up r82310. using mw.config instead

Adding function to check if the current page is the main space or (optionally) related to the main page (eg. talk page). Compatible with both Vector and Monobook. Both namespace and page title may contain spaces (eg. Wiktionary talk:Main Page)

followup r82181 and r82215 to fix the FIXME and botched fix for FIXME.
Patch supplied by DieBuche (bug #27338) along with screenshot
demonstrating fix.

Took out fix for bug #27458 (“<gallery> has a white background now”)
since bug it conflicts with a fix for bug #26470.

(Bug #27447) Added option to toolbar-item to switch off text-selection

Carsten Nielsen explains:

    A client of mine was a bit annoyed by the editor:
    Every time he inserted - lets say - an image, the default contents of the
    image-fields "thumbs|width|caption...." were automatically marked/selected. So
    with a tip on the keyboard all "helping" text was gone.

    I added an option to the toolbar-items to optional deactivate this behaviour.

Add support for namespace prefixed elements in svg. ie. <svg:svg>

Fixes bug 27465

Revert r78585.  While I definitely think this is an area where there's scope for improvement in clarity and security, I'm no longer convinced that this is the way to proceed.

Fix the issue where all links on Special:Contributions where marked as stubs (NS_MAIN pages, for users with a stub threshold) broken since r63584.
resetArticleId() is clearing the right data set by Title::newFromRow() and placing just the id, which in turn avoids to lookup the other items.
This was already done in 1.16 (since r46179, it was useful until r46181) but resetArticleID() did not reset everything (r63584 change) and so
the bug wasn't present there.

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

jQueryize the toggle upload source type code

Create a user.groups module in ResourceLoader, which bundles a CSS and JS page for each usergroup the user is a member of (MediaWiki:Sysop.js, MediaWiki:Autoconfirmed.css, etc).  Groups '*' and 'user' are not included.

revert r82283, loads of unrelated changes

Create a user.groups module in ResourceLoader, which bundles a CSS and JS page for each usergroup the user is a member of (MediaWiki:Sysop.js, MediaWiki:Autoconfirmed.css, etc).  Groups '*' and 'user' are not included.

Add new message key from r80240 to message file. This way it can be found in Special:AllMessages at least.

Moved Skin::makeGlobalVariablesScript() to OutputPage::getJSVars()
* merged <script> and if ( window.mediaWiki ) block with the one of mediaWiki.loader as stated in the doc
* removed dependency of $wgTitle
* the two only calls to this functions are in SemanticForms, but will not affect current version of MediaWiki:
** specials/SF_UploadWindow.php: this file is only used before 1.16
** specials/SF_UploadWindow2.php: the call is part of the else branch of a "method_exists( $wgOut, 'addModules' )" check, which means it's not called since 1.17

Fix fatal in r82258

Per Philip Tzou, move wgUserVariant from the set of config variables to the set of pageview-specific variables, because it depends on user settings and such

revert r77452, r80492 and r82246.

Expose $wgExtensionAssetsPath in JS

Follow r77452, r80492. Avoid to find sub-languages when load css/js messages.