w/s

In LocalFile:
* Removed bogus rmdir() call.
* Added lock() calls to upgradeRow()/getSha1().
* Use FileRepo getFileSha1() in getSha1() rather than via FSFile.
* Made purgeThumbList()/migrateThumbFile() use FileRepo::cleanupBatch().
* A few other minor cleanups.
* w/s cleanup in recordUpload2().
In FSRepo:
* Removed deleted zone config code from constructor; useless since r107028.
In FileBackend:
* Make sure 'latest' param gets passed through via $params for some functions.
* Cleaned up doMoveInternal() to use *_Internal() functions.

Localisation updates for core and extension messages from translatewiki.net

Fix fixme on r107328, attempting to use $this in a static method

Made said method non static, and updated caller

Followup r106554

Make init function static

Use WikiPage instead of Article to call updateRevisionOn()

[Core JS] Remove long deprecated globals 'ta' and 'akeytt'
* Not used anywhere in ./phase3/
* Not used anywhere in ./extensions/
* Deprecated in MediaWiki 1.16 according to HISTORY

* Use local context to get messages
* Pass the context to ContribsPager

[Core JS] wikibits.js / IEFixes clean up
* fix on-the-loose fixalpha() call
-- Calls should not be dangling loose like that
-- Removed local calls from skins and installer
-- instead calling from the IEFixes script (which is loaded raw by wikibits which is loaded from the bottom, as are all legacy scripts)
* Removing usage of 'skinpath' and 'skin' globals in wikibits.js, those aren't globals per se since introduction of mw.config and $wgLegacyJavaScriptGlobals
* Wrapping wikibits.js in closure to avoid leakage of "local" variables. This shouldn't break anything as it is loaded through resource loader which, in production mode, wraps it in a closure anyway.
* adding explicit posision=>bottom for wikibits. 'bottom' is default but repeating here since it must not change for legacy reasons.

In EditPage::showDiff():
* Group global declarations
* Only get 'currentrev' and 'yourtext' messages if needed

Updating jquery.qunit from upstream
* Source: https://github.com/jquery/qunit/tree/0aba72f723579082735782231a3503aa249618aa
* 1.2.0pre (master/trunk) -> v1.2.0 (release tag)

Note to self: pressing save before commiting is really a good idea.
Follow-up r107319.

* (bug 30711) WikiPage::replaceSection() now always the current text when adding a new section
* Made WikiPage::replaceSection() use getRawText() to get the current text
* Made EditPage::showDiff() also use the current text when diffing for adding a new section
* Removed the "You are editing an old version" when section=new since the oldid parameter has no effect new due to the above

Follow-up r106567: pass the User object to LogPage::addEntry(); was breaking protect.php maintenance script

Simplified Title::canExist()

Localisation updates for magic word localistion for core from translatewiki.net.

Localisation updates for special page aliases for core from translatewiki.net.

Localisation updates for namespace names for core from translatewiki.net.

Fixed undefined var from r106280

Bye, bye $wgExtensionAliasesFiles (deprecated in 1.16).

typo; follows-up r107289

[Core JS] Clean up and optimization mediawiki.util
* partial rewrite of updateTooltipAccessKeys, much shorter and a bit faster
-- Remove self-calling logic, totally unnecessary here
-- removing the .each() loop, using .attr()'s callback function instead
* directly calling util.toggleToc instead of triggering a click event
* strict comparison
* unquoted keys
* substitute single-use variable

Localisation updates for core and extension messages from translatewiki.net

Use 'newsectionheaderdefaultlevel' message when previewing a new section so that it renders the same as when saving the page

Use local context to get messages

And while I'm a it:
* Use the commented-out code ("MW 1.6" is way old and "Revision.php 1.26" sounds like CVS)
* Don't revert to a revision whose text was deleted

* Use WikiPage instead of Article to call doEdit()
* Pass the revision ID to doEdit() when reverting a page

* (bug 33367) Gendered namespaces for Czech

Whitespace

Fix incorrect encoding in RELEASE-NOTES-1.19 from r107258.

Rename no to nb (step 4/4): Change 'no' to 'nb' in remaining files.

Follow-up r107259: Also update $wgDummyLanguageCodes.

Rename no to nb (step 2/4): Re-add MessagesNo.php. Falls back to 'nb'.

Rename no to nb (step 2/4): Rename MessagesNo.php to MessagesNb.php and update Names.php and RELEASE-NOTES-1.19.

Remove MessagesNb.php pending rename of no to nb.

@since for r99863

Change require to require_once in extensions/README

Improve/add to documentation

Add braces

Explicitly define member variable

Followup r107011 per Krinkle - "mediawiki.api needs to declare the mediawiki.util dependency."

