Migrate api_slug missed before

Author: AlanCoding

ansible_base/rbac/migrations/0005_remote_permissions_data.py

@@ -27,6 +27,13 @@ def migrate_content_type(apps, schema_editor):
         cls = apps.get_model('dab_rbac', model_name)
         cls.objects.all().delete()
 
+    # DABPermission model had api_slug added in last migration
+    # if records existed before this point, it needs to be filled in
+    permission_cls = apps.get_model('dab_rbac', 'DABPermission')
+    for permission in permission_cls.objects.all():
+        permission.api_slug = f'{permission.new_content_type.service}.{permission.codename}'
+        permission.save()
+
 
 class Migration(migrations.Migration):
 

ansible_base/resource_registry/shared_types.py

@@ -79,14 +79,6 @@ class TeamType(SharedResourceTypeSerializer):
     )
 
 
-class RoleDefinitionPermissionsSerializer(serializers.Serializer):
-    permissions = serializers.SlugRelatedField(
-        slug_field='api_slug',
-        read_only=True,
-        many=True,
-    )
-
-
 class LenientPermissionSlugListField(serializers.ListField):
     child = serializers.CharField()
 
@@ -100,7 +92,6 @@ def to_representation(self, value):
 
 class RoleDefinitionType(SharedResourceTypeSerializer):
     RESOURCE_TYPE = "roledefinition"
-    ADDITIONAL_DATA_SERIALIZER = RoleDefinitionPermissionsSerializer
     UNIQUE_FIELDS = ("name",)
 
     name = serializers.CharField()