resourceloader: Never defer execution for top loading modules

There was a bug where if a module was in the top queue but had CSS,
its execution would be deferred until its CSS was ready. This would
cause the module's execution to occur after document ready and after
user scripts had run.

This is bad because top-loaded modules are supposed to be guaranteed
to run before document ready.

Fix this by skipping the wait-for-CSS logic when loading synchronously.
Top-loading modules' execution is now guaranteed to block loading the
rest of the page, but their CSS isn't guaranteed to be ready at this
point. Which is fine, because they shouldn't be measuring things in the
DOM before document ready anyway.

Change-Id: I5779cdef336be184741ad97ea84113eb7790dd39

Merge "Added new MWTimestamp::getRelativeTimestamp for pure relative."

index.php: Remove copyright line and use HTTPS url for mediawiki.org

None of the other entry points (or any file for that matter)
lists the main MediaWiki copyright holders. This is already in
README and SpecialVersion. Afaik there is no special value for
index.php to have this as well (plus, it has become outdated
and incomplete).

Change-Id: Ic3e1ab5b452002160563750fa1537dfbdd66e312

Merge "OutputPage->getJSVars: Maintain symmetry between JS & PHP vars"

Merge "Remove legacy scripts from .jshintignore"

OutputPage->getJSVars: Maintain symmetry between JS & PHP vars

The function is more difficult to read than it need be by dint of some things
two names. I can't be the only one who finds 'wgCurRevisionId' => $latestRevID
ugly, right? Anyhow, lots of JS code depends on the JS variable names, whereas
the PHP variables exist only in the scope of that method, so it's clear that
it's the PHP names that should be brought in line.

Change-Id: Ibddd5d2fc8d75e0ade18ff3433714d52605f2911

Merge "Clean-up of skins/common/IEFixes.js"

Remove legacy scripts from .jshintignore

- skins/common/config.js only needed a singlequote fix to pass JSHint.
- skins/common/IEFixes.js passes thanks to I1ee94b184.

Change-Id: If23ff6e13ea1e0060e7366346196ee61c4cbb841

Clean-up of skins/common/IEFixes.js

- Wrap script code in an IIFE.
- Create scope-local bindings for global vars.
- Use strict comparisons.
- Use single var statement to declare variables.

Change-Id: I1ee94b184f5bb2355d01fae554c50ef314147298

Support rate limiting backlink purges, including null edits

Change-Id: I86e45b7d0eba7d87f65150b34e190e04d8342b3e

Merge "Move LESS function definitions from DefaultSettings to ResourceLoaderLESSFunctions.php"

Move LESS function definitions from DefaultSettings to ResourceLoaderLESSFunctions.php

Intermittent error in production from top-level lambda functions:
https://bugs.php.net/bug.php?id=52144

Change-Id: I9785a601fd5d57c9d8d84b90ee29eef61a6f2e77

Merge "Use utf8 charset for searchindex MySQL table"

Add explicit plural support for limit report messages

Requested by Purodha Blissenbach in
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Limitreport-postexpandincludesize-value/ksh
and
https://translatewiki.net/wiki/Thread:Support/About_MediaWiki:Limitreport-templateargumentsize-value/en

Change-Id: I39bea8a45bdd941ebe417b823936aa599185ae03

Merge "Clear legacy restriction field on page restriction update"

Localisation updates from translatewiki.net.

Change-Id: I643a45c8f96b3c530a488c4a5ed2a0bbf6d7aa24

Fix typo in comment block in mediawiki.js

asynchrounous -> asynchronous.

Change-Id: I65cf79c14fdc8359875e7dda3821678e99712cb1

Use utf8 charset for searchindex MySQL table

This prevents the DB error "1283 Column 'si_title' cannot be part of
FULLTEXT index", which happens when the "binary" charset is used.

Note that 2ab7c9ec3b fixed the problem for creation of new DBs, though
not for reuse of existing DBs. It is still necessary to specify the
character set for the table itself.

Bug: 47191
Change-Id: Ib4f30fd2730839af1323295898e6dce47b27068d

Merge "Add wgRevisionId variable to ResourceLoader"

Merge "Rename lessc->embeddedImages to lessc->embeddedFiles"

Merge "Vary on forceHTTPS cookie"

Vary on forceHTTPS cookie

Varnish seems to be returning the cached version of pages for users
after they have logged in over https, but access an http page. This
seems to occure because only the forceHTTPS cookie is sent on the
request, but varnish doesn't vary the cache based on that cookie.

