User::addGroup could cause duplicate groups

Use array_unique to avoid this case.

Bug: 46844
Change-Id: I14eb6e5a5333d887d71d545e42570493437dd320

Moved CryptRand.php to MWCryptRand.php

Change-Id: I03e3742f270ccf05ea923a46e8f8171e6f73ffcd

StatCounter doc tweaks.

Change-Id: I8882e142170841c70cb620e1ea0246facc30f046

Renamed QoS_Atomic => QOS_ATOMIC in some places.

Change-Id: Ib6c4a1e36e8caddd08852d0fdce005ed0bc00cee

Make current year default value in the Special:Contributions

Bug: 45580
Change-Id: I720946e7a37ca5a639c3ac786fc863020929cf1a

HTMLForms: Support non submit buttons

There is now a new generic HTMLButtonField class which does
not need to be a submit button.

Change-Id: I451a952bba0c7339b5337e5201417beaee3f16fd

Move template header inside form so CAPTCHA works

Also wrap it inside a section tag so the form's styling of gray labels
and full-width input fields doesn't apply to the CAPTCHA.  The same fix
is in the new Create account form (though that uses JavaScript to
reposition the FancyCaptcha image).

CSS gives the section a small bottom margin to space it from the form
contents, small enough to not affect form layout if there is nothing
in the template header.

Bug: 47371
Change-Id: I1523fe6551585ba731b0ca4207e22d614d0a46d8

Replace weird usage of explode

Change-Id: Ic4021be695c9ed15dea0da0e94967a0d7d441670

Bumped RING_SIZE to use 28 bits to improve weight accuracy a bit.

Change-Id: I4dbecd292fd404e43ed296dd37892db113fa1026

mediawiki.util: Fix roundtripping of tooltip in portlet links

In 558985f72a ctrl-option- for Chrome on Mac was added, but it
didn't add "option-" to the regex. Since then the tooltip in
Chrome on Mac (and later when we started referring to option
instead of alt on all Mac browsers) was never detected by this
regex, sometimes resulting in double tooltips (when adding) or
outdated tooltips (when updating).

Now we always look for an accesskey hint in the tooltip and
strip it it's there.

Also fixed a bug where an undefined error can occur if accesskey
is given but tooltip not (the method unconditinally appended
text to the tooltip varible which might be undefined causing
a tooltip like "undefined[a]").

Change-Id: I0bde1a228983c58b20cad0c09a8e5efe8225ea23

Localisation updates from translatewiki.net.

Change-Id: Ic9d28bde4ca24c8e1a041b86ae48e7d6d24de522

Merge "Refactor MWTimestamp::getHumanTimestamp and add hook."

Merge "Avoid use of __METHOD__ in closure."

Merge "test: abstract parser test result"

Avoid use of __METHOD__ in closure.

Change-Id: Ib25063f41cc25647003442cbd5851c3b777b2a7a

Refactor MWTimestamp::getHumanTimestamp and add hook.

Changed logic in MWTimestamp::getHumanTimestamp so that
all the message and formatting was offloaded into the
Language class, keeping only actual timestamp logic in
the MWTimestamp class.

Also added a hook so extensions can override the
human timestamp format.

Change-Id: Ie667088010e24eb6cb569f9e8e8e2553005223eb

Cleaned up Hooks code, comments, and documentation.

Essentially rewrote Hooks::run() to get rid of the ridiculous
four levels of indentation. Also made some slight adjustments
to fix rare edge cases (for example, moved set_error_handler
after wfProfileIn in case Profiler triggers an error).

Change-Id: Iafdd4ceedac067b49ac597359ac456f4617da9e8

Merge "Swap port from 110000 to 11211 in $wgMemCachedServers"

(bug 25592) LogEventsList:showLogExtract should ignore WebRequest

LogEventsList:showLogExtract currently pays attention to WebRequest
parameters such as offset, dir, order, and limit (although limit is
typically overridden by callers, and offset sometimes too).

While these make sense for a pager that is driving the whole page, it
doesn't make much sense for the "most recent log entry" boxes that are
displayed using LogEventsList:showLogExtract. So let's have
LogEventsList:showLogExtract ignore these, and for good measure add in a
parameter to use the old behavior in case any caller really needs that.

Change-Id: Id4380cd863ba69fc9b9afb5a0034d82ad4d9cc34

Merge "Added more fine grained groups for parser tests"

