| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | |/ / / / / / / / / /
| |/| | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Bug: T310158
Bug: T386298
Bug: T386482
Bug: T388049
Bug: T388857
Bug: T389234
Bug: T390589
Bug: T390615
Change-Id: I1325618ae8ccd554f8f0891abe6a51df539ab6da
|
|\ \ \ \ \ \ \ \ \ \ \ \
| |/ / / / / / / / / / /
|/| | | | | | | | | | | |
|
| | |_|_|_|_|_|/ / / /
| |/| | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Migrated second ENotif Talk mail into Notification system. This
is a third iteration. At the end all different notifications cause
exactly the same Email.
To future proof the system, we can pass the Source, which means
what triggered this notification. Is it Talk, is it Watchlist or
is caused by fact that Recipient is on the ENotifAll list.
Bug: T387995
Change-Id: I1993208882097c591f93fb47d80076a93c34fcd9
|
|\ \ \ \ \ \ \ \ \ \ \ |
|
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Why:
- ParserCache refuses to return cache entries for redirect pages. So we
shouldn't write these entries into the parser cache in the first
place.
- If we write but refuse to read, this means there will be a cache write
every time a redirect page is retrieved via ParserOutputAccess.
What:
- Make ParserCache:save() skip entries for redirect pages. This is
only reachable via "redirect=no" since by default MediaWiki
pageviews render the destination page instead.
Note:
- It seems we should be able to store redirect pages like any other
page in the ParserCache. Why we didn't, and whether we should, is
not yet clear. While this is being investigated we should establish
consistent behavior for get() and save().
Bug: T389591
Change-Id: I880997193d6de66121c902de80a77c1305bd01d3
|
|\ \ \ \ \ \ \ \ \ \ \ \ |
|
| | |_|_|_|_|_|/ / / / /
| |/| | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Options are strings, cast them to bool
Change-Id: I66aef7856907d9bf80308eb9e32b7178b59040ba
|
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Change-Id: Ie9a992e9ad3a6f596c122d3d2747e5f9a0a22b51
|
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Bug: T390771
Change-Id: I2ef54405b0f27d665d11e387ff9b6a6e8633d460
|
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Bug: T385703
Change-Id: I7975b7443c03f9fbabcc7addcf6bba8656531360
|
|\ \ \ \ \ \ \ \ \ \ \ \
| |_|_|_|_|_|_|_|_|_|_|/
|/| | | | | | | | | | | |
|
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
There are dedicated classes and methods for this. No need to redo the
same manually.
Change-Id: I75f5c1f0911f5fcdc63414943747cb3fca664a8e
|
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Change-Id: I94adc67717ac66fc655866f4b8d93bcb758eaace
|
|\ \ \ \ \ \ \ \ \ \ \ \ |
|
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
Bug: T381727
Bug: T389536
Bug: T389562
Change-Id: If7d858ce69582cbec2ceaaf82dcd22bcf3d9e202
|
|\ \ \ \ \ \ \ \ \ \ \ \ \
| | |_|_|_|_|_|/ / / / / /
| |/| | | | | | | | | | | |
|
| | |_|/ / / / / / / / /
| |/| | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Setting conditional variable (and use it under the same conditional),
makes it hard for phan to see the correct type for the variables.
phan fails with "Suspicious array access" when formatDate() gets a
return type declaration `array`.
Unpack the array in the foreach and use the same variable to check.
Change-Id: I8408d10774cdf96ee59c0f1de3729aa41d2602b6
|
|\ \ \ \ \ \ \ \ \ \ \ \ |
|
| |/ / / / / / / / / / /
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Usage of the return value removed in 6e88bd35ff / r28086
Change-Id: Ie1a66528c409c79f49d8ad34792671c5d694f495
|
|\ \ \ \ \ \ \ \ \ \ \ \ |
|
| | |_|_|_|_|_|_|_|_|_|/
| |/| | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Change-Id: I328e7691cf620ddc9520670826a928a291e0dfe9
|
|\ \ \ \ \ \ \ \ \ \ \ \ |
|
| |/ / / / / / / / / / /
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Change-Id: Iba083aa271a911107477430e6831e349bcd2acfb
|
|\ \ \ \ \ \ \ \ \ \ \ \ |
|
| |/ / / / / / / / / / /
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Change-Id: I56b31870ce9be46d15fcd9095f9a12e0f2488ed9
|
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Added the linktrail for the Buginese script to MessagesBug_bugi and the Buginese script to the Release notes
Bug: T389916
Change-Id: I9e14944230fd78f3423426728d23b6e873ffbb40
|
|\ \ \ \ \ \ \ \ \ \ \ \ |
|
| | |/ / / / / / / / / /
| |/| | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Bug: T389916
Change-Id: Ie1c4c6bf1c3dcf5da6410bb282830390b0798405
|
|\ \ \ \ \ \ \ \ \ \ \ \
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
($haystack)"
|
| |/ / / / / / / / / / /
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
```
PHP Deprecated: strrpos(): Passing null to parameter #1 ($haystack) of type string is deprecated in /srv/mediawiki/1.42/includes/libs/filebackend/FileBackend.php on line 1588
```
Bug: T384851
Change-Id: I36c401587d94e7c1ab444d355b03ea88da8fb6e8
|
|/ / / / / / / / / / /
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | |
| | | | | | | | | | | |
Bug: T390221
Depends-On: Ic052dc51bdc6b5e8c2b2e7307359b0d5f07490d1
Change-Id: I4d2272540f9dd78d5b638929c06c917273f3797b
|
|\ \ \ \ \ \ \ \ \ \ \ |
|
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Change-Id: I8f55e929543b67ed66191a9421c5c365b625aecc
|
|\ \ \ \ \ \ \ \ \ \ \ \ |
|
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
Change-Id: Ic9cd94e8844043b87885b6c7f1768cdcd2e86837
|
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
Change-Id: I2f499c75ce82bd4bb8697b6a950017f0309b2427
|
|\ \ \ \ \ \ \ \ \ \ \ \ \
| |_|_|/ / / / / / / / / /
|/| | | | | | | | | | | | |
|
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
Add PLURAL to the "blocked $2 times" part of two messages.
Follows-up: I4e4151ff5b5ccc7efcd5cf94fa42c3b16a188837
Change-Id: I1c450c12a6f6f1b089fb8d0517c8527ac61f0234
|
|\ \ \ \ \ \ \ \ \ \ \ \ \ |
|
| | |_|_|_|_|_|/ / / / / /
| |/| | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
Parsoid has a set of Parsoid-specific native fragment handlers it
registers for parser tests; ensure the magic words for these are
registered before running parser tests.
Change-Id: Ib4e697f30d02a2e460e818cbde591fbe1bafd6fa
|
| |/ / / / / / / / / / /
|/| | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Almost everywhere else in MediaWiki core and extensions
(at least those used by Wikimedia), it's written
without a space.
Follow up from I4c40a9566642fb2fbcf2006e4b7e33501b7b2575
Change-Id: I952f9552a5300eb883df1b2fc8d9276c42a2c3a6
|
|\ \ \ \ \ \ \ \ \ \ \ \ |
|
| | |_|_|_|_|/ / / / / /
| |/| | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | |
| | | | | | | | | | | | |
Almost everywhere else in MediaWiki core and extensions
(at least those used by Wikimedia), it's written
without a space.
Change-Id: Ia70a197045e78a8ed79c81ebf3023b89e50df263
|
|\ \ \ \ \ \ \ \ \ \ \ \ |
|
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
When MediaWiki creates load.php URLs, in OutputPage and via RL\Client,
it sets 'lang', not 'dir'. The direction is automatically derived
from the language, which makes the library easier to use for developers,
and also avoids a potentially confusing scenario in which the two
are different (e.g. lang=he and dir=ltr would not make sense, and
would further risk poisoning LTR content in an RTL/Hebrew-related
cache somewhere).
Bug: T225845
Change-Id: I14905c85bb5e3bb47a681dc99668b92844d22f04
|
|\ \ \ \ \ \ \ \ \ \ \ \ \ |
|
| |/ / / / / / / / / / / /
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
XmlSelector exists primarily to create `<select>` menus,
whereas this is creating `<datalist>`. This caller bypassing half
the features (ctor args), overrides a half of what remains (tagname),
and leaves most of the rest unused (e.g. optgroup, defaults).
This in turn called Xml::option() which has more indirection, such
as an optional selected, which is never set.
Replace by idiomatic, simple, and diret use of Html::element() and
Html::rawElement().
While at it:
* Improve docs for the return value and make it clear that this is not
a list to append to, the exactly two chunks have distinct identities
and the callers depend on that exact [0] and [1] identity.
* There was an early return with empty array for when the feature is
disabled by site config. This violates the return shape but worked
because all callers in core check the return value for truethy-ness.
Support this by returning null instead, which is less likely to be
mistaken and communicates more clearly the need for checks on the
caller side, and e.g. nullable `?array` in the future.
Test plan:
* Edit buildTagFilterSelector() to force `if ( $ooui )` to `if ( false )`.
* Open Special:Contributions locally.
* Confirm visual rendering and HTML source are identical before/after.
Change-Id: I6d68b1846d31d861d6f4bacff0fca38e478e00de
|
|\ \ \ \ \ \ \ \ \ \ \ \ \ |
|
| | |_|/ / / / / / / / / /
| |/| | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | |
| | | | | | | | | | | | | |
Bug: T387255
Change-Id: I3c39db9d8394254b94f0828de0df4fb57e8d49e7
|
|\ \ \ \ \ \ \ \ \ \ \ \ \ |
|