lhc/web/wiklou.git
9 years agoProperly check whether a user exists
Kunal Mehta [Mon, 14 Jul 2014 05:03:00 +0000 (22:03 -0700)]
Properly check whether a user exists

User::newFromId() will never return false. We have to actually
load from the database to see whether the user still exists.

Change-Id: Id9329635717e6a4f1f34d0b2b665bb790685d2fa

9 years agoFold RevertFileAction class into RevertAction
withoutaname [Thu, 10 Jul 2014 01:26:16 +0000 (18:26 -0700)]
Fold RevertFileAction class into RevertAction

RevertAction was simply a dummy class for outputting an error message
if the action was not done in the file namespace. I've instead
moved RevertFileAction to RevertAction and threw an exception
from within checkCanExecute().

Change-Id: I7b87de60680009bf4e74d33342cbe81cc0d211b5

9 years agoGeneralize Installer::showStatusMessage()
withoutaname [Fri, 4 Jul 2014 07:34:17 +0000 (00:34 -0700)]
Generalize Installer::showStatusMessage()

Folds functionality from child classes back into a
concrete implementation in the parent class, reducing code duplication.

Change-Id: I697fe697dbd59d5db6fc77aba20af322e6011782

9 years agoEnforce type hinting for setTitle() in IContextSource subclasses
withoutaname [Tue, 8 Jul 2014 08:09:19 +0000 (01:09 -0700)]
Enforce type hinting for setTitle() in IContextSource subclasses

Change-Id: Iecd8535b04072823a6f4efe33697148f4d20118e

9 years agoRemove $wgEnableSorbs and $wgSorbsUrl
withoutaname [Sat, 12 Jul 2014 00:31:08 +0000 (17:31 -0700)]
Remove $wgEnableSorbs and $wgSorbsUrl

They were deprecated in version 1.17.

Change-Id: I8489c0cb4a041d081283fa3551377ef81ef36549

9 years agoVector: Use .mediawiki instead of .skin-vector in notification styles
Daniel Friesen [Tue, 13 May 2014 00:57:56 +0000 (17:57 -0700)]
Vector: Use .mediawiki instead of .skin-vector in notification styles

What was I thinking when I did this in I2b3d9cbc?

The extra class is present as mediawiki.notification's css is loaded later
thus overriding the skin's css. However using .skin-vector for that
purpose breaks Vector derived skins since they don't have it. The .mediawiki
class is always present so it should be an acceptable weight.

Change-Id: Ica2a3d3a2ef25aabb38fac36e9eaf142c8bbd388

9 years agomediawiki.feedback: Remove inline definition of link color
Fomafix [Sat, 12 Jul 2014 22:46:32 +0000 (22:46 +0000)]
mediawiki.feedback: Remove inline definition of link color

Change-Id: Iaf26b13011d3a46959fa3228792aece9f45a807f
Follows-up: b9d6456cb8ebf63244ca2fe1a48fafac0fbe543f

9 years agoMerge "installer: Recompute wgStylePath etc. after setting wgScriptPath"
jenkins-bot [Sat, 12 Jul 2014 22:23:23 +0000 (22:23 +0000)]
Merge "installer: Recompute wgStylePath etc. after setting wgScriptPath"

9 years agoMerge "installer: Move Server/Extension/Path preparations out of envChecks"
jenkins-bot [Sat, 12 Jul 2014 22:23:07 +0000 (22:23 +0000)]
Merge "installer: Move Server/Extension/Path preparations out of envChecks"

9 years agomediawiki.feedback: Resolve redirects when posting
Bartosz Dziewoński [Mon, 30 Jun 2014 21:57:01 +0000 (23:57 +0200)]
mediawiki.feedback: Resolve redirects when posting

To make this possible, extended mw.Api#newSection with an
'additionalParams' parameters.

Change-Id: I64102c920870e9d3ca6f1a6a712044b22029709e

9 years agoMerge "Localisation updates from https://translatewiki.net."
Translation updater bot [Sat, 12 Jul 2014 21:28:10 +0000 (21:28 +0000)]
Merge "Localisation updates from https://translatewiki.net."

9 years agoLocalisation updates from https://translatewiki.net.
Translation updater bot [Sat, 12 Jul 2014 21:07:11 +0000 (23:07 +0200)]
Localisation updates from https://translatewiki.net.

Change-Id: Ibf985c7870387ab6f7b47e626732edb359d2b9bb

9 years agojquery.ui.theme.css: Visible links in dialogs
Fomafix [Sat, 12 Jul 2014 17:12:11 +0000 (17:12 +0000)]
jquery.ui.theme.css: Visible links in dialogs