Bug: 54513
Change-Id: Ia97ed80622191669ee5ca37af809d307bbdb61ae

Merge "Replace Pakaran with Punjabi"

Replace Pakaran with Punjabi

These are Punjabi digits. I couldn't find a "Pakaran" language.

This was added in 5d00cad9.

Change-Id: I94649b7e5c3c6c9666accee561b12166552d50cb

Clear legacy restriction field on page restriction update

This causes problems where some pages can't be unprotected
as we still take the legacy field into account. See also
bug 33334.

Change-Id: Ibaec827289b1070ee110635d6c9d7b60e061384b

Merge "ResourceLoader: expose registry as mediaWiki.loader.moduleRegistry"

ResourceLoader: expose registry as mediaWiki.loader.moduleRegistry

This patch exports the ResourceLoader registry object as a public member of
mw.loader. I understand that it is not properly a part of the public API
ResourceLoader exposes to JavaScript code; it is an internal data structure and
ResourceLoader does not expect to have anything modify it except ResourceLoader
itself. At the same time, it provides a trove of extremely valuable data for
debugging and profiling, and it's annoying as hell not to be able to poke and
prod it from the JS console. So my hope is that we do the Python thing of
treating 'private' as a tip and social convention rather than something that is
enforced by the code.

Change-Id: I1c47970aa9fceb0b2ded9008efc926b46fa08f51

Merge "Add Special:CreateAccount to Special:Specialpages"

Merge "bug 23839: "Refreshing links table" -> "Refreshing links tables""

bug 23839: "Refreshing links table" -> "Refreshing links tables"

There's not a single links table; there are several tables. :)

Change-Id: Ie2a76da3d047a41d102442be1919796bcd7bf2d2

Merge "Add option to chose what language to fetch file description in."

Added new MWTimestamp::getRelativeTimestamp for pure relative.

Since MWTimestamp::getHumanTimestamp just makes a pretty
timestamp, and not a purely relative one, this commit adds
MWTimestamp::getRelativeTimestamp, which does just that.

Change-Id: I8b3e3fc6eeb4afd58f85fa7d05b5ea1a51b0afb6

Add wgRevisionId variable to ResourceLoader

mw.config provides wgCurRevisionId, which has the latest
revision number. This adds wgRevisionId, which indicates
the revision number currently being viewed. This is useful
when looking at old revisions.

Bug: 51594
Change-Id: Ie0d00f3a9a8af8533ab28204b5bb483a0092c710

Add option to chose what language to fetch file description in.

I want to get the file description in a language other then
the user language in an extension (I5e6bc45f9751).

Change-Id: Ifcae821a51f4207e7816e710d3b3857c7ed438b6

Merge "API: Check for nosuchsection in all code paths"

Update lessphp to upstream head (785ad53840)

This patch updates lessc.inc.php to 785ad53840:
<https://github.com/leafo/lessphp/commit/785ad53840b1777ef2120a10dcb2a5d5e6699f36>.

Differences from our version:

- Updated year in copyright notice
- Added support for 'precision' parameter to round().

Change-Id: If22afea96741c6d8fedefc05c49b24ae6e7809b0

Merge "MysqlInstaller: use default character set of utf8"

Merge "Improve wfEscapeWikiText, part 2"

MysqlInstaller: use default character set of utf8

Bug: 14931
Change-Id: Ied512a75d17b401206b6439f289fe36c01536cec

Remove LanguageEo::checkTitleEncoding()

In 2005, the code to perform the conversion from the Esperanto
x-system to Unicode was recognized as problematic and commented
out (bug 1512, r10997). Hence this function has done nothing
useful in the last eight years.

Change-Id: I9f4351c638645024b17a828144bba8575db14195

Rename lessc->embeddedImages to lessc->embeddedFiles

'embeddedImages' is a custom property we set on lessc compiler instances to
track the set of files that were embedded in the generated source as data URIs.
This is so done so we know to regenerate the CSS if the source file for an
embedded asset changes. (If my pull request[1] is merged, we won't have to use
a custom property at all.)

I realized that the property name should be 'embeddedFiles' rather than
'embeddedImages', because it's not just images that can be usefully embedded in
CSS, but fonts as well.

  [1]: https://github.com/leafo/lessphp/pull/472

Change-Id: Ief3afaa23e4532f4a536e0dfef943d4fa20db80d

Merge "Implement LESS image embedding"

Merge "Support LESS stylesheets in ResourceLoader"

Implement LESS image embedding

