Ejegg has uploaded a new change for review.
https://gerrit.wikimedia.org/r/174319
Change subject: Stop clobbering reason from alterImpressionData
......................................................................
Stop clobbering reason from alterImpressionData
My last change to impression recording overwrote reasons set by
banner-specific alterImpressionData functions. This should let
those back through.
Change-Id: Ifb4a6c01ce894408b0eb86587bcb5eb130d615bd
---
M modules/ext.centralNotice.bannerController/bannerController.js
1 file changed, 5 insertions(+), 1 deletion(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CentralNotice
refs/changes/19/174319/1
diff --git a/modules/ext.centralNotice.bannerController/bannerController.js
b/modules/ext.centralNotice.bannerController/bannerController.js
index 1069335..79eb55a 100644
--- a/modules/ext.centralNotice.bannerController/bannerController.js
+++ b/modules/ext.centralNotice.bannerController/bannerController.js
@@ -434,7 +434,11 @@
// ~~ as of 2012-11-27
if ( !bannerShown ) {
hideBanner = true;
- impressionData.reason =
'alterImpressionData';
+ // alterImpressionData should set a
reason, but we'll set a
+ // default if it didn't
+ if ( !impressionData.reason ) {
+ impressionData.reason =
'alterImpressionData';
+ }
}
}
}
--
To view, visit https://gerrit.wikimedia.org/r/174319
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ifb4a6c01ce894408b0eb86587bcb5eb130d615bd
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/CentralNotice
Gerrit-Branch: master
Gerrit-Owner: Ejegg <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
