mediawiki.searchSuggest: Enable for iPhone and iPod touch

* This removes iPhone and iPod touch from browser blacklist.
* Similar to f64a3c26c6 in WikiEditor.
* it requires iOS 6 or later to work.

Change-Id: I46913bb6d360d6b0994b0585c0064f16f1ed460b

Merge "Setup.php: Move hacky skin requires higher up"

Merge "Function type hints for LinkHolderArray.php"

Setup.php: Move hacky skin requires higher up

Some of the setup code in some of the configurations apparently depends
on all extensions and skins being already required by the point
Setup.php is being loaded. We ran into issues with LocalisationCache.

Bug: 67318
Change-Id: Idde13c2e835a9969593a4716a62b392d4c1388d6

Merge "Improve Database related documentation a bit"

Merge "Make the global objects documentation consistent in Setup.php"

Merge "Documentation improvements in ChangeTags.php"

Merge "Don't use isset() to check for null"

Improve Database related documentation a bit

- $wgDBport is also for MSSQL
- "type" key of $wgDBservers is not only "mysql" and "postgres"
- Note that $wgDBssl, $wgDBcompress and $wgDebugDumpSql only
  work in certain cases and mention how to emulate them otherwise

Change-Id: Id863da8c87308d06e0070877ac0af26d03be4faf

Function type hints for LinkHolderArray.php

Change-Id: I5e429baab774a790b3558732f9c87637adfbe4ce

Improved DB handling in LCStoreDB

* Only do the DB inserts in finishWrite(). Previously this would
  COMMIT there but would spread out queries throughout the set()
  calls. This pushes all the contention down to the last step.

Change-Id: Ia8afb79a8328c18a7d8a386bcd15a5074007d52e

Merge "mediawiki.feedback: Don't use deprecated calling convention for mw.Api#newSection"

mediawiki.feedback: Don't use deprecated calling convention for mw.Api#newSection

Change-Id: I82e69a19f1d4416d456521c1233d13a349fe457e

Add SpecialPagesWithProp::prefixSearchSubpages

This shows the existing props as subpages on search suggestion

Change-Id: I3d35474bcf840a6feede8079642d83074874c6f3

Merge "Localisation updates from https://translatewiki.net."

Localisation updates from https://translatewiki.net.

Change-Id: I77bfdae2686c736fbd1b9bd44cfc690d9e4bad13

Merge "qunit.completenessTest: Ignore mw.loader.moduleRegistry"

Merge "qunit.completenessTest: Suppress mw.log.warn in walkTheObject"

Merge "qunit.completenessTest: Unbreak regular functions with static methods"

Don't use isset() to check for null

$wgLocalTZoffset is defined in DefaultSettings.php so it is always set.

Change-Id: I86518176b30da4e13f6dbfde8f1c77c8ced58d9f

Do not show user blocked warning on included Special:Contributions page

When user included {{Special:Contributions}} page of blocked user,
do not show "this user is blocked" box.

Bug: 65781
Change-Id: I13b80f829dd26bf960b16efea588618c36898a5d

Merge "Salt the "nsToken" used for Special:Search namespace remembering"

Merge "Fix CSSMin url() remapping when comments in CSS contain curly braces"

Merge "Remove 'webkit-backface-visibility' rule which made iPads super blurry"

Merge "chmod 644"

chmod 644

Change-Id: I31dddeb4e6373f5817fdfaeba3ff95699aec86b6

Merge "Fix file permissions from Ic7fc3bd836241dce8f296237bbd80ed3e4d1ee0d"

Merge "Make Preferences tests language-agnostic"

Make Preferences tests language-agnostic

Change-Id: I2a0bd6e4306de11a8b3f13505bc3e65439211e23

Fix file permissions from Ic7fc3bd836241dce8f296237bbd80ed3e4d1ee0d

Change-Id: I5d147c6baa034057f95e69b78811b9011cc4355e

Merge "Added missing Cucumber tags to Selenium tests"

Fix CSSMin url() remapping when comments in CSS contain curly braces

Remapping of url()s was thrown off if the CSS contained comments
with curly braces. To fix, replace all block comments (except @embed
directives) with placeholders before url() processing and swap them
back in afterwards. Patch also includes tests.