Merge "Improved readability of MediaWikiParserTest::suite code"

Fix debug log line for $userLimit.

$userLimit was used inline in the message, causing an "array to string conversion"
notice and a mangled log entry.

Change-Id: I8c5199f1bf01b56e955d8d15c48061b5301b1745

Handle search engine returning status objects to the API

This is a followup to I54ab5207, which allowed search backends
to return status objects. We handled it in Special:Search, but
the API was overlooked.

Bug: 47353
Change-Id: Ib7abe8d5bc19d27c472053602c0e43cb1301b078

Swap port from 110000 to 11211 in $wgMemCachedServers

http://linux.die.net/man/1/memcached

"...the default is port 11211."

Change-Id: I923a1b3208c2ea1f29657cc51e813c547154eb32

Merge "Fixed Bug 40464 Placeholder attribute of searchInput element"

Fixed Bug 40464 Placeholder attribute of searchInput element

Removed the javascript code that sets the placeholder of #searchInput element.
Now the placeholder attribute of #searchInput element is set only in the original HTML.
On browsers that do not support placeholder attribute, placeholder.js will insert
the appropriate placeholder.

Bug: 40464
Change-Id: I6496716f2bd25307004ba16bceadca4c6723a5db

Fixed spacing in files direct in includes folder

Added spaces before if, foreach
Added some braces for one line statements

Change-Id: Ibb8dd102db045522d12ff939075ba7420d95ab6b

Fixed spacing in context/installer/media/templates/upload folder

Added spaces before if, foreach
Added some braces for one line statements

Change-Id: I9761be9fa47adc3554852a97b19792b4648466ad

Add doc for SpecialPage::postText

A combined doc does not work

Add some spaces to function bodys

Change-Id: I98db22fe5028c97ad257ddd86524c0356881ff57

Localisation updates from translatewiki.net.

Change-Id: I469b4f8fe3747d6f4b651905a4d26f0219bad45f

Merge "Minor CSS cleanup for Vector and Monobook skins"

Added more fine grained groups for parser tests

* Group "ParserTests" which allows to run all parser tests in PHPUnit without running other tests
  related to the MediaWiki parser from group "Parser"
* Dynamically generated groups to only run certain parser tests. All of those groups are the parser
  test file names ("." and "_" replaced) prefixed with "ParserTests_". This allows to easily run
  the parser tests of a certain extension separately without using "parserTests.txt".

Change-Id: I03b87820c2968cf16eba2ac39cd8444a0e4dc41a

Improved readability of MediaWikiParserTest::suite code

Change-Id: Ic74048bcce71c2eeb8adfca872b978c3f0de094b

Fixed spacing in db/debug/diff/externalstore/objectcache folder

Added spaces before if, foreach
Added some braces for one line statements

Change-Id: I32a43c547630ce31b6b25cc1bc9eedc50e67b2d7

Fixed spacing in actions/cache/filebackend/filerepo/job folder

Added spaces before if, foreach
Added some braces for one line statements

Change-Id: Idb93d34e314e5f314223b79208968d6bcd30c40e

Remove duplicate code from Setup.php

Change-Id: I6dadb12ec8410258d7c9c24e1d12badf6c1c2dea

Fixed spacing in logging/parser/profiler/rl/revdel/search folder

Added spaces before if, foreach
Added some braces for one line statements

Change-Id: I11bbcfa351e945b7bde10c2105d61a3cf5622205

Fixed spacing in specials folder

Added spaces before if, foreach
Added some braces for one line statements

Change-Id: I7873c656b16f6e905bb5abb2c2db6f6e84c11430

Update formatting

Change-Id: I0e334d910a311f067276215b6bff94cd814fc3b3

Fixed spacing in api folder

Added spaces before if, foreach
Added some braces for one line statements

Change-Id: Id7779dca4d1185245cf5764102b8de8b232c34b6

Merge "Separate language generation from outputPage"

Separate language generation from outputPage

Helps with readability - outputPage function is extremely long
Also helps with extensions of SkinTemplate that override outputPage
(for example skin in MobileFrontend)

See related change id: I04b625c8d76bb6a061e3201847bf97185c6b7a29

Change-Id: I4a49e704f6fe72c74ecb9103fb081aed93a86de7

[JobQueue] Fixed numbers given to redis queue stat calls.

Change-Id: Ia70e6ab947aab9ed807ea993b04647aa488fbf3d

