Features: 1) Add is_filterable field to Attribute model with migration; 2) Extend admin panel to include is_filterable in general_fields; 3) Update serializers to respect is_filterable for attribute querying; 4) Ensure object types enforce is_filterable in attribute filtering;

Fixes: None;

Extra: Improve `AlterUniqueTogether` constraint for attributes with `name` and `group`;

core/admin.py

@@ -204,7 +204,7 @@ class AttributeAdmin(FieldsetsMixin, ActivationActionsMixin, ModelAdmin):  # typ
     readonly_fields = ("uuid", "modified", "created")
     autocomplete_fields = ["categories", "group"]
 
-    general_fields = ["is_active", "name", "value_type"]
+    general_fields = ["is_active", "name", "value_type", "is_filterable"]
     relation_fields = ["group", "categories"]
 
 

core/graphene/object_types.py

@@ -250,7 +250,7 @@ class CategoryType(DjangoObjectType):
         ):
             distinct_vals = (
                 AttributeValue.objects.annotate(value_length=Length("value"))
-                .filter(attribute=attr, product__category=self, value_length__lte=30)
+                .filter(attribute=attr, attribute__is_filterable=True, product__category=self, value_length__lte=30)
                 .values_list("value", flat=True)
                 .distinct()
             )

core/migrations/0043_attribute_is_filterable_and_more.py

@@ -0,0 +1,26 @@
+# Generated by Django 5.2 on 2025-09-22 11:10
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("core", "0042_attribute_name_fa_ir_attribute_name_he_il_and_more"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="attribute",
+            name="is_filterable",
+            field=models.BooleanField(
+                default=True,
+                help_text="designates whether this attribute can be used for filtering or not",
+                verbose_name="is filterable",
+            ),
+        ),
+        migrations.AlterUniqueTogether(
+            name="attribute",
+            unique_together={("name", "group")},
+        ),
+    ]

core/serializers/detail.py

@@ -81,7 +81,11 @@ class CategoryDetailSerializer(ModelSerializer):
 
         request = self.context.get("request")
         user = getattr(request, "user", AnonymousUser())
-        attrs_qs = obj.attributes.all() if user.has_perm("view_attribute") else obj.attributes.filter(is_active=True)
+        attrs_qs = (
+            obj.attributes.filter(is_filterable=True)
+            if user.has_perm("view_attribute")
+            else obj.attributes.filter(is_active=True, is_filterable=True)
+        )
         attributes = list(attrs_qs)
 
         attr_ids = [a.id for a in attributes]