API edit: allow ConfirmEdit to use the merged parse

ConfirmEdit was tripling the amount of time it took to save a typical
page via the API, since it was assuming that the APIEditBeforeSave hook
was giving unmerged wikitext, requiring a full parse of the content
before and after the edit in order to check the addurl trigger.
Apparently nobody bothered to update it after Ia59fb6bb.

APIEditBeforeSave is unusable anyway, because it is given the serialized
content, without any information about the content model. It really
needs the Content object in order to do it properly. So instead, adapt
EditFilterMergedContent for the purpose. Incidentally, that avoids the
inelegant defined('MW_API') check in ConfirmEdit's
EditFilterMergedContent handler, since both API and normal edits are
handled by EditFilterMergedContent in the same way.

Unfortunately the interpretation of the 'value' member in the Status
object passed to EditFilterMergedContent is not extensible, being an
integer instead of an associative array. So we add a custom member in
order to get the result array back down the stack.

Another obstacle to this design was the lack of an EditPage object
passed to EditFilterMergedContent. ConfirmEdit was previously directly
calling EditPage::showEditForm() with a $formCallback which outputs the
necessary HTML. Instead, I hacked up runPostMergeFilters() a bit, to
allow the hook to request that the edit page be shown again even if
hookError is not set. Then a new EditPage::showEditForm:fields hook does
the necessary HTML output, instead of $formCallback.

Marked $formCallback as deprecated, since I think it is now unused.

Change-Id: I4b4270dd868a643512d4717927858b6ef0556d8a

Remove jQuery Migrate and $wgIncludejQueryMigrate

As indicated in MediaWiki 1.24, removing the jQuery Migrate bridging patch
that allows legacy Javascript code that depends ancient features of
jQuery removed in jQuery 1.9 to work.

Change-Id: I0934247182e2737716c9d40f4bf93415adcdadf3

Merge "Update redirect arrows to use SVG versions"

Update redirect arrows to use SVG versions

Included updated fallback PNGs (now with transparency).

Bug: T72538
Change-Id: I1bd865d3184fad59c7e6144ec1685fe65be97916

Merge "Add SVG versions of table pager icons"

Add SVG versions of table pager icons

Google Code-In 2014 - Ljudusika.

Bug: T71944
Change-Id: Ia9d9271fe0930a3826976585c694b561371dd32a

Fix DatabaseBase::replaceVars for PHP 5.3

Change-Id: I78a535abfe3f444b2ccb99638a63b6f16c9f18af

Merge "Hygiene: rename formheader to searchProfileTabs on Special:Search"

Merge "Revert "Remove deprecated profiling config parameters, clarify docs""

Hygiene: rename formheader to searchProfileTabs on Special:Search

It's not a header, its out of the form now, boiled down, it's
some search profile terms laid out in tabs.

Change-Id: Ic6ec29817ab7deadc6bc7125f8b4cd16686baa04

Revert "Remove deprecated profiling config parameters, clarify docs"

StatCounter still needs $wgUDPProfilerHost.

This reverts commit 88c42dc1df93533286dfbe969631403f77958dca.

Change-Id: I4a56f6fb6bc454d15a9dc752f45dfb29b2b47a84

Merge "Remove deprecated profiling config parameters, clarify docs"

Localisation updates from https://translatewiki.net.

Change-Id: I7f9787dfd402817f21686bcbe2ccafce6e224269

Remove deprecated profiling config parameters, clarify docs

We've already broken profiling completely in this release. Make
this abundantly clear in the RELEASE-NOTES and just remove the
awful back-compat attempt from I2af28cd3 and I49c0a83e.

Change-Id: Ib0b87192e2a6e87db19f7821906dd7b2063081e3

Revert wrapping sitenotice in google tags

Reverts change I95473e22de83e585044a4f917bbd1b00db7db71b

Change-Id: Ia522064df15b959e22e81abe2c607687ec2a29cb

Merge "Split things out of search element on Special:Search"

Merge "Prevent Google indexing of the SiteNotice div"

Hygiene: RemoveInvalidEmails maint script not in autoloader

Change-Id: Ib1a54898b07a1f47f99e7004a12e6829280a4857

Switched hook profiling to use scopedProfileIn

* Also made scopedProfileOut handle the case where the callback
  was null (e.g. when there are no frame methods for xhprof).

Change-Id: Ife242bda8e046990d0d8ac27d628975b7b4a14d7

Merge "Changed some DatabaseBase type hints to IDatabase"

Prevent Google indexing of the SiteNotice div

Site notice and CentralNotice should be omitted from search snippets.
We're diluting page content.