Test case:
$('<div>Examples: <a href="//example.com/foo">unvisited link</a>, <a href="//www.mediawiki.org/">visited link</a></div>').dialog({title:'Examples: <a href="//example.com/foo">unvisited link</a>, <a href="//www.mediawiki.org/">visited link</a>'})

Bug: 67851
Change-Id: I2280c87beb423d8b0ec53ffac53cd4cc5577a53c

10 years agoinstaller: Recompute wgStylePath etc. after setting wgScriptPath
Timo Tijhof [Thu, 10 Jul 2014 16:32:52 +0000 (18:32 +0200)]
installer: Recompute wgStylePath etc. after setting wgScriptPath

Bug: 67810
Change-Id: I95d02b41222d25fe216453c1d795475d4acdead5

10 years agoinstaller: Move Server/Extension/Path preparations out of envChecks
Timo Tijhof [Thu, 10 Jul 2014 16:27:11 +0000 (18:27 +0200)]
installer: Move Server/Extension/Path preparations out of envChecks

These aren't checks but preparations, they should always run
even when the user is further in the installer process or when
the server is generating the WebInstaller stylesheet (which needs
wgScriptPath to be set in order to remap image paths correctly).

Bug: 67810
Change-Id: I98eb15096ff4ef3e464b9d00c58b036a608deb41

10 years agoUpdate jQuery Cookie to v1.2.0
paladox [Thu, 12 Jun 2014 19:45:54 +0000 (12:45 -0700)]
Update jQuery Cookie to v1.2.0

Source

* https://github.com/carhartl/jquery-cookie/blob/v1.2.0/jquery.cookie.js

Change log

* https://github.com/carhartl/jquery-cookie/blob/v1.2.0/CHANGELOG.md

README

* https://github.com/carhartl/jquery-cookie/blob/v1.2.0/README.md

We're upgrading from an early alpha version of v1.1 (faa09dc).
The main change is the addition of $.removeCookie() for deleting cookies,
usage of $.cookie('key', null) has been deprecated.

We should eventually upgrade to v1.4.1, but that has a breaking change.
Namely, it drops support for $.cookie('key', null) which needs to be
replaced by $.removeCookie which didn't exist until v1.2.0.

Change-Id: I511138518fcd8bffc86fc3feaf5dbe253c7d786f

10 years agoMerge "Use htmlspecialchars not htmlentities to escape tags on Version."
jenkins-bot [Sat, 12 Jul 2014 08:34:52 +0000 (08:34 +0000)]
Merge "Use htmlspecialchars not htmlentities to escape tags on Version."

10 years agoCorrect out of date comment
Erik Bernhardson [Sat, 12 Jul 2014 03:07:00 +0000 (20:07 -0700)]
Correct out of date comment

The comment claims there is no cas method, but that was
implemented in 74f581e34871d3d2f711aaec32a09dfaaa46bd17

Change-Id: I6ffdc255204bcb3a03b0468c18609b0d30b6269c

10 years agoRemove broken by design maintenance test
Chad Horohoe [Sat, 12 Jul 2014 02:20:28 +0000 (19:20 -0700)]
Remove broken by design maintenance test

Any test that depends on the output of maintenance scripts is
asking for failure. This test fails quite often on HHVM and
is very very easy to break on single-database setups like most
users have. A harmless misconfiguration of SQLite will also
cause it to fail.

Change-Id: Ie90284eae847accea0569745005827f1af796940

10 years agoCSSMin: Add link to css-syntax spec for url tokens on w3.org
Timo Tijhof [Sat, 12 Jul 2014 00:00:55 +0000 (02:00 +0200)]
CSSMin: Add link to css-syntax spec for url tokens on w3.org

Change-Id: I9bfd7016ba4ec106e1b92602fd6d8188b8b183f3

10 years agoMerge "QUnit testrunner: Fix comment regarding animation and AJAX check"
jenkins-bot [Fri, 11 Jul 2014 23:30:52 +0000 (23:30 +0000)]
Merge "QUnit testrunner: Fix comment regarding animation and AJAX check"

10 years agomaintenance: Exclude OOjs UI's new RTL built CSS files from the pull-through
James D. Forrester [Wed, 2 Jul 2014 16:25:08 +0000 (09:25 -0700)]
maintenance: Exclude OOjs UI's new RTL built CSS files from the pull-through

Change-Id: I05967ab737afa76e92f98292cd95dd20c607678f

10 years agoQUnit testrunner: Fix comment regarding animation and AJAX check
Matthew Flaschen [Fri, 11 Jul 2014 22:40:55 +0000 (18:40 -0400)]
QUnit testrunner: Fix comment regarding animation and AJAX check

Follows-up b90c69cdf7.

This takes the approach of throwing an error to make the exact issue
obvious.

Unlike other versions (including the code in jQuery it is loosely
based on), this does not attempt to cleanup by stopping the effects
queue.

