Updated release notes and version number to 1.23.1

This is MediaWiki 1.23.1 security and maintenance release.

Change-Id: I66d92d53262f54699c8d4c43434481eb328d8e18

SECURITY: Prevent external resources in SVG files

On bug 65724, it was discovered that a user could upload SVG images
with embedded <image> elements that pulled in the resource via http.
This could allow an attacker to track all viewers of an SVG by having
the image embed another image hosted on their own server.

While testing the patch, I also identified 3 more element namespaces
that have been used on commons and seem harmless, so I added those to
the whitelist.

Change-Id: Iacc5d3d6248b73740510ba579be9484f4964ca13

Merge "MimeMagic: Don't seek before BOF" into REL1_23

Special:Watchlist: Don't try to render empty row

If no rows to render in table, show message and leave function before
try to render.

Bug: 67025
Change-Id: Ica4ad670a130f31124ff644a2e1e50beea481c9b

Don't allow some E_NOTICE messages to end up in the LocalSettings.php

There is probably a better, more comprehensive way to produce a
LocalSettings.php file so that E_STRICT and E_NOTICE errors don't end
up in the file, or to notify users that there is a problem with the
generated file, at least.   This only attempts to address those issues
reported as a result of c978cee8562f018dab158b41e15266fcf873bf1b.

Bug: 66922
Change-Id: I0fe70446d9fe23aed2a147e20e5ff3851371726d

filebackend: Avoid using popen() when "parallelize" is disabled

Bug: 66467
Change-Id: Iaf8eb2ecfad166e26e265a98933399dc4bc255a3

Some corrections to RELEASE-NOTES-1.23

* Fixed a typo: "When $wgJobRunRate is higher that zero [...]".
* Removed a reference to 1.22: "[...] not yet queried but will be
  in a future point release of 1.22."
* Fixed misspellings "wiki's" and "sql" in the same note.

Change-Id: I5fee88c459345ecd8b71f9e5b86c65afae183ba1

MimeMagic: Don't seek before BOF

This has weird side effects like only extracting the tail of the
file partially or not at all.

Cherry-picked from d96c6f01479ef

Bug: 66428
Change-Id: I182128c6958244f1515227ee742c3206a7484aee

chmod -x languages/data/plurals.xml

Bug: 66182
Change-Id: Ic8e15d94fb54b410d21b969b029fac685b2711d6

Updated release notes and version number for MediaWiki 1.23.0

This is MediaWiki 1.23.0 stable release.

Change-Id: Ibbfb27190d2d98c850849427c18526b898372ac8

Make convertUserOptions.php more self-contained

* Moved the code from User::decodeOptions() to the maintenance script,
  the only place it is (indirectly) used.
* Changed the script to insert new rows itself rather than calling
  User::saveSettings(), avoiding bug 63677.
* Removed FOR UPDATE and COMMIT lacking a matching BEGIN.
* Made ORDER BY explicit.
* Removed pointless "Do each user sequentially [...]" comment dating
  back to r48732, in which the script would increment a user's ID
  to get the next one.

Bug: 63677
Change-Id: I86365a7363af3376cc4f55fa528b050b44378656

Updated release notes and version number for MediaWiki 1.23.0-rc.3

This is MediaWiki 1.23.0-rc.3 release candidate.

Change-Id: Iad21d8fbd0bfc3f84e000eda649ca12bcd3b0568

jquery.suggestions: Handle CSS ellipsis better for IE

IE is not impressed by our puny hacks and still reports the width
"in context of" the position in the document, limited by the width
of ancestor elements.

Let's temporarily apply position: absolute; to the involved elements.
This pulls them out of normal document flow and lets us figure out the
real width at last.

Also wrote a proper comment on why we need this stuff.

Verified that this fixes:
* IE 8
* IE 11

Verified that it doesn't break:
* Firefox 3.6
* Firefox 29
* Opera 12
* Opera 22

It *does not* fix IE 6. I don't think that investigating why is a good
use of my time, so I didn't. I84fbae5a made the functionality usable
on IE 6, which feels good enough for me.

Bug: 65224
Change-Id: I4a7357543ca244585ade2061b92f5a6d1e439278
(cherry picked from commit 1a04fb81cfca8bde9b05ceb17aa7775d94fcf24c)

Merge "SECURITY: Don't parse usernames as wikitext" into REL1_23

Merge "Officially deprecate skin autodiscovery" into REL1_23

SECURITY: Don't parse usernames as wikitext

On Special:PasswordReset, don't parse the username as wikitext since
the wikitext is parsed according to the wiki's configuration (might
include wgRawHtml), and the wiki may be private.

Bug: 65501
Change-Id: Ic3e5d42e1be5acc42ba89ae853c5ecbfec04fa91