* Use local context to get messages
* Use Language::userTimeAndDate() instead of Language::timeanddate() to use user's timezone preference

Per Nikerabbit, follow-up r102024: removed uneeded variable

Followup r103945 - @since and whitespace

Fix r107197: removed 'lockmanager-fail-acquirelocks' key from messages.inc

* Follow-up r107195: these params are resource paths now, rather than hashes of the resource paths.
* Removed lockmanager-fail-acquirelocks message; a series of lockmanager-fail-acquirelock messages are now used instead.

Use hash key instead of key for lock manager messages. It was confusing translators.

FU r106752: fixed call to bogus function in LocalFile::getSha1()

In LockManager classes:
* Only use hash keys later on in the data flow rather than right when doLock() is called. This allows for error messages in Status objects to use human readable paths rather than ugly hash keys.
* Moved $locksHeld declaration duplication up to the base class.
* Fixed __destruct() in FSLockManager to not use bogus doSingleUnlock() lock type parameter.

Follow-up r107187, r107188: rebuild MessagesXx.php.

follow up to -r107187 - remove unused message key "column" from language file

bug24430 - patch from Chinchin, remove number of column for edit field in preference

Localisation updates for core and extension messages from translatewiki.net

* Use local context to get messages
* Use Language::userTimeAndDate() instead of Language::timeanddate()

Moving the flatlist option to HTMLFormField to prevent code duplication per hashar on r100551

In FileBackendBase/FileBackend:
* Changed concatenate to store to a specified temp FS file rather than a final storage destination. This makes it better fit the use case (chunked upload), so we can avoid extra copying around. Subclasses no longer have to implement this function now as well.
* Added extensionFromPath() helper function.
* Moved createInternal() up a bit and fixed @see comments pointing to the wrong functions.
In FSFileBackend:
* Use parent implementation of doConcatenateInternal().
In FileRepo/File:
* Added FileRepo::ALLOW_STALE and made thumbnail transforms use it.

Bugzilla bug11612 - Make watchlist max days preference setting reflect the max days in Recent Changes

[Core JS] scan all /phase3/, change certain constructors to literals
* new String, new Boolean, new Number
-- should not be used since they don't compare very well:
; 5 !== new Number( 5 ) // true
; !!new Boolean( false ) // true, objects are truethy
-- If type conversion is intended, they should be used as
   functions instead of constructors.
* new Array, new Object
-- literals are shorter, no point in not using them.

Localization update for magic words in he.

* Added 'latest' parameter to functions that get information about a file. This will get the latest version among all storage nodes. Non-distributed backends can ignore this.
* In doOperations(), ignore 'nonLocking' unless 'ignoreErrors' is set. Split out a doOperationsInternal() function so the sanity check could go in doOperations().
* Moved getFileList() down a bit.
* Documentation tweaks.

Use getPath, all operations are on filerepo level no need to get a local path. Follow up r107064

Follow-up r98698: Reduce amount of ipv6 spam in FileCacheBase::incrMissesRecent() per CR

Fix a session invalidation error when using LdapAuthentication.

Applying patch from bug 33335 (from Erwin Dokter).  Followup to r105280
and r106884.
* td.diff-deletedline gets a softer yellow background
* td.diff-addedline gets a lighter blue background,
* td.diff-context gets slightly lighter grey background.
* Instead of only the changed text being displayed with white-space:pre-wrap,
the entire line in a cell is.
* The font-size is changed from 'smaller' to 88%

Documentation fixes

Add comments explaining that the funky multiple Location headers stuff is a CURL artifact, and that the code will need updating if bug 29232 is done (high-level handling of http redirects)

Followup r94881, r106948.

rephrase r106992 release notes

Use local context to get messages

Localisation updates for core and extension messages from translatewiki.net

I hate eol w/s

(bug 12262) Follow-up r102026: update ul margin in shared.css

Use ⌘ in for Mac keyboard shortcuts

Looks better to the eye when playing with your User:foo/style.css

http://en.wikipedia.org/wiki/Command_key

apply CSS::Janus when user preview its CSS style

follow up r94421

- add comment to document us_chunk_inx in sql file
- fix chunk upload to work with recent filerepo changes
Follow up r104687

Fixed a typo in r107054

do not evaluate '<?php' line

This happen when you are sending to eval.php a php file. The first line
is the string '<?php' which is passed to eval(). Boom!
This patch skip that string.

Example usage:
 $ cat somefile.php
 <?php
 echo "Working!\n";
 $

 $ cat somefile.php | php maintenance/eval.php
 Working!
 $

remove bug 31921 from 1.19 RN

Fix was merged in REL1_18 by r107046

Moved SQLite test .sql files to the common test data directory