Change-Id: Ibb6ca0d083d4e8f1556e6f3493da17f7a5f44024

10 years agoUse htmlspecialchars not htmlentities to escape tags on Version.
Brian Wolff [Fri, 11 Jul 2014 22:02:20 +0000 (19:02 -0300)]
Use htmlspecialchars not htmlentities to escape tags on Version.

htmlentities assumes that the text is iso 8859-1 prior to php 5.4

Also we use htmlspecialchars almost everywhere else.

Bug: 67840
Change-Id: I73364ac34ef7a24c48c09ec1f87102a6cbee8d8f

10 years agoMerge "jquery.arrowSteps: Center arrowheads text"
jenkins-bot [Fri, 11 Jul 2014 21:59:23 +0000 (21:59 +0000)]
Merge "jquery.arrowSteps: Center arrowheads text"

10 years agoMerge "Remove ResourceLoaderGetStartupModules hook"
jenkins-bot [Fri, 11 Jul 2014 21:20:13 +0000 (21:20 +0000)]
Merge "Remove ResourceLoaderGetStartupModules hook"

10 years agoMerge "Note that jquery.json will be removed in 1.25"
jenkins-bot [Fri, 11 Jul 2014 21:17:45 +0000 (21:17 +0000)]
Merge "Note that jquery.json will be removed in 1.25"

10 years agoMerge "QUnit: Detect unfinished jQuery effects or ajax requests in teardown"
jenkins-bot [Fri, 11 Jul 2014 21:15:47 +0000 (21:15 +0000)]
Merge "QUnit: Detect unfinished jQuery effects or ajax requests in teardown"

10 years agoNote that jquery.json will be removed in 1.25
Matthew Flaschen [Fri, 11 Jul 2014 20:43:09 +0000 (16:43 -0400)]
Note that jquery.json will be removed in 1.25

This is a pretty straightforward migration, and improves performance
for users with modern browsers (since the skip function means
browsers with native JSON don't need to download a JSON module)

Stating a removal version will encourage this migration.

Change-Id: I8fe83f30fd27f7bfe50e756846486abdadb91189

10 years agoMerge "Warn when DISPLAYTITLE is used more than once"
jenkins-bot [Fri, 11 Jul 2014 21:01:54 +0000 (21:01 +0000)]
Merge "Warn when DISPLAYTITLE is used more than once"

10 years agoMerge "Adding a bit more documentation to mw.loader.using"
jenkins-bot [Fri, 11 Jul 2014 20:50:11 +0000 (20:50 +0000)]
Merge "Adding a bit more documentation to mw.loader.using"

10 years agoRemove ResourceLoaderGetStartupModules hook
Ori Livneh [Fri, 11 Jul 2014 20:49:00 +0000 (13:49 -0700)]
Remove ResourceLoaderGetStartupModules hook

Deprecated only since 1.23, but universally reviled, and used only by
MwEmbedSupport, which no longer uses it as of I43af3c87a).

Change-Id: I36c89b273fdb0ec3a76034c7a6d2f48a15d5457f

10 years agoMerge "Take redirect modifications into account for If-Modified-Since"
jenkins-bot [Fri, 11 Jul 2014 20:48:28 +0000 (20:48 +0000)]
Merge "Take redirect modifications into account for If-Modified-Since"

10 years agoTake redirect modifications into account for If-Modified-Since
Brad Jorsch [Fri, 11 Jul 2014 14:05:36 +0000 (10:05 -0400)]
Take redirect modifications into account for If-Modified-Since

When the pageview is following a redirect, MediaWiki is only considering
the page_touched timestamp for the target page. This causes issues with
handling If-Modified-Since if the redirect is retargeted after the
target page was last touched.

The simple solution is to use the maximum page_touched of the page or
the redirect (or any other redirect in the chain, if
$wgMaxRedirects > 1).

Bug: 67849
Change-Id: Ia359aaff6eab34c1d609254055c122a644fa7cce

10 years agoAdding a bit more documentation to mw.loader.using
kaldari [Fri, 11 Jul 2014 20:02:15 +0000 (13:02 -0700)]
Adding a bit more documentation to mw.loader.using

Change-Id: I3c9a838f5b110de898d532331e387b17901ccdea

10 years agoLocalisation updates from https://translatewiki.net.
Translation updater bot [Fri, 11 Jul 2014 19:42:59 +0000 (21:42 +0200)]
Localisation updates from https://translatewiki.net.

Change-Id: I72ac51b2eeec6e1b1d0711cddc42c75319b99dff

10 years agoAdded ApiResult::NO_SIZE_CHECK flag for addValue()
Yuri Astrakhan [Mon, 7 Jul 2014 18:07:14 +0000 (14:07 -0400)]
Added ApiResult::NO_SIZE_CHECK flag for addValue()