Add ar_text to the list from Revision::selectArchiveFields(). It is checked later.

See Revision::newFromArchiveRow().  This is called from
populateRevisionLength.php's upgradeRow() which ultimately populates
$row from the fields returned by Revision::selectArchiveFields().

Revision::newFromArchiveRow() has the following condition:

if ( isset( $row->ar_text ) && !$row->ar_text_id ) {

And from the bug report:

> for old archived revisions, the rows in the archive table do _not_
> contain a pointer to the text in the text table, but that the text
> is stored in the archive table directly. Obviously the above code
> cannot handle this situation properly.

Credit goes to Joerg who found this and figured out the problem.

Bug: 65765
Change-Id: Iadcaae5e531d439780f5b6095286df78926ccaa7

DerivativeContext::setConfig should take a Config object

Change-Id: Icc16a9c206c57fac2d26547cb874e02ef09c0485
Follows-Up: I13baec0b6d4ea7badf20b9c5f9b40846348838e4
(cherry picked from commit 07c748343cfe389b3c1a74b7aaa9bc222d588c94)

Make abstract Config class truly implementation-agnostic

Follow up to I13baec0b6 ("Config: Add Config and GlobalConfig classes"):

Config:
* Rather than returning Status objects, Config::set will now throw an exception
  if an error is encountered
* Config::factory was moved into it's own ConfigFactory class.
* Since there are no more functions in it, Config was turned into an interface.

GlobalConfig:
* Remove $prefix args from Config::set and ::get. The idea of having an
  abstract Config class is to abstract some notion of configuration data from
  the particular way in which it is currently implemented (global variables).
  So the abstract base class has no business dealing with variable name
  prefixes.
** Instead GlobalVarConfig's implementations of get and set call getWithPrefix
   and setWithPrefix internally, which are now protected
* Rename GlobalConfig to GlobalVarConfig, which makes it clearer that it isn't
  referring to the scope of the configuration value, but to the scope of the
  variable name which provides it.

ConfigFactory:
* ConfigFactory is where Config objects are registered, and later constructed.
* Config objects are registered with a given name, and a callback factory function.
  This allows for implementations to construct the object with the parameters they want,
  and avoids the overhead of needing an entire class.
** The name 'main' is the default object returned by RequestContext::getConfig(),
   and is intended to be used by core.
* This is a singleton class, the main instance can be obtained with:
  ConfigFactory::getDefaultInstance()

In addition to the above:
* $wgConfigClass was removed, and $wgConfigRegistry was introduced, which
  stores a name => callback. The name is to be what the Config instance is
  registered with, and the callback should return an implementation of Config.
* Tests were written for the new ConfigFactory, and GlobalVarConfig's tests
  were improved.

Co-Authored-By: Ori Livneh <ori@wikimedia.org>
Co-Authored-By: Chad Horohoe <chadh@wikimedia.org>
Co-Authored-By: Mattflaschen <mflaschen@wikimedia.org>
Co-Authored-By: Parent5446 <tylerromeo@gmail.com>
Co-Authored-By: Reedy <reedy@wikimedia.org>
Co-Authored-By: Daniel Kinzler <daniel.kinzler@wikimedia.de>
Change-Id: I5a5857fcfa07598ba4ce9ae5bbb4ce54a567d31e
(cherry picked from commit 8977e56a6b71e10e6046537cd82a3e32aea12f86)

Officially deprecate skin autodiscovery

Corresponding change in master (core skins no longer use autodiscovery
there, but it's too scary of a change to backport here at last minute):
I6fc30f97d4b35820c132c4892abf6d2c86f9482e.

Bug: 65748
Change-Id: If9125ef587ce11f0d69f12eacd00ae7236139798

Raised version number to MediaWiki 1.23.0-rc.2

This is MediaWiki 1.23.0-rc.2 release candidate

Change-Id: I362ebf7fe866beacd6af37d0239c0dc293d439cb

Avoid uncommitted transaction notices in thumb.php and img_auth.php

bug: 56269
Change-Id: I8cf5c070899d281c7efbac853f04c2fea9374e4d
(cherry picked from commit c993ec0095e80c939cdacea690d8677814cdbeb6)

Sync RELEASE-NOTES-1.23 between REL1_23 and master

* Change Ic915c50b0acd2e2 and I0ee55deb9b44b7 put the entry
  for bug 63444 in a different place in master.

* Commit d8b1b79ea423e was merged before branch cut, and then
  reverted in I7431eb83a09, but that left behind one of the two
  release notes. Release notes were fixed in master in I9ef5263ac8f7.

Change-Id: I3c9982a68d1759bd1f44d840a9414fea5d174be9

Removed 'Remember my login' preference

Removed 'Remember my login' from Preferences, as it was unwanted. It adds
to the complexity of the user preferences

Bug: 52342
Co-Author: Tyler Romeo <tylerromeo@gmail.com>
Change-Id: I7c957e1e1aaecf47f7c47bc063b5d3b364644afc
(cherry picked from commit 74756a24091d3b875a2fbf8759d8688609727586)

vector: Suppress watch star focus outline when animating it

It looks really funny, but that's not the effect we're aiming for.
The star is not supposed to look clickable during the rotation
animation anyway.

Bug: 65434
Change-Id: I3a7c91f5b7df0d70abd532cc234e8aa1bac9e129
(cherry picked from commit b3456db703ef66f3df5608fc8f2301b9273d2a9e)

jquery.textSelection: Don't throw errors on empty collections

All jQuery functions just do nothing or return "empty" values when
called on an empty collection (e.g. `$()`), the ones defined in this
module should behave in the same way.

This came to light when a change in WikiEditor combined with lousy
coding caused this code path to be called, breaking various gadgets
and extensions like SemanticForms.

Bug: 64289
Change-Id: Ib97f47ef1d66420682bd429c9c12e66c3392e77d
(cherry picked from commit 2828669dd40f63f60c2391b6d5ee30a3e255f3db)

Add space between two feed links

When more than one feed link was given, the links was concat without a
space, which looks odd. Adding now a space, between each link in the
toolbox.

Bug: 36356
Change-Id: Iaf61c88dde0a92c475a60c005e13222eb7618eef
(cherry picked from commit 853877f93af5056dc3162a7fc83957681674ca0c)

Merge "Revert "Introducing pp_sortkey."" into REL1_23

Revert "Introducing pp_sortkey."

Yep, I screwed this up.

This reverts commit f9e659cf0c03d2cb99527def8ca28585791d3e93.

Change-Id: I51095824730f90e7cc52ac47a9aebcbb01312419

Installer: properly override default $wgLogo value

Setup.php uses the value of $wgScriptPath in $wgLogo before the
installer determines the former, so it is necessary to override
the latter even before the WebInstaller_Options page is shown
(because it might never be shown).

Bug: 48084
Change-Id: Icc577c1bb1f78adc7a7cfb0581ae8ea2facba4cf

mediawiki.api: Fix API postWithToken method

Referred to "this" instead of "api" ("this" here refers to the Deferred object).
Fixes "Uncaught TypeError: Cannot read property 'ajax' of undefined".

Change-Id: I78ba47a614512f6218e23d03e7c688e2c9efbe45
(cherry picked from commit 93fc25c4a84916316c75c2b9b4065ee236a501df)

RELEASE 1.23.0-rc.1

Change-Id: I545756411db2db58056c611b76d270e9257d7af1

Profiler class refactoring

* Merged the "collate at end" and "running collate" logic
  into the base class to make it more unified and consistent.
* Pulled out DB code into ProfilerSimpleDB class.
* Removed the ProfilerSimple class and updated child classes.
* Made ProfilerSimpleTrace set the debug information too.
* Work around "ended by close()" bug from 99aef03f. Since the
  getRawData() method gets called while some wfProfileOut() calls
  have not yet happened, make it use the matched call data and let
  the final logData() method work on the full data as normal.
* Let ProfilerSimple classes use getFunctionReport() for the profiler
  debug log if it is setup instead of making it just return "".
* Made getRawData() work as best as possible with ProfilerMwprof.
* Removed $wgProfileToDatabase since it is now useless.
* Improved DB profile performance with sqlite.
* Updated visibility of various methods.

Hand picked from Change-Id: I1260bab2b5ba12dccbba701bcae4a637cb85c6e8

Change-Id: I832997187c20c95df406e6bfbe08d34b54531ef2

Introducing pp_sortkey.

This adds the pp_sortkey column to the page_props table.

pp_sortkeys allows for top-k queries for pages, e.g.
the 100 pages with the most language links, etc. It is also
possible to query for exact values.

For now, pp_sortkey will contain pp_value's numeric value if
the value was set to a float, int or boolean.

Associated tasks:
* create a maintenance script for populating pp_sortkey. Tricky,
because when reading from the database, all values are strings.
* create an API module for querying pages by property value.

bug: 58032
hand picked from Change-Id: I217c42656fb877ff35a36eb446a22bdaf119faac

Change-Id: I89b6fc719f70a225a0a71290cc6e051f464114b7

Parse versions in semver format

Change-Id: Icb3981736918154898cff0c98c472bc8bb36f1c6

Remove Vector from the default LESS import path

When Ia193571a added vector to core for VectorBeta's benefit it introduced a
regression that broke all skins that used imports like 'variables.less' which
happen to match a filename used by Vector.

This is a short-term fix that allows us to release 1.23 without this regression
by moving the bug from core to the VectorBeta extension.
It needs to be be backported to 1.23.
And we will still need one or two long-term fixes for the bug.

Bug: 60368
Bug: 64595
Dependency: I8ee928a8f60751f72f2a37250d274cf55bce6ee7
Change-Id: Iee47bdc23630e02ccfcbd28496ec5268892eb629

Merge "PostgreSQL: decode image.img_metadata" into REL1_23

Merge "Fix HISTORY hint in RELEASE-NOTES-1.23" into REL1_23

Restored method for clearing a watchlist in web UI, and rebuilt code.

People have been asking about this, as raw can cause problems for those with
massive lists (upwards of 1 million).

Manually cherry picked from I17dde71858fa65532fbe5d9c2ee2a65a39b2aa87
after short discussion with Brion.

Bug: 13250
Change-Id: I17dde71858fa65532fbe5d9c2ee2a65a39b2aa88

Fix HISTORY hint in RELEASE-NOTES-1.23

Was not updated the last time when RELEASE-NOTES-1.22 was deleted and
moved to HISTORY.

Change-Id: Iaec2b60091cd813c28bf9d0f35c444446023df1b

Make gallery properties publically accessible.

They were public in the previous release of MediaWiki (1.22), although
the visibility was not set explicitly.

See I779d5a683982dc25bc7cdd30addd5870f2a7efd0 and
https://github.com/SemanticMediaWiki/SemanticResultFormats/issues/12

Change-Id: I309cf46891223c5f1520e97b04d4ed315a300e7a
(cherry picked from commit d7f494c7be0718fef0f763e7a4286fcaf6a476c2)

PostgreSQL: decode image.img_metadata

The img_metadata field was not being decoded from bytea into text.

This change to core fixes the reported problem with PdfHandler against
1.22.6, origin/REL1_23 and git head.

Bug: 59147
Change-Id: I285a317a57a29713af1f7f630b340278bd5400da
(cherry picked from commit 8c62609dc877b6ed687e9a3835cfc24fb87e5be2)

Expand release notes for I82d26b64

Change-Id: Idd8f5d260e649338cf7903c7a1c157abd23cebaa

Fix Oracle 1.23 missing patches
(rebase)

also fixed wrong definition of password expire filed
DEFAULT was unnecessary and faulty (has to be before
NULLABLE clause)
and changed the null-user insert

Change-Id: Ia91ed19bd4e3380f396ef895745dbd2014e1eb8b
(cherry picked from commit 78922b9877e72c6962ac67f5afe28476914139ad)

Merge "PostgreSQL: Make l10n_cache.lc_value binary" into REL1_23

Merge "This reverts commit d8b1b79ea423ef3391c34f63aa382fd6bad6597e." into REL1_23

PostgreSQL: Make l10n_cache.lc_value binary

Change-Id I427c6de5a0a29b43cff755db0eb8a750db620173 increases the
probability that a null byte will attempt to be stored in the
lc_value column.  PostgreSQL does not allow that byte in a text
column, so convert the column to bytea.

If the column already contains corrupted data, the upgrade routine
might fail.  To prevent this, delete the contents of the table before
changing the type.

Manually cherry picked from Ie8368bde398b2ae4d3cfc9ee7bf35874bd2ded68
due to conflicts

Bug: 62098
Change-Id: Iaccfe71d38316f2445b786656d5c603be7bfb334

Fix missing import in mediawiki.ui

The .box-sizing() LESS function was moved in:

871875bed4b21e3b17d22046e121bf4e434b804b

And the usage in forms.less wasn't finding it anymore.

Change-Id: I7ae97bd5d4aa113af039182ff39817926bbbf3ed
(cherry picked from commit b38c1a2e374387330a8629128c88597ea35e77e3)

This reverts commit d8b1b79ea423ef3391c34f63aa382fd6bad6597e.

Keeping this "fix" in the release will hurt around 8.5% of MediaWiki
installations[1] while only about 30 or 40 people have complained about
the old behavior in the past.  In the meantime, they've been able to cope
with that behavior.

[1] https://wikiapiary.com/wiki/Extension:Semantic_MediaWiki

Bug: 62856
Change-Id: I7431eb83a0924cb571be5767c5b6309c848055de

Vector: Don't supress focus outline on watch star

This is an old remnant that was intended to remove outline from
images inside anchor links. However the watch star is no longer
implemented this way and this is doing nothing. Except when
the element is focussed (using keyboard tab index), then this
rule is overriding the default :focus and causing the user to
potentially get confused since it is no longer clear where the
focus has gone to.

Bug: 64934
Change-Id: I57ee089c1ae18bb7a6c55ae17eab3a06f9e1f7d2
(cherry picked from commit 678b4d55472bf7e6ffb3dd701c2c3daa3ca784ee)

mediawiki.mixins: Move .box-sizing mixin from mediawiki.ui utilities

Very simple, very useful.

Change-Id: Icd3d992aaad3116fd09164e2e003619677d051bb
(cherry picked from commit 871875bed4b21e3b17d22046e121bf4e434b804b)

API: Check return value from $gitInfo->getHeadCommitDate()

action=query&meta=siteinfo&siprop=extensions calls
$gitInfo->getHeadCommitDate() to attempt to find the commit date, but
does not check whether that function returned false before using the
value. This resulted in displaying the current date for "vcs-date" when
the function failed.

Bug: 64821
Change-Id: Ic39a74abe5160b3b7fbfb7c15323328d6b317560
(cherry picked from commit 986d3ef2c13aeaf8d172e89c30c2c2f93d9f1b55)

Use an actual URL for helppage in enotif_body

The helppage can be a full URL, which canonicalurl doesn't like.

This means that if a wiki
* has a custom enotif_body which uses canonicalurl + helppage, but
* doesn't have helppage defined locally (as local/interwiki title),
then it will still have to fix either of the two locally.

Bug: 63269
Change-Id: I82d26b648ef23b69ae4fbd0a74ffa23eb7a93dae

Merge "Update default contents of interwiki table" into REL1_23

Remove backwards-compatibility modules from I3947eaac

These were not included in any release.

Change-Id: Ieb927fa9ec1dd19e93ccdbf38595ac11b657d3d3

Rename basic skinning modules: skins.common.* → mediawiki.skinning.*

For consistency in module naming: skins.* contains the modules for
particular skins, mediawiki.* contains the modules for core MediaWiki
itself. Clearly this belongs more to the latter than the former.
'skinning' is probably the best thing to name these to not sound silly
and avoid confusion with actual skins.

(The fact that there are files in skins/ that are not skin-related is
a remnant of bad decisions in the past. I want to clean that up as
well this year.)

Updated release notes and usages, kept old definitions for HTML
backwards-compatibility (for 30 days on WMF deployment).

I intend to backport this to 1.23 (sans the compatibility modules).
Otherwise the current module names become "public API" and we either
break compatibility later or support them forever.

Change-Id: I3947eaacd389eaba32f6fc8370395caec4aa1edb

Update default contents of interwiki table

This list has obviously not been properly updated for many years. There
are many long-dead sites that are removed in this patch.

Removals will not affect existing wikis, only new wikis.

Also removing local designation for WMF wikis since this list is
intended for 3rd party users (WMF wikis have their own interwiki list).

Notes in square brackets should be checked again in the future.

acronym: update URL
annnotationwiki: remove, seedwiki has closed down
corpknowpedia: remove, spammy parked domain
disinfopedia: now called sourcewatch
docbook: update URL
drumcorpswiki: update URL
gej: update URL
gentoowiki: remove, site gone
hewikisource: remove, redundant to wikisource:he:$1
hrwiki: update URL
imdb: update URL
  [jargonfile: the given URL here is broken, but the site still exists: unsure how to proceed with this one]
jspwiki: remove, not useful as an interwiki (NB: now at jspwiki.apache.org)
keiki: remove, site gone
lugkr: update URL
mathsongswiki: remove, seedwiki has closed down
memoryalpha: update URL
moinmoin: remove, not useful as an interwiki (NB: now seemingly at moinmo.in)
mozillawiki: update URL
oeis: update URL
openfacts: remove, site gone
openwiki: update URL
patwiki: remove, wiki seems to have disappeared
pmeg: remove, site has been re-structured so it is now not conducive to interwiki linking (i.e. you may as well just use external links)
s23wiki: update URL
seattlewiki: remove, (now) redundant to wikia:seattle:$1
seattlewireless: update URL
susning: remove, site no longer contains a wiki
svgwiki: remove, seems to have disappeared
tavi: remove, broken site
tmwiki: remove, "This site has been overrun by spam" as of March 2006 (NB: now at www.easytopicmaps.com/wiki)
uea: update URL
unreal: update URL
vinismo: remove, broken site
why: remove, parked domain
wikicities: remove, legacy name for Wikia
wikinfo: update URL
wlug: remove, not useful as an interwiki (just a Linux user group from New Zealand!)
zwiki: remove, not useful as an interwiki
zzz wiki: remove, site overrun by spambots

Change-Id: I6c1eb3aba6cc55d1313f6d249510a03e4b8e6f13

Fix IE9's lack of support for console.warn.apply

Bug: 63770
Change-Id: I8c286af374df5b5c1e7fa6b9ea1bef2ddaf5d796
(cherry picked from commit 4c146968c843f74bf1b578ba78b614b5bd7f5457)

Fix some issues with Microsoft SQL Server support

- Add in missing schema update for 1.23
- Fix detection of table/field existence
- Fix INSERT IGNORE support for cases where the primary key was being duplicated

Change-Id: Ia15673c869c2cf732ffe96f1608cee884d106ccd
(cherry picked from commit 1781729aa2db38681b4dd4cfbcfde48558c60fc1)

wikibits.js: Remove false deprecation notices

We are clearly not removing these properties/functions in 1.23.

Bug: 63964
Change-Id: I5737c053d7e9eb5f6a3c6770475b6c2c8f48adab
(cherry picked from commit f73a49ae9aec7c0cd3938615efc4378673b62216)

mediawiki.Title: Correct documentation structure for Title.exist

Apparently the way we were using the documentation, with sub-properties
of properties, only worked due to the way that JSDuck v4.x happened to
be implemented, and was unsupported; as of JSDuck v5.x this no longer
works, so instead, change the form used so as to document the property
as a single item, as suggested in response to our complaint:

  https://github.com/senchalabs/jsduck/issues/551

Change-Id: If0c66e6e8a904769ee857c1ee94377dc13000d41
(cherry picked from commit bbf40d791ec6745a3a72e44dada5ace85bb59818)

The $CHANGEDORCREATED parameter to the 'enotif_body' message will not be removed in 1.23

It's used in 67 of our own translations and likely by end-users in
customized messages, and keeping it doesn't really hurt anything.

Per discussion on I885dc670.

Change-Id: Ia95ff4185d8f1a1e335b8c3f3d855289a66b8d3e
(cherry picked from commit b3352d7a9df12b2cd5400649d96aea705160f865)

Updated version number to MediaWiki 1.23.0rc0

This is the first release candidate of MediaWiki 1.23 branch

Change-Id: I4a723d3e454eda36e380da3c7aa9743a4ef88f21

SECURITY: escape sortKey in pageInfo

DEFAULTSORT isn't escaped before being added to the action=info table.

Bug: 63251
Change-Id: I087bfde8cbc69c3507f68ee3cb6e22aba0ffa7db

Resolve complex arguments to LESS helper functions

The icons in Flow and the Compact Personal Bar beta feature are missing,
their generated background-image 7 is empty.

The lessphp implementation passes full data frames into the helper
functions.  Utilize the now-public compileValue method of less
to evaluate the data frame into a string.

Additionally adds a small abstraction to run pairs of less/css files
as unit tests by executing the less file and comparing it to the
equivalent css file.

Change-Id: I1704f84638d86a0e6e6b9c190972ab19180bd484
(cherry picked from commit efabc31da242c7596116e5851a5384737ce3113e)

Update lessphp to 2cc77e3c7b

Source:
* https://github.com/leafo/lessphp/compare/b7cd5c79e8...2cc77e3c7b
* https://github.com/leafo/lessphp/blob/2cc77e3c7b9e84cf30ca64fccf2a45a84fbcf0d3/lessc.inc.php

Updates:
* 84b1fda0d4 Set visibility of lessc::compileValue to 'public'

Change-Id: I499ddb01a662896fdcfe56d18021de5314e4d5d2
(cherry picked from commit 5ad0cd9b89431c43ba5133602b31ef62b726d012)

Merge "Skip loading PHP shims for JSON i18n files" into REL1_23

Merge "Update documentation for $wgExtensionMessagesFiles" into REL1_23

Merge "JSON i18n shim: Only register LocalisationCacheRecache handler once" into REL1_23

Update documentation for $wgExtensionMessagesFiles

Follows-up e0043257c02e.

Change-Id: Id8212fca83fde09a89891ead1e0a107ff0d66398
(cherry picked from commit b67db3be8ef5cc9191c2dbdec8809af38596d06d)

PostgreSQL: Allow rc_cur_time field to be null.

The column recentchanges.rc_cur_time is no longer populated by SQL,
so PostgreSQL must be changed to allow the NULL value for it.

In MySQL, the empty string is used as a surrogate for NULL, but that
is not allowed in PostgreSQL.

Bug: 61318
Change-Id: I6733ef11152d545382bec087d992515dcf9aec0b
(cherry picked from commit 8e4b41a50866e9503ed1498e6ac70b31c88d66ca)

API: Don't rollback changes if the exception was a UsageException

In I1b7396ceb, we were trying to log failures to Special:Log, but
since $this->dieUsage throws an exception, the log entry insertion
was being rolled back.

This is similar to I36cd645d which was for index.php calls.

Change-Id: I136ba66d2f939fcdd0c9326e04d686a30449bf4c
Follows-Up: I8f1da51187b281fe4afc0d5a0c49f5caf3612e92
(cherry picked from commit 487e9dc599298b57e5583f19945987ce0b0f287b)

JSON i18n shim: Only register LocalisationCacheRecache handler once

Because LocalisationCache::recache() includes PHP shims each time it is
called (e.g. for a different language), in such cases as automated testing,
the shims could end up registering handlers many times instead of only
once, leading to a rather significant, progressively worse slowdown.

Checking whether the shim has already been loaded, and if so, not
registering the handler again, avoids this performance problem.

I also made the shim compatible with PHP 5.2 (minimum PHP version for
MW 1.17 - 1.19) by avoiding closures and __DIR__, and I added a
--shim-only option to generateJsonI18n.php to allow regenerating
existing shims.

Bug: 63928
Change-Id: I3bb39e7d2bb094873061b3b2adf7066bf26c1b71
(cherry picked from commit e0736bd1cdb3c2971baf344006f474580b8e326c)

generateJsonI18n.php: Use tabs for indentation

Bug: 63444
Change-Id: Iaaf6b1a08c316f29c18fd5c9494decee401a43f1
(cherry picked from commit cdf358807c8370648233c28d3e740a8c9aedb078)

Revert "Removed useless JobQueue return values"

This reverts commit bc8c89d2df39d64f15770d29874904ebcff2f131.

Bug: 64007

Change-Id: I4b4dbe4637dc50cd4630ef19d54f01efba10e138
(cherry picked from commit c76d5a95c1dd1c3086461feae8bc1c0b688bde1e)

Skip loading PHP shims for JSON i18n files

For now, non-message data (e.g. special page aliases, magic words) belong
in separate files having their own keys in $wgExtensionMessagesFiles. It
is unnecessary to read any PHP files listed under keys that exist in
$wgMessagesDirs; they are merely compatibility shims.

Bug: 63926
Change-Id: I92013d0a45e83ad0f5ac483a3db867806eb71f8a
(cherry picked from commit e0043257c02e866d176fc9cbb7721fa2defcf091)

Deprecate ResourceLoaderGetStartupModules

In bug 63240, Roan calls it a 'misfeature', which I think is exactly right. It
has been used as a lazy workaround for having to think about load-order,
dependency management, and race conditions.

Bug: 63240
Change-Id: Ic48ad39c6d3d166d6bb8f60dfdd5f95aa7ed16a6
(cherry picked from commit e8b46e5e6660c9fbce01c982e98c7662b135a130)

mediawiki.mixins.less: Correctly embed images in .background-image-svg

Using the /* @embed */ annotation twice for the same property
(background-image) confuses the LESS compiler, resulting in one of the
images not being embedded.

Let's use the embeddable() and embed() MediaWiki LESS functions,
like in the .background-image mixin.

And let's extend and clean up the documentation while we're here.

Bug: 61941
Change-Id: I39ea2004b9db34d096eed08b3ffdfcaf8d5fcd7d
(cherry picked from commit 127222af0b77650d752f5ba05165701fe7fd7895)

vector: Do styles the ResourceLoader way

It used to be done like this a few releases ago, but went
through some perturbation when the LESS rewrite happened
and afterwards during beta-testing the typography refresh.

The current way causes enough issues that I think the gain
from getting rid of it is worth losing some brevity.

Bug: 56592
Change-Id: Ibeff450fad789fbc67d95a1a9befd315f7405421
(cherry picked from commit e4f973bd4c6bb0b9c4d0d210fde18ec73c11aee4)

Prevent GC during phpunit shutdown

To keep phpunit from segfaulting during shutdown, turn off GC for
php 5.3.

Bug: 62623
Change-Id: Ie59780e37cbea027cc204a43df406667c3f432ab
(cherry picked from commit 90b3d6285481e5fab8fdbc0ad3a0415f82555df3)

Merge "Don't cache $wgResourceLoaderLESSVars in a static" into REL1_23

Don't cache $wgResourceLoaderLESSVars in a static

Caching the value of $wgResourceLoaderLESSVars in a static variable that is
enclosed in a closure makes it harder to reset global state for test runs, and
it does so in the name of a performance benefit that is miniscule.

Change-Id: I0958b03818f56ab73c8c9124daa9e54cc59f2428
(cherry picked from commit a2bf2346cdaa4b4dce7b005f20220c3ceb56f6bb)

jquery.suggestions: Handle CSS ellipsis when calculating suggestions' widths

This is so nasty :(

Bug: 64093
Change-Id: Ia6b1376dddf69a0faa214d5c9a756bdcd9658367
(cherry picked from commit 271be3d09acfd3eb1cea3c0729b813c3d193c7df)

WebInstallerOutput: Fix path to jquery.js

Follow-up to Id5ead145 and I335e4c08.

Bug: 64091
Change-Id: If40f479df7d68071d6ce5d2f3b8251e6bf3eb80e

Remove font choices from REL1_23 branch while the implementation is unstable

Especially as they are undocumented, against release policy (bug 63351).
Follows up to Ic5ba836364d04b2c3814777b69b5f47fce25292a etc.

Bug: 63942
Change-Id: I595feec9a525104e382f7730208b1cf32c496e8d

FormatJson: Make it possible to change the indent string

This is to allow consistency with MediaWiki PHP and JS files (e.g. when
generating JSON i18n files), not because tabs are "better" than spaces for
indenting code (both have advantages and disadvantages).

Because PHP's json_encode() function hardcodes the indent string, using tabs
has a performance cost (in post-processing the output) and is less suitable
for web output; thus the API and ResourceLoader debug mode will continue to
use four spaces. Adjusting the maintenance scripts and JSON files is left to
separate change sets.

Bug: 63444
Change-Id: Ic915c50b0acd2e236940b70d5dd48ea87954c9d5
(cherry picked from commit 1efdda25ee5cbaea2b7e2dd7afdbaf53063d4b68)

Add @since to Skin::getAllowedSkins

Follow-Up: I0b0fdfb5615c22aa876882c3ca344ebb1f10c4bd
Change-Id: Ia14edc64c57bd8a9da0f2bda2d061d1a67aa0101

ResourceLoader::makeLoaderImplementScript: Bind args as '$' and 'jQuery'

Make the function that wraps ResourceLoader modules bind the first and
second arguments it receives to '$' and 'jQuery'. This patch is a
follow-up to change I0c9edac35, which updated the invocation of
mw.loader#implement so that it passes jQuery as the first and second
argument.

Change-Id: I0f0c3a04c3b0e6a28115a10bf11a2a78aca66c21

Merge "document.createElementNS is null or not an object in IE8" into REL1_23

document.createElementNS is null or not an object in IE8

Having $wgDebugToolbar = true and visiting the wiki with IE8 will give
no debug toolbar, using ?debug=true gives a "document.createElementNS is
null or not an object" javascript fatal.

Throw a error. The function 'createSvgElement' will never be called,
because profiling gives a "An ES5 compatible javascript engine
is required for the profile visualization." in IE8.

Change-Id: I763dae7459cb88f892823b9458ed6dd1dbbf3e9d
(cherry picked from commit cd361413d80a6af61991da642ace1c12130d2c0f)

.gitreview: Adjust default branch for REL1_23

Change-Id: If52695d2415d03d45b492700aa1ddc074faf83bc

Fixed some @params documentation

Swapped some "$var type" to "type $var" or added missing types
before the $var. Changed some other types to match the more common
spelling. Makes beginning of some text in captial.

Change-Id: I64e8cfe478cb0ba438f40b0631d6e9049cdab567

Merge "Cache RecentChanges Atom feed in varnish for 15 seconds"

Cache RecentChanges Atom feed in varnish for 15 seconds

Prior to fc20c30d209, the feed was cached for 10 seconds. That
change accidentally removed varnish caching. Re-add the caching,
and upping to 15 seconds for good measure (15 second delay on the RSS
feed shouldn't matter, people generally read those asynchronously
anyhow. I was thinking of maybe even uping to 30 seconds). Users
can still manually set an smaxage parameter via the url if they
don't like the default cache time.

This assumes nothing private (e.g. per user) is ever on the RSS
feed. Reading through the ChangesList code this appears to be
the case, as everyone is getting the feed from a shared
memcache entry.

Bug: 63249
Change-Id: I268d0a53067738ba96bee74c593358b0b28cc083

Fix CS double space in TitleValue return

Change-Id: Ib7eb2b070eea9f8b2df860b6d16f9c769be78cf7

Merge "ChangesListSpecialPage: Implement two new hooks superseding 4 old ones"

Normalise comments in Setup.php

- Changed "#" to "//" for non-block comments
- Put the inline comment describing the file back
  in the file description

Change-Id: I0aec0f4c10def58b608a8f08455fccb2317e4538

Merge "Add columns option to Special:PrefixIndex"

Merge "PrefixSearch: Remove unnecessary wfSuppressWarnings()"

Add columns option to Special:PrefixIndex

Adds support for specifying the number of output columns to use in the
table generated by Special:PrefixIndex. Can be used as follows:

{{Special:PrefixIndex/{{FULLPAGENAME}}/|columns=2}}

Bug: 60759
Change-Id: Ic824351132ec8bf7c35116f6ab19f23386ead33e