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

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"

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."

Merge "Added protocol option to Linker and OutputPage::addReturnTo."

Localisation updates from translatewiki.net.

Change-Id: I300d5f11d881b764b80695b745bff5e9c7d3c404

Merge "(bug 29898) Set cookie to force HTTPS from HTTP"

Updates for MW 1.21

Change-Id: I582c1bcc50d694705b560fea893d0b2984072c08

Merge "Bring in 1.19 history, update 1.17 and 1.18 history too"

Bring in 1.19 history, update 1.17 and 1.18 history too

Change-Id: I098fdb5610011cbdd91012b8ed4d29e61ca80408

Add RELEASE-NOTES for g25605

Change-Id: Ibd09a52e98ea2e8fcdf0291b4a586870cb128007

(bug 39674) Fixed loading User from session when hook aborts.

Rather than have separate calls to User::loadDefaults()
every time User::loadFromSession() fails, there is now just
one call in User::load() if loadFromSession() returns false.
This fixes the case where a UserLoadFromSession hook aborts
loading from session, leaving the User object uninitialized.

Change-Id: I8d1a114d7ec361b27b260791f742c473a1497f26
Signed-off-by: Tyler Anthony Romeo <tylerromeo@gmail.com>

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

The fix for bug 34768 (r112710, regarding the lack of width and height
attributes for the Vector skin's search button) failed to include the
corresponding changes to SkinTemplate.php.

Because BaseTemplate::makeSearchButton() only special cased the src and
alt attributes, the width and height incorrectly went on the button
element, causing HTML validation errors (and also having no effect).
This patch moves the width and height attributes to the img element
in a similar fashion to src and alt.

Change-Id: I3561fc79364ce43521823bad691c9b61f87ba375

Add $wgCopyUploadProxy to be used for copy uploads only.

Means we don't break stuff on the cluster such as search by setting $wgHTTPProxy

Change-Id: I094b2bfd097e7e5c1c45b6332357d959ed95ac99

Copy plural rules from be to be-tarask

CLDR only defines rules for 'be'.
The bug was reported by Wizardist.

Change-Id: I459be3d2f488471eea2acfd3d3c4f79ff3ebc11c

Merge "Link the credits in Special:Version in all languages"

Link the credits in Special:Version in all languages

Add the link to {{SERVER}}{{SCRIPTPATH}}/CREDITS in Special:Version
in the code and not in the interface message, which is
way more straightforward, makes the message  easier and
assures that the developers are credited no matter of the current
interface language.

The following languages need interface changes after this is merged:
He, Ko, Bcl, Jv (the fact that this list is so short once again
proves the point that the link isn't set in many languages). I'll
take care of those.

Change-Id: If12c4eb8453da40ede4b9768b2dacfaf838aa2ab

Localisation updates from translatewiki.net.

Change-Id: I9e851866959da77ae184c3e07a95b07999027462

Merge "Catch and deal with UploadStashZeroLengthFileException when removing stashed files"

Merge "Style fix"

Added a pile of missing brackets

Change-Id: Id7b331e1b12875736c8e7e93660e71630360a723

Style fix

Change-Id: I6b900924b07b44152fa943d10400fe151b4046c8

(bug 40541) Fixed $wgSecureLogin functionality.

* Added parameter to login link so that wpStickHTTPS
  is set to true by default when the user is coming
  from HTTPS.
* Added redirect in Special:Userlogin so that when
  $wgSecureLogin is enabled it automatically redirects
  to HTTPS.
* Adjusted User::setCookies() to add a parameter for
  forcing secure/insecure cookies, and then added the
  appropriate argument to Special:Userlogin so that
  cookies are set appropriately.

Change-Id: I17ac68014840daa47bfd4768e978e9ff2edb00db

Catch and deal with UploadStashZeroLengthFileException when removing stashed files

Seen on the cluster and it halts script execution

reedy@fenari:/home/wikipedia/common$ mwscript cleanupUploadStash.php commonswiki
Getting list of files to clean up...
Removing 52958 file(s)...
File is zero length
Backtrace:

Change-Id: I740111ca20473c495a4a51edafa156169fe6dd4d

Added protocol option to Linker and OutputPage::addReturnTo.

Added new argument to the Linker options array to allow
the forcing of an HTTP or HTTPS protocol. In order to facilitate
this, a protocol argument was added to Title::getLinkURL.

Also, an options argument was added to OutputPage::addReturnTo
so that options can be passed to the linker and so that the
returnto URL can be forced to a certain protocol.

Change-Id: Ia9cc11e310ad6ef23c221bdba3a4834e7c5556e7

Merge "(bug 39793) #simpleSearch placeholder styles for Firefox, IE"

(bug 39793) #simpleSearch placeholder styles for Firefox, IE

Since some time ago Firefox and IE 10 also support styling for input
placeholders. (Opera supports placeholders themselves, but not (yet)
styling them.)

Firefox also applies regular input styles to placeholders, which is at
best weird, so we need to style the placeholder for it to display
reasonably (grayed out). (bug 39793)

Note the single colon for IE and Firefox and double one for Webkit:
these are pseudoclasses, not pseudoelements.

In Opera, the placeholder is always grey; #808080 on my system, but it
might differ on various setups. It's close enough to #999999 for us not to
care.

Further reading:
* http://stackoverflow.com/questions/2610497/change-an-inputs-html5-placeholder-color-with-css/2610741#2610741

Change-Id: I902abe16ac0cb7a55fe0d9ee7a2eec9a4e5a371a

Merge "Fix typo in docs."

Merge "The hooks are initiated from WikiPage instead of Article"