Merge "CologneBlue rewrite: remove hard dependency on SkinLegacy, kill Quickbar nonsense"

A few action=info bugfixes and enhancements.

(bug 39533) Merge action=credits into action=info.
(bug 39951) Remove redundant 'Page protection' text.
(bug 39953) Don't show irrelevant protection statuses.
(bug 39954) Use proper section headers.
(bug 39955) Link page creator and latest editor values.

Change-Id: I68308aac96b89c2aec296e2a4fa385fd1d42ab5a

Merge "Show HTTP error (if any) when scary transclusion fails"

Merge "(bug 16362) Add action=info link to toolbox."

Merge "Support explicit 0 and 1 forms for plural in PHP"

Merge "Minor parameter documentation improvements on External Store classes"

Show a "(blocked)" hint on Special:ListUsers/ActiveUsers

In ActiveUsers many of the code was there, but there was a mistake in
the query.
Allowing now also listing hidden active users for users with the
'hideuser' right.

Change-Id: I2a4db8433cb5a2dec497c0fa662459cb56df3c3c

Add 'expandurl' to prop=extlinks/list=exturlusage

Allow prop=extlinks and list=exturlusage to expand protocol-relative
urls

See also bug 40052

Change-Id: I3fd661b51c44020dad9e95ac2fe6994c6e22aabb

Minor parameter documentation improvements on External Store classes

Changed the comment about returning ExternalStore subclasses, as neither ExternalStoreDB or ExternalStoreHttp subclass ExternalStore

Remove trailing whitespace

Change-Id: I90b61f93a99d5088edfadf83c846ec5fa7e40c2e

(bug 32951) Do not register absolute internal externals

Setting $wgRegisterInternalExternals = false for proto server should not
store the http/https links in externallinks table

Also fix detection of own links for links with query or anchor or
nothing

new also detected:
//localhost
//localhost?query
//localhost#anchor

already detected:
//localhost/path

Change-Id: Idd03d309cc3b71728a8cbea460efa12b10348d64

Merge "[FileBackend] Suppress less warnings in doGetFileContentsMulti()."

Merge "Parsoid-centric tests to RT-test comments-ws-pre interactions."

Parsoid-centric tests to RT-test comments-ws-pre interactions.

Change-Id: I2974d1451acbaeae4bce1aa890f0becea2dca32a

[FileBackend] Suppress less warnings in doGetFileContentsMulti().

Change-Id: I7c03b4204bc2173c73e11ca9091907f17e93d586

Fixed documentation typo.

Change-Id: I31c7959d6feb2241984883e4a02882903a8475d4

Merge "[FileBackend] Optimized concatenate() to use getLocalReferenceMulti()."

Merge "[FileBackend] Added getFileContentsMulti() and improved it for Swift."

Merge "[FileBackend] Added getLocalCopyMulti() and getLocalReferenceMulti()."

Merge "Improved documentation"

Merge "Reorder keys for fileStoragePathsForOps() function since its a list."

Merge "Localisation updates from translatewiki.net."

Improved documentation

Change-Id: If17de8dadefeb6a2fdfe7073ba84553c07cec229

Localisation updates from translatewiki.net.

Change-Id: I766bdb2f0fd630144b1bd56f0fa5135af8fbdf5d

Merge "Changed notifyOnPageChange() to use onTransactionIdle()."

Merge "Removed useless begin()/commit() calls as DBO_TRX is not on in cli mode."

Updated comment and pre-interaction tests with more cases.

Change-Id: Icb631dd8692e4d169c2599598add30a495ea02d7

Merge "Convert valign HTML attribute into inline CSS"

Merge "Fix documentation of $wgProxyList"

Merge "Improve documentation of maintenance scripts and test suite."

Merge "Rebuild MessagesEn.php"

Merge "Stylize"

Rebuild MessagesEn.php

Change-Id: I09969e009076f3298dde8a0773771d057e404989

Merge "Defer the watchlist update query in EditPage."

Update messages.inc for Icd721011b4

Change-Id: I5b2fd3b3f913ee57cf26e2c079b2b3bece8df358

Merge "Removed user-agent string from mw.feedback posts"

Removed user-agent string from mw.feedback posts

Legal found this to be unacceptable

Change-Id: Ib8746ece3a34b5e577dc08596b0ff7b1b96b5d73

Add warning for unused params inside api

