Why is tis empty? Bug 34938

Transitional patch for bug 34832: introduce a CI-style option to allow deployment of 1.19 to converter wikis without disruption. The bug fix can be rolled out later by setting $wgBug34832TransitionalRollback = false. This is meant as a temporary measure, while we figure out a way to properly support Chinese wikis for inclusion in the 1.19 tarball.

Introduce a global variable which causes language conversion to not be disabled in interface messages (as before r94279). Use $wgContLang for conversion (as before r97849) since $wgContLang is set to the base language (e.g. zh) on converter wikis, whereas a typical user language (e.g. zh-tw) only has a FakeConverter.

TODO about 'class' throwing a fatal error

amend r113016 , that was for bug 34302

place holder to test bug 34919 when it is fixed

* Fix for r41340, r96405, r97849: introduce a function which gets the current content language, called Parser::getTargetLanguage() following the ParserOptions terminology, and added doc comments, to improve the chances of a human being able to work out what is going on. Just because a function happens to do the thing you want doesn't mean you should call it.
* The language object used for lc() in Parser::braceSubstitution() must match the one used in setFunctionHook() during firstCallInit(). It can't change depending on what message you are parsing. Use $wgContLang like before r97849.

[mediawiki.test] follow-up r112368: Append to fixture instead of body. Otherwise it stays in the dom even after the test is done

(bug 34922) If rev_parent_id is null, then the character delta displayed on Special:Contributions is wrong. In that case instead just display "x bytes". Kind of sticks out, but better than a wrong number (possible alternative is just display no number at all)

No release notes since issue introduced in 1.19 and I'm going to tag this commit for merging.

Fix missing * from start of comment block

Also minor documentation update

make file an argument, --title is script specific

--file did not make that many sense, it is easier to just pass the filename
as an argument

Call to parent constructor was misplaced, that made the script specific
options pretends they were generic maintenance parameters.

[ResourceLoader] Minor clean up
* Re-order special modules and add a few comments
* Single quote in QUnitTestResources.php
* Space in ResourceLoader.php

(Separated from another commit while skimming code)

[JSTesting] message fix

[ObjectCache.php] Add comment describing that CACHE_ANYTHING will use whatever is configured on the wiki for $wgMainCacheType etc., and that if those are set to CACHE_NONE (which it is by default) then CACHE_ANYTHING is CACHE_DB.

Added some super calls to constructors to be safe

Localisation updates for core and extension messages from translatewiki.net

grammar fix

(bug 28019) Make Special:MyPage and Special:MyTalk carry preloadtitle
url parameter to target page. Useful for inputbox extension.

Patch by Bagariavivek.

Perhaps should go through [[Manual:Parameters_to_index.php]] and add anything that's sane to the allowed list, at some point.

Localisation update for core and extension messages from translatewiki.net

DatabaseBase::selectRow() return a simple object, not a ResultWrapper instance

[FileBackend]
* Various documentation improvements.
* Moved a few protected FileBackendStoreShardListIterator functions down. Same with normalizeContainerPath().

r112918: Add new message key to maintenance file

Moved FileBackendStore and helper classes to their own file (no code changes). Updated AutoLoader.

* Removed check for invalid object in formatResult(), Title::makeTitle() always returns an object
* Don't call Title::exists() when result is not cached since the page must exist, and this was issuing a database query per row
* Reorder some variables for better clarity
* Don't execute the LinkBatch if there are now rows
* Call methods directly on ResultWrapper instead of DatabaseBase in preprocessResults()
* Use local context to get message

Change some DatabaseBase::query() calls to use more specific methods

* (bug 34937) Fix for r107636: call WikiPage::onArticleCreate() when moving a page an not overriding the redirect

[Title] In moveToInternal(), just use getParentId() on the null revision object as the base ID rather than $latest. They should conceptually be the same but the former was already fetched with master data.

[DatabaseBase] Made encodeExpiry() recognize 'infinity' and added a decodeExpiry() function

Added FIXME to formatExpiry()

fix for bug34781 - display old title text if the namespace has been changed

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

Use local context to get messages

Per happy-melon, fix for r107653: don't add the error class on success message

release-notes

[Resources.php] Fix missing dependency on jquery.textSelection (follows-up r81648, r81651)

Bug 34896 - Update jQuery.json

Has whitespace changes (bit of a null diff in many cases)

(bug 34832) Parse recentchangestext with the interface option off, since it comes from wfMsgForContent()

Fix for r86304: if MessageCache::parse() is called twice, once with $interface=true and then with $interface=false, actually use $interface=false the second time instead of using the cached value from the previous call. Same for $language. This is the reason why edittools was affected by bug 34832.

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: