fix(component): avoid crash on getting unused enforcements

weblate/trans/models/component.py

@@ -106,10 +106,12 @@
 from weblate.vcs.ssh import add_host_key
 
 if TYPE_CHECKING:
+    from collections.abc import Iterable
     from datetime import datetime
 
     from weblate.addons.models import Addon
     from weblate.auth.models import AuthenticatedHttpRequest, User
+    from weblate.checks.base import BaseCheck
     from weblate.trans.models import Unit
 
 NEW_LANG_CHOICES = (
@@ -1104,7 +1106,7 @@
         # Calculate progress for translations
         if progress is None:
             self.translations_progress += 1
             progress = 100 * self.translations_progress // self.translations_count
         # Store task state
         current_task.update_state(
             state="PROGRESS", meta={"progress": progress, "component": self.pk}
@@ -1753,7 +1755,7 @@
             from weblate.trans.tasks import perform_push
 
             self.log_info("scheduling push")
             perform_push.delay_on_commit(
                 self.pk, None, force_commit=False, do_update=do_update
             )
 
@@ -2340,7 +2342,7 @@
                 .order_by("-id")[0]
                 .auto_status
             ):
                 self.do_lock(user=None, lock=False, auto=True)
 
         if ALERTS[alert].link_wide:
             for component in self.linked_childs:
@@ -2358,7 +2360,7 @@
 
         # Automatically lock on error
         if created and self.auto_lock_error and alert in LOCKING_ALERTS:
             self.do_lock(user=None, lock=True, auto=True)
 
         # Update details with exception of component removal
         if not created and not noupdate:
@@ -2563,7 +2565,7 @@
                     )
                     self.handle_parse_error(error.__cause__, filename=self.template)
                     self.update_import_alerts()
                     raise error.__cause__ from error  # pylint: disable=E0710
                 was_change |= bool(translation.reason)
                 translations[translation.id] = translation
                 languages[lang.code] = translation
@@ -2627,7 +2629,7 @@
         if self.needs_cleanup and not self.template:
             from weblate.trans.tasks import cleanup_component
 
             cleanup_component.delay_on_commit(self.id)
 
         if was_change:
             if self.needs_variants_update:
@@ -2659,7 +2661,7 @@
             if settings.CELERY_TASK_ALWAYS_EAGER:
                 batch_update_checks(self.id, batched_checks, component=self)
             else:
                 batch_update_checks.delay_on_commit(self.id, batched_checks)
         self.batch_checks = False
         self.batched_checks = set()
 
@@ -2810,7 +2812,7 @@
             raise ValidationError(
                 {"filemask": gettext("The file mask did not match any files.")}
             )
         langs = {}
         existing_langs = set()
 
         for match in matches:
@@ -2878,7 +2880,7 @@
         if (not self.new_base and self.new_lang != "add") or not self.file_format:
             return
         # File is valid or no file is needed
         errors = []
         if self.is_valid_base_for_new(errors):
             return
         # File is needed, but not present
@@ -3249,7 +3251,7 @@
         if self.variant_regex:
             variant_re = re.compile(self.variant_regex)
             units = process_units.filter(context__regex=self.variant_regex)
             variant_updates = {}
             for unit in units.iterator():
                 if variant_re.findall(unit.context):
                     key = variant_re.sub("", unit.context)
@@ -3747,14 +3749,15 @@
             for glossary in self.project.glossaries:
                 sync_glossary_languages.delay(glossary.pk)
 
-    def get_unused_enforcements(self):
+    def get_unused_enforcements(self) -> Iterable[dict | BaseCheck]:
         from weblate.trans.models import Unit
 
         for current in self.enforced_checks:
             try:
                 check = CHECKS[current]
             except KeyError:
                 yield {"name": current, "notsupported": True}
+                continue
             # Check is always enabled
             if not check.default_disabled:
                 continue