Removes a CSS selector that is only used by Vector and makes red-links
in the personal menu a slightly different color than other red-links
in Vector (which are styled by rules in legacy.less).
Bug: T282884
Change-Id: Ie04182da57c968b34401a3811ce715424eaa181d
Refactor CollapsibleFieldsetLayout to work when not wrapped in
mw-htmlform-ooui-wrapper.
Bug: T278935
Change-Id: I89c17671de1c1c8b5897a75ad292c576a1e1bbb1
This commit was reverted before cause it has changed the initialization order
and broke FlaggedRevs. Now FlaggedRevs issue was addressed, so we can apply
it back.
Bug: T254838
Change-Id: Id29024caf06fcd0d7d4e039aea8142e8131d47a7
For MySQL/Sqlite:
- Nothing
For Postgres:
- Introduce the table. It's not used but it'll be used in the future (T220450)
Bug: T164898
Bug: T230428
Bug: T220450
Change-Id: I9f33132676344f8cd813f7a438b3a6a078fd281c
Join conditions are in the form of [ <table> => [ 'JOIN', <cond> ] ].
If they are given incorrectly as strings, e.g. in the form
[ <table> => 'JOIN', <cond> ], there won't be any error (courtesy
of PHP handling list($a,$b)='x' by setting $a and $b to null) and
the resulting SQL will probably even be valid, just completely
different from what's intended. Be more strict about verifying
the format.
Change-Id: Id09aff6193829972cdca3262dc8bb878512c6cc3
Some MediaHandler subclasses were setting custom properties on the File
object in order to cache file-associated state. So:
* Add File::getHandlerState() and File::setHandlerState().
* Put them in an interface, which will be used in a subsequent commit in
MediaHandler::getSizeAndMetadata().
* Use them in DjvuHandler.
* Provide a trivial implementation of the interface, for use in testing
and in the subsequent commit.
Change-Id: Ic365384ff13f7898c1203da38c4405abf03d7563
The value of content-media now depends on configuration.
These modules can be merged, as the expectation is both should
apply when the UseNewMediaStructure configuration flag is enabled.
For backwards compatibility map any content-thumbnail requests to
content-media
Bug: T278560
Change-Id: I10ba53b1ee217eae9e05182e989d56f5dc107722
Follows-up 663e4ce6bb.
SqlModuleDependencyStore::remove() gets called unconditionally by
RL::saveModuleDependenciesInternal() and lots of time with empty value,
this opens a master connection in GET requests where it's not needed.
Let's remove that for now.
Change-Id: I9cc658d5282b3cfa8664406497f24dedcb688369
Add restriction support to LoginSignupSpecialPage::__construct,
and set mRestriction to 'createaccount', removes the need to
inject a PermissionManager
Change-Id: Ifd7d6e687a3c78d300cc612526458afb6934a4c0
SqlModuleDependencyStore::storeMulti() gets called unconditionally by
RL::saveModuleDependenciesInternal() and lots of time with empty value,
this opens a master connection in GET requests where it's not needed.
Let's remove that for now.
More info in I462055ce9caa
Change-Id: I6f618d396d64c687ba6cc68cfa99bf3f4327e798
Create new SessionProviderTestTrait and
SessionProviderTestTrait::initProvider method. Replace uses of
SessionProvider::init with new method in tests.
Bug: T283709
Change-Id: I2770188faf3e4f374f9df885ff4deff7189063fc
Instead of using a PermissionManager
Required some changes to the creation of
the mock users in WatchedItemQueryServiceUnitTest,
simplified those changes by combining the various
helper methods that returned user mocks.
Change-Id: Id8382a5884133013e3da0d3318a41f8c451c3039