This patch adds two custom LESS functions: embeddable() and embed(). Both
functions take a single image file reference as their sole argument.

- embeddable() is a predicate function (= returns a LESS boolean value) that
  checks whether an image is suitable for embedding. An image is suitable if it
  exists, has an appropriate MIME type, and is not too large.
- embed() generates a CSS url() value that contains the reference image encoded
  as a data URI.

The existence of the predicate function allows embeddable() to be used as a
mixin guard. This provides an elegant means of generating a single rule when
the image is not embeddable and multiple rules (the data URI & fallback) when
it is. This technique is used to implement a .background-image mixin, included
in this patch.

Change-Id: I3e06b6d6e8630b7923fa42b3daf1ced3e656bbe7

Support LESS stylesheets in ResourceLoader

This patch adds support for the LESS stylesheet language to ResourceLoader.
LESS is a stylesheet language that compiles into CSS. The patch includes
lessphp, a LESS compiler implemented in PHP. The rationale for choosing LESS is
explained in a MediaWiki RFC which accompanies this patch, available at
<https://www.mediawiki.org/wiki/Requests_for_comment/LESS>.

LESS support is provided for ResourceLoader file modules. It is triggered by
the presence of the '.less' extension in stylesheet filenames. LESS files are
compiled by lessc, and the resultant CSS is subjected to the standard set of
transformations (CSSJanus & CSSMin). The immediate result of LESS compilation
is encoded as an array, which includes the list of LESS files that were
compiled and their mtimes. This array is cached. Cache invalidation is
performed by comparing the cached mtimes with the mtimes of the files on disk.
If the compiler itself throws an exception, ResourceLoader constructs a
compilation result which consists of the error message encoded as a CSS
comment. Failed compilation results are cached too, but with an expiration time
of five minutes. The expiration time is required because the full list of
referenced files is not known.

Three configuration variables configure the global environment for LESS
modules: $wgResourceLoaderLESSVars, $wgResourceLoaderLESSFunctions, and
$wgResourceLoaderLESSImportPaths. $wgResourceLoaderLESSVars maps variable names
to CSS values, specified as strings. Variables declared in this array are
available in all LESS files. $wgResourceLoaderLESSFunctions is similar, except
it maps custom function names to PHP callables. These functions can be called
from within LESS to transform values. Read more about custom functions at
<http://leafo.net/lessphp/docs/#custom_functions>. Finally,
$wgResourceLoaderLESSImportPaths specifies file system paths in addition to the
current module's path where the LESS compiler should look up files referenced
in @import statements.

The issue of handling of /* @embed */ and /* @noflip */ annotations is left
unresolved. Earlier versions of this patch included an @embed analog
implemented as a LESS custom function, but there was enough ambiguity about
whether the strategy it took was optimal to merit discussing it in a separate,
follow-up patch.

Bug: 40964
Change-Id: Id052a04dd2f76a1f4aef39fbd454bd67f5fd282f

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I80c44c7678b86b6890e1afda2924b986c39d52a0

Merge "libxml_disable_entity_loader() just in case..."

libxml_disable_entity_loader() just in case...

Change-Id: I5c6394f811f9cd14dc549d005b4583d7575e85aa

Fix missing wfProfileOut call in HtmlFormatter

Followup to missing call in Ic276e1604c5718e8568e120ddfb9a8fc13a682fc

Change-Id: I69623d565826aacaa884c1c0a3ffec46ca1fb465

Fix doc for HtmlFormatter

Bug: 54473
Change-Id: I96d76eb2d372fb3997c432196e7c4cc4e2984bfb

Merge "Move HtmlFormatter from MobileFrontend"

Merge "Changing default copyright notice per WMF Legal's suggestion."

Merge "Replace single to double-quotation marks"

Localisation updates from translatewiki.net.

Change-Id: If490bc68434b06359037fac93ec613d0e7265f44

Replace single to double-quotation marks

Give grep a chance to find the usages

Change-Id: I95431f8e5cc684c322fdd7cda041388120b7c49b

Localisation updates from translatewiki.net.

Change-Id: I82effb99c8a9b2f7bedc874008835e29f1d9ae4d

Merge "Deprecate assertException in MediaWikiTestCase as it adds no value"

Merge "SearchEngine::userHighlightPrefs needs no user"

Merge "Simplified some code and broke lines in thumb.php"

update.php clears caches twice

There was a bad rebase on I96b4cfd4c02e9cbf46cc6a0499b87fb3b89020a0,
because it partically reverts Ice42a31dee1e6b41da4aa0a47e8786579382aff1

