Use this class to track the entry point and handler used for requests,
making it available for use in profiling, stats, and logging code.
This makes it possible for periodic and/or shutdown profiling callbacks
to know the basic action handler that applies to the request (if any).
Metric names can easily include this string along with MW_ENTRY_POINT
to create per-action profiling dashboards.
This info cannot otherwise be acquired from things like excimer stack
traces since the router and handler classes do not appear in the stack
during PRESEND deferred updates and variations like ApiMain/SpecialPage
"inclusion mode" would have to be detected somehow.
Bug: T330810
Change-Id: Icca5a7a343faeeb18652994c96752acb61a61fd1
Follow-up to I89e1a9b901aff6ff1bde729df8958ba42a45f3d5 where I forgot
to remove this line after initialization was moved to MessageCache::get.
Change-Id: Iae441ccedba08eb12685d81381259aad49b7b42f
Special:RevisionDelete is using OOUI and is limited to priviliged users,
so its javascript can be merged into mediawiki.misc-authed-ooui.
The moved script is about 520 bytes minified uncompressed,
mediawiki.misc-authed-ooui grows to around 4800 bytes minified
uncompressed. The removed module saves up to 52 bytes minified
uncompressed in the startup module on every page view.
Change-Id: If0380e7a71e8bce215a21336f61644fd52c45ef6
If you store a WeakMap in the object, containing only circular
references in the values, then nothing is keeping the WeakMap alive and
it can be deleted along with its parent TempFSFile by the garbage
collector.
A WeakMap contains normal references to its values, so those normal
references can be marked by the GC like any other references.
Instead, store the WeakMap as a static property.
Bug: T332461
Bug: T332397
Change-Id: Idbaa1019c2bac3d1dd9b001753c2f4c0aaf66e98
Make it use a power function and increase LOW_TTL from 30 to 60 so
that higher regeneration times (e.g. 1+ seconds) are more resistant
to regeneration stampedes.
Also fix code that was using LOW_TTL instead of TTL_LAGGED.
Bug: T331914
Change-Id: I2c0b35fbbce6b5a692b6d7885a2a921d7ffdd031
Make it more obviously correct and straight forward, as you want
as little doubt and confusion as possible when debugging a test.
Doubt and confusion from our source code is... enough to make ones
day interesting.
Change-Id: I440be81d4ee7c505c0f2c11af7edaf513b8ff2d7
Applying central tokens to HTML elements, starting with a number of
color and border values.
Depends-On: I3c5c8f9fc4b1841f8f8bdbcf27de82291d1fabbd
Change-Id: I3461cd21866332e5f1329f11c27873191f43ed29
Enforce underscores as used by the most languages in the namespace
aliases and special page aliases
Change-Id: I53d895f4b1d241954bd31973ad4996beed91bfb2
If the messages are blanked (e.g. via site interface messages)
they should not be wrapped in elements.
Follow up to
Id2114a65635c050a2017926274622f0a3c2d58d6
Bug: T311891
Change-Id: I99dd7ee77bd9a29f318ab4b209f2b198376636e9
The 7% increase in css shipped is because of styles added for the
content-media feature, without dropping content-thumbnail. This is
noted in T297984#8525146
Getting rid of content-thumbnail won't be possible until templates and
extensions that mimic the parser output have been migrated, which is
being worked on in T318433.
Bug: T314318
Bug: T297984
Bug: T318433
Depends-On: Ib54b49d546423dd1b4416c82585bbb686950b29c
Change-Id: Ic587b1b9b8d22f3aef38e8823613815faaa09d9d
wgCanonicalServer is false by default. Setup.php tried to set it to the
result of a wfExpandUrl() call, which initialised the UrlUtils
service with the current value of wgCanonicalServer, i.e. false.
UrlUtils had a fallback for null but not false. So subsequent calls to
wfExpandUrl() would always use the empty string for the canonical
server. Probably since 472a914c63.
So, have UrlUtils handle the fallback, and have Setup.php ask UrlUtils
for the final value.
Bug: T325529
Change-Id: I99329e75bd53049ae84a8ae7a2536ad12cfeeb3b
* Wait until we know the action before choosing the temp directory.
Use the configured temp directory if the action is verify, since the
arguments for using a subdirectory of the source tree do not apply on
verify.
* Use the configured cache directory in precedence over the source tree.
Change-Id: I0431db332238c8e2de14ec71fb611e0a8d4e959b
* ParserTestRunner: LocalisationCache needs to be reset since it has a
reference to LanguageNameUtils which has a copy of
$wgUsePigLatinVariant. Also factor out some
MediaWikiServices::getInstance() calls.
* In some other tests, set the variable.
Change-Id: I6c1e9bfad9790cf805809c28a3f8d45952cbb981
The added flag makes no functional difference. It informs
executeQuery() and SQLPlatform->isWriteQuery() that this is not a
write query, which it would otherwise find out by running a regex
on the SQL text.
QUERY_IGNORE_DBO_TRX is documented as being for USE queries,
which in isWriteQuery() return false, just as it did already
based on the regex previously.
This follows-up to Ic91412ffdaab6b7d which made isWriteQuery warn
when this flag is missing, precisely to find this inefficiency.
Bug: T332228
Change-Id: If510415dbcf407353612f7b8e425d8a8189e26d9
