Links created by Linker::makeExternalLink didn't include rel=nofollow

The links created by Linker::makeExternalLink didn't obey $wgNoFollowLinks
(and $wgNoFollowNsExceptions, $wgNoFollowDomainExceptions). To
fix this I made the function call Parser::getExternalLinkRel which
I implemented earlier.

Fixes bug 41983.

Change-Id: I2ff35f89502db2e1f8266f3a943e38c0ea67aced

Revert UIDGenerator commits.

Changes seemed to be merged without anyone really looking at it
and the authorship in git is wrong as well. All related changes
can later be batched as one commit.

Following commit / change are reverted:

 - 7ac3edd https://gerrit.wikimedia.org/r/36803
 - 036d03b https://gerrit.wikimedia.org/r/36813
 - 2b168e3 https://gerrit.wikimedia.org/r/36812
 - 6030d92 https://gerrit.wikimedia.org/r/36801

See wikitech-l thread
http://lists.wikimedia.org/pipermail/wikitech-l/2012-December/064801.html

Change-Id: Ie39fde2ee4121372b6d790943f306e56fbacc2b5

Merge "Add wfDeprecated to wfMsg* methods from 1.21 on"

Merge "Add 1.20 RELEASE-NOTES to HISTORY"

Merge "Add type hint"

Merge "(bug 37263) better handling of minor/bot in enhanced changelist"

Merge "(bug 36211) add cursor:pointer to .mw-enhancedchanges-arrow"

(bug 37263) better handling of minor/bot in enhanced changelist

It makes more sense, to have the whole group as minor/bot, when all of
the group are minor or bot edits, not when only one of the group is
minor/bot.

Change-Id: I62bf1099f2e3fb73e9b50374cbb7ba8bfd8a009e

(bug 36211) add cursor:pointer to .mw-enhancedchanges-arrow

With this, the arrow is shown with a pointer and looks like clicking on
a link. Without this, the user has no feedback, that he can click the
arrow.

Change-Id: I8bee6e51b1786785128d56931dc1ff9a09ba1d78

(bug 42619) Adjust comment for log_params database field

Change-Id: Ib78fd5ad7bef78b07460200d5d8b143e5a9a8802

Merge "Improve documentation for "pipe trick""

[JobQueue] Removed redundant DBO_TRX handling in JobQueueDB.

* The onTransactionIdle() function already handles this.

Change-Id: I4851e75a63a270984399cef288f370dcc3138e30

Add type hint

Change-Id: If529cfbf568a10e8cea184fa229959701268ebcb

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I41401477973247555f5be35d76620d7b9dc71510

Merge "Treat message cache loading as fatal"

Cleaned up UIDGenerator docs a bit.

Change-Id: I32ca575ca4d7b446b4a27b91d1e8aaa7173c575b

fix sort order

Change-Id: I07e29fbdae88ce5164ba4fb60dbb597dbc2c4dcf

Add UIDGenerator to AutoLoader

Change-Id: I288eef52590033002d7e36f0ca2f667f46dd6aec

Class for getting statistically unique IDs NOT a RFC4122 standard string representation

Change-Id: Ia4133b65afb63931b5f887feb247dac1b1e42360

Merge "add method to check if ORMTable exists"

New parser tests to spec wikitext behavior found on en:WP

* Added tests for:
  - table-list nesting interactions
  - templated-table tags
  - templated-list tags
  - stray *include* tags
  - autoclosed *include* tags

* These tests should help catch regressions from recent
  Parsoid fixes.

Change-Id: I54f0c13a16f4330fa258ff3718921b71bdafa6f0

Merge "Added a few tests to spec HTML-pre behavior and test Parsoid."

Merge "(bug 42483) Avoid serializing database object in update.php"

Added a few tests to spec HTML-pre behavior and test Parsoid.

* Renamed existing pre tests to use Indent-Pre to distinguish
  them from the HTML-pre tests.

Change-Id: I3500c3ba7db19b19baa1d9ab4e3bfb5ccbc491d2

add method to check if ORMTable exists

Change-Id: I7ef34545a9b4cfa6ed8bcdd302ad49faa7747974

Treat message cache loading as fatal

I suspect this is the primary cause we occasionally have
* Sidebar stuck in default values
* No gadgets at all
* Main page full of <int:custom-message> occurances

Change-Id: I811755d4c250800177c4ea9e509bf6e1d13443a0

Add wfDeprecated to wfMsg* methods from 1.21 on