Merge "Localisation updates from translatewiki.net."

Added abandoned job stats.

Change-Id: I3ef8c56e1a3e16875c7282afc4d3d0b135bd776d

Localisation updates from translatewiki.net.

Change-Id: I4445cabf897d43873aac8750f1e0a1585cff6b14

API: Add new hook 'APIGetPossibleErrors'

This allows extensions to modify the array.

This completes the serie of hooks for similar get methods in ApiBase.

Change-Id: Ib398f5815ab57f25d56356b0997c55a03fd96874

Only allow fields from isFieldSortable to be sorted on.

Prevents people from doing stupid things like:
https://commons.wikimedia.org/w/index.php?title=Special%3AListFiles&limit=50&user=bawolff&sort=img_description&asc=&desc=1
Which don't have indexes.

Change-Id: I7aeb89032e89283dd7dfe5d7bf12872203ca5520

[FileBackend] Made stat cache factor in last-modified time.

* This should lesson the chance of consistency problems
  if there is any memcached flapping.
* Also cleaned up some documentation.

Change-Id: I89401892c7e848fcff0b093eaa861dfe02387a5c

[FileBackend] Disallow files ending in / and clarified some docs.

Change-Id: I162ca34fe9f92d862345004bd184013531da9e8d

[JobQueue] Added JobQueueFederated class for partitioned queues.

* This lets queues be horizontally partitioned onto
  different servers, with weights assigned to each.
  The queue classes used by the different partitions
  can be hetereogenous or homogeneous.
* How partitioning is done is setup similar to LBFactory,
  where wikis belong to sections and sections have config.

Change-Id: I44d59b67cf417dca28a3e9b25371dac5a7ffcb47

[LockManager] Added a RedisLockManager class.

Change-Id: I7ade74eb307a5075533f36836768af60f106a6b9

[JobQueue] Deprecated confusing Job::getId() function.

* Also slimmed down $job->metadata use by JobQueueRedis to just the ID.

Change-Id: If9fdb33afd6d1ce2be957a1eea107f54bd6dac53

Factored JobQueueRedis::redisEval() into RedisConnRef::luaEval().

Change-Id: I1b4bb4f4bb5e25fc3f358c9e2b16e4672584c68b

Merge "UIDGenerator: better support for 32-bit machines"

Merge "Wrap job execution around profiling calls."

Merge "[JobQueue] Added per-type stat counter calls for better graphs."

Moved Block and Title purgeExpired() functions to onTransactionIdle().

Change-Id: I82b2f4689a42481cd2a476351354f55ded581ff5

UIDGenerator: better support for 32-bit machines

Replace one exception with another that is less likely. Integer
multiplication is exact up to the limit of the double precision
mantissa size, and wfBaseConvert() will work up to any size of string.
So it's not necessary to require bcmath/gmp to be installed. The modulo
operation is equivalent to truncation of the binary form.

Perhaps mantissa sizes will be extended before the year 144680, made
possible by improved computing hardware. The exception message should be
altered when such hardware becomes available.

Change-Id: I8116dac1958b5dbef977d3900973bf13be3d9d17

Fix @return type hint for Title::newFromID

Change-Id: I7f9ff81b91dcabb68c9c86eedf391b3e6160c6d4

Merge "Clean up, comments, remove obsolete mw-ui-formlist"

Clean up, comments, remove obsolete mw-ui-formlist

* Remove obsolete ".mw-ui-formlist div label" selector, replaced by
  ".mw-ui-vform > div label".
* Rename mw-ui-block-label to mw-ui-label since it doesn't impose
  block styling.
* Remove overspecified ".mw-ui-vform > div label.mw-ui-checkbox-label"
  selectors, since the bare class applies the same styling.

Also in the CSS for the vertical forms,
* reduce the margin-bottom of #languagelinks
* allow word wrap to break words in the errorbox (bug 46966).

Change-Id: I03bdf3b1c50be732435e10193f004230a6a3bacf

Merge "Clean up SpecialFilepath."

Merge "Fixed some spacing in maintenance folder"

[JobQueue] Added per-type stat counter calls for better graphs.

* Also refactored doBatchPush() for the DB job queue to reduce
  the amount of PHP closure work-arounds. I'd rather not add a
  $type variable to the bloated 'use' clause.

Change-Id: I265effc061558d96048dcc331ac2338a8b95aff5

