Merge "Always return something nice in Language::translateBlockExpiry()"

Test un-closed noinclude, onlyinclude and includeonly tags

Change-Id: I328d1e2f5c1362caa0cb32efcd7b81dab7bab8c5

Always return something nice in Language::translateBlockExpiry()

Change-Id: I30a1950df5ae018cb9124392dc8d6e99ca3b98b8

Merge "(bug 36053) Login returnto doesn't work if title isn't in the URI"

(bug 36053) Login returnto doesn't work if title isn't in the URI

Returnto after login didn't work if title wasn't given as URI
parameter. I'm using $this->getTitle() to find it out, in case
the user got read rights (per code comment above my change).

Change-Id: I14d92581ce790355404d3c184fa6542a24f7a130

Merge "(bug 40632) Remove CleanupPresentationalAttributes feature"

Test non-ASCII / URL pseudo-tags

Make sure these are rendered and round-tripped properly.

* Tags with names outside [a-zA-Z][a-zA-Z0-9]* are rendered as plain text
* Angle brackets around URLs render as angle brackets around URL links

Change-Id: I7f0653cbbbb6ccb9a9aa0ecfa29a0c2e78091343

JavaScriptContentHandler: Fix typo in comment block (actual: 'CSS'; intended: 'JS')

Change-Id: Iaef21f0c864222aee7c970012e8163741d6310ca

(bug 40632) Remove CleanupPresentationalAttributes feature

Removed $wgCleanupPresentationalAttributes, the associated
code it toggles and references to those in src and tests.

Also fixes bug 40329.

This was originally introduced in r94465 (released in REL1_19) but
disabled by default. Then enabled in r98053, after which several
bugs were filed and eventually the decision was made to remove
this feature.

Removed obsolete release-note entry, as this is to be backported
to REL1_20.

Change-Id: I4e86305520a3b22ef88381caab55d24abac932e3

Merge "Fix PHP notice in Category.php"

Merge "Follow-up I214a988e (42a8daa): move documentation to the correct block"

Fix PHP notice in Category.php

Change-Id: I52838a1125298da330f60ef1e761130ff0e4faba

Follow-up I214a988e (42a8daa): move documentation to the correct block

These lines belongs to the whole file, not only to that class.

Change-Id: Ib61eae1e8f7d50d29a929c6034ede08a851105c9

Merge "(bug 26280) all files should have GPL header"

Merge "profileinfo links now have anchors"

Merge "profileinfo now has lower lines heights"

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I9083fea439c7bf545a7d2971d204513fab51c04c

Merge "Lint: Pass jquery.tablesorter."

Lint: Pass jquery.tablesorter.

Added tolerance for "onevar" (hard to dissect, many out-of-scope
usage, probably better to be done by someone who knows this code
better).

Errors:
* Strings must use singlequote
* '$cell' is defined but never used.
* 's' is defined but never used.
* Unnecessary semicolon.
* '$document' is defined but never used.
* 'sortOrder' is defined but never used.
* 'shiftDown' is defined but never used.
* 'table' is defined but never used.
* 'table' is defined but never used.

Change-Id: I2e0abc41a79922a1810519705237ae3bd60ed679

Lint: Pass mediawiki.language.

Maybe they'll get closures soon, but for now just fix the
warnings so we can get on with enforcing jshint from jenkins like
we do phplint/puppetlint.

Added tolerance for "onecase" and disabled "noempty" in a few
files. Keeps these files easier to update. Though they should be
restructured, I'm not going to do that now.

Change-Id: Ie3535f22fc88ea5272d7387cef870c8c042a9393

Merge "Add release note for Ieec65c90"

Add release note for Ieec65c90

Gerrit change Ieec65c90 did not update RELEASE-NOTES-1.21.

Change-Id: I4820060b8459682ed28c05c2b69a89321b01183e

Cleanup: Fix yoda and indentation in SpecialUserlogin.php