These were marked soft deprecated in 1.18.

Change-Id: Ifa82da981c69b2ea9d5b298e9dfb04fc29483206

Add 1.20 RELEASE-NOTES to HISTORY

Change-Id: I2f5062c518017bdb26688df3278132befeb80826

Merge "(bug 42638) Fix API action=options&reset=1 & unit tests"

Make Language::isValidBuiltInCode() return bool

Make Language::isValidBuiltInCode() return true or false, as
documented. Previously returned 0 or 1. Hacked around in I81ba3228.

Change-Id: Iaa7515095d687d745d878faaa957aae51737abf7

Merge "(bug 42601) Make deleteOldRevisions.php no longer throw database errors on empty sets."

Localisation updates from translatewiki.net.

Change-Id: Ied9bc631fb1feac6b31ce015f9507537f4f16b7b

(bug 42483) Avoid serializing database object in update.php

Change I96b4cfd4 changed the data being stored in updatelog, trying to
store the mangled parameters (which may included an added copy of the
DatabaseUpdater object itself) instead of the input parameters. This
causes warnings because database objects cannot be effectively
serialized.

Go back to storing the original input parameters instead.

Change-Id: I96e6dffed98772ebb9b812773ff2f608e7e40cb1

Merge "(no bug) Fix per-connection db name in ORMTable."

[Upload] [Schema] Added us_props column to avoid expensive GETs.

* We already compute the props when stashing, so including that in a column
  lets us reuse it later when the stashed file is to be published.

Change-Id: I3d6adb6528f1e992d8986058806e745c4e1c0300

[JobQueue] Use SELECT for deduplication rather than more expensive DELETEs.

Change-Id: Iff5dbc0ca6dc7032596d1b4ffddb3cc1848b9e10

(no bug) Fix per-connection db name in ORMTable.

ORMTable was bypassing the mechanism used by DBAccessBase to
ensure that the correct database name is selected when getting
a database connection.

Change-Id: Ic82ce46f1852dcaa5f52b40660ba4a9f52df22f6

(bug 42601) Make deleteOldRevisions.php no longer throw database errors on empty sets.

Also changes DatabaseBase::query() calls to more specific ones.

Change-Id: I7b1eef9ac7cfc89c535a932477fcf1598d3bc0b2

(bug 27202) Add timestamp sort to list=allimages

New parameter 'sort' that defaults to 'name' so if not specified it will
act exactly as before.

Changes in the access to $params['dir'] to a boolean variable
$ascendingOrder in a try to be more meaningful.

Some existing filtering parameters are mutually exclusive with new ones.
The reason is because the table is sorted by name, as primary key, while
sorting by timestamp use the timestamp or usertext_timestamp index which
is not the primary key, thus filtering and ordering by different indices
doesn't make an efficient index usage.

Exceptions are:
* mime: disabled in Miser Mode
* size: although there's apparently no use for this when sorting by
  timestamp, there's an index on that and the EXPLAIN doesn't use filesort
  algorythm (unlike other combinations):

EXPLAIN
select img_name, img_user_text, img_timestamp
from image
where img_size>=1000 and img_size<=20000
and img_timestamp > '20120916153705'
ORDER BY img_timestamp\G

* Same for sha1 (see comment on Patchset 1)

In function run:

* Added USE INDEX query options when sort=timestamp to ensure the proper
  index is being used.
* Table selection and result fields moved at top, then filter parameters,
  and finally order by and options.

Parameter description and definition sorted by purpose: sort and dir
first, then pagination, result fields, filters and limit.

Change-Id: Ide2ff3dbc3e617e749685d558444854617535bab

test: raise timeout for DatabaseSqlite

DatabaseSqliteTest does file access and hence needs a slightly higher
timeout in PHPUnit.

Change-Id: I4b5eb7d7c5a3b9d4790f11cdb4efa9f578763183

test: raise timeout for ExifRotation

ExifRotationTest does file access and hence needs a slightly higher
timeout in PHPUnit.

Change-Id: I871c4a4fde5e71708e55b3f28e7267e1023c2df5

(bug 42638) Fix API action=options&reset=1 & unit tests

Change I98df55f2 broke action=options&reset=1, causing it to return an
error "No changes were requested" rather than resetting the options as
it should. Unfortunately, that change also broke the unit test that
would have caught this regression.

This changeset fixes the bug and the unit tests.

Change-Id: I7fe63640d54efab4572538e9d08f5b75c61243a4