This way we no longer need to disable size checking just for one operation
(enable|disable)SizeCheck functions were depricated.

Overall, this is a much better practice than disabling than re-enabling
the flag, as it might lead to accidentally forgetting to re-enable it,
just like the issue with the dangling file handlers, etc.

Example:

disable, do some complex logic, re-enable.  And later, by accident,
the complex logic is changed to return a value half-way, or throws
an exception that gets handled as part of normal operations. This
results in the unsafe disabled state of the result object,
which is not good (tm).

Change-Id: I389a334d35f52f23a1847aca4aef5e96b262f589

10 years agoMerge "Made getMaxLag() use caching to reduce connection spam"
jenkins-bot [Fri, 11 Jul 2014 18:02:10 +0000 (18:02 +0000)]
Merge "Made getMaxLag() use caching to reduce connection spam"

10 years agoMerge "Made getMaxLag() skip checking the master DB"
jenkins-bot [Fri, 11 Jul 2014 17:54:24 +0000 (17:54 +0000)]
Merge "Made getMaxLag() skip checking the master DB"

10 years agoMerge "Removes deprecated $wgUseCommaCount variable"
jenkins-bot [Fri, 11 Jul 2014 17:34:43 +0000 (17:34 +0000)]
Merge "Removes deprecated $wgUseCommaCount variable"

10 years agoRemove MWInit class file Init.php
withoutaname [Tue, 1 Jul 2014 00:01:45 +0000 (17:01 -0700)]
Remove MWInit class file Init.php

Change-Id: I370b9d68d412c8f47b68be2b54ec1e86b7c06fd7

10 years agoMerge "Set autofocus on Special:Unblock"
jenkins-bot [Fri, 11 Jul 2014 14:39:47 +0000 (14:39 +0000)]
Merge "Set autofocus on Special:Unblock"

10 years agoAdded VERP functionality hook to core
Tony Thomas [Tue, 10 Jun 2014 18:32:42 +0000 (00:02 +0530)]
Added VERP functionality hook to core

* By default, wiki should keep the envelope sender as $wgPasswordSender
* Requires BounceHandler extension. Added hook to core
* Alters the return path, to bounce-{key}@domain
* The envelope sender is set using the 5th param in mail()

Bug: 46640
Needed By: I9463ae33ae327405725ea9693d45ad61b8ecf798
Change-Id: Ic5c1231611a5c4984ddf81fd716e6f3a3e82fd57

10 years agoRemoves deprecated $wgUseCommaCount variable
Rohan [Mon, 16 Jun 2014 14:52:27 +0000 (20:22 +0530)]
Removes deprecated $wgUseCommaCount variable

Also sets the default value of $wgArticleCountMethod to 'link'

Bug: 60454
Change-Id: I31c3fe837b59e693d03ed403c8cb52e50beccddf

10 years agoQUnit: Detect unfinished jQuery effects or ajax requests in teardown
Timo Tijhof [Fri, 11 Jul 2014 01:25:19 +0000 (03:25 +0200)]
QUnit: Detect unfinished jQuery effects or ajax requests in teardown

Based on jQuery test suite
https://github.com/jquery/jquery/blob/bcca4f041/test/data/testrunner.js#L113-L125

Change-Id: I1dba146a6795887c23c806f292c6425662ea4aad

10 years agoRemove $wgFileStore
withoutaname [Thu, 12 Jun 2014 19:45:54 +0000 (12:45 -0700)]
Remove $wgFileStore

Change-Id: I58cf67ac1f2754a472b49b0b6d7f2b289deb2d4a

10 years agoUpgrade e5-shim to v4.0.0
Timo Tijhof [Thu, 10 Jul 2014 20:44:32 +0000 (22:44 +0200)]
Upgrade e5-shim to v4.0.0

Source code:
 https://github.com/es-shims/es5-shim/blob/v4.0.0/es5-shim.js

Follows-up 61c81dedcdca49, and ea307ea.

Notable changes;
* Removes breaking polyfill for Array#splice
  https://github.com/es-shims/es5-shim/issues/255

Change-Id: Id1bf4b5f35fed05817f77e087cd489aed719ecd5

10 years agoUpdate Sinon.JS to 1.10.3
Timo Tijhof [Thu, 10 Jul 2014 12:55:04 +0000 (14:55 +0200)]
Update Sinon.JS to 1.10.3

Sinon.JS 1.10.2 had a regression causing a fatal https://github.com/cjohansen/Sinon.JS/issues/506
Fixed in version 1.10.3.

Source code:
 http://sinonjs.org/releases/sinon-1.10.3.js
 http://sinonjs.org/releases/sinon-ie-1.10.3.js