The awkward thing about this patch is that we can only confirm the
googleoff tag is supported for the search appliance, not the public
web crawler.  See here for those docs, though:
http://static.googleusercontent.com/media/www.google.com/en/us/support/enterprise/static/gsa/docs/admin/70/gsa_doc_set/admin_crawl/admin_crawl.pdf

Change-Id: I95473e22de83e585044a4f917bbd1b00db7db71b

removeInvalidEmails.php: Fix batching continuation for wikis with >500 users

Change-Id: I6343aa204ddd68e9685630b67ac0b0ca535b426b

Merge "Darken the text color in successbox"

Make LocalisationCache::getMessagesDirs() public

So it can be used inside LocalisationUpdate

Change-Id: I2399ddd7fd4462f2c6d1bf81036af451a67c58b1

Merge "Remove Profiler::isStub()"

Merge "Fix up interwiki search results in API"

Remove Profiler::isStub()

Just check for ProfilerStub directly.

Change-Id: I503916599f182df4206da5282193ae6ec9324ee6

Don't attempt to stash new section edits

The parser output of edits produced via the 'new section' interface varies on
both the edit body and the edit summary (since it determines the new section's
name). So there isn't an early point in the edit process where we can start
computing the edit output. We have to wait for the user to be completely
done, by which point edit-stashing provides no advantage.

Change-Id: Ida461e08189b75c5a508628d11dc929a6138814a

Make ApiStashEdit use the StashEdit log group, rather than PreparedEdit

An early version of what ultimately became commit 3a6c9d36c used
'ApiPreparedEdit' instead of 'ApiStashEdit'. The class / file were renamed but
not the log group.

Change-Id: Ib4de1f1f21f058d3be627048bc93e7730515d340

Split things out of search element on Special:Search

While keeping these in the same order, pulling them out of the form
helps mobile display search suggestions on non RL devices.

Change-Id: Id267f66485b95788e682b912e8d2540c826d83fd

Merge "Use quickUserCan() for the undelete link message"

Use quickUserCan() for the undelete link message

* This was taking 267.304ms on https://en.wikipedia.org/wiki/Tammar_wallaby for admins

Change-Id: I0e669869e1990502331a80de168562421ac5336d

Merge "Move core message dirs from $wgMessagesDirs to LocalisationCache::getMessagesDirs()"

Removed useless "use" clause from closure

Change-Id: Ie2e0328548b185e77b919c0b8bd18a5005d94037

Added ApiStashEdit module for pre-emptive edit parsing

* This lets edits be prepared while users enter edit summaries.
* The edit form will now make use of this API, controlled by
  $wgAjaxEditStash.

Change-Id: I4f4057bc0d1d4a66a8f7cfb7cdc26d443a8eb0c4

Merge "If a user logs in while not on https, then the user should be sent back to the non-secure website."

Move core message dirs from $wgMessagesDirs to LocalisationCache::getMessagesDirs()

If $wgMessagesDirs is initially empty, we can optimize when batch loading
extensions:

    if ( !$wgMessagesDirs ) {
        $wgMessagesDirs = $cache['MessagesDirs'];
    }

With APC, this should be O(1) CPU time.

This was suggested by Tim in the code review of I7074b65d07c5.

Change-Id: I66fa907cdaafe18b74b5b9afaa8b6b1db069bea3

Removed low signal wfDebugLog call in DatabaseBase

Change-Id: I231766b6a1a1651508bb03fa137e6b2b55c55e6a

Log backtraces for DB reconnections

Change-Id: Ibe7badde80bf6a44e292b89fe6c085cbda5748e2

Merge "Before pseudo elements are not always perfectly in line"

Merge "Database: Remove use of strencode() in replaceVars()"

Merge "Add removeInvalidEmails.php maintenance script"

Merge "Deprecated $wgProfilePerHost"

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

Merge "Deprecated $wgUDPProfilerHost, $wgUDPProfilerPort and $wgUDPProfilerFormatString"

Localisation updates from https://translatewiki.net.

Change-Id: I38970b3004ea686f55507fdb36bd701a511287d8

Add removeInvalidEmails.php maintenance script

The script scans the user table and removes emails
that are technically invalid which don't pass
Sanitizer::validateEmail(), and wouldn't be allowed
today. Confirmed emails are skipped entirely since
they had to be valid at some point.

Bug: T76512
Change-Id: I3cc6396ff6d8b738846b7716b4b0cddc9bf9e1a4

Merge "OutputPage: Add accessors for some protected properties"

Deprecated $wgProfilePerHost

Change-Id: I2af28cd3083228136789cff5feadd7b16c509e5b

Deprecated $wgUDPProfilerHost, $wgUDPProfilerPort and $wgUDPProfilerFormatString

All can be passed as options to $wgProfiler now.

Change-Id: I49c0a83e0d386be0f66d6703fc358089e4b1f59f

Before pseudo elements are not always perfectly in line

This results in the psudo element's top inline with the element's
top

Change-Id: Iadd741f02155a9b5ddeb6cd8ff64fa13ae5506d7

Stop using $wgProfileToDatabase

Was not completely removed and the remaining bits were mostly
broken. $wgProfiler['output'] = 'db' is the proper way.

Fixes T75917
Change-Id: I36565e2372db2ed49b219cf533ec433e8111c52f

OutputPage: Add accessors for some protected properties

Bug: T76168
Change-Id: Iee11c217b6b0d35acfdfac916814521acda2b1a4

Database: Remove use of strencode() in replaceVars()

Almost all known uses of the /*$var*/ syntax are for either the
table prefix or table options, neither of which need string
encoding. Use of that syntax inside strings has been discouraged
for a while anyway.

This is to allow deprecating strencode().

Also changed the method to replace in one pass (to not find
variables inside values of other variables).

Change-Id: Ia697ce5f14082e737b3e1c4e174bd3f6223acc5c

Merge "content: Deprecate TitleIsCssOrJsPage and TitleIsWikitextPage hooks"

Changed some DatabaseBase type hints to IDatabase

Change-Id: Id945aaabbf5b467106676ea9f3eab3a6161c7f3d

content: Deprecate TitleIsCssOrJsPage and TitleIsWikitextPage hooks

They're currently documented as a 'compatibility hook'
in docs/contenthandler.txt. No longer used in any Wikimedia-hosted
git repository.

Superseded by the ContentHandlerDefaultModelFor hook.

Change-Id: I212230da7d6080cf500f930d4aa5a9024959d5f9

content: Re-arange tests to declare data provider first

For consistency with other test suites.

Change-Id: Ic4b4d8e228a9ce82a42a07d88d690a9c932960c3

Merge "Update OOjs UI to v0.2.4"

Merge "API: Add Access-Control-Allow-Headers in CORS preflight response"

Merge "Adding check for lessphp compiler to getLessCompiler()"

Merge "Add jquery.fn.serializeObject"

Merge "Document external libraries changes in release notes"

Localisation updates from https://translatewiki.net.

Change-Id: I06194421786553b0ee867ccd722a0f32a15ba594

Add jquery.fn.serializeObject

Teeny-tiny jQuery plugin to get form data as an object mapping each form
control's name to its value. I needed this for something and I saw this
pattern already in use in mediawiki.searchSuggest.js, so that's two use-cases.
I thought about calling this 'jquery.formData', but since it's a close analog of
jquery.serializeArray, I think this name is appropriate.

Change-Id: I00b6a7fb75a4d5132e7d6715c8867091a161dbb6

Support offsets in prefix searching

Fixes T75522
Change-Id: I7a27a64e295a1efcb1d9728d95cf254bb8bfbe92

Update OOjs UI to v0.2.4

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

Change-Id: Ic1afd83022ad8e6eee0ca89d94f6df23ba938684

Consistently use "T1234" format for release note bug refs

This will be the format for 1.25 and beyond as we've now moved
to Phabricator.

Change-Id: Ib555a2cbe72928b697dcbfc10f8168294691f4cb

Set optimize-autoloader: true and prepend-autoloader: false in composer.json

Quoting Tim from 240fd31cfa6b in mediawiki/vendor (prepend-autoloader):
 Composer's autoloader is slow, taking about 63µs per class, whereas
 MediaWiki's is fast, taking about 7µs per class. This adds up to an
 overhead of about 13% of CPU time on short requests such as API calls.
 On those same short requests, MediaWiki's autoloader finds most of the
 classes. So it makes sense to run it first, before the two Composer
 autoloaders. So change the config so that composer passes
 $prepend=false to spl_autoload_register().

optimize-autoloader is also just a good idea in general.

Change-Id: I383c72a85eda415da92e8a9253e83b4ed591768b

Merge "InstallDocFormatter: Hyperlink Phabricator task numbers"

Merge "Handle redirects during prefix search exact match"

resourceloader: Clear module storage if disabled by configuration

To free up space for other things that might want to use localStorage
for users where it was previously enabled.

Bug: T66721
Change-Id: Idd62a5eab72ef79c47cfe54fcd5c240d1cbb0e44

Merge "Avoid FOUC on API help and pretty-printed output"

content: Minor clean up to make JsonContent match other classes

Compared to JavaScriptContentHandler:

* Add missing license header.
* Add @ingroup.
* Move class description to @class.

Change-Id: I88c6e231756ce9f6d0d3aa2cb2bb80848c8bc5b9