Localisation updates from translatewiki.net.

Change-Id: Ib22bb939459977f160056d83697434b5a456474b

Merge "Fixed documentation"

Merge "(Bug 40367) MediaWiki:Contributions should support gender"

Merge "Show comment not as block in tables"

Merge "(bug 37755) Set robot meta tags for 'view source' pages"

Merge "(bug 42598) Database: fname was not logged for BEGIN/COMMIT."

(bug 37755) Set robot meta tags for 'view source' pages

Apply the same robot behavior of the edit & history pages (noindex,nofollow), on view source pages
(case of ?action=edit when having no edit permissios).

Change-Id: Ifc162332798a3dea93db4096d26e928545fbf4b6

Merge "Add some __METHOD__ to begin/commit/rollback calls"

(bug 42598) Database: fname was not logged for BEGIN/COMMIT.

Relied on presence of whitespace. From git-blame looks like $commentedSql has been this way for a long time. Which is odd
since I've seen many commits since then specifically improving
callers of commit() and begin() to pass the __METHOD__, which
doesn't appear to have been logged anywhere up until now.

Change-Id: I48dc8b3f68a6764327b4b1a8f7c06c59a25a31bb

Show comment not as block in tables

Special:BlockList and Special:ListFiles showing comments inside a table
cell. In that case the parentheses not needed, because the table cell
splits it from the rest. Using the Linker method, which does not produce
the parentheses.

The file history of an file pages does this the same way.

Change-Id: I7fbb068bd2acd90f8117719706c32ca05b27b09f

Merge "mediawiki.Uri: Update doc comments."

Merge "Fixing typo"

Fixing typo

Change-Id: I930c37ad7550c7dd0c4856c22921155dec6ccb3b

Add some __METHOD__ to begin/commit/rollback calls

Change-Id: I20dad8d6bb7a523e8a6f50bc0af5cdba57d7160f

mediawiki.Uri: Update doc comments.

Change-Id: I2c49ae07587b23fc522f8c2dda9be955fa4602be

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: Iad76ee37cb4dfd50f8bc9e7b61b5a7cf31238d4c

Merge "(Bug 40145) Changed the wording of the protectedpagetext message to include moving and other actions."

Merge "Implement static public Parser::getExternalLinkRel"

(Bug 40145) Changed the wording of the protectedpagetext message to include moving and other actions.

Change-Id: Ia8a478586128c647a6957a125fdf5e6ae85ab150

Implement static public Parser::getExternalLinkRel

I've implemented the function Parser::getExternalLinkRel which
gives the 'rel' attribute for a given link in a given NS. Per Tim's
suggestion, as it's currently impossible to invoke the logic in
Parser::getExternalLinkAttribs externally.

Change-Id: Id0bfed81e2afd6730d820b6c9a4a09155a557f37

Refactor edit notice rendering so that it's not baked into EditPage

* Move edit notice rendering to Title class
* Use new getEditNotices method in EditPage

This opens the door for alternative editors (such as the VisualEditor) to use the same notice systems already in use.

Change-Id: Ib0e40714f5433f4d75c54a3c3d60b1590fded7f1

Test / spec handling of HTML entities in link tails

Change-Id: I97ed5e17efb1d783797c46161be56e83b94307a7

Merge "Added assertException method to MediaWikiTestCase"

Fixed documentation

Change-Id: I799304e44af405b58207b0f4f1ec4996f17a9c71

Added assertException method to MediaWikiTestCase

Change-Id: I3ce667b4405241c66c5a979863d1cea2cf39956b

(bug 42202) Validate preference values in action=options

Previously, there was no validation whatsoever and the module would
happily write any preference you asked it to. This, combined with the
fact that the code using the 'editfont' preference didn't perform any
validation or escaping, led to a CSS injection vulnerability.

Using Preferences::getPreferences breaks some existing test cases
because a MockUser doesn't have groups for preferences.

Change-Id: I98df55f2b16ac1b6fce578798b6f58b5dad96775

(Bug 40367) MediaWiki:Contributions should support gender

Change-Id: I80fea312e6f84b2aefaca23c751e6d44dd2ee787

(bug 42202) Validate editfont before embedding it in CSS

If the editfont preference somehow had a value like "foo; color: blue",
we have a CSS injection problem. Normally preference validation should
protect against that, but the API module for setting preferences doesn't
perform any validation.

Change-Id: I5c12aa9a48bf4f6ea4a8fb44554d13189e7757fb

