Send content rules with pattern_type to clients (#14356)
This commit is contained in:
parent
b5ab2c428a
commit
63cc56affa
|
@ -0,0 +1 @@
|
||||||
|
Fix a bug introduced in 1.66 which would not send certain pushrules to clients. Contributed by Nico.
|
|
@ -44,6 +44,12 @@ def format_push_rules_for_user(
|
||||||
|
|
||||||
rulearray.append(template_rule)
|
rulearray.append(template_rule)
|
||||||
|
|
||||||
|
pattern_type = template_rule.pop("pattern_type", None)
|
||||||
|
if pattern_type == "user_id":
|
||||||
|
template_rule["pattern"] = user.to_string()
|
||||||
|
elif pattern_type == "user_localpart":
|
||||||
|
template_rule["pattern"] = user.localpart
|
||||||
|
|
||||||
template_rule["enabled"] = enabled
|
template_rule["enabled"] = enabled
|
||||||
|
|
||||||
if "conditions" not in template_rule:
|
if "conditions" not in template_rule:
|
||||||
|
@ -93,10 +99,14 @@ def _rule_to_template(rule: PushRule) -> Optional[Dict[str, Any]]:
|
||||||
if len(rule.conditions) != 1:
|
if len(rule.conditions) != 1:
|
||||||
return None
|
return None
|
||||||
thecond = rule.conditions[0]
|
thecond = rule.conditions[0]
|
||||||
if "pattern" not in thecond:
|
|
||||||
return None
|
|
||||||
templaterule = {"actions": rule.actions}
|
templaterule = {"actions": rule.actions}
|
||||||
templaterule["pattern"] = thecond["pattern"]
|
if "pattern" in thecond:
|
||||||
|
templaterule["pattern"] = thecond["pattern"]
|
||||||
|
elif "pattern_type" in thecond:
|
||||||
|
templaterule["pattern_type"] = thecond["pattern_type"]
|
||||||
|
else:
|
||||||
|
return None
|
||||||
else:
|
else:
|
||||||
# This should not be reached unless this function is not kept in sync
|
# This should not be reached unless this function is not kept in sync
|
||||||
# with PRIORITY_CLASS_INVERSE_MAP.
|
# with PRIORITY_CLASS_INVERSE_MAP.
|
||||||
|
|
Loading…
Reference in New Issue