Bug: 60077
Change-Id: If18a93c17ea9bcd529f6a664aa2dcc51d4a37f38

Vector: Remove translations for non-existent message 'vector-desc'

Follow-up to Id7bde76b.

It seems that l10n-bot will only remove them if anything else changes
in any of the messages (see e.g. change Id40eee57), and this message
key is conflicting with the one used by the old Vector extension.
Letting it stay here is at best confusing and at worst problematic.

Change-Id: I649e305a5eaa019b9afeb4fd9aa1cbe06259db66

Name implicitly created CHECK constraints

to ease future maintenance work. The pattern for catching the right
CHECK constraint is admittedly vague but the order in which auto-created
CHECK constraints of a table is removed doesn't really matter assuming
the update is always completed before another update is applied. If
updates are run through Update.php, that should be always the case.

Furthermore eliminating a bug causing application of several patches each
time when running Update.php

Bug: 65757
Bug: 65813
Change-Id: Ic7fc3bd836241dce8f296237bbd80ed3e4d1ee0d

Merge "Less false positives for MEDIATYPE_VIDEO"

Less false positives for MEDIATYPE_VIDEO

Remove the ffmpeg2theora String to suppress false positives
when doing type detection on ogg files

(Importing pull 23 from github)

Bug: 63584
Change-Id: Ib39ed06f895624b55d15a001cb0a2cd3129f4bb0

Remove 'webkit-backface-visibility' rule which made iPads super blurry

An extra CSS rule was snuck into the .transition mixin in I895679ff
which caused the entire content area in Vector skin to render blurry
on iPads, especially visible on non-Retina devices.

There was no mention of it in the commit message, but a comment in
the mixin source said something about a 'Chrome 1px movement bug'.

With no source for said Chrome bug and a definite Safari bug, let's
try removing it for now.

Bug: 67286
Change-Id: If6e88d2b5fa7adaa3cd8198686422a65689fa68b

qunit.completenessTest: Ignore mw.loader.moduleRegistry

It was iterating over it and considering its 'script' properties
to be methods that had to be tested.

The CompletnessTest before this change is filled with stuff like:
- loader.moduleRegistry.skins.vector.js.script
- loader.moduleRegistry.mediawiki.api.script
- loader.moduleRegistry.mediawiki.Title.script
- ..

And was actually causing it to hit hard limit and thus actual
lots of actual methods were no longer inspected by the test.

Change-Id: I13fbf1224b48aa34a7bb01fdc0543023e502bb4f

qunit.completenessTest: Suppress mw.log.warn in walkTheObject

Lots of false positives where triggered for deprecation notices
while iterating over the object.

Change-Id: I710946062edd38b8d72ced6d67bd4bb7613246e3

qunit.completenessTest: Unbreak regular functions with static methods

While it already skipped constructor functions during the injection
phase (and thus didn't affect their properties), regular functions
can have properties, too.

Due to the QUnit CompletenessTest injecting a spy in mw.log,
things like mw.log.warn became undefined and causing exceptions
to be thrown sometimes.

Fix breaking of mw.log methods (and potentially other things) by
giving letting the spy function object inherit from the original
function object.

Also reduced and simplified logic further to accomodate this new
approach.

* Give injectCheck obj/key instead of masterValue/currPath.
* Pass around obj/key instead of value/key.
* Remove 'action' parameter, only ACTION_INJECT was ever used.
* Remove logic for 'prototype', this is now included in the type function/object case.
* Remove logic for 'prototype.constructor', this exclusion works naturally
  due to constructor functions being filtered out.

Change-Id: If16db9337a99865ada0fe1c482dd62a572239030

Localisation updates from https://translatewiki.net.

Change-Id: Id40eee5754cd13f0573f841c8ea42cb1ad67d4f4

Remove comment about externaledit from EditAction

The ExternalEdit class was removed in
Ic3791d188688f556753ab5556a4d9733bbf2e85e

Change-Id: I3cec96c0fe2a597ba5360c08d6bde55397fc9204

Remove internaledit, externaledit, mode params from My* special pages

The corresponding code was removed with
Ic3791d188688f556753ab5556a4d9733bbf2e85e, so there is no need to allow
these params on such redirects.
The redirect will not break, only the param is removed from request.

Change-Id: Ibb85bf3b23c9d17336f0a8b37a7315da8dcb7eef

Fix return doc for UploadBase::isAllowed

Method can return bool or string

Change-Id: Ib290843ce97926bfc529d510b1898f35c7e3f6c6

Use Database methods in RevisionDeleteUser::setUsernameBitfields

The bit handling in RevisionDeleteUser::setUsernameBitfields was not
using Database::bitAnd, bitOr or bitNot.

Also changed one condition to use Database::addQuotes

Change-Id: I898ab5afcb628bb7ebe81bae1522153434c7fe98

Merge "Replace hardcoded square brackets by message brackets"

Merge "Fix uncaught ApiFormatXml exception with api debuginfo"

Replace hardcoded square brackets by message brackets

* Rename $temp into $attribs

Change-Id: Id8792f9e6d0de6460a5c6a4ae34b8a745fe2409e

HTMLSelectOtherFields: Have 'other' always as first element

The other field is in most case the default item selected in the drop
down box when a page is loaded. But when the last item is selected you
have to scroll to the top to select one of the first item, which are
often the common or most used items.

So adding the 'other' always as first item to the list.
For the HTMLSelectAndOtherField this was before
If4175332405d26c7ff2e8fbe100bcad61762ce6f, so that is fixing a
regression. In my opinion it should be the same for both types.

Follow-Up: If4175332405d26c7ff2e8fbe100bcad61762ce6f
Change-Id: I82f31e260d63bfaa6a4a94908e019feb9e0f2421

Add comment to TableDiffFormatter::blockHeader

magic strings should have a comment

Change-Id: I23fc7a4de31e22383f0a6ffd1db56717615cfa0c

Merge "Combine SpecialPageBeforeFormDisplay set of hooks into one"

Merge "Move QueryPage and abstract subclasses to specialpage folder"

Combine SpecialPageBeforeFormDisplay set of hooks into one

Hooks with dynamically generated names cannot be documented in the normal
manner, and indeed this set, added in r86482 / 10d93c34cb17, was not.

In contrast, every other hook in core has a name that is a fixed string
and thus is easy to grep for, create a manual page title for, and so on.
Conceptually, each hook's name should correspond to one or more locations
at which code can be inserted.

So I changed the code to use hook name "SpecialPageBeforeFormDisplay"
for all FormSpecialPages and added the special page name as a parameter,
which is consistent with existing hook "ActionBeforeFormDisplay".

From extensions in Gerrit, the only use I found was in StopForumSpam,
which is updated to use the new hook in Id474915a. So I chose to break
backward compatibility instead of using a hack to hide the old hooks
from maintenance/findHooks.php.

After three years, the script again reports, "Looks good!"

Change-Id: I7ea6521b47fb034bc367a1d06d477a7654035a5f

Fix initSiteStats.php maintenance script

It appears not to have worked for some time, as it either calls a protected
function ($counter->refresh()) or an undefined function
($counter->update()) depending on the parameters specified.

Bug: 65214
Change-Id: Ia7d867792b84c98714ec6dbbfef09745e875c8bc

Merge "Tweaks to Parsoid CSS styles for citations."

Merge "installer: Remove unnecessary param to outputCss()"

installer: Remove unnecessary param to outputCss()

WebInstaller::outputCss no longer requires an argument
as of I717cb299b3639024.

Change-Id: I85c1624eb99affd5df809a041cfc9f22fc46945a

Remove unneeded else brach from SpecialAllMessages::execute()

Spotted while reviewing I49cf4f294a (8b4c170)

Change-Id: I5b18106bbd892a22c0d48aefecc1ae246b577480

Fix uncaught ApiFormatXml exception with api debuginfo

Also added tests to cover this.

Bug: 67246
Change-Id: Ifb2e392d3277a4702832727f70c77b170d4b2bf5

Added missing Cucumber tags to Selenium tests

Also:
- updated Ruby to version 2.1.2
- added missing ruby-gemset section to Gemfile

Change-Id: I2e6e7213ce51e8a15ecea34f76dd265d9d2f8594

De-globali[sz]e AllPages and Prefixindex

Change-Id: I2ea969b3bf2573448562fd98b9306997bcfcfea4

De-globali[sz]e Special:Allmessages

Change-Id: I49cf4f294a6543c7ee9ca56d9d6e9635996de30d

Update Moment.js from 2.6.0 to 2.7.0

Project site

* http://momentjs.com/ and https://github.com/moment/moment/

File source

* https://github.com/moment/moment/blob/2.7.0/moment.js

README

* https://github.com/moment/moment/blob/2.7.0/readme.md

Changelog

https://gist.github.com/ichernev/b0a3d456d5a84c9901d7

difference between 2.6.0 and 2.7.0

new languages

• #1678 Bengali (bn)
• #1628 Azerbaijani (az)
• #1633 Arabic, Saudi Arabia (ar-sa)
• #1648 Austrian German (de-at)

features

• #1663 configurable relative time thresholds
• #1554 support anchor time in moment.calendar
• #1693 support moment.ISO_8601 as parsing format
• #1637 add moment.min and moment.max and deprecate min/max instance methods
• #1704 support string value in add/subtract
• #1647 add spm support (package manager)

bugfixes

• #1651 fixes #1627 fix timezone parsing in some edge cases
• #1641 fixes #1626 fix creation from week/weekday/weekyear for weeks starting monday
• #1616 fix language code of cyrillic Serbian (sr-cyrl) and latin Tamazight (tzm-latn)

language bugfixes

• #1699 fixes #1326 use lowercase months in Italian
• #1650 partly fixes #1640 implement meridiem parsing for Russian
• #1673 fix lithuanian weekday names for weeks not starting Monday
• #1433 Better "time ago" expression for European Portuguese
• #1671 Improve welsh past expressions
• #1449 Use dot after the day number in Danish
• #1559 Update arabic numbers, meridiem and short weekdays
• #1622 Fix previous Saturday in Greek

tests

• #1687 fixed flaky test in Russian timezone

Change-Id: If8e065bf9e0962059697e6f08100e7bc966cd32f

Merge "Replace deprecated Title::escapeCanonicalURL call"

Tweaks to Parsoid CSS styles for citations.

* This brings Parsoid rendering quite close to PHP rendered output.

Change-Id: I22e1fad26e917a84a24193c53bf9522b383b849d

Fix release notes

Reference to a more appropriate bug in the release notes
for I0f82b146fbe948f917c1c5d29f7469644d797e80

Change-Id: I1c7fe2e3244cb509fdd72539632d0a8440d454e7

Update OOjs UI to v0.1.0-pre (a63f03882e)

New changes:
0dafebd build: Only include source maps for test builds
a63f038 build: Move source map option from git-build to pre-test

Change-Id: I2b06b256e48106eac2090df6d4bf5f45349222f9

Merge "Follow-up I81663929: Use grunt git-build to ensure we get the hash"

Merge "First version of Page Language selector"

Follow-up I81663929: Use grunt git-build to ensure we get the hash

Change-Id: Idd140a9027ad4f1891e139c83b19de33f4a19d23

First version of Page Language selector

Special page PageLanguage to set the page language of a page.
To enable the feature, set $wgPageLanguageUseDB to true
and assign the 'pagelang' user right to a user group.

Bug: 35489
Change-Id: I0f82b146fbe948f917c1c5d29f7469644d797e80

maintenance: Simpler OOjs UI pullthrough-script

No need to run grunt test, given that the grunt build from npm install will
suffice for our needs.

Change-Id: I816639290874e4962d5dac7d580d314e3bd1cb16

Don't unserialize integer reply from Redis incrBy

It does not work with unserialize because RedisBagOStuff->unserialize
expects either strings of digits, or serialized strings.  A plain integer
is neither.

incr docs were incorrect, so fix and expand by copying from parent class.

Change-Id: Ifc9c7a36f221b251ceea0d9a79ae6c3ce5753dd4

Merge "Fix RequestContextTest screwing up $wgUser"

Fix RequestContextTest screwing up $wgUser

Change-Id: Ib3fabc1b016f33bf188682f7f894e8f52417ee40

Merge "Localisation updates from https://translatewiki.net."

Merge "Use separate PoolCounter config for expensive thumbnails"

Localisation updates from https://translatewiki.net.

Change-Id: Ib16f36dc258233455cea345f579a98055529c078

Standardize version comment in OracleUpdater.php

All updaters have a space between // and version number, do this also
for oracle updater

Change-Id: I0f44bd5d1ab5e131cca3cd7ebfaaf1fe91e4d9d8

Do not mention deprecated User::isValidEmailAddr() in hooks.txt

Replaced by Sanitizer::validateEmail()

Change-Id: I7b4ef438dd31a2dbc641bb06bea8e256e0f5dd1f

Use separate PoolCounter config for expensive thumbnails

Depends on Ic62984e0f4a761642b2bdd1bfa362301ed94c284
and If40e223c328ddd15310d13f2a49000bf6e08200e

Bug: 65691
Change-Id: Ic737673b5a31974d866b9bf5afa4b4036baa03bb

Salt the "nsToken" used for Special:Search namespace remembering

Token is used as a GET parameter, so we shouldn't use the raw edit
token.

Bug: 67210
Change-Id: Ie69d9f6557eb9a08b21361aad0ed95bee994c05f

Merge "Added some sanity warnings to TransactionProfiler"

Merge "objectcache: Actually unserialize integers as integers in RedisBagOStuff"

Merge "Reset RequestContext between tests"

Added some sanity warnings to TransactionProfiler

* The use of $id should avoid any nesting, unless something is broken

Change-Id: I01941a186d1e74ddd6c4610b0eb9e15f54e28347

Merge "Add some casts for sanity before BagOStuff::add() calls"

Reset RequestContext between tests

Its state can change when people do something with objects it returns or
when they alter globals like $wgRequest. By resetting this singleton, we
ensure that no such change will propagate outside of a test.

Change-Id: I7e8598716d810a09c17f80a05deecab617b62346

Merge "Port preferences test from /qa repo"

Port preferences test from /qa repo

Add tests for preferences, make  core tests language-independent.
Change-Id: I1ba4e2c53a9e1bdb7b832efec4c2c8f6f64d2b86

Replace deprecated Title::escapeCanonicalURL call

Change-Id: Icd3e2304d6188babe2d887aa32fc0c398352b7b6

Merge "Remove ApiBase::createContext() (deprecated since 1.19)"

Merge "Add parser test for "extra interlanguage links"."

Merge "Update OOjs to v1.0.10"

Merge "Make SpecialPreferencesTest::testBug41337 non-risky"

Make SpecialPreferencesTest::testBug41337 non-risky

PHPUnit treats tests that make no assertions as "risky" because they make it
difficult to distinguish successful test runs from programmer error. To make
SpecialPreferencesTest::testBug41337 non-risky, assert that the execute method
returns null.

Change-Id: I4c199d2b81037e6ca2b2862418c4f8a8352c777e

Update OOjs to v1.0.10

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fcore.git/v1.0.10/History.md

Change-Id: Ib154a0c9cfc7c8b428671c9d77892361ea8d7e1b

Update OOjs UI to v0.1.0-pre (b9d403a678)

New changes:
a725505 Work around element sizing bug in Chrome
d7930fb build: Implement source maps for js build target
75ed129 build: Use isoUtcDateTime instead of yyyy-mm-dd in banner
b9d403a Make ConfirmDialog interpret clicking X as 'cancel', not as 'OK'

Change-Id: I7d4ba9521935412b780cb19897990801ed51f654

Add some casts for sanity before BagOStuff::add() calls

Change-Id: I6ecb609c88e4fc1ef614e94c551199bd610ca360

objectcache: Actually unserialize integers as integers in RedisBagOStuff

Before:
> SET   (stored)    GET
>  5     5          (string) "5"
>  '5'   5          (string) "5"
>  'x'   s:1:"x";   (string) "x"

After:
> SET   (stored)    GET
>  5     5          (int) 5
>  '5'   s:1:"5";   (string) "5"
>  'x'   s:1:"x";   (string) "x"

Follows-up 2ceda41c5783ddc3cfb.

Bug: 60563
Change-Id: I2bb09381b2bb733ac5a89175e053cb10eca68b08