Added some lock call profiling

(bug 34863) Make it so an image that was previously deleted and currently does not exist (locally or foreignly), shows a snippet from the deletion (and move) log.

Currently things outside of file namespace display the snippet, so file ns should display snippet for completeness. As far as I can tell, the only reason files don't is because ImagePage class overrides the Article class, and it was unintentional that the snippet was not included in ImagePage class.

Added missing space :)

Add some {}

Ping r111605

Improves on r112836 by only using borders on the left and right of the changed text, which makes multi-line changes render much nicer.

Localisation update for core and extension messages from translatewiki.net

(bug 34873) List contributions of existing users that have invalid names

r112839: missing file change

Error log readability is nice

Actually enforce the FileBackend name requirements instead of just having a comment

* Pass number with numParams() to Message object and let that object format the number
* Added missing escaped() call from my last commit

Resolves bug #11374 and improves on r94429, r94461, r101147, r105280, r106884, r107127, r108975 and r109932 which were reverted by r112750.

Squishing resources/ images

Tidy up return values of envChecks

False is used to to signify a failure mode we need to do stuff for, so returning true is ok in other cases, and makes it explicit

fix typo in r112479

* (bug 34849) Diff when editing an old version show the comparison with the current text again

Hrrrmp *grumbles*

Credit for Christian Aistleitner

Bug 34818 improves regex to capture DB error message

Patch by christian at quelltextlich dot at

Bug 34734 - compressOld.php ignoring -s option

This is fixing a regression caused by r90549.
Patch by christian at quelltextlich dot at

Whitespace cleanup

Restore many missing spaces from r112589
Also fixed indentation by spaces
And please don't ever use != ''. It's misleading.

[mediawiki.action.history.diff.css] Revert 1.19 style changes back to how it was in REL1_18
* After the 1.19wmf1 deployment several people have complained about various aspects of the new diff styling
-- The contrast being too low in the highlighted part (darker background behind bolded text)
-- The colors not being obvious perhaps (orange/blue)
-- Color blind users not seeing the difference very well between the light tones of the orange and blue
-- Trevor mentioned something about W3C Accessibility guidelines

To play it safe for now I think we should revert these changes to the status quo, and take the next few days (or weeks) to carefully check the concerns, perhaps look at other diff tools out there for inspiration (GitHub, Gerrit, LocalWiki, ..).

* Goes back to r91762 (last rev before REL1_18)
-- reverting: r94429, r94461, r101147, r105280, r106884, r107127, r108975, r109932
* Re-opens bug 33335

In populateImageSha1.php:
* Added 'force' parameter to run the script on rows with sha1 existing values (and where the file exists)
* Do the all the old versions of files after doing their current versions
* Broke long line and made a few minor cleanups

Fix comment from r54410, add rationale for using MySQL command line pipe

Added unsetVal() method as antithesis to setVal()

More image shrinkage!

Fix double indenting

* (bug 34768) Set width and height for search-ltr.png (and search-rtl.png) in Vector skin

<img> in <button> didn't have width or height set.

Followup r112374/bug 34653

We still get legacy log entries, so grab the data from parameters as applicable

Should probably be encapsulated into DatabaseLogEntry itself, with a getParamValue esk wrapper

This fixes issues noticed on live site for the moment

TODO: Check if rights/block are still ok (probably not?)
TODO: If (especially if above needs doing) encapsulate grabbing of old/new parameters to logging code

Localisation update for core and extension messages from translatewiki.net

[mediawiki.js] minor improvements based on recently added features to JSLint

* Fix with help of JSLint:
-- else statement after a return at the end of a connected if statement
* Using arguments.length incorrectly
* Added JSLint settings to the file

r111588: refactored proxy/noProxy code slightly for simplicity

making sure colored buttons get white icons, implementing a smarter css selector scheme for large colored buttons (less redundancy)

