Merge "Added autopatrol parameter to MarkPatrolled and MarkPatrolledComplete hooks"

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"

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

Adjust file name in style module after rename

Follows I65b9343ea002c332323ededf4e328e4463faf698

Change-Id: I436f80041a8b30964755040a699166f182c1736c

Remove outdated style for span.minor

There is no span.minor on pagehistory to style. It is called .minoredit
and already has the style.

Change-Id: I832b394e475104394700fe215b218115220d6c61

Add process cache support to WANObjectCache

* Make getWithSetCallback() accept a TTL field for this
* Make ChangeTag callers use this flag to avoid hundreds of
  duplicate queries at Special:Tags

Change-Id: Ic1ed28294f5d557e02f39a7f20d36766244b9ded

Skip user namespace prefix for rights log entry

The target of the right log entry is shown with namespace prefix, but it
is always a user, so skip the namespace prefix.

Change-Id: Ibe6005cc70ff3933836166d97c544267fb7d9951

Merge "Minor getWithSetCallback() example formatting tweaks"

Merge "Fix broken min() call in OutputPage"

Fix broken min() call in OutputPage

Change-Id: Ia2d895371f6f114c1cb821dd1d2aaaef7935033f

Fix more LBFactory __construct() IDEA errors

Change-Id: Iec0d108d6ad405f77710e4ce111d0adefea3832b

Merge "Add ObjectCache::getMainClusterInstance()"

Merge "Set correct parentid on import"

Add ObjectCache::getMainClusterInstance()

Provide a complement to ObjectCache::getMainWANInstance() and
ObjectCache::getMainStashInstance() which gets the default ObjectCache
instance.

Change-Id: Ib16ee40908b159e60be41a308db49a9291b5de0d

Better explain WANObjectCache::TTL_NONE constant

Change-Id: I2ce363c957f35e75a17f05b2160979a6fa4aa642

Fix "installer started" detection

NoLocalSettings.php tries to detect whether the installer has been
started or not. But this detection has been broken since I4cf237d when a
change was made to stop setting the session name when NO_SESSION is
defined, causing NoLocalSettings.php to be looking at a different
session-cookie than is actually being used by the installer.

The complete fix is twofold:
* Have WebInstaller::startSession() call session_name()
* Have NoLocalSettings.php not call session_name() when PHP's
  session.auto_start configuration setting is set.

Change-Id: I618d11df902b5d1f70e175bc94137621e9195c2f

Localisation updates from https://translatewiki.net.

Change-Id: I630ffad122995402233190be172a5f250c4f24d2

Merge "mediawiki.jqueryMsg: Strip leading colon in wikilinks"

Set correct parentid on import

When importing over an existing page the parentid is set to the latest
rev id of the page, which makes the size diff in history unusable.

The import constructs the Revision object without a parentid and than
Revision::getPreviousRevisionId is using the page_latest field to
propagate the missing parentid.
Avoid this bad propagate by select the previous revision id depending on
timestamp before construct of the Revision object.

Bug: T114806
Change-Id: Iee44d5a74de459f733ea62373cdbe9911e77083f

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

* When the input loses focus, or when we fire new queries, abort
  any current ones.
* Never display the menu after input loses focus.
* Use PendingElement to indicate that we're waiting for responses.
  (Depends on I604fff9a9e5bfbb584b3926802dab445e6131aaa in OOjs UI
  for correct styling.)
* Do not send needless queries for empty input.

Bug: T114945
Change-Id: I80f4d0143279f1768ed2a3bdcb3f731526597577

Add abort method to mw.api

The abort method allows you to cancel any requests currently pending
in the current class. Useful for cancelling no longer needed requests
due some user action (for example a user switching back to edit mode
from preview or a user cancelling multiple searches across different
lookup widgets)

Bug: T111245
Change-Id: Ie614b05fbfbddca38ea201e90053bebdd58da949

mediawiki.jqueryMsg: Support {{ns:}}

Often used in links like [[{{ns:template}}:Foo]], in order for the
link to use namespace name in wiki content language rather than
localised to user language.

Change-Id: Icc360724c75755a6ffc31792d8439cba142642ae

mediawiki.jqueryMsg: Strip leading colon in wikilinks

Strip leading ':', which is used to suppress special behavior in
wikitext links, e.g. [[:Category:Foo]] or [[:File:Foo.jpg]].

Also added tests for empty wikilinks.

Bug: T114130
Change-Id: Idc78310d6106c2badc0fde933f901d672556ab26

Merge "LocalisationCache::recache(): Use array_fill_keys()"

Minor getWithSetCallback() example formatting tweaks

Change-Id: I6fc7164770fbbd32374b01d1aa27dbaa9263c940

LocalisationCache::recache(): Use array_fill_keys()

LocalisationCache was added in 1.16, so in order to retain PHP 5.1
compatibility, array_fill_keys() was avoided. In 1.17, support for
PHP 5.1 was dropped, so we can use that function now.

Change-Id: I435705639f1a470324a4ba46153351aadc0d40e2

Localisation updates from https://translatewiki.net.

Change-Id: I02f0defd61aa359a17b6d5cfc75c9e6db0149124

Merge "Implement ApiPageSet::setRedirectMergePolicy()"

Implement ApiPageSet::setRedirectMergePolicy()

This allows generator implementations to define how generator data
about a redirect gets merged into the redirect destination.  This
does not change any defaults, only modules that explicitly set a
merge policy will have a change in their results.

This functionality is initially used for the search api modules to
retain search positions within the final result set.

Bug: T92796
Change-Id: If2f49e0fc3176288c95e870240754ee320a6bf91

Merge "Consistent TODO spelling in JavaScript"

Merge "Change "Subject/headline" to "Subject""