Change-Id: I0bde21a8da679350559779132dbd7b1ba85116d1

profileinfo links now have anchors

Whenever expanding a section in profileinfo, the browser would sent us
to the top again which would force us to browse down to where we were.
Adding basic anchors send the user directly where he was browsing.

Note that HTML5 let you specify anything as an id as long as there is at
least one character.

Change-Id: I5b55297ce25d5c07b29610798fb39eae16d27f6a

profileinfo now has lower lines heights

The 8px padding was a bit too tall, "reduced" that to 0.1em which is
good enough. I have also dropped the line-height which does not have any
purpose when you already apply a padding.

Change-Id: I271eb932e4c70f940ebe3da39731f09c998610bc

Merge "add SpecialUndeleteLogEntry hook"

Merge "Log profiling data when tests have finished."

Merge "Clean up database cloning for tests."

update.php now create profiling table when needed

When enabling $wgProfileToDatabase, one would have to manually apply a
patch to the database that would add the `profiling` table. This patch
let update.php creates the table whenever $wgProfileToDatabase is true.

This also provide a SQL patch for SQLite backend and update
profileinfo.php to give some clue about enabling the global and running
update.php

Change-Id: If68a25f7ec2b0fbb61f82a318427abe58a89dae7

add SpecialUndeleteLogEntry hook

- allows extensions to perform some action at time that a undeletion log entry is performed,
or skip the insertion of a log entry.

Change-Id: I5b0caaddfe5d7262612a60946d9f5c4af2679141

Fix typo (manuall -> manually)

Change-Id: Ia6101ea84faf9a4420afe9568b5e3904fffa61b1

Merge "Use wfBCP47() to format language code for IE in Accept-Language."

(bug 39067) Add support for HTML5 <mark> element.

* whitelist <mark> in tidy and sanitizer
* provides a default styling for mark elements

Change-Id: I23fc2fc558ff0590be04771ef1e75fcfdf240aac

support for HTML5/Microdata elements

Patch let us handle the <data>, <time>, <meta>, and <link> elements.

* handles one part of bug 32545 requesting us to support the <time>
  element in WikiText.
* Partially fix bug 28776 about whitelisting global HTML5 semantic
  attributes and inline meta element.
* <meta> and <link> are only permitted when Microdata is enabled using
* the global $wgAllowMicrodataAttributes. For for security reason, the
  links are only allowed to be actual elements when they have a
  strict set of attributes set.

Change-Id: Ica11be186bd62eb154e1ebc400acb515c10fb65f

Test horizontal rulers followed by content on same line

Change-Id: I165975fe5faf0205f8f94ab9f55f083b7d48dcaa

Merge "(bug 26617) clean up usages of list-style in Vector&Monobook CSS"

(bug 26617) clean up usages of list-style in Vector&Monobook CSS

* remove list-style-*: none from non-list elements
* remove duplicate declarations
* always use list-style-type:none; list-style-image:none; instead of
  list-style:none to work around an IE bug
* set list-style-type to square for Monobook and to disc for Vector
  to resemble the icons used

Change-Id: I0d10acc9f07f1bf4239c70b476baea9a0ae40391

Localisation updates from translatewiki.net.

Change-Id: Idddaf093a4bc45e3cb634176f83a40ae1f047a4e

(bug 26280) all files should have GPL header

MediaWiki is licensed under GPL so we should really add the header to
our PHP code unless reusing code licensed differently.

Tested using:
  ack-grep --php -L 'This program is free software' \
    includes/ maintenance/ mw-config/ resources/ skins/

Output the following files:
 includes/ExternalEdit.php (public domain)

Various other licenses:
 includes/json/Services_JSON.php
 includes/libs/CSSMin.php
 includes/libs/IEContentAnalyzer.php
 includes/libs/JavaScriptMinifier.php
 includes/libs/jsminplus.php
 includes/normal/Utf8Case.php
 includes/objectcache/MemcachedClient.php

