diff options
author | David Baker <dave@matrix.org> | 2015-02-09 14:17:52 +0000 |
---|---|---|
committer | David Baker <dave@matrix.org> | 2015-02-09 14:17:52 +0000 |
commit | 784d714a3f2be2cf15151eee8723377a0e3eea11 (patch) | |
tree | 23df275e074d1bd612f558884399058e56dbf37d /synapse | |
parent | glob *s should probably be non-greedy (diff) | |
download | synapse-784d714a3f2be2cf15151eee8723377a0e3eea11.tar.xz |
Fix server default rule injection (downwards, not upwards!)
Diffstat (limited to '')
-rw-r--r-- | synapse/push/baserules.py | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/synapse/push/baserules.py b/synapse/push/baserules.py index 8d4b806da6..37878f1e0b 100644 --- a/synapse/push/baserules.py +++ b/synapse/push/baserules.py @@ -4,24 +4,24 @@ def list_with_base_rules(rawrules, user_name): ruleslist = [] # shove the server default rules for each kind onto the end of each - current_prio_class = 1 + current_prio_class = PRIORITY_CLASS_INVERSE_MAP.keys()[-1] for r in rawrules: - if r['priority_class'] > current_prio_class: - while current_prio_class < r['priority_class']: + if r['priority_class'] < current_prio_class: + while r['priority_class'] < current_prio_class: ruleslist.extend(make_base_rules( user_name, PRIORITY_CLASS_INVERSE_MAP[current_prio_class]) ) - current_prio_class += 1 + current_prio_class -= 1 ruleslist.append(r) - while current_prio_class <= PRIORITY_CLASS_INVERSE_MAP.keys()[-1]: + while current_prio_class > 0: ruleslist.extend(make_base_rules( user_name, PRIORITY_CLASS_INVERSE_MAP[current_prio_class]) ) - current_prio_class += 1 + current_prio_class -= 1 return ruleslist |