Merge pull request fedora-infra#40 from fedora-infra/feature/bugfix

Order of operations matters.
jeremycline · Feb 21, 2015 · 219f0c5 · 219f0c5
2 parents 1c2edd6 + bb4e4d4
commit 219f0c5
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/fmn/rules/generic.py b/fmn/rules/generic.py
@@ -44,7 +44,7 @@ def not_user_filter(config, message, fasnick=None, *args, **kw):
     if not fasnick:
         return False
 
-    fasnick = fasnick or [] and fasnick.split(',')
+    fasnick = (fasnick or []) and fasnick.split(',')
     valid = True
     for nick in fasnick:
         if nick.strip() in fedmsg.meta.msg2usernames(message, **config):