refactor: adds _check_taxonomy and _check_tag methods to the ObjectTag subclasses

Author: pomegranited

openedx_tagging/core/tagging/models.py

@@ -490,6 +490,24 @@ def valid_for(cls, taxonomy: Taxonomy = None, **kwargs) -> bool:
         """
         return taxonomy and taxonomy.allow_free_text
 
+    def _check_taxonomy(self):
+        """
+        Returns True if this ObjectTag has a valid taxonomy.
+
+        Subclasses should override this method to perform any additional validation for the particular type of object tag.
+        """
+        # Must be linked to a free-text taxonomy
+        return self.taxonomy and self.taxonomy.allow_free_text
+
+    def _check_tag(self):
+        """
+        Returns True if this ObjectTag has a valid tag value.
+
+        Subclasses should override this method to perform any additional validation for the particular type of object tag.
+        """
+        # Open taxonomies don't need an associated tag, but we need a value.
+        return bool(self._value)
+
     def _check_object(self):
         """
         Returns True if this ObjectTag has a valid object.
@@ -509,14 +527,10 @@ def is_valid(self, check_taxonomy=True, check_tag=True, check_object=True) -> bo
         If `check_tag` is False, then we skip validating the object tag's tag reference.
         If `check_object` is False, then we skip validating the object ID/type.
         """
-        # Must be linked to a free-text taxonomy
-        if check_taxonomy and (
-            not self.taxonomy_id or not self.taxonomy.allow_free_text
-        ):
+        if check_taxonomy and not self._check_taxonomy():
             return False
 
-        # Open taxonomies don't need an associated tag, but we need a value.
-        if check_tag and not self.value:
+        if check_tag and not self._check_tag():
             return False
 
         if check_object and not self._check_object():
@@ -540,6 +554,24 @@ def valid_for(cls, taxonomy: Taxonomy = None, tag: Tag = None, **kwargs) -> bool
         """
         return tag and taxonomy and not taxonomy.allow_free_text
 
+    def _check_taxonomy(self):
+        """
+        Returns True if this ObjectTag has a valid taxonomy.
+
+        Subclasses should override this method to perform any additional validation for the particular type of object tag.
+        """
+        # Must be linked to a closed taxonomy
+        return self.taxonomy and not self.taxonomy.allow_free_text
+
+    def _check_tag(self):
+        """
+        Returns True if this ObjectTag has a valid tag value.
+
+        Subclasses should override this method to perform any additional validation for the particular type of object tag.
+        """
+        # Closed taxonomies require a Tag
+        return bool(self.tag_id)
+
     def is_valid(self, check_taxonomy=True, check_tag=True, check_object=True) -> bool:
         """
         Returns True if this ObjectTag is valid for use with a closed taxonomy.
@@ -550,20 +582,16 @@ def is_valid(self, check_taxonomy=True, check_tag=True, check_object=True) -> bo
         If `check_tag` is False, then we skip validating the object tag's tag reference.
         If `check_object` is False, then we skip validating the object ID/type.
         """
-        # Must be linked to a closed taxonomy
-        if check_taxonomy and (not self.taxonomy_id or self.taxonomy.allow_free_text):
-            return False
-
-        # Closed taxonomies require a Tag
-        if check_tag and not self.tag_id:
+        if not super().is_valid(
+            check_taxonomy=check_taxonomy,
+            check_tag=check_tag,
+            check_object=check_object,
+        ):
             return False
 
         if check_tag and check_taxonomy and (self.tag.taxonomy != self.taxonomy):
             return False
 
-        if check_object and not self._check_object():
-            return False
-
         return True