Bug: 44690
Change-Id: Iff2c8d3cb1a10da7cd0ab18ee189454761d7db6f

Added {{REVISIONSIZE}} variable to get the current size of a revision.

This avoids giving the own page name to {{PAGESIZE:}} and allows
PAGESIZE on subst: to give the size before save.

Viewing a oldid page will give the corresponding size to that revision.
(Like REVISIONUSER or REVISIONTIMESTAMP)

This partially reverts Idfac13de37d05317f65e4131534543e66cf74873
because there is no need to let PAGESIZE handle extra, when the given
title is the own page.

Change-Id: If9e608d54904c21ac6b095e91ff6e0a15da0fb4c

drop unused fields rc_moved_to_ns/rc_moved_to_title

Follow-up I68e8c0bb23f185c0f996a8905f6d437db3080aa1

Deleting those fields also from tables.sql definition
for mssql, oracle and postgres

Bug: 34960
Change-Id: I1867536c9161abed0b23428d3c840d16a05cba56

Tidy up log deletion rows which didn't get subpages

Bug: 37714

Change-Id: I904988c864d777d039b2aae956aef7b19092cf83

Cleanup Preprocessor_DOM::preprocessToObj wfProfileOut()s

Simplify the multiple if levels used for profiling out.

Change-Id: Id0530207f99daca49a6a76ce256476b677a4108f

Add a way to redact certain function parameters from exception stack traces

Bug: 30714
Change-Id: I0a9e92448f8d9009dd594cb4d7f5dc6fdd4bcc86

Remove reduntant regex from calls to StringUtils::isUtf8()

I've cautiously moved the regex out of the most used code path.
There is no string that will match that regex check that will not also be
passed by mb_check_encoding.  I think the regex was intended as a shortcut
evaluation, but it is no faster than mb_check_encoding which will often
need to be run anyway.

I think it could just be deleted, but I have limited motivation to
risk introducing a bug to improve performance on old PHP vesions and
unusual configurations, so I've moved it to the fallback code path.

Change-Id: Ie9425cc23ba032e5aff42beeb44cbb1146050452

optipng -o7 "$png" && advpng -z -4 "$png" && advdef -z -4 "$png"

Change-Id: I2b267d23da582e4daf95c6898e3c687ad63d6e6b

Changing default copyright notice per WMF Legal's suggestion.

Changing from "Content is available under $1." to...
"Content is available under $1 unless otherwise noted."

This is especially relavent to wikis that use InstantCommons.

Change-Id: I878372de6c42bcb7660b111911d0e12bb351d2c5

Note about removing portal-url, currentevents-url and helppage from default Sidebar

A note about the removal of portal-url, currentevents-url and helppage from
the default Sidebar as requested at the related bug report.
Follows up: Icb0e1b91e40bd682629f8a6f7291dbf8588b586

Bug: 53862
Change-Id: If7a624cd3a62c31af84a711f79ea396b53868fd2

Deprecate assertException in MediaWikiTestCase as it adds no value

setExpectedException, provided by PHPUnit itself, can and should be used instead

Change-Id: I06833eeeb492d2cc4e66b14aaebd5b2061a29e3a

Localisation updates from translatewiki.net.

Change-Id: I4d3fce0836a6e2bd2c7c79159fb74411a02b40b6

SearchEngine::userHighlightPrefs needs no user

Was removed with r87215 from the function,
but not from the function call

Change-Id: Iacb247d5962fe77db5a29084ece941261bf306ec

Move HtmlFormatter from MobileFrontend

This class is needed by CirrusSearch, another application will be API
prop=extracts which doesn't really belong to MobileFrontend and will also
be integrated into core soon.

Change-Id: Ic276e1604c5718e8568e120ddfb9a8fc13a682fc

Simplified some code and broke lines in thumb.php

Change-Id: I99fc63ab473a3f2fe33e0227e5e283e0c195f01b

Improve wfEscapeWikiText, part 2

The following are now correctly escaped:
* Blank lines (including those with only tabs)
* ---- at the start of a line

Bug: 53658
Change-Id: I300e15a22c93ca542913135c9c83913c18c99faf

Pass user to Title::userCan in ApiFeedContributions

Change-Id: I15b95f4e4bf4a07b55b98baa360078411c0f42cc

Merge "Collect stats about RL cache hit rate"

Collect stats about RL cache hit rate

Change-Id: I7cf2f2248f60b0e686e90faf9cb9989b723c88d1