Remove the throw from Message::extractParam()

Per Tim Starling's comment on bug 41400, I replaced the throw (when a
message parameter is invalid) with a call to trigger_error().

The string [INVALID] is used as a placeholder to make it clear from
the output that an error has occurred.

Change-Id: I974d55d44d392c956e7de6d243da9d8dc07d8833

Add comments to Linker::formatLinksInComment()

Per Tim Starling's suggestion, added comments to the regex in
Linker::formatLinksInComment().

Change-Id: If85567e5a2e57dd56f5f263f3512e9ed2f9c940c

Optimized regex in Linker.php

Optimized the regex in Linker::formatLinksInComment().

Change-Id: Idea5554bcb8d7e78ddf972ce4c9827fa738ff837

(bug 40995) Refresh SessionId on login

SpecialUserlogin updated to refresh the user's session_id on each
successful login.

Change-Id: I1bd76f2c199b515f570e18669ca2138668bf847e

Merge "Change msg check sp-contributions-explain from exists to isBlank"

Localisation updates from translatewiki.net.

Change-Id: I554504cd3d622f5eccc27feaf55057c1af443212

Merge "Add a hook to Special:Version so we can customize the version URL"

Add a hook to Special:Version so we can customize the version URL

Change-Id: I02fe38c7b13dcfe570e78d28783098fc18756204

Test handling of templates with invalid template target

Those templates are not expanded, and their contents are treated regular
wikitext.

Change-Id: I3cbc0c8bed8d44435b83c2523f26db09b27215fa

Merge "(bug 42215) "Welcome, X" account creation title #2"

Merge "[FileBackend] Actually unset the "conn" field in closeConnection()."

Skin: Make portlet rules backwards compatible with html cache.

Follows-up I9a2ebd50.
Caused regression with 1.21wmf5.

Change-Id: Iabda81555d2af6d91447a51ff3d8b6ea1f1f00c2

Change msg check sp-contributions-explain from exists to isBlank

This avoids adding the empty message.
Also add a missing text() call.

Change-Id: Idb85abecdf2fa83e89bf6ebe1161e482d1ab2ce7

(bug 42215) "Welcome, X" account creation title #2

Follow-up to 8842727 https://gerrit.wikimedia.org/r/#/c/34063/ .
Renamed welcomecreation-agora to welcomecreation-msg since it's not
agora-specific; actually use this message in SpecialUserLogin.php;
improve Qqq docs.

---- Patch 2: fix alignment nit

---- Patch-3: rebased, and remove 'welcomecreation'
'welcomecreation-msg' obsoletes it

Change-Id: I3f5552b0e60ea89461744c823c646d35891ecdb6

Localisation updates from translatewiki.net.

Change-Id: I838c080f6e060e42daf9c5e889a213dff9f6e83b

Merge "Improve type hint"

Merge "Additional table tests to spec parser behavior."

Merge "API: Fix description for YAML format"

Genderize be-tarask namespaces, fix spaces in definitions

Previously, gender-related namespace aliases existed,
but $wgExtraGenderNamespaces wasn't used at all.

Also, spaces replaced with underscores in alias definitions
where needed.

Change-Id: Idc5e9e6928a21c8e5fb20aba9aae0a93acee71d3

Additional table tests to spec parser behavior.

Change-Id: I4f4852744d463d77fc622c1bcce27e3d9751bc6b

Merge "Localisation updates from translatewiki.net."

Localisation updates from translatewiki.net.

Change-Id: I104647cc6b24e8f6b3717c7230b4c53799670ab7

Merge "Test table / list interaction"

Merge "MWTimestamp::getHumanTimestamp uses 'just-now' message, which did not exist"

Improve type hint

Eclipse and phpstorm where showing 'User' as return type before, which causes me to not check for false somewhere and thus fatals happening :)

Change-Id: Ibd5b5598f05e6b08481ad65060c7cae18762dc4e

Merge "Add 'dimensions' and 'thumbmine' to resultproperties in prop=imageinfo"

Test table / list interaction

Add a test for indented tables with lists in table contents in start-of-line
context.

Change-Id: I6e81e4148873b3ae97287f72630a8ef353cd6e25

API: Fix description for YAML format

The description for format=yaml in the auto-generated docs is coming out
as "Output data in YAML formatOutput data in JSON format". That needs
fixing.

Change-Id: I20185fba6b70cc995a41eb6f42565da4cc7c338d