Preferences: Use mediawiki.confirmCloseWindow module

This logic was duplicated into the preferences page module before we
had refactored it into a separate module.

Bug: T89819
Change-Id: I67a25ff8758f009123526d5194929bfcd7c3039b

Merge "Maintenance script for importing site info."

Merge "Avoid double word-separator when using Linker::userToolLinks"

resourceloader: Simplify log message for getDefinitionMtime() cache miss

* Add quotes around context (which has pipe characters in it).
* Remove hash (not useful).

Change-Id: Iff80c914add0c5a2fe27fff6398bb3384102e3d2

Merge "API: Add authz features for RESTBase"

Removed doCascadeProtectionUpdates method to avoid DB writes on page views

* Use special prioritized refreshLinksJobs instead, which triggers when
  transcluded pages are changed
* Also added a triggerOpportunisticLinksUpdate() method to handle
  dynamic transcludes

bug: T89389
Change-Id: I8e5a6ddb643c12e0fb5c1c68bc83f912944e6e8d

Merge "Adding TemplateParser class providing interface to Mustache templates"

Made TransactionProfiler log regardless of Profiler

* Also avoid the use of slow closures for profiling

Change-Id: Id437e39038fd9ba80b899329a61e9f14229481f2

Adding TemplateParser class providing interface to Mustache templates

The TemplateParser class provides a server-side interface to cachable
dynamically-compiled Mustache templates. It currently uses the
lightncandy library to do compilation (which is already included in
the vendor repo).

Also converting NoLocalSettings.php to use it as a proof-of-concept.

Bug: T379
Change-Id: I28cd13d4d1132bd386e2ae2f4f0d1dd88ad9162b

Sync up with Parsoid parserTests.

This now aligns with Parsoid commit b3127b83a8923c7d73c7e00eb0c651e039b84521

Change-Id: I0d8160bc93e5912c2466d1a3051d2a69cca6c41a

Merge "Add clickable link for section headers"

Add clickable link for section headers

Changed the Linker::makeHeadline function to additionally generate an
anchor that uses a section symbol. Created mediawiki.sectionAnchor
module to style the anchors and included it in SkinTemplate so that
other skins automatically get those styles.

Bug: T18691
Change-Id: I562e437ec0bd337c9db0406f5dcab504dfa37034

API: Add authz features for RESTBase

The RESTBase team has requested the ability to check the validity of a
CSRF token and to interface with Title::userCan().

The former is accomplished by the new action=checktoken module. The
latter by a new parameter ('testactions') to the existing prop=info.

Bug: T88010
Change-Id: I2530f1315ec93f5be9fb437137992150fdc305f2

Merge "database: Move LoadBalancer::getConnection debugmsg to getReaderIndex"

database: Move LoadBalancer::getConnection debugmsg to getReaderIndex

Most of the time the early $this->mReadIndex inside getReaderIndex()
should be a hit, in which case this message isn't very useful.

This one message was filling up over 60% of the build logs (496/720 lines).
> LoadBalancer::getConnection: using server  for group ''

Move it to further down in getReaderIndex() where it actually
tries and finds one.

Change-Id: Id572f2bb3a905e095fcc929a9e2c17f40f7b1b6e

Localisation updates from https://translatewiki.net.

Change-Id: I8af63eaa5674731fc32c5a8cbdc1df7fe1cf577f

Merge "database: Avoid use of PDO transaction system for SQLite"

database: Avoid use of PDO transaction system for SQLite

This is in preparation for changing doBegin() to use BEGIN IMMEDIATE.

PDO::beginTransaction() doesn't support any parameters for mode.
Use direct queries instead so that modes (such as IMMEDIATE) can
be used.

Since PDO's transaction system maintains state internally (similarly
to what MediaWiki Database classes do) we have to abandon use of
PDO commit() and rollback() as well (in favour of direct queries
already provided by DatabaseBase).

Bug: T89180
Change-Id: I2ad2a3c2a6d4737f5ef8822fba7cbcf5e36021f4

Merge "mediawiki.skinning: Remove italics from q tag for IE7 and below"

Merge "Made BloomFilterTitleHasLogs refresh odds linear to be smoother"

Merge "Html::openElement: Don't omit closing tags."

SpecialLinkSearch: clean up munged query variable handling

- Drop m prefix, it's silly
- Give it a default value so it doesn't complain about undefined variables

Change-Id: I378ffe304a5b29d4f35821c1f867d3a26caf5223

Fix hook situation for Skin::doEditSectionLink

The old hook supplied a way to override the HTML used for the section link,
but two extensions both trying to use it was obviously not going to work.
Deprecate it in favour of a hook that goes around gathering info to build
the initial HTML, then shoves it through the old hook for back-compat.

So that WikiEditor can add in extra URL parameters as well as VE adding in it's
own link.

Bug: T88027
Change-Id: I5a7a23709805625bdefb69cd9379be0c95acd982

Merge "DatabaseMssql: Don't duplicate body of makeList()"

Disable edit section buttons on live preview

The classical preview also does not show edit section buttons.

Bug: T89941
Change-Id: I5f9e06f33253cd155e063305b822d4ae6d155011

DatabaseMssql: Don't duplicate body of makeList()

Instead keep only the code that creates blobs for specific columns
and call DatabaseBase:makeList() to handle the rest.

This will allow the fix made in 26235c73376b to work also for MSSQL.

Change-Id: I74ae55a52fcd1dad8b857385396bd8f5d81370ba

Resources: Use OOjs UI distribution with vector→raster fallback

Bug: T87501
Change-Id: Iff139bf2601d517d91f922c033f1e3e793757943

Update OOjs UI to v0.8.0

Release notes:
 https://git.wikimedia.org/blob/oojs%2Fui.git/v0.8.0/History.md

Change-Id: If14d382beae3f42d2ffb3a43ee0c3e2a488a5fa1

Merge "Made ActiveUsers work a bit better on small wikis"

Merge "Fixed a path issue in NoLocalSettings"

Merge "Made TransactionProfiler affected-row warnings not trigger for JobRunner"

API: Unconditionally load ipb_timestamp in ApiQueryBlocks

Continuation breaks without it.

Bug: T89893
Change-Id: I7f29e7b65d49ac3764bb3584c586cbd575929592

Html::openElement: Don't omit closing tags.

Logic added with commit eefe1b13a382a7d11c2137bbf900b783ee445323
for both openElement and closeElement, removed with commit
d950da075dcb6c6dd3f68a50a0d427569350763a but only from closeElement.

Reported by Petr Bena on Wikitech-l:
https://lists.wikimedia.org/pipermail/wikitech-l/2015-February/080861.html

Change-Id: I6ed45763f7e95f7f39b73432e4f6152431891963

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

Localisation updates from https://translatewiki.net.

Change-Id: I1c6633f6b0182696cbb7341d8010965b02fb0594

Adjusting patrol checks for RC and watchlist prefs

This uses the NP patrol check instead of the RC patrol check for the
preference option that allows to hide unpatrolled new pages by default.
And this checks use of RC patrol for the user instead of the config for
the watchlist option (so that users who cannot patrol don't have it).

Bug: T89716
Change-Id: I2895a96f8c21ade83b0174d0d02d1b7e00625737

Merge "Detect duplicate archived files by SHA1 search on upload"

Detect duplicate archived files by SHA1 search on upload

- As of now, we detect duplicate files by a combination of sha1
  and file extension.
- Since multiple file extensions for the same MIME are permitted
  this method does not reliably work.

Bug: T74070
Change-Id: If13059441097799227f23ece36a96c8375f17aab

ResourceLoaderImage: tighten up svg handling

- Check $wgSVGConverter to see if it starts with rsvg, instead
  of just being rsvg. Other things like rsvg-secure are also ok.
- Make sure SvgHandler::rasterize() returned sanely before attempting
  to use the file it produces.
- Clean up temp SVG file if we return early
- Add some debug logging when rasterization fails

Bug: T89505
Change-Id: I9483c8c54a30e328565182b00d50dbf3b83076cd

Maintenance script for importing site info.

Bug: T87176
Bug: T87183
Change-Id: I3936417bc79e08cf3d04270158a6e483b5515246

Merge "Parse "retrievedfrom" and "lastmodifiedat""

Merge "PostgreSQL: Improve dropping of foreign key constraint"

Merge "skin: Use Title::isKnown() / Title::exists() instead of Title::getArticleID()"

Parse "retrievedfrom" and "lastmodifiedat"

The two system messages should allow wikitext.
The messages have been customized on many wikis to
include additional links and other HTML features.
Previously they allowed raw HTML.
Follows up Ifd696ecd and Ibcc1b499.

Bug: T86702
Bug: T87269
Change-Id: I6e67906a8c2a06c63620b0fb49226335ccb84650

Flush any implicit transactions in RecentChangesUpdateJob

Bug: T89806
Change-Id: Ibd71966225de3dfb392002fc3fd181b2ff521b9d

PostgreSQL: Improve dropping of foreign key constraint

Change I2abd650c8ce83c5b725aec054 to fix bug T76254 was
inefficient upon subsequent updates, as update.php would
first add the constraint, and then remove it again.

Remove the older code responsible for adding the doomed
constraint.

Also, make the "do nothing" output message more consistent
with other messages of the same nature.

Change-Id: Id6a05c1234fbdc5849480d13fb7d6476eac52857

Fixed backwards $group check in LoadBalancer

* Fixes 9190427

bug: T89507
Change-Id: I192ac3b00160c85e22e18d69cea5d9c46d2a301a

Made TransactionProfiler affected-row warnings not trigger for JobRunner

* The CLI check did not work since jobs can work via HTTP requests

Change-Id: I524e79fd51275777c1505e29ecf0725b998eabcc

skin: Use Title::isKnown() / Title::exists() instead of Title::getArticleID()

Some of these calls pre-date the existence of isKnown() and exists(), which
should be used instead of calling getArticleID() directly. For places where
we're just trying to determine link color, I used isKnown(), and for other
places we're trying to figure out where an article exists, I used exists().

Bug: T85550
Change-Id: I47c8af8485857cc0f3190b4ada1578ab672580af

Fix TOC anchor name collisions in edge cases

Currently, the parser adds a "_2" to the second of two identical headlines to
avoid collisions, but there's still a collision if another headline actually
ends in "_2". This change causes the new headline to also be checked for a
collision, and advances to "_3" or beyond if there is one.

Bug: T26787
Change-Id: Id0a55aa4c1917bac2f8f0d4863fcb85bd3dff1ca

Localisation updates from https://translatewiki.net.

Change-Id: Iaf0e89004c1d0358b1bad108fbbafed7f9004637

Merge "mediawiki.skinning: Merge toccolours into css rule for toc and mw-warning"

Made ActiveUsers work a bit better on small wikis

bug: T89027
Change-Id: If0fc4b51b89170e000cf42bdc3e782e42323a440

mediawiki.skinning: Merge toccolours into css rule for toc and mw-warning

Exactly the same, line for line.

Change-Id: I3076d733514665a55463f087d1eb189929d44b26

Merge "Make showDiffPage() protected"

Merge "[schema] Make column user_newtalk.user_id an unsigned int"

[schema] Make column user_newtalk.user_id an unsigned int

The primary key of the user table is an unsigned int, the column
user_newtalk.user_id refer the id and should have the same typ to store
the same values as the user table does.

No pre-deployment activity necessary; no need to sync deployment
and merge; can be done at any time (now or later)

Bug: T77005
Change-Id: I641dc1fad0a5bb14815f55e0357243a562fd672b

Merge "mediawiki.ui: button: Use the correct background color for disabled buttons"

Merge "jquery.tablesorter: testcases for the parsers"

Merge "Improve title text in jquery.confirmable"

Merge "jquery.makeCollapsible: Align toggle based on content language"

Merge "htmlform: Simplify implementation of loadDataFromRequest()"

Made BloomFilterTitleHasLogs refresh odds linear to be smoother

Change-Id: I25a4de8cf3537c9b3f27002e8bbb210322decec5

mediawiki.ui: button: Use the correct background color for disabled buttons

Bug: T88483
Change-Id: I366129bdb87cffa080f5b2f39024834ef188b64c

Document mw.libs better

Per Krinkle in IRC. This is a comment-only change.

Change-Id: Id5b8be26557683ab82b4c664e8d5271e7a32a52b

registration: Improve merging of arrays

Currently we use array_merge_recursive when merging any array, which is really
only needed for merging $wgHooks entries, and causes issues when trying to
merge default settings if the config is already set.

$wgHooks and $wgGroupPermissions are now special cased when merging, and all
other arrays are just +='d.

Bug: T88665
Bug: T89364
Change-Id: I773a9463d4428aa618c17f848c01b24e04610e95

Made EditPage avoid querying the master block table on form view

* Refactored getUserPermissionsErrors "expensive" checks flag to be
  a bit more general.

bug: T51419
Change-Id: Ic1882aa2957eed2b978761b5fc34ea9bdd8981b5

Fix URL comparison used for cacheability decision

HTTPS hits were never marked as cacheable, because
getFullRequestURL() (https://...) was being compared to
getSquidURLs() (which are all http://...).

Instead, expand the request URL with PROTO_INTERNAL and
use that for the comparison.

Bug: T89673
Change-Id: I3402820613bc6839fa2e2d053ddffcbf1b17ddb5

Localisation updates from https://translatewiki.net.

Change-Id: I063611d4a2759d3d367175068f8e2de90717d2a6

test: Remove several jscs over-rides and make the code pass

Rules newly instated from the coding standards:

* disallowKeywordsOnNewLine
* disallowImplicitTypeConversion
* requireLineBreakAfterVariableAssignment
* requireSpaceAfterLineComment
* validateIndentation

Change-Id: I3d1845b1ed68037b6a4275ba9f333c3ccc36359a

Improve title text in jquery.confirmable

Don't clone the title text for the "No" button, and make the title text
customizable for both yes and no buttons.

Bug: T89572
Change-Id: I7e1e5c6ee78aa54a97205d6682f9520008cc11dd

Merge "mediawiki.mixins.less: Do not serve SVG to Opera 12 when fallback available"

mediawiki.user: Clean up crypto version of generateRandomSessionId

Follows-up 4860ea3ca6.

* Documented how the byteToHex padding works.
* Move for-statement to after var and function declarations.
* Use permalink for git url (master will change, making the link
  less useful).
* Remove dead ", r" comma statement.
* Substitute 0x03 to match the other 3.
* Use 8 instead of arr.length. (Matching the other loops.)
* Use jQuery from closure instead of global $.
* Use $.trim instead of str.trim (new in ES5).
* Add test to assert consecutive return values are different.

Change-Id: I9f59cf60316091e435e4bc9dbd700b9c6e431dac

mediawiki.mixins.less: Do not serve SVG to Opera 12 when fallback available

We support it as a Grade A browser, which means that we should do our
best to provide the best possible experience. In my opinion, the best
possible experience involves not giving it any SVGs.

Opera 12 has issues when rendering SVG background-images together with
border-radius or background-size rules (see task T87504 for details
and examples), and both of these are becoming increasingly common in
our codebase.

Uses in CSS are not worth changing, as they are probably the simple
cases where it works correctly anyway.

Bug: T87504
Change-Id: Icdc2e1dcbcbc356042a064d703e1fe469004280b

Merge "Chunked upload: Return expected offset on offset error"

Merge "Make HTMLTagFilter disappear totally when $wgUseTagFilter = false"

Make HTMLTagFilter disappear totally when $wgUseTagFilter = false

Previously the label displayed but the input field didn't.

Bug: T78432
Change-Id: Ia84ac611f76c98495a5b3f90c0ce01b6e1e404a3

Use both Latin and Cyrillic for Uzbek in Names.php

Change-Id: I19c3dcae6ea5900a492a963f8ac9aa0b7a43fab7

Merge "update.php: Add option to not check if external dependencies are up to date"

Localisation updates from https://translatewiki.net.

Change-Id: I7dc4a94ec0977157d89cc73f0b550afaa1d87180

mediawiki.skinning: Remove italics from q tag for IE7 and below

The q tag is stylized as italic using the star hack for IE7 and below.
Since those browser do not support before and after pseudo elements
(and so, quotation marks cannot be added in), a more appropriate
styling is to just leave the default styling for quotations as is.

Bug: T89595
Change-Id: I382a9268d2d98fbaed6c46081b09152be8d6670c

jquery.makeCollapsible: Align toggle based on content language

The collapsible toggle button is in user interface language but the alignment
of this button should fit to the direction of the content language.

Change-Id: I31927b8e091bdf81458c7d3d0cd5f6040af5f0f4

Check use of RC patrol for watchlists

This checks use of RC patrol instead of NP patrol to determine when to
display show/hide patroled edits in watchlists. RecentChanges already
does this, and there's no use in displaying a setting that only affect
new page creations in the watchlist. The filtering cannot work anyway
since ChangesListSpecialPage checks for RC patrol on its own.
Bug: T76054
Change-Id: I9c587f56297881d58551ee302c4de01e9452b960

Pass the right variable to recordQueryCompletion()

Change-Id: I0d75ed7dd55e40b63bd823ec43b8d3a8f5bed53b

test: Don't run the npm pipeline for the now-empty skins directory

Change-Id: I426835c83c0dd73fa7247ed2ec299cdec1e43fcb

Merge "Replace dev.w3.org with more permanent or stable urls"

Merge "Avoid undefined index when query old block logs with api"

Replace dev.w3.org with more permanent or stable urls

* Sanitizer: dev.w3.org/html5/spec-preview
  Follows-up 8e8b15afc6.
  Use stable reference to www.w3.org/TR/html5 instead (currently
  from October 2014) instead of an old preview branch from 2012.

* parserTests: dev.w3.org/html5
  Follows-up 959aa336a1.
  Url is now a dead end. Replaced with link to a draft from around
  that time. The relevant section no longer exists in the curent
  spec as it got split off into a separate spec. Maybe this one:
  https://url.spec.whatwg.org/#percent-encoded-bytes

* Parser, HTMLIntField: dev.w3.org/html5
  Use stable reference to www.w3.org/TR/html5 instead.

* HTMLFloatField.php: dev.w3.org/html5
  Url is now a dead end. Draft from around that time:
  http://www.w3.org/TR/2011/WD-html5-20110525/common-microsyntaxes.html#real-numbers
  The section "Real numbers" no longer exists in the current spec,
  but the Infrastructure chapter has a section on floating point
  numbers that describes the same sequence now.

Change-Id: I7dcd49b6cd39785fb1b294e4eeaf39bda52337b2

Merge "Handle invalid grouppage on Special:Statistics"

mediawiki.ui: Fix .mw-ui-input padding to be centered vertically

Changed the padding information in
resources/src/mediawiki.ui/components/inputs.less
from .4em .3em .2em .6em to .3em .3em .3em .6em
so that vertical spacing is centered properly.

Bug: T85284
Change-Id: I1e90e3f1922ebee205f6101e186ddb4ba7c19c10

htmlform: Simplify implementation of loadDataFromRequest()

Change-Id: I30d1c186c9e3eb67130b798af987f275316828de

Merge "Add .list-style-image-svg"

Merge "Using cryptoAPI if available in generateRandomSessionId"

Merge "Update MWLoggerMonologHandler for Monolog 1.12.0"

Update MWLoggerMonologHandler for Monolog 1.12.0

Monolog 1.12.0 "fixed" Handler::isHandling() so that instead of a full
log record it is only passed an array with the log event's level.
MWLoggerMonologHandler was relying on a full record to allow inspecting
the channel name and looking for a 'private' flag in the context
information.

Update MWLoggerMonologHandler to do legacy processing checks in
Handler::write() where the full log event is present for inspection.

Bug: T89313
Change-Id: Ia878c2cb6bff47d6b35ff38ba3b7ac2ea5556565

Merge "mediawiki.cookie: Implement expiry option as time from now"

Merge "Add File::getDescriptionTouched() method"

mediawiki.cookie: Implement expiry option as time from now

The current option (Date object) isn't user-friendly as it requires
users to construct a Date and set the time offset accordingly.

They likely do that using an offset, not an exact date. Optimise
for the common case by supporting this directly (similar to the
server-side cookie expiration logic, as well as our ObjectCache
classes and other systems).

Support for exact dates is kept as this is still useful for the
native transport layer where timestamps are used. But the API
surface now supports durations as well.

Note that there was an undocumented fall-through of options.expires
as number of days. This is implemented by upstream jquery.cookie
and mw.cookie previously didn't recognise or reject this value so
it fell through.

Change-Id: Ia2e51d3a3e4d19484367cb74003c4f756c87345d

Add File::getDescriptionTouched() method

Fetch page_touched timestamp of file description page via a DB query
(possibly to a foreign DB).

Not sure about the performance implications; could add a memcached
layer on top, but the same DB lookup was already done for local
files.

Bug: T88648
Change-Id: I891c806aab235ff2c0e73c98b06b64fbe78e1517

Merge "Fix magic getter for $status->ok"