Wrap job execution around profiling calls.

Change-Id: Idf184b7c2f7003032d3a28830ee9c4a1eaeaa98b

Localisation updates from translatewiki.net.

Change-Id: Idb721fab7d2bd80ccd07a02365407763585865c0

Fixed some spacing in maintenance folder

Added spaces before if, foreach
Added some braces for one line statements

Change-Id: I9657f72996358f8c1c154cea1ea97970d973723c

Pass a limit to BackLinkCache::getNumLinks() in some places to avoid OOMs.

Change-Id: Idf1042c083837a9e4a380bb20ecdf3043ea9ab09

Merge "Normalise user names passed to Special:BlockList"

Normalise user names passed to Special:BlockList

Special:BlockList now handles correctly user names with spaces when passed as subpage

bug: 47218
Change-Id: I3a52c454c64fdc599c7abcdca05b182e09640c59

Use a combined const, instead of string concat in Exif.php

Fix some spacing while at it

Change-Id: I47ff4aef5125dae6308720680641f617b5795a41

Merge "Modifying ArticleEditUpdateNewTalk hook so that it passes the recipient"

Merge "Add non DBMS depending SQL tests for DatabaseBase"

Cleaned up test cleanup for FileBackend and avoid use of @.

Change-Id: Ie9f0090f626384a90b0139b9f8c2d94bf0bb8f23

Merge "Make UserCache only look up a user once."

[JobQueue] Simplified a bit of code in JobQueueRedis.

Change-Id: Ia0329a9721b126a3573cfee0d67c9a54c911196f

Merge "Do not bother with ForkController for procs=1."

Merge "Fixed spacing in skins folder"

Merge "Got rid of some global $wgContLang usage in SpecialAllpages"

Merge "Fixes to Special:ConfirmEmail form"

Merge "Fixed spacing in languages folder"

Merge "else if -> elseif"

Merge "Add missing braces to for in wfGetCallerTest"

Merge "Fix HTML output of TablePager"

Merge "Fixes to Special:BookSources form"

Make UserCache only look up a user once.

I noticed on special:listfiles/username, UserCache was doing queries
like select user_whatever from user where user_id in ( '1', '1', ...)
with the same user id 50 times (one for each result returned in the
special page). That seemed a little insane, so put the list of
users to query through a array_unique. (Quite likely the db would
optimize that query to not literally look up the same user 50 times,
but nonetheless it seems better to filter the list before then)

Change-Id: I80c8a359d0f7a53b2420ebcda641e594dd3c56e9

Do not bother with ForkController for procs=1.

Change-Id: I84d4dd5369c5640b849d2fb58e01b8565b2abd65

Merge "Make concurrent runs of FileBackendTest independent"

Merge "Add gender to 'passwordreset-emailerror-capture' message"

Got rid of some global $wgContLang usage in SpecialAllpages

Using $this->getContext()->getLanguage() instead now. In SpecialAllpages::showline
the global is still being used and a comment added. It could be argued about whether
the content language or the user language should be used in that place.

Also added some missing documentation to Language::getNamespaces.

Change-Id: If511e2d9edb0f0092f3ca0ceadabd7fdb72f6c06

Add gender to 'passwordreset-emailerror-capture' message

Bug: 46646
Change-Id: Ib775597d7d5b5afd84717fc7064a73721bc1a01c

Make concurrent runs of FileBackendTest independent

Make each run of the FileBackendTest be for a different "wiki"
so you could have multiple runs at the same time without them
interfering with each other.

There's been intermittent failures on jenkins for this test. I'm
not sure if its some race condition (which wouldn't really make
sense given each run should be independent I thought), but this
can't hurt.

Change-Id: I3b639f1c783efc140028c3acdce35ef3f58bf66f

Fixed spacing in languages folder

Added spaces before if
Added some braces for one line statements

Change-Id: I980771894369499646532b13b801db6447381773

Localisation updates from translatewiki.net.

Change-Id: I29386c98e8d9308a6cbebe8b654834996ac36b0c

Add missing braces to for in wfGetCallerTest

Change-Id: I14b7abb90e2574022c792a6874390eee87d48708

Merge "Set up the profiler in CLI mode."

Merge "[FileBackend] Simplified the shard list iterator to use AppendIterator and FilterIterator."

Merge "Reduced DB contention in User::saveOptions()."

Merge "Small doc fix to JobQueueRedis."