Krinkle has uploaded a new change for review.
https://gerrit.wikimedia.org/r/198459
Change subject: Update mock wgEchoConfig to also run on
Special:JavaScript/qunit/plain
......................................................................
Update mock wgEchoConfig to also run on Special:JavaScript/qunit/plain
Follows-up 8d12ef9.
Tests were failing because wgEchoConfig was undefined and ext.base.js
accesses wgEchoConfig.eventlogging.
This mock should not be needed. Modules should be loadable at any
time and react to the current environment. If they should not do
anything, the client should be able to determine that. Perhaps
an isAnon() or some property wgEchoConfig to indicate this.
It only being loaded from beforePageDisplay() when logged-in is a
load optimisation, not business logic. wgEchoConfig should always
be there, or the code should handle the case of it not being there.
Change-Id: Ide90adf20e583921c9f3bd5cd37057995aa3d2c3
---
M Hooks.php
1 file changed, 5 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Echo
refs/changes/59/198459/1
diff --git a/Hooks.php b/Hooks.php
index c048399..053a4d3 100644
--- a/Hooks.php
+++ b/Hooks.php
@@ -771,7 +771,11 @@
);
$vars['wgEchoHelpPage'] = $wgEchoHelpPage;
$vars['wgEchoConfig'] = $wgEchoConfig;
- } else if ( SpecialPage::getTitleFor( 'JavaScriptTest', 'qunit'
)->equals( $outputPage->getTitle() ) ) {
+ } else if (
+ $outputPage->getTitle()->equals(
SpecialPage::getTitleFor( 'JavaScriptTest', 'qunit' ) ) ||
+ // Also if running from /plain or /export
+ $outputPage->getTitle()->isSubpageOf(
SpecialPage::getTitleFor( 'JavaScriptTest', 'qunit' ) )
+ ) {
// For testing purposes
$vars['wgEchoConfig'] = array(
'eventlogging' => array(
--
To view, visit https://gerrit.wikimedia.org/r/198459
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ide90adf20e583921c9f3bd5cd37057995aa3d2c3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Echo
Gerrit-Branch: master
Gerrit-Owner: Krinkle <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
