Make Special:Redirect work for blocked users and in read-only mode

Bug: T115570
Change-Id: Id26af6efe9dc7eddb2066a2f61c86116305fff7c

Merge "objectcache: Rename WANObjectCache TTL_NONE to TTL_INDEFINITE"

Use specified pageId for LinksDeletionUpdate→DeleteLinksJob

Not page->getId() which may now be 0 due to the page having been
deleted.

Bug: T115586
Change-Id: I5a3482bbe1df23dda0f33e7d675b06d616395bb6

Added a parenthetical to fileextensions description

Changed "fileextensions for more clarity of the description.

Bug: T62121
Change-Id: Ida361a370069e6d5565a5ef5b03f0976dfd3515b

objectcache: Rename WANObjectCache TTL_NONE to TTL_INDEFINITE

"None" has a somewhat unobvious meaning, also ambiguous with "uncachable".

Change-Id: I799de385427aeb8f581d51800606552bdc998252

Localisation updates from https://translatewiki.net.

Change-Id: I821a5724c2f14c6d1697d60454cd17618998e2bd

Re-enable last phpcs rule

Bug: T102609
Change-Id: I49eb778ccb5f0a3662fe4573872d1235c52fdd81

Merge "Move comment in backup_PageTest.php"

Merge "Fix array index spacing in ResourceLoaderEditToolbarModule"

Merge "Change some comment style in userOptions.inc"

Merge "Add curly braces to while"

Merge "Remove empty comment line from GlobalTest.php"

Add $suppress to ArticleDelete hook

We should be telling extensions whether a deletion is a suppression or not, so
they can behave appropriately.

Change-Id: I2cb6ffd61dd12766fe0266514c9360ff0c90b788

Use the frame 'type' in wfFormatStackFrame, like MWExceptionHandler::prettyPrintTrace

Instead of assuming '::'

Change-Id: I2452bc5ebd3716eb61f53da41a0050f4833fb5a4

Mark Title::setFragment() as @private

It is deprecated for public use, so indicate that it should only be used
privately.

Change-Id: I1b32f52f427a36ddb81bd7befadc9099ebc4f191

adding 7zip archives to the MIMEs

Change-Id: Ib3392b7fcec475c153099a1b92fe195c5eb4b195

Handle $frame['function'] not being set in stack frames

Bug: T115107
Change-Id: I9fbfcd53fbcba1800a924acb1a1702c3c9037cc5

Merge "Add recentChangesLine to ChangesList"

Use correct @codingStandardsIgnore in DairikiDiff.php

@codingStandardsIgnoreFile is for whole files, use
@codingStandardsIgnoreStart instead

Also correct a MediaWiki.WhiteSpace.SpaceBeforeSingleLineComment.
SingleSpaceBeforeSingleLineComment sniff in the same file

Follows I06cdab4616b5bff47c85152df28f18c861730a23

Change-Id: I75e6e936e2fb4453bd56848ff39cabd92ae171fc

Remove empty comment line from GlobalTest.php

Remove empty line comments as found by the
MediaWiki.WhiteSpace.SpaceBeforeSingleLineComment.EmptyComment sniff

Change-Id: Ibd60165ae8ddef2a2fd443856e7476f32a0c6392

Fix array index spacing in ResourceLoaderEditToolbarModule

Code style is not using spaces in array index

Follows Ic3eab71691e502bfe19bdf4eb6f82cc679a7782f

Change-Id: I0e20787c41d2edbc2899308647092ed6831d2489

Add curly braces to while

Follows Iaa1a479b5eefcae0fc8f82b1d20e9575d609c0e9

Change-Id: I7f2b9b12e3282f5b8ba54a315fa12f1a97985572

Revert "Add // @codingStandardsIgnoreFile to FormatMetadata"

For testing.

This reverts commit df706d2457e675ce3896b6b2aace27dac3fb9ff1.

Change-Id: If63199293ac3539030f3503fbcbd5e0f4aa2eece

Merge "Show a confirmation message on Special:UserRights"

Change some comment style in userOptions.inc