Now MediaWiki knows all used params, it can be helpful to get a list of
unused params. This can be spelling issues or the missing 'g' for
generator modules. When the client knows about this, he can correct
them.

Change-Id: I221b7cd02bed4a38aba161dc3f85eb8fd2aad52c

Merge "Truncate SHA-1 indexes"

Use User::getGroupsWithPermission

Change-Id: Iba44440600484b9f2525805a1c80df50d5335c1a

Defer the watchlist update query in EditPage.

Change-Id: I9441a2778065b03593a0e42154bb0a6e44f0a196

ResourceLoader: Refactor style loading

Fixes:
* bug 31676: Work around IE stylesheet limit.
* bug 35562: @import styles broken in modules that combine
  multiple stylesheets.
* bug 40498: Don't output empty "@media print { }" blocks.
* bug 40500: Don't ignore media-type for urls in debug mode.

Approach:
* Re-use the same <style> tag so that we stay under the 31
  stylesheet limit in IE. Unless the to-be-added css text from
  the being-loaded module contains @import, in which case we do
  create a new <style> tag and then re-use that one from that
  point on (bug 31676).

* Return stylesheets as arrays, instead of a concatenated string.
  This fixes bug 35562, because @import only works when at the
  top of a stylesheet. By not unconditionally concatenating files
  within a module on the server side already, @import will work
  in e.g. module 'site' that contains 2 wiki pages.

  This is normalized in ResourceLoader::makeCombinedStyles(),
  so far only ResourceLoaderWikiModule makes use of this.

Misc. clean up and bug fixes:
* Reducing usage of jQuery() and mw.html.element() where
  native DOM would be very simple and faster. Aside from
  simplicity and speed, this is also working towards a more
  stand-alone ResourceLoader.
* Trim server output a little bit more
  - Redundant new line after minify-css (it is now an array, so
    no need to keep space afterwards)
  - Redundant semi-colon after minify-js if it ends in a colon
* Allow space in styleTest.css.php
* Clean up and extend unit tests to cover for these features
  and bug fixes.
* Don't set styleEl.rel = 'stylesheet'; that has no business
  on a <style> tag.
* Fix bug in mw.loader's addStyleTag(). It turns out IE6
  has an odd security measure that does not allow manipulation
  of elements (at least style tags) that are created by a
  different script (even if that script was served from the same
  domain/origin etc.). We didn't ran into this before because
  we only created new style tags, never appended to them. Now
  that we do, this came up. Took a while to figure out because
  it was created by mediawiki.js but it calls jQuery which did
  the actual dom insertion. Odd thing is, we load jquery.js and
  mediawiki.js in the same request even...
  Without this all css-url related mw.loader tests would fail
  in IE6.
* mediawiki.js and mediawiki.test.js now pass jshint again.

Tested (and passing qunit/?module=mediawiki; 123 of 123):
* Chrome 14, 21
* Firefox 3.0, 3.6, 4, 7, 14, 15, 16beta
* IE 6, 7, 8, 9
* Safari 4.0, 5.0, 5.1
* Opera 10.0, 11.1, 11.5, 11.6, 12.0, 12.5beta
* iPhone 3GS / iOS 3.0 / Mobile Safari 4.0
  iPhone 4 / iOS 4.0.1 / Mobile Safari 4.0.5
  iPhone 4S / iOS 6.0 Beta / Mobile Safari 6.0

Change-Id: I3e8227ddb87fd9441071ca935439fc6467751dab

Additional template and pre-wrapping interaction tests.

Change-Id: I84f5b223d2f7abba3668be6f13784664380cf82f

Localisation updates from translatewiki.net.

Change-Id: I0b758741deecd9ab4836085ca9728b7dc274bd71

Improve documentation of maintenance scripts and test suite.

Change-Id: If5d695dc3665635fbed73a713583c0b03c14ebb1

Reorder keys for fileStoragePathsForOps() function since its a list.

Change-Id: I83cf6f6db92c9a0b39c3229d688463c5cda61d69

Merge "release note for bug 34495"

Merge "Removed __NOHEADER__ magic word"

Sane whitespace

Change-Id: I849e9e672e50f033a24b5b42db93659d6e5fe899

release note for bug 34495

When [r111965] got backported in 1.19wmf branch, it was not applied to
the REL1_19 branch and thus is missing from the 1.19 releases. The fix
is already in 1.20, this merely add the forgotten release notes entry.

[r111965]
https://www.mediawiki.org/wiki/Special:Code/MediaWiki/111965