File autogenerated from a GPL file and Unicode data:
 includes/ZhConversion.php

A few lines coming from stack overflow:
 maintenance/mwdoc-filter.php

Message files do not have any license :-/

Change-Id: I214a988edfa9e2dfcc762612a0f7d47eee8bcb74

Merge "updater now shows the SQLite file being used"

Merge "Replace deprecated HTML attributes"

Merge "Undefined offset in ApiQueryLogEvents"

Replace deprecated HTML attributes

Replace deprecated bgcolor="" with style="background-color:;", and remove deprecated cellpadding and cellspacing (which is not needed since the wikitable class is used).

Change-Id: I0f332fb520ea88e3d3f1abce3c423f95a5c224c8

Merge "Set correct language for page content in action=historysubmit&revisiondelete=1"

Set correct language for page content in action=historysubmit&revisiondelete=1

Change-Id: Iabfb9a58e023b56556100b340f8ebdb0bdc51d8f

Undefined offset in ApiQueryLogEvents

Follow up Ie188bc6f: Necessary changes to api for rights log changes

Getting for new style logs:
Notice: Undefined offset: 1 in \includes\api\ApiQueryLogEvents.php on
line 245
Notice: Undefined offset: 0 in \includes\api\ApiQueryLogEvents.php on
line 245

Added a legacy part as seen by patrol and move for the rights log action

Change-Id: I1d0bdfb483dba30572e8dcf8faac331a77eb04c6

Localisation updates from translatewiki.net.

Change-Id: Ifa31cb11727543cf607fc06aea877dd4d11bc346

Use wfBCP47() to format language code for IE in Accept-Language.

It's reported that IE is actually sending language code in BCP 47
format (like zh-Hans), instead of uppercasing the whole 2nd part.

Change-Id: I6761191f299373d9267660a7eb14dc0b34cd5a38

RecentChangeTest now uses 'colon-separator' msg

LogFormatter uses "colon-separator" message instead of an hardcoded
string. Since the RecentChangeTest use hardcoded colon, tests would
break whenever the message in the site language is not exactly the same
as the english version.

Change-Id: I6360584caff53fda0b2688964ae614bc9fdcc97e

(bug 35204) set wgLanguageCode in unit tests

Setting $wgContLang without changing $wgLanguageCode accordingly is a
very bad idea.

This caused about 20 exceptions when using LanguageCode set to 'fr'
on my installation:

"MWException: Error in MediaWikiLangTestCase::setUp(): $wgLanguageCode
('fr') is different from $wgContLang->getCode() (en)"

Follow-up of I2987db68 (2b0edc8).

Change-Id: I56d3f1bfc78d6a2e40460533524e82f360490206

Merge "Delete more thumbnails after (new) parser tests."

Merge "Output special page name before attempting to run function"

Fix typo (cointent => content)

Change-Id: I4d6702d0b2a6c4fe224a8c59808a579b6c6916a0

Merge "Update jQuery to 1.8.3"

Update jQuery to 1.8.3

http://blog.jquery.com/2012/11/13/jquery-1-8-3-released/

Change-Id: I0bda71044ff6ea41e946d34e48fa279101f4bf90

Additional tests to spec behavior of wikitext in extlink-brackets.

* Mostly to keep Parsoid in line.

Change-Id: I1629b156f66cb3f1b511fbe26feb1bee1e1e0e59

Output special page name before attempting to run function

Related to bug 42210, we don't have a blank line, not knowing what it's actually running

QueryPage updates do this already!

Change-Id: I18eb0fb4c316744d92a81f5e64e201508b27eb27

Delete more thumbnails after (new) parser tests.

Same as I0a0e757a only for the new parser tests (under phpunit)

Change-Id: If2198676da7e8a621d02227b42da0713f67a1de0

Merge "(bug 37158) display personal menu on top of page tabs on Vector"