Pass context to Action::factory

Fixes a PHP Notice that was introduced on every page (follows-up 01d81d88)

Bug: 54344
Change-Id: I24f01c7244089e73b0cf2e0af37c365f9dd051b2

Reduce use of FORCE INDEX in LogPager

* This currently hurts some change tag queries by orders of mangitude.
  MySQL/Maria query planning is also a fair bit smarter than it used to
  be. Lastly, WMF table statistics are better maintained now.

Change-Id: I4d0ac7a35c9cec6b47771aae718b44bfc03bdfbe

Localisation updates from translatewiki.net.

Change-Id: Ifc090ee9782be50af70146738908549ca31b6f96

Merge "Reduce complexity of SiteSQLStore saveSites method"

Allow RedisConnectionPool to be used with HHVM's Redis library

As of commit 2811e9d, HHVM includes a pure-PHP Redis client library with
method-for-method compatibility with phpredis: <http://git.io/ylDAyQ>.
RedisConnectionPool appears to work well with it save for the
extension_loaded() check. Practically a freebie!

Bug: 54158
Change-Id: I3ac12293a51a45e8f6a61a35bff7a3c063e16964

doc: Fix grammar in GitInfo documentation comment

Change-Id: I67da49398f1281b83700884475d364f6b30c05c8

Merge "Adapt StringUtils::isUtf8 to the top of Unicode at U+10FFFF"

Fix initialization of Context in Action class.

When Action::getActionName calls Action::factory, $context
must be passed explicitly, since WikiPage does not have a getContext()
method (only Article does).

As a failsafe in Article::getContext(), use $this->page->getContext()
only if $this->page is an Article. If $this->context is null and $this->page
is not an article, fall back to RequestContext::getMain()

Change-Id: I6d8db29dfd1aaa68f3de4f8f418841205c2379af

Add Special:CreateAccount to Special:Specialpages

Bug: 53447
Change-Id: I4423e5beeb7b72814f4ee839880ec4b835d14640

jquery.suggestions: Fix up trailing comments from 2f5125e8

Change-Id: I8f2f8a9f1e0da3e460928003b56a2c99668473da

Factor out ChangeTags::tagUsageStatistics() from SpecialTags

It's prettier if we keep all explicit DB queries to those tables
in one place.

Change-Id: I85b6854e26ffc656b990079d463a085ba39556df

CSSMin: factor out image encoding into encodeImageAsDataURI method

The code for encoding suitable images as base64 data URIs was previously part
of the remap method of CSSMin, which greps CSS files for /* @embed */
annotations. This patch moves it into its own method, encodeImageAsDataURI.

Change-Id: I6f57116c6a6c18ff9d4e28760dd484ea2c8fc493

Adapt StringUtils::isUtf8 to the top of Unicode at U+10FFFF

RFC 3629 defines the legal range of characters as U+0000..U+10FFFF
and forbids overlong forms (encodings of a character that use more
bytes than necessary). Let's make StringUtils::isUtf8() match the
specification.

* Changed the maximum value in the pure PHP code path and added a
  check for overlong forms.
* Added another check, specific to PHP 5.3's mbstring extension,
  for values above U+10FFFF.
* Fixed the mbstring test errors in PHP 5.4 using changes to
  StringUtilsTest by Platonides <platonides@gmail.com>.
* Uncommented some other tests that could fail because of the
  missing check for overlong forms.
* Added additional tests for extra continuation bytes, overlong
  sequences/forms, and values in the UTF-16 surrogate range.

The changes to the function were so extensive that I might as
well say I rewrote it.

Bug: 43679
Change-Id: I56ae496d17ffc3747550e06a72dacab3ac55da61

Merge "Separate RevDel logic from UI"

Reduce complexity of SiteSQLStore saveSites method

split code that generates an ORMRow from a Site object

Change-Id: Ie1f7ed834a33befb43536355ad2b9b98a13d4c79

Merge "Add gzdecode fallback to GlobalFunctions"

Merge "Localisation updates from translatewiki.net."

Merge "Changed tableName so it returns uppercased table names (+prefix) Changed tableExists so it returns boolean (and closes resource when it's done in case it fails)"

Merge "Changed quoting function for oracleDB."

Localisation updates from translatewiki.net.

Change-Id: Ia363d8fc374b04d948cb0b6599269f9c0e1c4117

Fixed check of BagOStuff set() return value in mctest

Change-Id: I6e69cdc24b9b40132641205cb40f64bb5db45f70