Change log:
 https://github.com/cjohansen/Sinon.JS/blob/v1.10.3/Changelog.txt#L1-L25

Change-Id: Ib667afc89c54b435b9d37e0d83a9aacdc7bb9807

10 years agoMerge "New 'UserResetAllOptions' hook from the wikiHow codebase."
jenkins-bot [Thu, 10 Jul 2014 21:51:41 +0000 (21:51 +0000)]
Merge "New 'UserResetAllOptions' hook from the wikiHow codebase."

10 years agoNew 'UserResetAllOptions' hook from the wikiHow codebase.
Jack Phoenix [Tue, 8 Jul 2014 13:14:27 +0000 (16:14 +0300)]
New 'UserResetAllOptions' hook from the wikiHow codebase.

Location is a tad bit different now than on the wikiHow codebase and some
arguments have been added, as per code review.

Because sometimes you have things that are stored in the user_properties
database table that should *not* be reset even when the user has requested
to reset all prefs back to the site defaults.

Live example of a thing using this hook (well, its previous iteration) is
wikiHow's WikihowPreferences extension.

Change-Id: I1da936c786adb21e2c1802ef405bb904c9cf4918

10 years agoMerge "Change message on Special:Wantedfiles based on if foreign repo on."
jenkins-bot [Thu, 10 Jul 2014 19:29:55 +0000 (19:29 +0000)]
Merge "Change message on Special:Wantedfiles based on if foreign repo on."

10 years agoMerge "Handle invalid language code gracefully in Language::fetchLanguageNames"
jenkins-bot [Thu, 10 Jul 2014 18:50:25 +0000 (18:50 +0000)]
Merge "Handle invalid language code gracefully in Language::fetchLanguageNames"

10 years agoMerge "thumb.php: Fix undefined variable $width"
jenkins-bot [Thu, 10 Jul 2014 18:39:37 +0000 (18:39 +0000)]
Merge "thumb.php: Fix undefined variable $width"

10 years agoMerge "API: Fix list=deletedrevs with pre-1.5 revisions"
jenkins-bot [Thu, 10 Jul 2014 18:37:26 +0000 (18:37 +0000)]
Merge "API: Fix list=deletedrevs with pre-1.5 revisions"

10 years agoSync up with Parsoid parserTests.
C. Scott Ananian [Thu, 10 Jul 2014 17:46:45 +0000 (13:46 -0400)]
Sync up with Parsoid parserTests.

This now aligns with Parsoid commit 29cdb1ddab0ee8884abe1062c5f9510855988ed4

Change-Id: If25a2e8472ef7660470d0760938a5cb54fe2a0c2

10 years agoMade getMaxLag() use caching to reduce connection spam
Aaron Schulz [Thu, 10 Jul 2014 17:46:42 +0000 (10:46 -0700)]
Made getMaxLag() use caching to reduce connection spam

* This method is currently hit when bots use the maxlag API parameter
* Also added a $fallback param to ObjectCache::newAccelerator so code
  does not have to keep reinventing that logic.

Change-Id: I45bf1085301c5cf69b29b494a146c6a067534581

10 years agoUpdate es5-shim to latest master (40181d720e07)
Kevin Israel [Sun, 6 Jul 2014 12:08:38 +0000 (08:08 -0400)]
Update es5-shim to latest master (40181d720e07)

Includes a fix for <https://github.com/es-shims/es5-shim/issues/253>.
The bug makes es5-shim unusable in the browsers we are most interested
in using it for: old versions of IE.

Source code:
 https://github.com/es-shims/es5-shim/blob/40181d720e/es5-shim.js

Bug: 67565
Change-Id: Iee964bea6fcc1071871e457643b77ffd452e1040

10 years agoMerge "Remove WikiPage::isBigDeletion() (deprecated since 1.19)"
jenkins-bot [Thu, 10 Jul 2014 17:29:38 +0000 (17:29 +0000)]
Merge "Remove WikiPage::isBigDeletion() (deprecated since 1.19)"

10 years agoMerge "Remove Language::getFallbackLanguageCode() (deprecated since 1.19)"
jenkins-bot [Thu, 10 Jul 2014 17:28:57 +0000 (17:28 +0000)]
Merge "Remove Language::getFallbackLanguageCode() (deprecated since 1.19)"

10 years agoMerge "Remove Title::getEscapedText() (deprecated since 1.19)"
jenkins-bot [Thu, 10 Jul 2014 17:28:33 +0000 (17:28 +0000)]
Merge "Remove Title::getEscapedText() (deprecated since 1.19)"

10 years agoMerge "Remove User::isValidEmailAddr() (deprecated since 1.18)"
jenkins-bot [Thu, 10 Jul 2014 17:27:13 +0000 (17:27 +0000)]
Merge "Remove User::isValidEmailAddr() (deprecated since 1.18)"

