There's no need to do any fancy isset() on every setting - extension.json
will automatically take care of that.
For $wgMediaViewerEnableByDefaultForAnonymous which depended on the value
of $wgMediaViewerEnableByDefault, treat true/false as explicit values and
null as a fallback to the other configuration setting.
Keep the documentation associated with each setting in MultimediaViewer.php
for IDEs.
Change-Id: Ib34bbf75d027b8b96b2ec1a5975fa19061fadd55
multimediaviewer-enable default option does not exist in
$wgDefaultUserOptions, but is added in a UserGetDefaultOptions
hook callback.
User::getDefaultOptions will trigger that hook (and merge it
with $wgDefaultUserOptions), so that's the result that should
be used to figure out the value for multimediaviewer-enable.
Bug: T166058
Change-Id: Ib16a779430886305582dfce7db6600edd87ace11
To be consistent with $wgDefaultUserOptions logic
Migration of bugzilla bug id to maniphest
Bug: T148492
Change-Id: Ie371a0c2445c9aededb5c9e26018d19cbd616358
Depends-On: If53a735458703f0bd2c094349edf86f38f05ccd7
The following sniffs are failing and were disabled:
* MediaWiki.Commenting.FunctionComment.MissingParamComment
* MediaWiki.Commenting.FunctionComment.MissingParamName
* MediaWiki.Commenting.FunctionComment.MissingParamTag
* MediaWiki.Commenting.FunctionComment.MissingReturn
* MediaWiki.Commenting.FunctionComment.ParamNameNoMatch
* MediaWiki.Commenting.FunctionComment.WrongStyle
* MediaWiki.FunctionComment.Missing.Public
Change-Id: Iab9881706ae88d106baefed1460ed5317eb8a6ae
Some of the code encountered accessed mw.config directly,
I cleaned that up by migrating to mmv.Config, which is an
abstraction added to avoid peeking at mw.config directly.
Bug: T132064
Change-Id: I2a95ba703e6c7f46296f8e435bceec769dceebf9
Having many modules makes the startup module in the head
unnecessarily longer. Things should only be in their own
module if they're going to be loaded on demand on their
own or are a shared dependency of separate modules that
are.
This change brings down the amount of modules declared by
Media Viewer from 53 to 8. The ones that remain are
mostly things loaded on demand.
Bug: T103706
Change-Id: I5b0d29209599285b93217e96def818e641646c73
When media viewer is enabled, go through the latest images in full-screen
mode making it look neat.
Bug: T63085
Bug: T73372
Change-Id: I8d92689eba580a88c98207e9197f1af0ea8ac96b
This is complete, but it would be better if the HEAD request
was actually aborted by Varnish when the viewDuration parameter is
present, or if the hit pointed to a script that does that.
Change-Id: I66cafd97427756411e967de1901324af2215e3ae
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/1001
This will be needed by Erik Zachte for compiling per-file image view data.
Since Media Viewer does preloading, it skews the HTTP request-based
statistics. By marking image requests coming from Media Viewer,
it allows us to remove that bias.
Change-Id: Iac8e7770c1a379691547de4b6d47b7d54467f54d
Mingle: https://wikimedia.mingle.thoughtworks.com/projects/multimedia/cards/1002
Ensure that changing the preference via the quick link and
Special:Preferences is stored identically in the DB.
Change-Id: Ia37da1c6bfbb3edf0eba56f01105e4a5f3a5e4ba