Change-Id: I2de04fbe22a0b18a15755eac2db9683e5e179b0a

Removed __NOHEADER__ magic word

Followup that was not done by anybody in 4 years, see r41994
Reported in http://translatewiki.net/wiki/Thread:Support/MediaWiki_magic_word_"noheader"

Change-Id: Ieded9ca281762c2c42e26c7cfe7669580cbbac62

Fixup release notes

* This section shouldn'tve been moved to the 1.21 file, because
  the change that added it (4d4bfc27a1bedf6b0d1670707ac812fe36fe85ad)
  added it to REL1_20 for back porting (which happened).

Change-Id: I1128691e330bf816cd1bb7d8c8e3e665f919272c

Merge "Additional indent-pre and template interaction tests."

Merge "Bug 40668 - "Return to Array." when logging in"

Additional indent-pre and template interaction tests.

Change-Id: I445247130dea266beb8b94e2fa1f039f1886446a

Merge "(bug 40448) Replace legacy mwsuggest with mediawiki.searchSuggest"

jquery.byteLimit: Rewrite (fix bug 38158, bug 38163)

It used to be fairly simply and that seemed good enough. Listen
on keypress, get current value, add character from key code in
event object, and calculate the byteLength. If it is too long,
preventDefault().

However there were so many edge cases (too many of them to be
considered edge cases) where this failed whereas the native
maxLength handling was fine.

For example:
 - Cut and paste
 - Selecting text and replacing or removing it (by any of the
   methods on this list)
 - Custom javascript input methods (e.g. Narayam)
 - Spelling suggestions and corrections
 - Drag and drop
 - etc.

Now it acts on changes after the fact.

I considered building in a timeout loop in addition to this so
that when javascript gadgets change the input value it would
detect it, however I'd like to hold that off for now. We can
add that later if needed. For now scripts are expected to fire
the 'change' event if they change it, which seems reasonable.

The only challenge that it creates, however, is the need for
basic history reconstruction. Figure out what was already there
and what is new, and cut down the new part until the total sum
satisfies the limit.

This is the same behavior that WebKit browsers (and others) have.
e.g. of "barbaz" is pasted between "f" and "o" in "foo" with a
maxLength of 5, then the resulting string will be "fbaoo".

Also fixed bug 38158 while at it. Filed an upstream bug against
webkit/Chrome but was wont fixed. Either way, we'd have to be
compatible for a little while with old WebKit builds out there
(iOS 3, Safari 5, etc.).
http://code.google.com/p/chromium/issues/detail?id=136004