Merge "Fix namespaced class references for Doxygen"

Merge "mediawiki.Upload.BookletLayout: Add setFile method"

Merge "Fix French special pages titles inconsistencies"

Merge "Fixed Special:ChangeEmail spelling in French"

Fix namespaced class references for Doxygen

Doxygen doesn't quite grok PHP's use of the reverse solidus (backslash)
character as a namespace separator. The C++ based parser it uses needs
them to be escaped in comments just as if they were being used in
a literal string context in PHP.

Change-Id: I9aff9dd0fb74a95039da1091c2f247cf71fd085a

mediawiki.Upload.BookletLayout: Add setFile method

Change-Id: I30ec98be0e28e65d84b57cb78c9a89dd82d19c30

Merge "mw.ForeignStructuredUpload.BookletLayout: Reset forms after use"

Merge "mw.widgets.DateInputWidget: Close calendar after a date is picked"

Merge "Add get() flags support to BagOStuff::getWithSetCallback()"

Merge "Update getWithSetCallback() callers to the newer signature"

Merge "Update getWithSetCallback() examples to use the new signature"

Merge "Handle edge case in WikiPage::lock()"

mw.widgets.DateInputWidget: Close calendar after a date is picked

It guess it makes sense to do this. I can't think on any use case in
which the user would want to keep clicking on random dates on an open
calendar. It could be helpful if it were a filter for some
visualization, but in that case we should probably have a config to
keep the calendar always open.

Bug: T114942
Change-Id: I1c81464a1835eb488a75920334a3cf3b92ef34fc

Merge "Effectively remove EDIT_DEFER_UPDATES flag"

Merge "Various WikiPage code cleanups"

Merge "Move SquidPurgeClient under /clientpool"

Merge "Convert updateChunkStatus to using a flush commit()"

Merge "Made LinksUpdate on edit use the job queue"

Merge "Show password-required message only if $wgRequirePasswordforEmailChange = true"

Remove .php5 entrypoints and $wgScriptExtension

* The '.php5' entrypoints were deprecated in I68b1ae842, $wgScriptExtension
  in I3690f78bc.
* Drop the associated ResourceLoader configuration variable, too. `mwgrep`
  shows no usage in the MediaWiki namespace.
* Keep the scriptExtension configuration parameter for FileRepo for people who
  would like to interoperate with older MediaWiki installations that still use
  '.php5'.

Change-Id: I17c8a15484b7e82cd5970d34e688109a2aae3840

Merge "Allow change tags to be added via API action=delete"

mw.ForeignStructuredUpload.BookletLayout: Reset forms after use

Bug: T114936
Bug: T114947
Change-Id: I96a6440c32316a077069bb744fff8564e91a04b2

Merge "Provide a default "max lag" value for LoadBalancer"

Provide a default "max lag" value for LoadBalancer

* No MediaWiki site can tolerate unbounded slave lag
  without editing and account creation and such being
  broken and unusable
* Also fix a few trivial __construct() IDEA errors

Change-Id: I95b8b73be2df3b5a87e7adbf53988cfbeb85c260

Made LinksUpdate on edit use the job queue

* LinksUpdate is now an EnqueueableDataUpdate
  and can yeild a prioritzed refreshLinks job.
* DeferredUpdates::runUpdates() now takes an enqueue
  flag to try to use jobs. This is set in restInPeace().
  Updates that change many links will be less likely to
  increase lag, as the runners are more strict about that.
* Also made the LinksDeletionUpdate job enqueue happen
  post-send on page deletion for consistency

Bug: T95501
Change-Id: I8863caef9c8f03234699d33e4d47d2310a0c8446

Merge "Make ForeignDBRepo::getDBFactory() actually work on 5.3"

Merge "Make DeferredUpdates::doUpdates use DataUpdate::runUpdates"

Make ForeignDBRepo::getDBFactory() actually work on 5.3

Change-Id: Id790ccb1728a6a163f2e2149dd56ec1b14fc33af

Move SquidPurgeClient under /clientpool

Each class has its own file now too.

Change-Id: I11593d6efbfce8e3981895e84edb4e0dea3998e4

Allow change tags to be added via API action=delete

ApiDelete::delete() and ApiDelete::deleteFile() are altered from public
to protected visibility. This is not believed to affect any extensions.

Also get rid of the token variables, as they are not doing anything useful.

Bug: T108564
Change-Id: I6143043dafa6425a0df4baefe970620cbe69466c

Make DeferredUpdates::doUpdates use DataUpdate::runUpdates

All DataUpdate objects will be managed by runUpdates() while
the other DeferrableUpdate objects will still be run here.
This respects the transaction sematics of DataUpdate a bit more.

Change-Id: Ia0d2dd26a38b0e8911589407b533b58d04d2b084

Effectively remove EDIT_DEFER_UPDATES flag

* All DeferrableUpdates will be deferred as normal.
  In CLI mode this is just post-commit and would have
  happened before the doUpdates() call anyway. For web
  requests this will be at request-end (post-send on HHVM).
  Note that deferral was already the default for EditPage.
* Previously, it was easy to accidently not set EDIT_DEFER_UPDATES.
  Deferred updates are supposed to be deferrable by definition,
  and there is no longer the problem of them piling up in CLI in
  recent versions. Not deferring mostly just forfeits performance.
* Also, it is not really the responsibility of WikiPage to be
  running DeferrableUpdate instances itself.

Change-Id: I2bc47b82635e0a24b1df5d502ba66f6de737697e

Merge "SpecialEditWatchlist: Use 'parentheses' message instead of hard-coded ()"

Small clean-ups for OutputPage::userCanPreview

Change-Id: Ibd973750b60cbcc8d1289686de6cabcfdca5c5d9