Change-Id: Ida429683ee5b6194c96d782cf5fb4e0527c89103

Move comment in backup_PageTest.php

Move the comment at the begin of the function out of it to a doc
comment.

Change-Id: I6febf51a74de88204d54b1e3251df187197c1819

Merge "Change some line comments to multi line comments"

Merge "Remove empty line comments"

Remove empty line comments

Remove empty line comments as found by the
MediaWiki.WhiteSpace.SpaceBeforeSingleLineComment.EmptyComment sniff

Change-Id: I5d694f7a7d3bc97e16300ba03c60ad17f3c912a5

Change some line comments to multi line comments

This allows empty lines inside the comment as found by the
MediaWiki.WhiteSpace.SpaceBeforeSingleLineComment.EmptyComment sniff

Change-Id: Iac155bbda4a84562db2b452baeae9b8973899453

Merge "Move the "shit" out"

Move the "shit" out

There is no need for such words

Change-Id: I0294c0e2381b96fd4377d7edcdf4d7e7fc0d06d3

Show a confirmation message on Special:UserRights

If the rights was saved, show a Special:Preferences-like success-box on top
of the page, which indicates, that the user rights was saved successfully.

Bug: T103361
Change-Id: I19646e9a1b0b31b4fbfe0fa2c8f775f8a2a67a83

Merge "Revert "mediawiki.widgets: Add temporary workaround for upstream oojs-ui bug""

Move release notes about mw.Upload and friends to 1.26

This was originally meant for 1.26 release, but we didn't quite make
the deadline and currently 1.26 contains a half-broken implementation
of some parts. We're going to be backporting all the additions and
fixes since branch cut to 1.26.

Change-Id: I24100009d8a0c7c691b7ef44f33bf42f093a514a

User: Mostly remove password handling

AuthManager is coming, which will make it easier to add alternative
methods of authentication. But in order to do that, we need to finally
get around to ripping the password-related bits out of the User class.

The password expiration handling isn't used anywhere in core or
extensions in Gerrit beyond testing for expired passwords on login and
resetting the expiry date on password change. Those bits have been
inlined and the functions removed; AuthManager will allow each
"authentication provider" to handle its own password expiration.

The methods for fetching passwords, including the fact that mPassword
and other fields are public, has also been removed. This is already
broken in combination with basically any extension that messes with
authentication, and the major use outside of that was in creating
system users like MassMessage's "MediaWiki message delivery" user.

Password setting methods are silently deprecated, since most of the
replacements won't be available until AuthManager. But uses in unit
testing can be replaced with TestUser::setPasswordForUser() immediately.

User::randomPassword() and User::getPasswordFactory() don't really
belong in User either. For the former a new PasswordFactory method has
been created, while the latter should just be replaced by the two lines
to create a PasswordFactory via its constructor.

Bug: T47716
Change-Id: I2c736ad72d946fa9b859e6cd335fa58aececc0d5

Merge "Remove unused mw.Upload.BookletLayout messages"

Merge "Do not double load gallery styles now cache has expired"

Remove unused mw.Upload.BookletLayout messages

'upload-process-error' and 'upload-process-warning' are no longer
in use since 3e596a9185724793bb8ff4e47373a9d75c17914a.

Change-Id: I6af1bc962901d5be89584795a5521be830199ab4

Do not double load gallery styles now cache has expired

This reverts commit 4a9aab29c1ba0725271898bd60138336f17fa67e.
now that galleries are rendered correctly on category pages.

Dependency: I32697c2c65824d7622c
Change-Id: I0ef5c2d5a9019355aa3a158f9e489df02975e0af

Update OOjs UI to v0.12.12

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

Change-Id: If241c883b8c079ea9f1d73c8eb6f51c99dd615ac

Revert "mediawiki.widgets: Add temporary workaround for upstream oojs-ui bug"

No longer needed.

This reverts commit ca751872aac2be673d88fe9067472808c6500bb5.

Change-Id: Iefa3386d448826a220fe0e861fe38fc9f1809d4d

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

Merge "jquery.tablesorter: use of expand-child class"

Localisation updates from https://translatewiki.net.

Change-Id: I3a2e3736065603ab00a1a29703c0f391c6531a06

jquery.tablesorter: use of expand-child class

Rows with class expand-child are now skipped in detectParserForColumn().
This is necessary after change I5180296.
Add a test for expand-child class.

Bug: T114721
Change-Id: I88a01208889fbb461b1f42855ed6494d479440fa

Add recentChangesLine to ChangesList

This method is implemented in all sub classes.

Not having this method here looks odd as
ChangesList::newFromContext returns a ChangesList
and parts of the code base then call recentChangesLine
on that object which may not exist..

In the future we might even have some interface here?

Change-Id: Iad00a956862c078a2bcaf3ef0602abcf3fedb7d2

Merge "Add missing scope to ChangesListSpecialPage methods"

Merge "Of course EmailNotification::updateWatchlistTimestamp returns users ids"

Of course EmailNotification::updateWatchlistTimestamp returns users ids

Change-Id: I04b2a8f97527c2bcd3f993b60e4100915b7f4c18

Use explicit methods instead of the jQuery constructor's second argument

As recommended by
https://www.mediawiki.org/wiki/Manual:Coding_conventions/JavaScript#Pitfalls:
"As of jQuery 1.4 the jQuery constructor has a new feature
 that allows passing an object as second argument, like
 jQuery( '<div>', { foo: 'bar', click: function () {}, css: { .. } } );.
 Don't use this. It makes code harder to follow, fails on attributes
 (such as 'size') that are also methods, and is unstable due to this
 mixing of jQuery methods with element attributes. A future jQuery
 method or plugin or called "title" might convert an element into a
 heading, which means the title attribute can also no longer be set
 through this method. Be explicit and call .attr(), .prop(), .on()
 etc. directly."

Change-Id: Iaf456b4b76dd4fc260dad0e4c0ec8f2976b59b83

Merge "mw.ForeignUpload: Provide an edit summary to help track these uploads"

Merge "mw.ForeignStructuredUpload: Use the right license template for Commons"

mw.ForeignUpload: Provide an edit summary to help track these uploads

The summary is "Cross-wiki upload from xx.wikixedia.org". We don't
know the content language of the foreign wiki, so it's not translateable.

Bug: T115328
Change-Id: I4b36cad20ffe318f9a01126a5b9ed70f5d806b57

Merge "mw.ForeignStructuredUpload: Use '{{own}}' template for 'source' field"

mw.ForeignStructuredUpload: Use the right license template for Commons

When uploading locally or to different repo, keep the field empty,
since we don't really know what licenses are appropriate (and we don't
name a specific one in the UI).

Change-Id: I5c8f55653dae42e47ac696d44a7b2082adf6d6f2

Merge "Caching documentation tweaks and doxygen cleanups"

mw.ForeignStructuredUpload: Use '{{own}}' template for 'source' field

Change-Id: Ib939b6c8b0ae3afdb93465721de8de995687b81a

Merge "OOjs UI: Backport aadaa8a1, 9aba218a"

Merge "jquery.tablesorter: Match clobbered iso dates and fix edgecases"

Merge "Added autopatrol parameter to MarkPatrolled and MarkPatrolledComplete hooks"

OOjs UI: Backport aadaa8a1, 9aba218a

Important tweaks for mw.widgets.CategorySelector, which will rise to
prominence soon when we deploy mw.Upload.BookletLayout in VisualEditor.

Bug: T114945
Bug: T115010
Change-Id: I4236a7c64469faa7476bcab9bd0a0f8497101fe2

Merge "Update comment about enabled extensions"

Update comment about enabled extensions

The comment message given in enabling the extensions
are based on for the earlier versions of mediawiki which
uses the requireonce() and as at present all the versions
since 1.25 uses wfLoadExtensions() the comment message
have been changed accordingly.

Bug: T100940
Change-Id: I915a4893cc2d89624c9d8dd6504d7dc8bf166cd9

Merge "Clean up zh-tw.json"

Add missing scope to ChangesListSpecialPage methods