* Don't execute the LinkBatch if there are now rows
* Call methods directly on ResultWrapper instead of DatabaseBase

fix r112591: re-add whitespace between message name and (talk)

Use local context to get messages

Pass __METHOD__ to DatabaseBase::commit() and DatabaseBase::rollback()

Follow-up r112565: fix code duplication

(bug 34702) More localised parentheses.

(bug 34702) More localised parentheses. Patch contributed by Yusuke Matsubara.

[mediawiki.action.edit] Clean up and bug fixes
* Bugfix: Locally alias the toolbar object and replace usage of 'this' with 'toolbar'. Calls to window.addButton were failing due to it referring to the dynamic 'this' context which changes when the function is a member of a different object.
* Bugfux: Move currentFocused = $( '#wpTextbox1' ) into the dom-ready hook. When executed before dom-ready the element doesn't exist yet. r111983 did this for $toolbar, but not for currentFocused (bug 34538)

* Move onReady and isReady to the local scope (introduced recently in r111983, not used or meant to be used publicly)
* Merge onReady with the $(document).ready hook function, same purpose
* JS conventions (closure arguments, var hoisting, whitespace)

* Touch r111983, r112451, r112567

Fix r110209: Move the invalid title check above the first use of the title object, to avoid a fatal error, for example when navigating to [[Special:MovePage]] with no target.

Fix scoping in r112451: isReady was accidentally used as a global variable

r112563: Removed random bogus user IDs passed into to Block constructor in tests

(bug 34762) Calling close() on a DatabaseBase object now clears the connection. Based on patch by Christian.

By the way, the duplication here is stupid.

Force the target user ID to be the correct foreign ID on the relevant local wikis for crosswiki suppress blocks.

[IRC Log Message] Fix for protect/protect and protect/modify

* protect/protect and protect/modify were still failing because the output on IRC is not matching the i18n messages. The bots look for comment.match(<i18n-message> + (optionally) followed by ": user comment").
* The problem with protect/protect and protect/modify is that the protection settings are also inserted in somewhere in the message. Before MediaWiki 1.19 these were included in the IRC message as part of $1 (title). That may be ugly, but that's how it has always been. Since the point is to reproduce the old output we need to fix this one as well. This will make the output like:

  Krinkle * protected "[[Sandbox [edit=autoconfirmed)]]"

which is exactly how it has been before and that's the the bots rely on (which was presumably done originally like that since it was the only way to make the output match the i18n message)

* Fixes more bug 34508

