zephyr_mirror: Fix case issue in subscriptions syncing.

(imported from commit e87ddab3684f4acae940ee831caadbc346458a11)
This commit is contained in:
Tim Abbott 2012-12-03 15:33:33 -05:00
parent dd90c73f33
commit 24b8a7eccc

View file

@ -570,7 +570,7 @@ def add_humbug_subscriptions(verbose):
zephyr_subscriptions = set() zephyr_subscriptions = set()
skipped = set() skipped = set()
for (cls, instance, recipient) in parse_zephyr_subs(verbose=verbose): for (cls, instance, recipient) in parse_zephyr_subs(verbose=verbose):
if cls == "message": if cls.lower() == "message":
if recipient != "*": if recipient != "*":
# We already have a (message, *, you) subscription, so # We already have a (message, *, you) subscription, so
# these are redundant # these are redundant
@ -584,7 +584,7 @@ def add_humbug_subscriptions(verbose):
# instead of subscribing to stream "message" on humbug # instead of subscribing to stream "message" on humbug
zephyr_subscriptions.add(instance) zephyr_subscriptions.add(instance)
continue continue
elif cls == "mail" and instance == "inbox": elif cls.lower() == "mail" and instance.lower() == "inbox":
# We forward mail zephyrs, so no need to print a warning. # We forward mail zephyrs, so no need to print a warning.
continue continue
elif len(cls) > 30: elif len(cls) > 30: