refactor(serializers): Update serializers to use ACLAssignment model

Replaces `ACLInterfaceAssignmentSerializer` with
`ACLAssignmentSerializer` to align with the unified ACLAssignment model.
Removes legacy fields and validation logic that depended on outdated
models. Enhances maintainability by consolidating serializers under the
updated model structure.

BREAKING CHANGE: Legacy ACLInterfaceAssignmentSerializer is replaced
with ACLAssignmentSerializer.

Author: pheus

netbox_acls/api/serializers.py

@@ -11,17 +11,17 @@
 from rest_framework import serializers
 from utilities.api import get_serializer_for_model
 
-from ..constants import ACL_HOST_ASSIGNMENT_MODELS, ACL_INTERFACE_ASSIGNMENT_MODELS
+from ..constants import ACL_ASSIGNMENT_MODELS
 from ..models import (
     AccessList,
     ACLExtendedRule,
-    ACLInterfaceAssignment,
+    ACLAssignment,
     ACLStandardRule,
 )
 
 __all__ = [
     "AccessListSerializer",
-    "ACLInterfaceAssignmentSerializer",
+    "ACLAssignmentSerializer",
     "ACLStandardRuleSerializer",
     "ACLExtendedRuleSerializer",
 ]
@@ -45,10 +45,6 @@ class AccessListSerializer(NetBoxModelSerializer):
         view_name="plugins-api:netbox_acls-api:accesslist-detail",
     )
     rule_count = serializers.IntegerField(read_only=True)
-    assigned_object_type = ContentTypeField(
-        queryset=ContentType.objects.filter(ACL_HOST_ASSIGNMENT_MODELS),
-    )
-    assigned_object = serializers.SerializerMethodField(read_only=True)
 
     class Meta:
         """
@@ -61,9 +57,6 @@ class Meta:
             "url",
             "display",
             "name",
-            "assigned_object_type",
-            "assigned_object_id",
-            "assigned_object",
             "type",
             "default_action",
             "comments",
@@ -75,14 +68,6 @@ class Meta:
         )
         brief_fields = ("id", "url", "display", "name")
 
-    @extend_schema_field(serializers.JSONField(allow_null=True))
-    def get_assigned_object(self, obj):
-        if obj.assigned_object is None:
-            return None
-        serializer = get_serializer_for_model(obj.assigned_object)
-        context = {"request": self.context["request"]}
-        return serializer(obj.assigned_object, nested=True, context=context).data
-
     def validate(self, data):
         """
         Validates api inputs before processing:
@@ -103,26 +88,26 @@ def validate(self, data):
         return super().validate(data)
 
 
-class ACLInterfaceAssignmentSerializer(NetBoxModelSerializer):
+class ACLAssignmentSerializer(NetBoxModelSerializer):
     """
-    Defines the serializer for the django ACLInterfaceAssignment model and associates it with a view.
+    Defines the serializer for the django ACLAssignment model and associates it with a view.
     """
 
     url = serializers.HyperlinkedIdentityField(
-        view_name="plugins-api:netbox_acls-api:aclinterfaceassignment-detail",
+        view_name="plugins-api:netbox_acls-api:aclassignment-detail",
     )
     access_list = AccessListSerializer(nested=True, required=True)
     assigned_object_type = ContentTypeField(
-        queryset=ContentType.objects.filter(ACL_INTERFACE_ASSIGNMENT_MODELS),
+        queryset=ContentType.objects.filter(ACL_ASSIGNMENT_MODELS),
     )
     assigned_object = serializers.SerializerMethodField(read_only=True)
 
     class Meta:
         """
-        Associates the django model ACLInterfaceAssignment & fields to the serializer.
+        Associates the django model ACLAssignment & fields to the serializer.
         """
 
-        model = ACLInterfaceAssignment
+        model = ACLAssignment
         fields = (
             "id",
             "url",
@@ -148,34 +133,6 @@ def get_assigned_object(self, obj):
         context = {"request": self.context["request"]}
         return serializer(obj.assigned_object, nested=True, context=context).data
 
-    def validate(self, data):
-        """
-        Validate the AccessList django model's inputs before allowing it to update the instance.
-          - Check that the GFK object is valid.
-          - Check that the associated interface's parent host has the selected ACL defined.
-        """
-        error_message = {}
-        acl_host = data["access_list"].assigned_object
-
-        if data["assigned_object_type"].model == "interface":
-            interface_host = data["assigned_object_type"].get_object_for_this_type(id=data["assigned_object_id"]).device
-        elif data["assigned_object_type"].model == "vminterface":
-            interface_host = (
-                data["assigned_object_type"].get_object_for_this_type(id=data["assigned_object_id"]).virtual_machine
-            )
-        else:
-            interface_host = None
-        # Check that the associated interface's parent host has the selected ACL defined.
-        if acl_host != interface_host:
-            error_acl_not_assigned_to_host = "Access List not present on the selected interface's host."
-            error_message["access_list"] = [error_acl_not_assigned_to_host]
-            error_message["assigned_object_id"] = [error_acl_not_assigned_to_host]
-
-        if error_message:
-            raise serializers.ValidationError(error_message)
-
-        return super().validate(data)
-
 
 class ACLStandardRuleSerializer(NetBoxModelSerializer):
     """