From 365f4419243fcd80c84acb1952bf29e7573334c0 Mon Sep 17 00:00:00 2001 From: Neil Lalonde Date: Wed, 8 May 2019 11:01:01 -0400 Subject: [PATCH] FIX: groups with mixed case names not working --- assets/javascripts/discourse/components/ad-component.js.es6 | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/assets/javascripts/discourse/components/ad-component.js.es6 b/assets/javascripts/discourse/components/ad-component.js.es6 index c70cbb1..8efd72f 100644 --- a/assets/javascripts/discourse/components/ad-component.js.es6 +++ b/assets/javascripts/discourse/components/ad-component.js.es6 @@ -14,9 +14,10 @@ export default Ember.Component.extend({ return true; } - const noAdsGroupNames = this.siteSettings.no_ads_for_groups.split("|"); + const groupNames = groups.map(g => g.name.toLowerCase()); + const noAdsGroupNames = this.siteSettings.no_ads_for_groups.split("|").map(g => g.toLowerCase()); - return !groups.any(group => noAdsGroupNames.includes(group.name)); + return !groupNames.any(g => noAdsGroupNames.includes(g)); }, isNthPost(n) {