WikiPage: Remove premature Content::isValid check

Content classes that implement the isValid method are causing
an "Internal error" to appear when saving from EditPage because
WikiPage throws an exception that isn't handled anywhere.

Turns out that ContentHandler already has infrastructure for
this in place in Content::prepareSave (which returns a Status
object with a relevant message informing the user), and WikiPage
is actually calling this already right after this.

Content::prepareSave defaults to using Content::isValid, so this
code is simply obsolete and premature as it calls an internal
method directly without handling it accordingly.

Change-Id: I69f18d9e101c2261b0ff7bd1b0459ea7fdd7ae9f

Darken the text color in successbox

A user mentioned that there isn't currently enough contrast in successbox,
which impairs accessibility. This fixes that.

Change-Id: Id819524377108ec04fe5f3faef7979bc22fbd891

Merge "Fix group separator in installer's sidebar"

Adding check for lessphp compiler to getLessCompiler()

Otherwise, PHP fatal errors will be thrown for anyone who installs
without it.

Change-Id: Ia027b7a829bbdae88b750c8119c546c846f3ae54

Merge "Hide 'redirectedfrom' notice when printing articles"

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

Localisation updates from https://translatewiki.net.

Change-Id: I2039215565965839327236df77eccfd226fe08c8

Populate revision data when expanding templates

 * Add a $revid parameter to the expandtemplates api action so that,
   when passed, the correct revision information is set for {{REVISIONID}}
   and similar variables.

Bug: T73306
Change-Id: I93ef39fdabbaa5f394efce9b5c5e080666ff6119

Fix group separator in installer's sidebar

Uses explode() to split the sections and puts correct separators between them.

Bug: T39362
Change-Id: I9a287b6ac206debb9bd93dbfad0703a8fad0931f

Merge "build: Upgrade grunt-jscs and remove legacy jshint camelcase/nomen"

Removed bogus argument in frame profiling method

* Caused "Warning: xhprof_frame_end() expects exactly 0 parameters, 1 given in includes/libs/ScopedCallback.php on line 70"

Change-Id: I1ab5a973a185738cae97de4cfdc3cca78598d35e

Avoid FOUC on API help and pretty-printed output

Bug: T76268
Change-Id: Ie1e087cad764cd7c8f84e2a7b3fb2dc5083cabaf

API: Add Access-Control-Allow-Headers in CORS preflight response

Otherwise a CORS request won't be able to properly make use of the new
header.

Bug: T76340
Change-Id: I1dbccdf928b85a4b194174d38f505787dd18f745

Hide 'redirectedfrom' notice when printing articles

Wrap the 'redirectedfrom' message in "mw-redirectedfrom" id
and then add it to commonPrint.css to hide it when printing.

Bug: 16296
Change-Id: I077bda9596c37b122a1f6964470d9d85108d834e

Localisation updates from https://translatewiki.net.

Change-Id: Ib0e7163ff33af2d163b55db583b71195d07571c0

build: Upgrade grunt-jscs and remove legacy jshint camelcase/nomen

Follows-up 4ec6b0cce.

grunt-jscs v0.7.0 upgraded node-jscs to v0.6.0 which introduced
the ability to disable rules via inline comments.

This means we can finally remove the deprecated jshint rules for
coding style rules and restrict jshint to code quality only.

Unsuppress the equivalent rules from jscs.

Change-Id: Idacf7e552d656ee139863d927e5bfc1f86af8283

Merge "Remove RELEASE-NOTES-1.24 and merge into HISTORY"

Merge "Add missing visibility keywords to all LoadBalancer methods"

Remove RELEASE-NOTES-1.24 and merge into HISTORY

1.24 has been released, so any changes that need release notes
should be targeting 1.25

Change-Id: Ie55c554131d750e22e6da1bf137279623325e67d

Document external libraries changes in release notes

Bug: T1355
Change-Id: If3432d513e053742ef09f1330604e57918ab90b2

Localisation updates from https://translatewiki.net.

Change-Id: I2765755ba154f1666a75da5d3ad7a95ba73d2d1c

Merge "Link to mediawiki.org's SkinDistributor from the fallback skin"

Merge "Style visited links correctly"

Style visited links correctly

If a link is styled as a neutral button, the text color will change
if the link is already visited. Style the text like a real button (same
color for visited and not visited links)

Bug: T76240
Change-Id: I18d985c1ada10becee15888b7f31d56976c7a2eb

Add IP::isInRanges()

cf https://github.com/wikimedia/operations-mediawiki-config/blob/master/wmf-config/throttle.php#L43-L58

Change-Id: I8ae145035de9d349710971f17551da637e4d2242

Merge "Inject cache as constructor param of SiteSQLStore"