* (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

Localisation updates for core and extension messages from translatewiki.net

r112128: Comment out dupe message keys

avoid test suite file leaks

r100315: removed now redundant code

wikistandard.css is not a common skin file, it is only used by the classic/standard skin. moving from skins/common/ to skins/standard

In FSFileBackend:
* Removed some error suppression as display_errors should never be enabled for production sites and the suppression hid useful log information.

follow-up to r111861, fixing classic skin

Test that verifies uniqueness of query module prefixes

More __METHOD__ in our madness

Improve documentation

More __METHOD__ into db related methods

better structure filename error output

When dumping an array, PHPUnit limit the number of characters shown. That would
just hide the relevant part of the path:

1) StructureTest::testUnitTestFileNamesEndWithTest
[...]
 Array (
+    0 => '/some/path/tests/phpunit/incl...ge.php'
+    1 => '/some/path/tests/phpunit/incl...ki.php'
 )

By stripping the common path ( /some/path/tests/phpunit ), we end up with a
more useful output:

 Array (
+    0 => 'includes/RecentChange.php'
+    1 => 'includes/Wiki.php'
 )

avoid exif tests file leakage

Exif test are leaking files heavily on our test server. This quick patch make
it sure we delete temporary files.

Ideally, we should have something like a temporary filesystem backend that
would self destruct :-D

requires r112326: wfRecursiveRemoveDir()

move wfRecursiveRemoveDir to global functions

That useful method was hidden in SeleniumWebSettings. I need it to clean
up test files leakages.

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

Use local context to get message

Fix bug 34684 in my PathRouter code:
- Update the tests to test extra characters and patterns like like \\ and $1
- Also update the tests to make sure that matches that don't have enough data to work fail
- Replace the str_replace and preg_match based code with code based on preg_replace_callback.

ContribsPager can't accept multiple namespaces, passing an array causes a warning in DatabaseMysql::strencode(). Not sure if/how this worked in 1.18 but it's broken in 1.19wmf1. Tested.

Examples can just be a string

So if it's not false, it can be string|array, if it's a string, just put it in an array

[JSTesting] mock/restore mw.messages as well, like mw.config

[JSTesting] Refactor mw.loader test to be more extendable

Improved profiling of a few functions a bit

fix doc, this is not an array

docs++

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

Use local context to get messages

Bug 34604 - [mw.config] wgActionPaths should be an object instead of a numeral array JavaScript

release-notes-1.19
* Update some bug summaries to clearer versions (to lessen ambiguity as to whether it describes the removed behavior or the added one).

* Removed "Some deprecated presentational html attributes will now be automatically
  converted to css." duplicate entry of "Most presentational html attributes like valign are now converted to inline css style rules."

* Removed bug 24037, which was reverted.