Unset DirectoryIterator in the FileBackend

This is necessory to close all handlers on the directory and than it is
possible to rmdir the directory without a 'permission denied' on a
windows machine.

Corresponding unit test failure:
1) FileBackendTest::testRecursiveClean
Dir mwstore://localtesting/unittest-cont1/e/a no longer exists
(FSFileBackend).
Failed asserting that true matches expected false.

Change-Id: Ide58c6d0ec101bf0ddcd6b7414c51075fc28825b

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: Iaa7b07c4e450b6d6ddbc53b9c095c8f16409f59c

Merge "[FileBackend] Stat caching improvements."

Merge "(bug 32368) Add ParserCloned hook"

[FileBackend] Stat caching improvements.

Callers often tend to end up calling getFileStat(), at least indirectly,
or in various successive function on the same path. This created RTTs
when the file didn't exist since negatives were not cached. This change
does the following:

* Cache definitive negatives (404s) in the process cache.
  Nothing is cached on failure (like network problems).
* Ignore process cache entries after a brief time period
  so long running scripts do not have overly stale entries.

Change-Id: I356bd9f48281e3c7e7a273778b2aca59c521a0c7

Merge "(bug 42173) Standardize categories special pages output"

Merge "Parser parse requires title, make a caller require valid title too"

Log profiling data when tests have finished.

Previously, no profiling data was recorded from unit test runs.
That made it impossible to a) use unit tests for selective profiling
of individual functions, and b) made it impossibel to profile
the tests themselves.

This change fixes this problem by calling wfLogProfilingData()
after the test runner has finished.

Thaks to Hashar for some ideas, especially the fix in GlobalFunctions.

Change-Id: Iaa295115f3c4eb3b529388dcd953fe8932448b3e

Parser parse requires title, make a caller require valid title too

Change-Id: I8a9cd18c5aa27493a0945765f80136cd0cc4f218

Clean up database cloning for tests.

This change makes the operation for overlaying the wiki database
with a structurally equivalent test database statically accessible,
and adds a function to revert that operation.

This is a precondition to logging profiling info to the database
after the test run has finished.

Change-Id: I2e8335d50eefcc7a97c1cd122b509f0939eacc92

Merge "(bug 40850) jquery.byteLimit: Always update prevSafeVal."

(bug 40850) jquery.byteLimit: Always update prevSafeVal.

Not doing this sometimes caused trimValForByteLength to compare
the new value to an empty string instead of the old value,
resulting in trimming always at the end instead of at the position
of insertion.

Change-Id: I2e46961efa4f82732d577f7e5f98fc80719c88bb

Merge "Add tests for empty attribute values in tags and table syntax"

Add tests for empty attribute values in tags and table syntax

The PHP parser implements this very inconsistently and not in line with the
HTML5 spec. We follow the HTML5 spec in Parsoid, so mark these tests as
disabled for the PHP parser and add FIXMEs.

Change-Id: Iee5d95f3ee5fcb596282097c52f729d33ccb1474

Merge "Regression test for detecting edit conflicts."

(bug 32368) Add ParserCloned hook

We store various bits of data as "expando" properties on the Parser
object, to pass information from one stage of the parser to another. If
the parser is cloned, however, we can run into trouble because two
different Parser objects are now manipulating the same extension data
structure; this often shows up when ParserClearState is called on one
clone and clears the state of the other as well.

Since a deep clone might be too expensive and still might be wrong in
some cases, it seems most useful to simply provide a ParserCloned hook
so extensions can just do The Right Thing.

Change-Id: Ieec65c908d71e89b9a66f83b9a626f842aadacbb

Merge "Let TextContentTest extends MediaWikiLangTestCase"

Merge "SearchEngineTest.php must extends MediaWikiLangTestCase"

(bug 42173) Standardize categories special pages output

