Ejegg has submitted this change and it was merged. Change subject: Fix check of banner logged-in status properties ......................................................................
Fix check of banner logged-in status properties Change-Id: I8036e34f9367b33dd2f2b81bb5be1d117f58a815 --- M modules/ext.centralNotice.bannerController/bannerController.lib.js M tests/data/AllocationsFixtures.json 2 files changed, 3 insertions(+), 6 deletions(-) Approvals: Ejegg: Looks good to me, approved diff --git a/modules/ext.centralNotice.bannerController/bannerController.lib.js b/modules/ext.centralNotice.bannerController/bannerController.lib.js index 8548f70..84d9e08 100644 --- a/modules/ext.centralNotice.bannerController/bannerController.lib.js +++ b/modules/ext.centralNotice.bannerController/bannerController.lib.js @@ -311,12 +311,10 @@ } // Filter for logged-in status - if ( mw.centralNotice.data.anonymous && - ( banner.display_anon != 1 ) ) { + if ( mw.centralNotice.data.anonymous && !banner.display_anon ) { continue; } - if ( !mw.centralNotice.data.anonymous && - ( banner.display_account != 1 ) ) { + if ( !mw.centralNotice.data.anonymous && !banner.display_account ) { continue; } diff --git a/tests/data/AllocationsFixtures.json b/tests/data/AllocationsFixtures.json index 4606f81..07b5673 100644 --- a/tests/data/AllocationsFixtures.json +++ b/tests/data/AllocationsFixtures.json @@ -16,8 +16,7 @@ "banners": [ { "name": "b1", - "weight": 5, - "display_anon": 1 + "weight": 5 } ] } -- To view, visit https://gerrit.wikimedia.org/r/175786 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8036e34f9367b33dd2f2b81bb5be1d117f58a815 Gerrit-PatchSet: 2 Gerrit-Project: mediawiki/extensions/CentralNotice Gerrit-Branch: master Gerrit-Owner: AndyRussG <andrew.green...@gmail.com> Gerrit-Reviewer: Awight <awi...@wikimedia.org> Gerrit-Reviewer: Ejegg <eeggles...@wikimedia.org> Gerrit-Reviewer: Mwalker <mwal...@khaosdev.com> Gerrit-Reviewer: Ssmith <ssm...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits