diff --git a/src/commands/subscription_helpers.py b/src/commands/subscription_helpers.py index e122654..c73c146 100644 --- a/src/commands/subscription_helpers.py +++ b/src/commands/subscription_helpers.py @@ -325,8 +325,6 @@ async def _fetch_clan_basic_card( def _standardized_rule_string(rule: dict[str, Any]) -> str: - fresh = "yes" if bool(rule.get("requireFresh")) else "no" - completed = "yes" if bool(rule.get("requireCompleted")) else "no" raid_ids_raw = rule.get("raidIds") raid_ids: list[str] = [] if isinstance(raid_ids_raw, list): @@ -347,7 +345,13 @@ def _standardized_rule_string(rule: dict[str, Any]) -> str: raid = f"raids:{','.join(raid_ids)}" else: raid = "raids:all" - return f"`fresh:{fresh}` `completed:{completed}` `{raid}`" + filters: list[str] = [] + if bool(rule.get("requireFresh")): + filters.append("`require:fresh`") + if bool(rule.get("requireCompleted")): + filters.append("`require:completed`") + filters.append(f"`{raid}`") + return " ".join(filters) def _player_rule_line( diff --git a/tests/test_subscription_helpers.py b/tests/test_subscription_helpers.py index 8cf334b..7da2be1 100644 --- a/tests/test_subscription_helpers.py +++ b/tests/test_subscription_helpers.py @@ -154,8 +154,8 @@ def test_registered_player_rules_show_per_rule_filters(self) -> None: ) fields = {f["name"]: f["value"] for f in out["embeds"][0]["fields"]} self.assertNotIn("Rule Filters", fields) - self.assertIn("`fresh:yes`", fields["Player Rules (1)"]) - self.assertIn("`completed:no`", fields["Player Rules (1)"]) + self.assertIn("`require:fresh`", fields["Player Rules (1)"]) + self.assertNotIn("`require:completed`", fields["Player Rules (1)"]) class SubscriptionRulesSuffixTests(unittest.TestCase):