This change removes namespace prefix from [[Special:UncategorizedCategories]],
so the output is consistent with the other categories pages.

Change-Id: I033e570754ca81be900a1f5482b8429583ab9172

Merge "Fix typo (seams => seems)"

Fix typo (seams => seems)

Change-Id: Ied64fa71b9da9762ee6bad7a4bae5a7082258f05

Merge "Fix prop=userid in list=protectedtitles"

Merge "drop orm_test table on teardown"

[Database] Fixed case where trx idle callbacks might be lost.

* (bug 41656) LoadBalancer always commits if there are callbacks pending.
  This works by checking a new DatabaseBase::writesOrCallbacksPending() function.
* Made transaction idle callbacks run in autocommit mode. Generally callers already
  want autocommit mode or do quick begin()/commit() calls anyway. The docs already
  make stat that callbacks should close any connections they start, but this makes
  it harder for people to forget about implicit transactions. Since the callbacks
  often may happen in commitMasterChanges(), if transactions are left open, they
  might not get committed by anything afterwards.
* Added sanity exceptions and warnings if callbacks get lost for some reason.
* Renamed trxIdleCallbacks -> mTrxIdleCallbacks for consistency.

Change-Id: I00e1e0fcdd7deeee1fbac6d0f295160479cb8962

Fixed uncaught thumbName() exceptions and use $_GET.

* Using $_REQUEST is bad since cookie data is also in there.
* Also made a few tiny code cleanups.

Change-Id: Ib7064b95a2788484d05cf457034fe1d5be298bc4

Add test for table row followed by newlines and table heading

Change-Id: I208f5464523ef3e76cd9ddc5c1198aa231bca9b8

Localisation updates from translatewiki.net.

Change-Id: I620f7ee98965b8d07caeeaf19d7db40efcfc7975

Regression test for detecting edit conflicts.

Change-Id: I9f8f3850ff0d6d51f173f5be73ffcb3b64886235

Merge "Made SSL validation in Curl HTTP requests the default."

Merge "Bug 42039 - Fix some file-related issues in the distribution Minor issues."

Merge "Uniform $rtl usage in Arabic-script Messages*.php"

Merge "Add 'contentmodel' to resultproperties in prop=revisions"

Merge "When returning rev content, always include model."

Merge "Fixed action=query list=blocks for IPv6 addresses"

Fix typo: exipration => expiration

Change-Id: I73d19cfeed0596b66e16e4c383b3658594c211e3

Moved "create" function definitions before "store" for consistency.

Change-Id: Ic44470d7a178cb8dcd8f6a9343f961c26fb92988

[JobQueue] Fixed de-listing updates to pending job cache.

Change-Id: Ied0f2962fcbcde62a041592933c667c861f54715

Add a test for indent pres with content wrapped in inline element

A line of content fully wrapped in an inline element was an issue in Parsoid
round-tripping, so make sure this is detected in the future.

Parsoid fix: https://gerrit.wikimedia.org/r/#/c/33496/

Change-Id: I2497c90174fac69e9c4dc1cc898844e94e604b1b

Fix intermittent QUnit failure

Occasionally the test 'mediawiki: mw.loader.implement( styles={ "url": {
<media>: [url, ..] } } )' will fail because it only ran 5 of 7 expected
assertions, while at the same time a later test will fail because it had
two extra assertions.

The problem is that the test is performing two external CSS loads, but
calls QUnit.start() after the ''first'' one is loaded. If the second
doesn't manage to get loaded at close to the same time, QUnit moves on
to the next test, and then when the second external CSS file finally
does load it will get counted towards whichever test is running at that
time. This can be reliably reproduced by arranging for
tests/qunit/data/styleTest.css.php to take longer for one than the other
of these two external CSS loads.

The solution is to wait until *both* of those external CSS loads to
complete (or time out) before calling QUnit.start().

Change-Id: Id695e95733b4c3f58234d9688c6b6e1b9ba591cc