Change-Id: Ia4d916a06178eef235c764703c9c4e65133b45df

Merge "build: Upgrade grunt-banana-checker from v0.3.0 to v0.4.0"

Added autopatrol parameter to MarkPatrolled and MarkPatrolledComplete hooks

Change-Id: Ibc832ba8cec8eaef673ef07534c5522f678e8f58

Merge "mediawiki.Upload.BookletLayout: Handle errors from TitleBlacklist (HACK)"

mw.Upload.BookletLayout: Correct some message parameters

* 'fileexists', 'filepageexists': Requires prefixed page name.
* 'api-error-duplicate', 'api-error-duplicate-archive': Takes the
  number of duplicates, not name of duplicate. The list of duplicates
  is supposed to follow this message.

Change-Id: Idfa978469f4a99526100f6c24a67bf18abf7ec89

mw.Upload.BookletLayout: Use appropriate messages for warnings

...and properly handle the 'page-exists' warning.

Bug: T115056
Change-Id: I5c393ee034e100d4fa415e7e215920d8af465a78

mediawiki.Upload.BookletLayout: Handle errors from TitleBlacklist (HACK)

This code really really shouldn't be here, or in fact shouldn't be
needed at all. But we're currently offering really awful user
experience, and given how expansive Commons' title blacklist is, we
have to do better now and not when we get to rewrite everything.

Since the default TitleBlacklist error message really sucks and
Commons replaces it with something that jqueryMsg can't handle, reuse
a message from UploadWizard, `mwe-upwiz-blacklisted`, duplicated here
as `api-error-blacklisted`.

Bug: T114940
Change-Id: I8a7723abde124e8ff21e404dfc2efb8506f8dd75

mw.Upload: Refactor error handling for the umpteenth time

When I started, I just wanted mw.Upload.BookletLayout to be able to
display more information about errors than just the default message
(like it can do for warnings). And down the rabbit hole I went...

mediawiki/api/upload.js:
* Simply throw errors when our methods are called with bad parameters,
  rather than return a rejected promise.
* Always call .notify( 1 ) when upload is complete, regardless of
  whether it succeeded or failed.
* Reject promises with error code and error details, for consistency
  with api.js. Previous behavior meant that we did not let callers
  know the details in some cases. It was also problematic when we
  passed-through promises rejected in api.js (which had different
  parameters given).
  * Made some effort to return sane codes when something intricate
    fails in iframe upload, but no guarantee that this works well. The
    codes are inspired by what api.js returns in similar circumstances.
  * When rejecting because of warnings, use the first warning's key as
    error code.
  * Always ignore the warnings when uploading to stash and 'filekey'
    is present in response, never ignore when uploading directly.
* When the upload succeeds, never check for 'result.upload.error'
  (which just isn't a thing) nor for 'result.error' (which api.js
  detects and rejects the promise before we get to it). We only need
  to check for 'result.upload.warnings'.

mediawiki.Upload.js:
* Update for the above changes in mediawiki/api/upload.js.
* More reliably distinguish warnings from errors in all cases, not
  only when finishing a stash upload.
* Store machine-readable error codes, not mw.Message objects. This
  lets callers do something sensible when we encounter an unknown
  error (especially one that has no corresponding message).
* Store full result as state details for warnings, as well as errors.

mediawiki.Upload.BookletLayout.js:
* Update for the above changes in mediawiki.Upload.js.
* Give errors/warnings generated during upload to stash the same
  loving treatment as errors/warnings during publishing.
  * Extract the code to a new method getErrorMessageForStateDetails().
* Handle 'stashfailed' warning (which is really an error).
* Handle unknown errors, now that mw.Upload lets us do something
  sensible with them. (See, this is the thing I set out to do.)

Bug: T114940
Change-Id: I4c0f619a4e483cca296c2fa2907ed1f81a99fdd6

Localisation updates from https://translatewiki.net.

Change-Id: Iba14673f9c6fb330e84b0777670dbeb7db88714e

build: Upgrade grunt-banana-checker from v0.3.0 to v0.4.0