Make apibase of $wgForeignFileRepos (InstantCommons) protocol relative too

Add some letters per https://incubator.wikimedia.org/w/index.php?diff=666382&oldid=664736

* FU r106752: de-uglified Setup.php by moving most of the b/c code into FileBackendGroup. Deferred registration of file backends and lock managers to the respective singleton() functions for the group objects.
* Refactored FileRepo::initZones() to require the specific zones; nothing was using it differently.
* Removed deleted zone check in deleteBatch(), a similar error will instead trigger with the initZones() call as needed.
* Added $wgLocalFileRepo comment.
* Updated tests.

Update documentation on params method of Message class to indicate it can either be varags, or it can also
take an array of all the parameters as the first argument.

FU r105512: urldecode() the file and thumb name in wfExtractThumbParams() for 404 handling as needed.

added missing UploadStash exceptions to AutoLoader

[mediawiki.api] bug fixes

* error callback of some convenience functions was broken. Second argument to get() is ajaxOptions, there is no third argument.

[mediawiki.api] clean up

* Adding return values to most of the ajax functions so that the jqXHR object (originally returned by jQuery.ajax) is available. Some functions documented they already did this, some don't. Now they all do and are also documented as such.

* Renaming a few 'callback' arguments to 'success' to avoid confusion with 'error'.

* Removed unused variables
** mw.Api's instance this.url was unused
** var cachedToken was unused in mediawiki.api.titleblacklist.js

* Making closure argument order the same and referencing mediaWiki as a global (mw is something thought not to be a global but mediaWiki certainly is, no need to access window from the global scope and then the mediaWiki property of it). Also renamed any used of $j to $. Adding 'undefined' to the closure arguments where missing.

* Re-written the mw.Api constructor's logic for options. Now keeping a defaultOptions privately "statically" cached outside the constructor and using that to build the options object.

* Made all non-block comments the same comment style (some were /* */ or /* \n */)

* Completed parameter documentation and wrote TODO as @todo and added an example use for mw.Api in the documentation.

* Some other random whitespacing, line breaking, merged var statement, and moved them out of blocks (since JS doesn't have block scope) into the main function body for clarity. And some other random JS Lint/JS Hint stuff.

* Added comment to api.titleblacklist module about the module not being in core but in the TitleBlacklist extension.

* (bug 23427) Introduced {{PAGEID}} variable to expose page.page_id [patch by Waldir]

follow-up to r106912, making change to correct branch this time

This isn't a doc comment, so shouldn't start with /** (and it messes up my syntax highlighting)

changed instances of sample.com to example.com per RFC 2606

[Core JS] mw.util.addCSS: Insert style tag into dom before setting cssText
* Fixes bug 33305

[Core JS] Applying conventions to mw.util
- new recommended closure format
- fixing repeated var statements, or var statement inside blocks (there is no block scope in JavaScript) and moving them to the top of the function
- line breaking, indention, white space
- and more.. see also https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript

* FU r106752: unbreak urls to ForeignAPIRepo file thumbnails. FileRepo no longer uses bogus <public root URL>/thumb default for the thumnail URL when the public root URL wasn't even set. This was making ForeignAPIRepo not set it since it saw that it was already set.
* Cleaned up and added some missing sanity checks for scriptDirUrl member in FileRepo. Made some related documentation tweaks.
* Removed pointless getRepo() call in File.

Localisation updates for core and extension messages from translatewiki.net

* Added FileRepo::SKIP_LOCKING constant and made storeBatch() check it.
* Made File::maybeDoTransform() use the FileRepo::store() and a new File::getThumbRel() function. Looks cleaner and loosens FileBackend coupling.
* Also made storeTemp() use FileRepo::SKIP_LOCKING for performance.
* Killed some useless initZones() calls in FileRepo. Extensions may not even use these zones. Likewise, it could make tests fail even though they don't those zones. We already do the sanity with some prepare() calls in storeBatch().
* Removed FileRepo::SKIP_VALIDATION, not used by anything now.
* Moved getUrlRel() down a bit.

Code style: move a couple htmlspecialchars() down to usage so it's clear where escaping happens

(bug 33246) News icon shown for news:// URLs but not for news: URLs. Patch by Erwin Dokter

Declare constant default values directly in the class definition and removed the constructor since it's useless now

Use LinkCache::singleton()->clear() instead of using a local variable so that I can see what LinkCache::singleton() will be used for directly in grep

* Use local context to get messages
* Use Language::userTimeAndDate() and related instead of Language::timeanddate() where needed
* Factorised some calls to member functions

Use accessor instead of the member variable directly and factorise the calls to it

tests for r94881 which interprets relative Location: headers