10 years agoRemove WikiPage::isBigDeletion() (deprecated since 1.19)
umherirrender [Thu, 10 Jul 2014 17:03:58 +0000 (19:03 +0200)]
Remove WikiPage::isBigDeletion() (deprecated since 1.19)

Change-Id: If47bae3cfd22e7d584d189c761ccb1bef8ca4c2e

10 years agoRemove Language::getFallbackLanguageCode() (deprecated since 1.19)
umherirrender [Thu, 10 Jul 2014 16:59:59 +0000 (18:59 +0200)]
Remove Language::getFallbackLanguageCode() (deprecated since 1.19)

Change-Id: Ie3a1505763d7c7358a7b913bb2a8177b18c5c5bd

10 years agoRemove Title::getEscapedText() (deprecated since 1.19)
umherirrender [Thu, 10 Jul 2014 16:57:29 +0000 (18:57 +0200)]
Remove Title::getEscapedText() (deprecated since 1.19)

Change-Id: I54dfbe024fee9e82f99a394f7f41141761375ca4

10 years agoMerge "Html: Throw exception if array is used for an attribute not supporting it"
jenkins-bot [Thu, 10 Jul 2014 16:53:31 +0000 (16:53 +0000)]
Merge "Html: Throw exception if array is used for an attribute not supporting it"

10 years agoRemove User::isValidEmailAddr() (deprecated since 1.18)
umherirrender [Thu, 10 Jul 2014 16:52:34 +0000 (18:52 +0200)]
Remove User::isValidEmailAddr() (deprecated since 1.18)

Change-Id: I3440e32768d03e9160e23b038b6f4dc830f78647

10 years agoMerge "Remove outdated unused jquery.tipsy styles from common/shared.css"
jenkins-bot [Thu, 10 Jul 2014 16:52:28 +0000 (16:52 +0000)]
Merge "Remove outdated unused jquery.tipsy styles from common/shared.css"

10 years agoHtml: Throw exception if array is used for an attribute not supporting it
Thiemo Mättig [Sat, 10 May 2014 08:58:19 +0000 (10:58 +0200)]
Html: Throw exception if array is used for an attribute not supporting it

Previously the behavior was more or less undefined for most attributes
except for the ones in $spaceSeparatedListAttributes (currently 'class',
'accesskey' and 'rel'). If an other attribute is set to an array (no
matter what it contains) the method produces broken HTML like
'key=' and only triggers a warning if error_reporting is enabled. If
error_reporting is not enabled a developer may overlook this.

To clarify: The method always *ALLOWS* array values. This is *NOT*
about unexpected types in the call signature but unexpected
combinations of nested values. These combinations are already
checked in the method but the check was incomplete.

I considered several solutions:
* Simply use the first array element. But we can't know if the first
  element is what the caller expected.
* Silently drop all arrays if the attribute doesn't allow lists. This
  is close to the current behavior of always returning 'key=' but is a
  breaking change for boolean attributes like 'checked' and 'selected'.
  Browsers accept the current 'checked=' as true while omiting the
  attribute means false.

Choosing to always throw an exception. As above, this is a
breaking change in some cases.

Change-Id: Id5fcbdef2696d0a81a91d54338939ee678475ca3

10 years agoRemove outdated unused jquery.tipsy styles from common/shared.css
Thiemo Mättig [Thu, 10 Jul 2014 14:59:19 +0000 (16:59 +0200)]
Remove outdated unused jquery.tipsy styles from common/shared.css

This code is really, really old and not compatible with jquery.tipsy
anymore. Look at the image. It's a single arrow. That's not how
the tipsy version we are currently using works. It needs to be a
star with arrows pointing to all four directions.

Luckily this is just dead code. The jquery.tipsy.css file is loaded
after shared.css and overrides the outdated styles anyway.

Wikibase uses it's own styles for the tipsy tooltips. So there are
three places that try to style the tipsy tooltips: shared.css,
tipsy.css and Wikibase' wbtooltip.css. For some reason this broke
the ULS tooltips on wikidata.org. I would like to fix this mess by
removing all individual styles. I will submit an other patch that
removes the individual styles in Wikibase.

Bug: 67802
Change-Id: Idfd3a5e30dd2f24dbd588a30c1f516d5d219ced5

10 years agoMerge "Use PHP_VERSION constant instead of phpversion() function call"
jenkins-bot [Thu, 10 Jul 2014 16:19:55 +0000 (16:19 +0000)]
Merge "Use PHP_VERSION constant instead of phpversion() function call"

10 years agoBackport SinonJS bugfix for broken splice() in IE8 and es5-shim
Timo Tijhof [Thu, 10 Jul 2014 13:36:30 +0000 (15:36 +0200)]
Backport SinonJS bugfix for broken splice() in IE8 and es5-shim

