Merge "Replaced Linker::link() usage with LinkRenderer in some special pages"

Replaced Linker::link() usage with LinkRenderer in some special pages

Bug: T149346
Change-Id: I73c2b7402012a49e4419b6214f1d03f2e819220b

Merge "Add $magicWords translation for Bengali (bn)"

Merge "Clarify in emailuserfooter that by responding one reveals their email address"

Clarify in emailuserfooter that by responding one reveals their email address

Bug: T152242
Change-Id: I686f7bb8dd1d225620266ac9cbecadcf06bd7705

Add $magicWords translation for Bengali (bn)

Bug: T151134
Change-Id: I81ce473668e3712ea575dc38f5eb76abb45498ac

ArticleViewCustom, EditPageGetDiffText and ShowRawCssJs were removed

Bug: T145728
Change-Id: Iddf29005abbaceaa1e3cc2c79e4becfb46294414

Article::getAutosummary() and WikiPage::getAutosummary() were removed

Bug: T145728
Change-Id: I233571658b902c93a148a992ea99eb159551e732

Remove ContentHandler functions with no usages at all

Bug: T145728
Change-Id: Ia0d7a5eb7e79b586c3b30688c80bb8b4b723fff6

Merge "Localisation updates from https://translatewiki.net."

Merge "Avoid use of DatabaseBase in /includes"

Localisation updates from https://translatewiki.net.

Change-Id: Ifb9f00278f2dc7589efa501dd4ee8fba3087ea60

Avoid use of DatabaseBase in /includes

Change-Id: Ia3b7f9cb7d40f4ec9f19c924904e8a096880ec6a

Merge "Ignore query cache in LoadMonitorMySQL"

Merge "Set token properly when creating in User::newSystemUser()"

Move link-underline user preference rule to body class

ResourceLoaderUserCSSPrefsModule is now empty and can be removed.

Change-Id: I806e117131765b859e990c14735bb9494db9a37e

Replace Linker::link() usage with LinkRenderer

Replaced deprecated Linker:link() with LinkRenderer

Bug: T149346
Change-Id: Iab158d49b186835a14c357c91a1c5c4f78986238

Set token properly when creating in User::newSystemUser()

Otherwise callers that don't use 'steal' is going to break because it'll
think it needs to steal the user.

If such a user exists on a wiki, it can be fixed by setting the token to
the invalid token. The easiest way is probably to just call
User::newSystemUser( $name, [ 'steal' => true ] ) with eval.php.

Note there's no way for anyone to use these users unless they steal the
token from the DB, since they still don't have a password, email, or any
other method of authentication or account recovery set up.

Change-Id: I9efd2d2f5fffb4e4411a894f9514cdf2c66663a9

Merge "mediawiki.notification: Improve scroll performance"

slave -> replica in SessionConsistentConnectionManager

Change-Id: Ie59c90fa5f4b18d4321f7924a4facd7ea9049f4a

Ignore query cache in LoadMonitorMySQL

Change-Id: I0b3c27ca6d2f8805297d91557cc40bd4bbc1a677

Removed Xml::escapeJsString()

Change-Id: Iead8385a72a79632a90594a8828080a6d572e039

User::edits() was removed

Change-Id: I3ebe2cceb94a0d61a9e0ab5bb0599e8a03146046

Merge "SpecialActiveUsers: escape group names"

Merge "Remove Atomic methods from ConnectionManagers"

Remove Atomic methods from ConnectionManagers

Change-Id: I697f63f45975b13af52bc22fced6e4d07e35115f
Depends-On: I4341a1b4ff6a67e4c1770faae38e9b126f1bf0bf

resourceloader: Add test coverage for ResourceLoaderContext::msg()

Brings ResourceLoaderContext coverage to 100%.

Change-Id: I3ccc6aa87c89f69158b8d8684be565403afb4036

Don't hard fail when we couldn't find an entry point for an extension

It doesn't make a ton of sense, it's just pointing out that we have
a weird extension for which we cannot detect a standard entry point
for. These (unfortunately) exist, but they're easily worked around
using --list-file

Removing the hard failure allows you to use the two options in
tandem... --extension-dir for the initial pass and then --list-file
for the weirdo outstanding ones

Change-Id: I3d9cf1d614dacaa91fb2092019ccf1d14d61ccab

Merge "SpecialFewestrevisions: Allow pages with only 1 revision to be shown"

Remove $purpose parameter from password validity check

This was added in I56b6600 in an attempt to work around a bug in
CentralAuth, but the bug has since been fixed in a better way. No hook
functions in Gerrit use the parameter (or ever have, as far as I can
tell), and anything that was passing a value other than the default
'login' has since been removed. So let's just get rid of it instead of
keeping it around doing nothing.