[RecentChangeTest] Refactor unit test to test better test the issue
* assert used message-keys/variables instead of hardcoded values
* Pass $comment to the assertion helper instead of using it by default so that actions that don't use a comment (such as patrol) don't get forced a comment, to better replicate the IRC behavior
* Adding placeholder for Auto-Edit-Summary things that should also be checked in here, not sure how to do those yet
* Adding tests for block/* and protect/*

Fix exception from list=recentchanges rcprop=loginfo: update for new log system

Dummy proof TempFSFile purging when the constructor is used directly to avoid data loss.

In ResourceLoaderContext, normalize invalid skin names to $wgDefaultSkin . This should help a lot with the pollution of the module_deps table, which is currently littered with invalid skin names from people trying to hack the site. I found 3,897 (!!) distinct values for md_skin

Sample from the query result:

| md_module                   | md_skin                          |
|-----------------------------|----------------------------------|
| ext.vector.collapsibleNav   | vector'                          |
| ext.vector.collapsibleNav   | vector' and 1=1--                |
| ext.vector.collapsibleNav   | vector' and 1=2--                |
| ext.vector.collapsibleNav   | vector')waitfor delay'0:0:20'--  |
| ext.vector.collapsibleNav   | vector',0)waitfor delay'0:0:20'- |
| ext.vector.collapsibleNav   | vector',0,0)waitfor delay'0:0:20 |
| ext.vector.collapsibleNav   | vector',0,0,0)waitfor delay'0:0: |
| ext.vector.collapsibleNav   | vector'waitfor delay'0:0:20'--   |
| ext.vector.collapsibleNav   | vector../../../../../../../../.. |
[...]
| ext.vector.sectionEditLinks | vector<script src=               |
| ext.vector.sectionEditLinks | vector?.tri.co.id/               |
| ext.vector.sectionEditLinks | vector??id=jCustomerWAPProv      |
| ext.vector.sectionEditLinks | vector??id=wap.mauj.com....      |
| ext.vector.sectionEditLinks | vector?id=202.87.41.147....      |
| ext.vector.sectionEditLinks | vector?java                      |
| ext.vector.sectionEditLinks | vector?m.vuclip.com/             |
| ext.vector.sectionEditLinks | vector?toyota.co.id              |
| ext.vector.sectionEditLinks | vectorGET                        |
| ext.vector.sectionEditLinks | vector]]>>                       |
| ext.vector.sectionEditLinks | vector`ping -c 20 127.0.0.1`     |
| ext.vector.sectionEditLinks | vector|echo 9e7f7fd5750593ab cef |
| ext.vector.sectionEditLinks | vector|ping -c 20 127.0.0.1||x   |

(bug 34542) Calling mw.loader.load('someurlhere') sometimes calls document.write() from inside an asynchronously loaded script. This is because we added the async parameter in 1.19, and made it default to false, which works for 1.18 HTML but doesn't work for user/site/Gadget JS written for 1.18 . So make the async parameter default to true when a URL is passed and to false otherwise. This is kind of evil but it's the only sane way of preserving b/c that I can think of

Revert r110045: well-meaning but broken attempt to apply preemptive XSS protection on messages

This caused bug 34708: broke usage examples in WikiEditor which include <a href> or <img> tags as-is, which is perfectly legit HTML.

Localisation update for core and extension messages from translatewiki.net

They don't want this customised - reported on IRC

* partly fixes two issues exposed by bug 34280
** forcing recreate if creating user
** parent::createTables was setting DB to a wrong setting

Combine getLanguageNames and getTranslatedLanguageNames into one function which is much easier and more logical to use. Does not contain any actual changes yet in what the function does.
* Reduces the overly long code in r107002, and reduces code for {{#language:}}
* Fixes the language list in Special:Translate which contained languages that gave "invalid code" when selecting

(bug 34736) empty limit on special pages causes navigation issues

On special pages, the older link showed a limit='' query parameter which
made the page show nothing ('' is evalued to a limit of 0 articles).

The cause is r105809 which changed the way we interprets parameters in
wfArrayToCGI() and wfCGIToArray(). Previously, An empty string value would skip
the key, after r105809, we need to use null.

Typofix, ping r112427

Make userAgent also work as an option to MWHttpRequest::factory() and MWHttpRequest::__construct(), not just Http::request().

Remove empty messages introduced in r108445. Translation is present in fallback tt-cyrl.

[mediawik.special.changeslist] Remove ugly arrow swapping hack
* Use CSS and change the background image, instead of displaying 2 <img> tags and hiding one based on the parent element's class
* touches bug 17616 a bit, although it doesn't address the focus issue yet, but does improve the overal structure of this module
* Also speeds up by:
- Outputting less markup
- Having 1 embedded images in the CSS file instead of (N changes)*<img> tags
* jquery.makeCollapsible:

Release notes for added feature in r112451 which fixed bug 33952

Re-add the exception log removed in r88392: unlike Chrome, the current Firebug displays exceptions dumped in this way with full details, including a backtrace

* Fix r111983 (bug 34662): make mw.toolbar.addButton() work even after DOM ready
* Don't use an empty array as the default for $toolbar, use false which will throw an exception on a logic error.
* Fix exception handling in mw.loader: have handlePending() rethrow its exception so that the exception handler in execute() can run, so that the exception gets logged. But don't rethrow the exception in execute(), because that may break other modules.

Localisation updates for core and extension messages from translatewiki.net

Pass __METHOD__ to DatabaseBase::begin(), DatabaseBase::commit() and DatabaseBase::rollback()

Add lang and hreflang tags to the variants in the variant menu's. This fixes bug 24742.

I have not added the rel 'alternate' links, because they are also present in the <head>

[mediawiki.api.watch] Re-use watch() in unwatch().
- Fix dependency issue. In practice not an issue due to user.tokens being in the HTML output before the first mw.loader.load()
- Follows-up r107969, r107350

Added @since annotations

Use local context ( $this->msg() ) instead of global method wfMessage() per IAlex. Follows-up r112433.

Use local context to get messages

(bug 34702) Localised parentheses are now used in more special pages.

Follow up r112423, added release note.

Wow. Messy. Follow-up r112426, r112424: Fix incorrect calls. Should have been escaped() instead of escape().

Follow-up r112424: Per IAlex on IRC: Use escape() instead of plain() for improved security.

Follow-up r112410, r112412: Cheaper processing using plain().

Fixes bug 34723: applies the English language to CSS/JS both as a user subpage and a MediaWiki space page, and consequently applies LTR to JS and CSS in the textarea.

rename test file to end with Test.php :D

basic tests for bug 34508

Follow-up r112410: (bug 34702) Use localised parentheses for comment, too.

(bug 34702) Use localised parentheses in Special:Contributions.

Localisation updates for core and extension messages from translatewiki.net

Fixes bug 34715 Image redirects show the "redirected from" subtitle twice.

ImagePage no longer needs to set the "redirectedfrom" header by itself. Seems Article.php is taking care of it now. I suspect this was caused by the refactoring of WikiPage, Article, etc.

release-notes 1.19: Mark .wikitable change as BREAKING-CHANGES

Forgot to commit this file in r112396

* (bug 34246) Display MediaWiki:Whatlinkshere-summary again in Special:Whatlinkshere

* (bug 34528) Don't parse section name in edit section tooltips

Pass __METHOD__ to DatabaseBase::begin(), DatabaseBase::commit() and DatabaseBase::rollback()

Use local context to get messages

fixing bug 34609 - datepicker should always be set to display:none by default

Fixed $headers check in streamFile() to use isset()

More file streaming related profiling

Added some StreamFile profiling

Attempting to address bug 34653; It seems ApiQueryLogEvents has not kept up with changes to the Logging classes. I've changed some aspects of the ApiQueryLogEvents to use a DatabaseLogEntry object when extracting log row information and updated addLogParams to use this new scheme for handling parameters. I have not been able to sufficiently test this and I am still concerned about backwards compatibility with old parameter formats for older log records as well as parameter handling for the 'rights' and 'block' log types.

[mw.loader] Remove 'media' type support in addInlineCSS

* This was never used in practice because the server only gives css text strings for media type "" or "all". Because server side concatenates everything together during pre-processing and e.g. print-stylesheet parts of it are wrapped in @media print {}

* Follows-up r110988

[mw.loader] Refactor addInlineCSS's logic, fixing various bugs

* Using mw.util.addCSS as base to instance of bug 33305 automatically
* Expose as mw.loader.addStyleTag
* Re-use the code in mw.util.addCSS
* Drop the "text > Cdata > element > jQuery object > innerHTML", in favor of setting cssText or appending a text node (html escapement isn't a problem when working with text nodes directly).
* Appending this way also works in IE, no need for the dispose/re-create style-tag logic in the try-catch().

* Follows-up r110988 (fixme)

[JSTesting] actually commit the file (follows-up r112368)

[JSTesting] Add more mw.loader tests
* Follows-up r12279, r112281
* Follows-up r110988: Adds test for @import (currently failing: bug 34669)
* Adding StyleTest.css.php for making this easier and re-usable