SinonJS 1.9.1 made use of a rare feature in Array#splice that is
implemented in most browsers but not actually part of the ES5 spec.

And in IE8 and es5-shimmed environments, the method was 'broken'
and thus always yielded an empty array causing none of the XHRs
to be responded to.

We can't upgrade to SinonJS 1.10 right now due to
https://github.com/cjohansen/Sinon.JS/issues/506.

Patch source:
 https://github.com/cjohansen/Sinon.JS/commit/60ab124d21

Change-Id: I8882198b1609cb78f16a8269d40bb19df2df127a

10 years agoMerge "jquery.makeCollapsible: No tabindex="0" for default buttons"
jenkins-bot [Thu, 10 Jul 2014 12:28:31 +0000 (12:28 +0000)]
Merge "jquery.makeCollapsible: No tabindex="0" for default buttons"

10 years agoMerge "Remove unused message 'postcomment'"
jenkins-bot [Thu, 10 Jul 2014 07:24:50 +0000 (07:24 +0000)]
Merge "Remove unused message 'postcomment'"

10 years agoMerge "GlobalFunctions: Deprecate swap()"
jenkins-bot [Thu, 10 Jul 2014 00:50:28 +0000 (00:50 +0000)]
Merge "GlobalFunctions: Deprecate swap()"

10 years agoUse PHP_VERSION constant instead of phpversion() function call
Chad Horohoe [Wed, 9 Jul 2014 23:46:35 +0000 (16:46 -0700)]
Use PHP_VERSION constant instead of phpversion() function call

Change-Id: Ifb3d1bd92d6abaa561e7337b311b4cb10c38a2b6

10 years agoMerge "Web installer: check for low PHP versions before executing"
jenkins-bot [Wed, 9 Jul 2014 23:41:38 +0000 (23:41 +0000)]
Merge "Web installer: check for low PHP versions before executing"

10 years agoMerge "If $wgEnableEmail is false, disable other email settings"
jenkins-bot [Wed, 9 Jul 2014 23:38:50 +0000 (23:38 +0000)]
Merge "If $wgEnableEmail is false, disable other email settings"

10 years agothumb.php: Fix undefined variable $width
Sergio Santoro [Wed, 9 Jul 2014 21:29:41 +0000 (23:29 +0200)]
thumb.php: Fix undefined variable $width

Follows-up 5ca94d2d26e80bb2a4ea9339a3adc31baf186d3a.

In thumb.php an undefined variable ($width) was used, triggering
fatal errors while processing requests that were passing thumbnail
width with px (pixel) suffix.

Change-Id: I2dc4f4d2979f8225d2a6211ec50cdc79438539a9

10 years agoMerge "Refactor Preferences code to allow better reuse."
jenkins-bot [Wed, 9 Jul 2014 21:34:32 +0000 (21:34 +0000)]
Merge "Refactor Preferences code to allow better reuse."

10 years agoRefactor Preferences code to allow better reuse.
jdlrobson [Mon, 16 Jun 2014 17:26:15 +0000 (10:26 -0700)]
Refactor Preferences code to allow better reuse.

Break up getPreferences into smaller subfunctions to
allow for reuse.

Change-Id: Idd33bab61b3ef5a262fee9361105f1c5d8024044