Change-Id: Ie604e03d268706221161ac93eb866f477e466fb4

installer: Restore link text in default main page content

Accidentally got dropped in b85a17b7543.

Change-Id: I30b7a8d42877a54749792e97e49670087a85ca2b

Merge "ApiSandbox: Better handling of parsed messages"

IDatabase::delete() table name parameter should be a string

Change-Id: I7ba2cdb6cdddf04819ff80e27814f704f41d15f1

Merge "Replace Linker::link() usage with LinkRenderer"

Merge "http: Support HTTP Basic Authentication"

Replace Linker::link() usage with LinkRenderer

Bug: T149346
Change-Id: Ie384566617f18ad6c3bf836b39702365d59f714d

Merge "Replace Linker::link() with LinkRenderer in some special pages"

http: Support HTTP Basic Authentication

Adds two new options (username and password) to the
MWHttpRequest (and HTTP helper class) to enable
support for HTTP Basic Authentication on outgoing HTTP
connections.

Change-Id: If83f025bbe63769ba7bb4a824c5f12d5f1ec640a

Merge "Replace Linker::link() with LinkRenderer in includes directory"

Merge "mediawiki.storage: Provide a wrapper for sessionStorage too"

Merge "Fix typo in $fallback so Russian fallback is applied"

Fix typo in $fallback so Russian fallback is applied

Change-Id: I25ab7b58dedff895b60737d45b6fcf48f9484001

Merge "Align search result CSS with Wikimedia UI color palette"

Merge "Rank aliases in search in order they appear in the messages file."

Replace Linker::link() with LinkRenderer in some special pages

* SpecialEditTags
* SpecialEmailuser
* SpecialFewestrevisions
* SpecialImport
* SpecialListgrouprights
* SpecialMediaStatistics
* SpecialMergeHistory
* SpecialMIMEsearch
* SpecialMostcategories
* SpecialMostinerwikis
* SpecialMostlinked

Bug: T149346
Change-Id: Idee566dbff87f2e8bf909ba8371d551aaf0dd08b

Replace Linker::link() with LinkRenderer in includes directory

* CategoryViewer
* OutputPage
* Preferences

Bug: T149346
Change-Id: I9f9bcd9b461884817e8ceefbc6757c436221e331

mediawiki.storage: Provide a wrapper for sessionStorage too

T119146 provides a use-case for using sessionStorage. So far mw.storage
was localStorage-specific. With a small modification, we can allow the
Storage object to passed to the constructor, which allows us to create a
wrapper around sessionStorage (mw.storage.session) with minimal code duplication.

Bug: T121646
Change-Id: I73bc82d9fa2359148fe1e50b6535bfa0dbe8bd3e

Merge "Clean up http classes a bit"

Clean up http classes a bit

* added integration tests. We probably don't want automated tests
  to make external requests but these make manual testing more
  convenient. Documented some oddities discovered by testing.