Take opportunity to drop no-longer-needed options.

Change-Id: I11250c12d2123d4729e12cd0dce07b24fc5008e5

Merge "Improve error message for missing content model on import"

RELEASE-NOTES-1.27: Add mw.Upload, its dialog, booklet layouts and subclasses

Change-Id: Ie010a9ba2be9a7fde9a73e6e62d731874e862876

UploadBase: Return 'was-deleted' warning in addition to 'exists-normalized', not instead of

The 'was-deleted' warning was generated by getExistsWarning(), which
was returning immediately if this was found to be the case. A bunch of
later checks were incorrectly skipped, in particular 'exists-normalized',
which was resulting in UploadWizard incorrectly ignoring that problem.

I'm not sure why that was part of getExistsWarning() at all, it
doesn't seem very relevant. For that matter, neither do the 'thumb',
'thumb-name' and 'bad-prefix' warnings that it also generates, but
this should not be a problem in practice and so I'm leaving them alone.

Other than by allowing some more warning types to appear together or
in different order, this should not affect action=upload API output or
Special:Upload (which was updated appropriately). It does affect
'action=query&prop=imageinfo' output's 'html' property (used for AJAX
checks on Special:Upload), which no longer includes the 'was-deleted'
warning; this was never specified anywhere and just a side-effect.

Bug: T48741
Change-Id: I3686ee8ffd635f5f06f51971b6f16e3e66f33a9e

Merge "mw.widgets.CategorySelector: Indicate pending requests and abort useless ones"

Merge "Add abort method to mw.api"

Merge "mediawiki.jqueryMsg: Support {{ns:}}"

Merge "Show different message if image thumb in different format than img"

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

Merge "Enable Squiz.Classes.ValidClassName.NotCamelCaps"

Localisation updates from https://translatewiki.net.

Change-Id: I257ed32a1323008f5a27f548c497e08407f97302

When using fopen on https streams, disable weak ciphers and compression

Per recomendation of
http://www.docnet.nu/tech-portal/2014/06/26/ssl-and-php-streams-part-1-you-are-doing-it-wrongtm/C0

Change-Id: I69d063ff4aa4248dd4f3d03de5a168c4b5a99c50

Localisation updates from https://translatewiki.net.

Change-Id: I8dd69f55dd50f97819b21574a80b6f925e9c1680

Show different message if image thumb in different format than img

Users on commons feel that the other resolution links on the image
page can be confusing to users if they are in a different format
than the original image. Use a different message in that case
to clarify the situation.

See discussion at https://commons.wikimedia.org/?oldid=165672017

Change-Id: I76d7155e1c1e48a9824819135943d2385a0ee41a

Enable Squiz.Classes.ValidClassName.NotCamelCaps

Change-Id: Ie413b403992ba8598b3dea03adc9733c30414fe9

Merge "Use line comments for @codingStandardsIgnoreStart"

Merge "Fix class name of WfEscapeShellArgTest"

Merge "Change 1.26 to 1.27, mostly in doc comments"

Clean up zh-tw.json

Change-Id: I8930136a2feabdd5ce229e31927678f01bc53fab

Merge "Clear previously left-over PQresult before calling pg_get_result()"

Clear previously left-over PQresult before calling pg_get_result()

Bug: T52091
Change-Id: Iaa1a479b5eefcae0fc8f82b1d20e9575d609c0e9

OutputPage: Fix typo in documentation

Change-Id: If89f249de3049eb37c261a7417910637b7464961

Merge "Remove outdated style for span.minor"

Merge "Skip user namespace prefix for rights log entry"

Merge "Adjust file name in style module after rename"

Improve error message for missing content model on import

I was initially going to refactor out the error message into Import.php,
but it quickly became apparent that WikiImporter's error handling needs a
LOT of work. In particular, to localise the error message into the user's
language is sadly non-trivial.

Although not used currently, the MWUnknownContentModelException subclass
will help with error handling improvements in the future.

Bug: T49270
Change-Id: I9f53c9d6a8a2ea842cb2ba94d4131e10a8b08f5d