diff --git a/core/serializers/detail.py b/core/serializers/detail.py
index bc2d471e..058d17f7 100644
--- a/core/serializers/detail.py
+++ b/core/serializers/detail.py
@@ -78,10 +78,7 @@ class CategoryDetailSerializer(ModelSerializer):
         request = self.context.get("request")
         user = getattr(request, "user", None)
 
-        if user is not None and user.has_perm("view_attribute"):
-            attributes = obj.attributes.all()
-        else:
-            attributes = obj.attributes.filter(is_active=True)
+        attributes = obj.attributes.all() if user.has_perm("view_attribute") else obj.attributes.filter(is_active=True)
 
         for attr in attributes:
             distinct_vals = (