* made ::$status, ::proxySetup() and ::getHeaderList()
  protected; they were not referenced in any gerrit-hosted extension
  and they provide no useful functionality to external callers.
  Similarly, marked ::read() and ::errorHandler() as internal
  (these are used as callbacks so can't be protected)
* removed inheritance abuse in ::execute()
* documented ::execute() as returning a StatusValue (but
  keep returning a Status for now)
* changed setCookie argument defaults to ones that make sense
* replaced MWException
* moved unit tests to the correct location
* fixed some code style issues

Change-Id: I5852fc75badc5d475ae30ec2c9376bde7024bd95

Merge "SpecialBrokenRedirects: fix up LinkRenderer conversion"

SpecialBrokenRedirects: fix up LinkRenderer conversion

$options is no longer supported, use makeBrokenLink() instead.

Follows-up d8c56ec84f.

Change-Id: If2025232a2e09ab99b1bdc6d58a77f347f3dbf5e

Merge "ExtensionRegistryTest: Illustrate priority in array_replace_recursive test case"

ExtensionRegistryTest: Illustrate priority in array_replace_recursive test case

When two keys have conflicting values, the value from extension.json wins
when using this merge strategy.

Change-Id: I0222682f3b3fb08f80afc7dda677da7bbfe8e5d1

Fix SessionConsistentConnectionManagerTest class and file name

Change-Id: If60f081946b8f3bb564f2bb17fd1261af0cb7e3a

Localisation updates from https://translatewiki.net.

Change-Id: I028c1cb92a6b14cc4ca72f6cff0b75f74bc68058

Revert "Localisation updates from https://translatewiki.net."

Local twn repo seem outdated

This reverts commit 683e05d4413a9727ec882b4b15d04edcc562f93f.

Change-Id: I3e38317ef0ef2a7e9080ee7a50c2101a921d632c

Localisation updates from https://translatewiki.net.

Change-Id: I83a9c227bab5b29162367eed71ccabf513ac5423

Align search result CSS with Wikimedia UI color palette

Bug: T152025
Change-Id: Idc58deca48327d82372f5136a2fb07146ab1f371

Merge "DatabaseUpdater: Stop inserting junk into 'updatelog' every time the updater runs"

DatabaseUpdater: Stop inserting junk into 'updatelog' every time the updater runs

Nothing ever looks at these rows, and if you have update.php in a Cron
job or a Git hook or something, they can accumulate in huge numbers.

Also remove a test which verified that the rows get inserted.

If you reached this place while trying to find out what the weird
"updatelist-1.28.0-14628929450" rows in your database are, rest
assured they are useless. If they're causing disk space problems,
you can delete them:

  DELETE FROM updatelog WHERE ul_key LIKE 'updatelist-%';

Change-Id: If94f243810e82ea3724cbd47b301bb96ba9a49d9

Merge "Almost invisible change on UI colors to align with Wikimedia color palette"

Almost invisible change on UI colors to align with Wikimedia color palette

 - wikitable
 - thumbnails
 - gallery
 - TOC
 - catlinks

Bug: T152025
Change-Id: I8ca0f94ad66a5d6cec649f65e87b145a9a9aba78

Merge "CategoryPager: Allow categories with no members to be shown"

Merge "ImageHistoryPseudoPager: Only display 10 revisions at once by default"

Merge "Throw an exception if password hash would be truncated by DB"

Merge "mediawiki.language: fix convertNumber( ..., true )"

Merge "Update OOjs UI to v0.18.1"

Merge "Add DB ConnectionManagers"

Merge "SpecialUserrights: Make new fetchUser() parameter optional"

SpecialUserrights: Make new fetchUser() parameter optional

This in, unfortunately, a public API, apparently.
Follow-up to d87a4eaa19aad80e109307eb0088fd6a25dd5f51.

Bug: T151961
Change-Id: Ia0e583a5cbb9ed143bf5dff9b513b76ca2790d72

Merge "Fix MediaTransformError message handling"

CategoryPager: Allow categories with no members to be shown

This affects the listing on Special:Categories.

Bug: T12915
Change-Id: I6265d20b8afd199832de6dd3b2e126a1c45d4b11

Merge "Prevent login-only local password provider from removing passwords"

Fix MediaTransformError message handling

Give access to the raw Message instead of only to the HTML or text in
the RequestContext language.

Pass Message objects instead of strings from calling ->text() as the
parameters of Messages so if the outer Message's language is changed
things get parsed sensibly.

Change-Id: Ibd6c1217b6fed839c888b66e02900f8e21ed3e6b

Add Message::listParam()

This allows for passing a list of values that will be turned into a list
in the context of the language for which the Message is being processed.

For example, currently you'd have to do

 $msg = new Message( 'something', [ $language->commaList( $list ) ] );

which isn't going to give correct results if the message is later
changed to a different language with a different value for
'comma-separator'.

Now, you can do this instead

 $msg = new Message( 'something', [ Message::listParam( $list, 'comma' ) ] );

and it will be listified properly no matter what language is later used to
parse $msg.

Change-Id: I66868c61832260870449998fef14c842f17753ee

Merge "Added warning for improper ending of a token"

Merge "Accept 'OK' status results from search engine"

SpecialActiveUsers: escape group names

Change-Id: I1a4d1501b8481d9f670916818fe7f75e983c2800

mediawiki.language: fix convertNumber( ..., true )

It was not handling separator unformatting.

In mediawiki.jqueryMsg.test.js fix broken test case:
* Hindi does not use comma as decimal separator!
* The number was not formatted as it would be by the
  formatting code. By definition back-conversion is
  only defined for input that we generate. For other
  input the output is undefined. There are other tests
  with same issue, but I did not touch them now.

Bug: T151198
Change-Id: Icc9bc09675151645055c7e7f706472a15cc81b0f

Add DB ConnectionManagers

This moves and refactors the ConsistentReadConnectionManager
from Wikibase into the core rdbms lib.
The refactoring also creates a generic ConnectionManager.

This relates to Iff20a22f9f2bc7ceefd6defc0ed9a494a6fe62c0
which introduced a DB factory / connection manager in
an extension revealing the need for this in multiple places.

Change-Id: I0c58e15aed5bed88323d18cb95e5008f8d3381c5

Merge "Replace Linker::link() with LinkRenderer in some special pages"

Replace Linker::link() with LinkRenderer in some special pages

* SpecialAllPages
* SpecialAncientpage
* SpecialBlock
* SpecialBrokenRedirects
* SpecialDoubleRedirects

Bug: T149346
Change-Id: Ib9997ec29b5c0079acb79dc2772196b38665145d

MessageCache: some inline comments

Change-Id: I42427eef73d86c1a2e99dfa22b5dae159542e243

Added warning for improper ending of a token

Added warning if token ends with Token::SUFFIX by comparing result of substr to Token::SUFFIX.
Simplified Token class calls.

Bug: T122280
Change-Id: Id405dcc7b5e4d28995edf334aa6e183efdb749de

qunit: Don't assume synchronous Deferred.resolve

* mw.loader.test: Most test did this correctly already by returning
  a Promise to QUnit.test, or by using assert.async().
  However two tests did not.

* mediawiki.jqueryMsg.test: Previously the async() handles were
  obtained within the task execution loop. Before jQuery 3.0,
  simple .then() operations when they are already resolved happen
  synchronously, so the current handle was resolved and the next
  handle obtained in the same go and QUnit never saw the state
  as having no unresolved async handles.

  With jQuery 3.0 the test fails because QUnit would end the test
  after the first iteration.

  Fix by simply obtaining all the async() handles when creating the
  list of tasks, instead of within the individual task execution.
  This way they're all reserved and they'll count down as we go.

* mediawiki.api.test: Consistently use 'respondImmediately' for
  the tests that were missing it, and return promise to QUnit.test()

  Also update hardcoded API urls that encoded space as '+', which may
  now be encoded as '%20'.

Bug: T124742
Change-Id: If7ee1c6025be70fecc0a93d4ac155da4db6571ab

Merge "mediawiki.api: Fix getToken() incompat with jQuery 3.0"

Merge "Remove $wgSessionsInMemcached"

Update OOjs UI to v0.18.1

Release notes:
 https://phabricator.wikimedia.org/diffusion/GOJU/browse/master/History.md;v0.18.1

Change-Id: I5fe3129c9ccf3fa20494b5934bca7487d8d36879

Remove $wgSessionsInMemcached

Change-Id: Ia2b4d62ea504a3cb04f4c2bec49b16f155c8b0c9

Merge "Make protect.php maintenance script not ignore --user and --reason parameters"

Accept 'OK' status results from search engine

Historically code that talks to the SearchEngine has been treating
anything other than a 'Good' result, such as results but with a warning
(perhaps the query timed out and only gave partial results) as a
complete failure. Adjust the handling to report results from an 'OK'
status response while attaching any warnings to the api response, or
in a notiable red box on Special:Search.

Bug: T134157
Change-Id: Ic5e0db727790f4fd189caa54ea5f01672d6a8ea4

Rank aliases in search in order they appear in the messages file.

This will ensure the local page name (which is the first alias) is prioritized.

Bug: T151660
Change-Id: I3cc266024f5e052d20ff55c4b2c6cbd2945e3dd3

Merge "Use isDisabled() instead of isBlank() in getGrantName in User.php"

Merge "Allow API endpoint customization for mw.widgets.TitleWidget"

Merge "Rewrite logstash key conflict warning from I6677dbf6"

Merge "List grant codenames next to grants on Special:ListGrants"

Use isDisabled() instead of isBlank() in getGrantName in User.php

Bug: T151956
Change-Id: Ic23c75b784c725f8f73eb1cf98d2bdb3763ab09d

List grant codenames next to grants on Special:ListGrants

Bug: T145351
Change-Id: Ia07de6269fa6192071aaea5d750362685389001c

mediawiki.notification: Improve scroll performance

This fixes the problem with random scroll lag when on a page that had
1 or more notifications at some point. The scroll handler was fairly
expensive and kept being called on every scroll event.

The actual frame where a frame is detected does take more than 16ms to
paint but that's unavoidable. It only happens for 1 or 2 frames, which
is perfectly within the response guidelines for RAIL.

https://developers.google.com/web/fundamentals/performance/rail

(MacBook Pro, Latest Chrome, CPU throttle 5x)

1. Before
* Duration of no-change scroll event
  (not crossing offset.top in ither direction) ~5ms
* Duration of changed scroll event
  (crossing the offset.top and swapping classes) ~10ms
* Paint:
  - 16ms for no-change frames (50-60fps)
  - 40ms for frames that paint a change. (25-30fps)

2. After storing isFloating as boolean and not checking
   className if nothing changed.
* no-change scroll event: ~0.13ms
* changed scroll event: ~2ms
* Paint:
  - 15ms for no-change frame (60fps)
  - 33ms for no-change frame (30fps)

3. After changing $window.scrollTop() to window.pageYOffset.
* no-change scroll event: 15μs (0.015ms)
* changed scroll event: ~1ms
* Paint:
  - 11ms for no-change frame (60fps)
  - 20ms for no-change frame (48fps)

Change-Id: Ia48b00cd0834700ce79e72d4599c51ff8c6179b5

Merge "Localisation updates from https://translatewiki.net."