10 years agoes5-shim: Support PhantomJS (has strict mode, but no Function#bind)
Timo Tijhof [Wed, 9 Jul 2014 20:58:39 +0000 (22:58 +0200)]
es5-shim: Support PhantomJS (has strict mode, but no Function#bind)

Follows-up fff9a814f.

The es5-shim wasn't being loaded in PhantomJS causing tests to
fail due to absence of Function#bind.

Change-Id: I84b8ebe382bfc4825accc31a0a5da2fb1717536d

10 years agoMerge "Create valid HTML on Search page"
jenkins-bot [Wed, 9 Jul 2014 20:50:42 +0000 (20:50 +0000)]
Merge "Create valid HTML on Search page"

10 years agoLocalisation updates from https://translatewiki.net.
Translation updater bot [Wed, 9 Jul 2014 20:27:56 +0000 (20:27 +0000)]
Localisation updates from https://translatewiki.net.

Change-Id: Ib3532d709f98117b2e23ccfda1a9b74d3db398a2

10 years agoMade getMaxLag() skip checking the master DB
Aaron Schulz [Wed, 9 Jul 2014 19:11:32 +0000 (12:11 -0700)]
Made getMaxLag() skip checking the master DB

Change-Id: I81a9102570bf9a45af56f31d04e3a0f2be72b919

10 years agoCreate valid HTML on Search page
florianschmidtwelzow [Wed, 9 Jul 2014 18:29:29 +0000 (20:29 +0200)]
Create valid HTML on Search page

Create valid HTML on search page and close clearing div.

Follows up: If0b6df347315bfe5b6f9a7aa9c6730cbdf44fd38

Bug: 67740
Change-Id: I3ee40b29e54aec91b627fe19de1c685c058aa184

10 years agoMerge "Minor: Remove unnecessary @noflip annotation"
jenkins-bot [Wed, 9 Jul 2014 18:29:06 +0000 (18:29 +0000)]
Merge "Minor: Remove unnecessary @noflip annotation"

10 years agoMerge "Use the user from the context to format the date in Block::getPermissionsError()"
jenkins-bot [Wed, 9 Jul 2014 18:18:46 +0000 (18:18 +0000)]
Merge "Use the user from the context to format the date in Block::getPermissionsError()"

10 years agoMinor: Remove unnecessary @noflip annotation
Gabriel Wicke [Wed, 9 Jul 2014 18:17:14 +0000 (11:17 -0700)]
Minor: Remove unnecessary @noflip annotation

Follow-up to 9573eb29678.

Change-Id: If5a289e67ad110c2489087f9dde995391f851e98

10 years agoIf $wgEnableEmail is false, disable other email settings
withoutaname [Fri, 9 May 2014 08:24:27 +0000 (01:24 -0700)]
If $wgEnableEmail is false, disable other email settings

Bug: 63678
Change-Id: I8464f540f4bd850d6be1c578b043d9702ed8e773

10 years agoMerge "Allow third party code to hook-up MIME type detection"
jenkins-bot [Wed, 9 Jul 2014 17:47:44 +0000 (17:47 +0000)]
Merge "Allow third party code to hook-up MIME type detection"

10 years agoGive "mergehistory" to sysops
Aaron Schulz [Wed, 9 Jul 2014 00:22:27 +0000 (17:22 -0700)]
Give "mergehistory" to sysops

Follows-up cf8e994053af and 9b070bf461ddf1 (r27823).

Bug: 66155
Change-Id: Iaaa30afbdc834faf0fb5f68bc82c7d8aef8fcec8

10 years agoRemove Title::escapeFullURL() (deprecated since 1.19)
umherirrender [Wed, 9 Jul 2014 17:19:42 +0000 (19:19 +0200)]
Remove Title::escapeFullURL() (deprecated since 1.19)

Change-Id: Ie3f50fa38b92294ffb67daaaba8560a640bf2eeb

10 years agoRemove Title::escapeLocalURL() (deprecated since 1.19)
umherirrender [Sat, 28 Jun 2014 13:41:39 +0000 (15:41 +0200)]
Remove Title::escapeLocalURL() (deprecated since 1.19)

Change-Id: I588e1946f6dc9200261469ae8134f7cf885bba3f

10 years agoRemove Title::escapeCanonicalURL() (deprecated since 1.19)
umherirrender [Sat, 28 Jun 2014 13:40:16 +0000 (15:40 +0200)]
Remove Title::escapeCanonicalURL() (deprecated since 1.19)

Change-Id: Ib451c25643e65c5812046324d998f5286b36d21d

10 years agoMerge "Remove ParserOptions::setSkin() (deprecated since 1.19)"
jenkins-bot [Wed, 9 Jul 2014 17:20:08 +0000 (17:20 +0000)]
Merge "Remove ParserOptions::setSkin() (deprecated since 1.19)"

10 years agoMerge "Fix MediaWikiPageLinkRendererTest failing in non-English setups"
jenkins-bot [Wed, 9 Jul 2014 17:01:12 +0000 (17:01 +0000)]
Merge "Fix MediaWikiPageLinkRendererTest failing in non-English setups"

10 years agoMerge "Fix TitlePermissionTest failing on non-English setups"
jenkins-bot [Wed, 9 Jul 2014 17:00:57 +0000 (17:00 +0000)]
Merge "Fix TitlePermissionTest failing on non-English setups"

10 years agoMerge "Skip 64 bit MWMessagePack tests on 32 bit machines"
jenkins-bot [Wed, 9 Jul 2014 16:59:58 +0000 (16:59 +0000)]
Merge "Skip 64 bit MWMessagePack tests on 32 bit machines"

10 years agoMerge "Change some mixed @param docs in api classes"
jenkins-bot [Wed, 9 Jul 2014 16:59:24 +0000 (16:59 +0000)]
Merge "Change some mixed @param docs in api classes"

10 years agoChange some mixed @param docs in api classes
umherirrender [Thu, 3 Jul 2014 19:29:02 +0000 (21:29 +0200)]
Change some mixed @param docs in api classes

Change-Id: I2ab4b0ae452325392c4ea72ab3d985107d429ac2