Use removeAttribute('maxlength') to let the browser internally
reset the maxLength property. Resetting it directly (to undefined
or with `delete el.maxLength') does not work and casts it to 0,
which blocks the input field entirely.

Updated unit tests, and also removed the "type=text" code
afterwards which that doesn't work in IE (Internet Explorer does
not support dynamic changing of input types), needs to be in the
HTML directly. So far it was passing in IE because 'text' is the
default. Tests now use attr('maxlength') everywhere for
consistency, though properties work the same. Browsers
synchronize these things (just like the "class" attribute and
"className" property).

References:
* Fixes bug 38158: Unexpected 0 maxLength.
* Fixes bug 38163: Incorrect limiting of input when using input
  methods other than basic per-char typing.
* Follows-up: 39cb0c19
* Supersedes: I70d09aae

Change-Id: I9c204243d0bdf7158aa86a62b591ce717a36fe27

(bug 40448) Replace legacy mwsuggest with mediawiki.searchSuggest

The module has been broken for a while now, but nobody noticed
because in plain core it is disabled by default, and in the
bundle we ship with Extension:Vector (and its SimpleSearch).

This commit removed the mediawiki.legacy.mwsuggest module (and
related components that become obsolete with its deletion) and
replaces it with the new mediawiki.searchSuggest module, which is
based on SimpleSearch from Extension:Vector (where it will be
removed soon).

The following and all references to it in core have been removed,
I also made sure that they weren't used in any of extensions/*.
Only matches in extensions/Settings and some file that dumped the startup module, and in extensions/Vector which are addressed in
I1d5bf81a8a0266c51c99d41eefacc0f4b3ae4b76.

Had to make a few updates to jquery.suggestions to make it work
in other skins. So far it was only used in Vector, but now that
it is used in mediawiki.searchSuggest, I noticed several issues
in other skins. Most importantly the fact that it assumed the
default offset was from the right corner, which isn't the case in
Monobook where the search bar is on the left (in the sidebar).

It now detects the appropiate origin corner automatically, and
also takes directionality of the page into account.

It also uses the correct font-size automatically. Previously it
used font-size: 0.8; but that only works in Vector. Every skin
seems to have its own way of making a sane font-size. In Monobook
the <body> has an extra small font-size which is then fixed in
div#globalWrapper, and in Vector it is extra large, which is then
fixed as well deeper in the document. Either way, the size on
<body> can't be used, and since this suggestions box is appended
to the <body> (it is a generic jQuery plugin without knowledge of
the document, and even if we could give it knowledge inside
the configuration, it'd have to be per-skin). So I removed the
Vector specific font-size and let it handle it automatically.
This was needed because it is now used in all skins.

Removed modules:
* mediawiki.legacy.mwsuggest:
  > Replaced with mediawiki.searchSuggest.

Removed messages:
* search-mwsuggest-enabled
* search-mwsuggest-disabled
  > No longer used.

Removed mw.config.values:
* wgMWSuggestTemplate
  > Obsolete.

* wgSearchNamespaces
  > Obsolete.

Removed server-side settings:
* $wgEnableMWSuggest
  > Suggestions are now enabled by default and can be disabled
    through the user preference `disablesuggest` still.
    They can be disabled by default site-wide or hidden from
    prefs through the standard mechanisms for that.

* $wgMWSuggestTemplate
  > Obsolete.

Removed methods
* SearchEngine::getMWSuggestTemplate()
  > Obsolete.

Filters:
 $ ack mwsuggest -i -Q --ignore-dir=languages/messages
 $ ack wgSearchNamespaces -Q

Message changes:
* vector-simplesearch-preference
  > It was wrong, it didn't activate search suggestions, that
    was handled by the Vector extension. This preference in
    MediaWiki core controls whether the SimpleSearch bar HTML
    and CSS will be used (e.g. the rectangle search box with
    the magnifying class instead of the browser-default input
    field with the plain submit buttons).

* searchsuggest-search
* searchsuggest-containing
  These come from Extension:Vector message and should be imported
  by translatewiki:
  - vector-simplesearch-search
  - vector-simplesearch-containing

Change-Id: Icd721011b40bb8d2c20aefa8b359a3e45413a07f

Merge "Added test for parsing &nbsp; in wikilinks."

Bug 40668 - "Return to Array." when logging in

Change-Id: I847684cdc474d0694bf4fa5eddaa0d0b19e7be4e

Localisation updates from translatewiki.net.

Change-Id: Ib96d779a5d6b8a7cff6a181b5530d1dc2abe191f

Merge "Use hasSubjectNamespace in Skin.php instead of namespace tests."

Merge "more public accessors for Title class"

Merge "Bug 38955 - Don't include job_timestamp in checks for duplicate jobs"

Fix documentation of $wgProxyList

Change-Id: Iaac13bee34a978061605182123a90b3167e78129

Added test for parsing &nbsp; in wikilinks.

Change-Id: I3aa2c76e6fb03c3acaf2edb9fbeca6fd70c556c2

Use hasSubjectNamespace in Skin.php instead of namespace tests.

Change-Id: I11c5657de868a99429d400e0edb05707ec6069fb

more public accessors for Title class

This patch adds new public accessor to the Title class. They are merely
returning the already existing conventions.

Added tests for the four new methods:
 - Title::getBaseText()
 - Title::getRootText()
 - Title::getRootTitle()
 - Title::getSubpageText()

The later does not test with $wgNamespacesWithSubpages variants.

Change-Id: I4f7230c1a5487b82d06c78a45c50436085df57be

Bug 38955 - Don't include job_timestamp in checks for duplicate jobs

Change-Id: I28bffdd8612b7a87cb35983c3a0db89eaa018262

Convert valign HTML attribute into inline CSS

Change-Id: I62c14f3373e6f8b35c73a02fd042c79a01fab814

Added test to allow "|" char between url and text in extlinks.

Change-Id: Ic86f6fff600650207afc52a87d4f92b1c9347130

Stylize

Change-Id: Ia9532f06db8331da1ea5ff9c9398073d7d084484

Do not register Special:JavaScriptTest, if disabled

It is better when the special page is not register,
instead of giving a hint, that this is disabled on the wiki.

See other special pages like Special:PopularPages or Special:ChangeEmail

Change-Id: I43fb118d61ddcb2536192f54ee888d55b2fbd49d

(bug 40541) Fixed minor code style problems.

Follow-up to I17ac68014840daa47bfd4768e978e9ff2edb00db.
Replaces some ==/!= with ===/!== and other code style
changes mentioned in Gerrit comments on previous patchset.

Change-Id: I50da16bc62241491ac5c4948e0d3059b21f113dc

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: Id807056afde3025ac14fff8d2d9fee6de906066d

Merge "(bug 34960) drop unused fields rc_moved_to_ns/rc_moved_to_title"

Merge "(bug 40384) Move width/height from button to img"

(bug 36151) mw.Title: Don't limit extension in title parsing.

This allows for e.g. "*.properties" files.

Previously, the extension could be no more than 5 characters. If
it was, it was considered as not having any extension (part of
the main title text).

Change-Id: Ib27792b661666ac5f704eeaaadcdd9668dd81b56

Fixed mismatched profile calls in LoadBalancer.

Change-Id: I1ac81347b8af9b30dc03df94a76483a4ae4a6503

(bug 34960) drop unused fields rc_moved_to_ns/rc_moved_to_title

Change-Id: I68e8c0bb23f185c0f996a8905f6d437db3080aa1

Merge "Drop unused database field ss_admins"

Merge "Revert "Revert "Release notes: Get back lost history"""

Revert "Revert "Release notes: Get back lost history""

This reverts commit bd8d6492dc0122c12cf8e7ca1d883cd6e6521dd1

Merge "Revert "Release notes: Get back lost history""

Revert "Release notes: Get back lost history"

This reverts commit b6d5645a920041e246e268dad842ddc4be397246

Merge "Revert "Release notes: Get back lost history""

Revert "Release notes: Get back lost history"

This reverts commit b9e3916b80fe7220ed2a4fbe045df09cd253c812

Drop unused database field ss_admins

no longer updated since 1.5

Change-Id: Iebdce084d178b5003105b2d2b7dcc499c274ff56

Merge "Release notes: Get back lost history"

Release notes: Get back lost history

REL1_20 was branched from master as it was on 2012-09-15, but
release notes was removed from latest master. So there's stuff
missing.

I copied over parts from this diff:

$ git diff gerrit/REL1_20...e4f62bbb0b393e7f01ee5ee8390e1c5e890f7ead -- RELEASE-NOTES-1.20

Fixed existing entries, and copying new entries to
RELEASE-1.21 as they never made it to 1.20, because we branched
REL1_20 from a past master.

Change-Id: Ic07c57c4b3fb27074ea8a0d85b56837f8db81bd1

Revert "Release notes: Get back lost history"

This reverts commit b9e3916b80fe7220ed2a4fbe045df09cd253c812

Release notes: Get back lost history

Whoever branched it had an outdated copy of master (2012-09-15
instead of 2012-09-27).

Now resetting back to how the file was right before it was
removed:

$ git co e4f62bbb0b393e7f01ee5ee8390e1c5e890f7ead -- RELEASE-NOTES-1.20

Change-Id: Iadb900a087188ff84c6115d91e0d6765b178f9b8

Release notes: Maintain 1.20 notes in master, revert history.

* The one in history isn't final yet, which makes maintenance
  harder as it would have to be synced between 2 different file
  names across 2 different branches.

* For backport commits its easier if the 1.20 file is still in
  master so that they can be written together with the commit
  (and so that the history is in master as well), as opposed to
  merging it in master without notes and then adding them later
  on in the (amended) merge version.

Change-Id: I3fa4b88a5845a51e83221d78facaba1ddb3b1acb

Merge "Wrap long lines"

(bug 16362) Add action=info link to toolbox.

MZMcBride asked me to split this away from https://gerrit.wikimedia.org/r/#/c/22547

Change-Id: I392d823e02a26e45134a175fc215b658a90ebb41

Removed unnecessary whitespace in tpl name/arg class tests.

Change-Id: Id0346d16fcecd0d44d85f29a45923c8401f47a9c

Wrap long lines

Also remove default setting of $p done in else statement at end

Change-Id: I5024c1fc1748ad1add145d0696b33a6d7ef3223b

Merge "Adding wildcard support to $wgCopyUploadsDomains"

Merge "Clean up: Declare variables with public instead of var"

Merge "(bug 40541) Fixed $wgSecureLogin functionality."