fix: conditional fields

Author: francesco-filicetti

uniticket/uni_ticket/dynamic_form.py

@@ -57,7 +57,7 @@ def __init__(self, constructor_dict={}, custom_params={}, *args, **kwargs):
             "help_text": TICKET_SUBJECT_HELP_TEXT,
             "initial": subject_initial,
             "max_length": subject_max_length,
-            "min_length": subject_min_length
+            "min_length": subject_min_length,
         }
         subject_field = getattr(
             dynamic_fields, "CustomCharField")(**subject_data)
@@ -95,6 +95,7 @@ def __init__(self, constructor_dict={}, custom_params={}, *args, **kwargs):
             captcha_field.define_value(custom_value="", **custom_params)
             for single_field in captcha_field.get_fields():
                 self.final_fields[single_field.name] = single_field
+
         super().__init__(
             initial_fields=self.initial_fields,
             final_fields=self.final_fields,
@@ -137,6 +138,7 @@ def serialize_form(form, api_source = 'api_source'):
         field_dict['initial'] = field.initial
         field_dict['disabled'] = field.disabled
         field_dict['required'] = True if field.required else False
+        field_dict['visible'] = True if field.visible else False
         field_dict['error_messages'] = field.error_messages
         field_dict['help_text'] = field.help_text
         field_dict['api_source'] = getattr(field, api_source, '')

uniticket/uni_ticket/forms.py

@@ -128,6 +128,7 @@ class Meta:
             "pre_text",
             "aiuto",
             "is_required",
+            "visible",
             "ordinamento",
         ]
         labels = {
@@ -836,3 +837,47 @@ class Meta:
 
     class Media:
         js = ("js/textarea-autosize.js",)
+
+
+class TicketCategoryModuleRuleForm(ModelForm):
+    """ """
+    def __init__(self, *args, **kwargs):
+        module = kwargs.pop("module", None)
+        super().__init__(*args, **kwargs)
+        allowed_types = [
+            'CustomRadioBoxField',
+            'CustomSelectBoxField',
+            # ~ 'CheckBoxField'
+        ]
+        source_choices = []
+        target_choices = []
+        if module:
+            fields = TicketCategoryInputList.objects.filter(
+                category_module=module,
+            )
+            for f in fields:
+                if f.field_type in allowed_types:
+                    source_choices.append((f.pk, f.name))
+                target_choices.append((f.pk, f.name))
+                self.fields['source_field'].choices = source_choices
+                self.fields['target_field'].choices = target_choices
+            
+    class Meta:
+        model = TicketCategoryModuleRule
+        fields = [
+            "source_field",
+            "condition",
+            "action",
+            "target_field",
+            #"value",
+            "is_active"
+        ]
+        labels = {"visibility": _("Visibilità"),}
+        widgets = {
+            "source_field": BootstrapItaliaSelectWidget,
+            "target_field": BootstrapItaliaSelectWidget,
+            "value": forms.Textarea(attrs={"rows": 2}),
+        }
+        
+    class Media:
+        js = ("js/textarea-autosize.js",)

uniticket/uni_ticket/migrations/0036_ticketcategoryinputlist_visible_and_more.py

@@ -0,0 +1,69 @@
+# Generated by Django 5.2.8 on 2026-04-10 10:07
+
+import django.db.models.deletion
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("uni_ticket", "0035_alter_compiledticket_id_alter_log_id_and_more"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="ticketcategoryinputlist",
+            name="visible",
+            field=models.BooleanField(default=True),
+        ),
+        migrations.CreateModel(
+            name="TicketCategoryModuleRule",
+            fields=[
+                (
+                    "id",
+                    models.BigAutoField(
+                        auto_created=True,
+                        primary_key=True,
+                        serialize=False,
+                        verbose_name="ID",
+                    ),
+                ),
+                (
+                    "action",
+                    models.IntegerField(
+                        choices=[
+                            (1, "Nascondi il campo"),
+                            (2, "Mostra il campo"),
+                            (3, "Modifica il valore"),
+                        ]
+                    ),
+                ),
+                ("value", models.TextField(blank=True, default="", max_length=20000)),
+                ("created", models.DateTimeField(auto_now_add=True)),
+                ("is_active", models.BooleanField(default=False)),
+                (
+                    "category_module",
+                    models.ForeignKey(
+                        on_delete=django.db.models.deletion.CASCADE,
+                        to="uni_ticket.ticketcategorymodule",
+                    ),
+                ),
+                (
+                    "source_field",
+                    models.ForeignKey(
+                        on_delete=django.db.models.deletion.PROTECT,
+                        related_name="source_rules",
+                        to="uni_ticket.ticketcategoryinputlist",
+                    ),
+                ),
+                (
+                    "target_field",
+                    models.ForeignKey(
+                        on_delete=django.db.models.deletion.PROTECT,
+                        related_name="target_rules",
+                        to="uni_ticket.ticketcategoryinputlist",
+                    ),
+                ),
+            ],
+        ),
+    ]

uniticket/uni_ticket/migrations/0037_ticketcategorymodulerule_condition.py

@@ -0,0 +1,19 @@
+# Generated by Django 5.2.8 on 2026-04-10 11:00
+
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("uni_ticket", "0036_ticketcategoryinputlist_visible_and_more"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="ticketcategorymodulerule",
+            name="condition",
+            field=models.CharField(default=1, max_length=255),
+            preserve_default=False,
+        ),
+    ]

uniticket/uni_ticket/models.py

@@ -23,7 +23,7 @@
 
 from django_form_builder.forms import BaseDynamicForm
 from django_form_builder.models import DynamicFieldMap, SavedFormContent
-from django_form_builder.utils import get_as_dict, set_as_dict
+from django_form_builder.utils import format_field_name, get_as_dict, set_as_dict
 
 from organizational_area.models import (
     OrganizationalStructure,
@@ -36,6 +36,7 @@
 from .settings import (
     CATEGORY_CONDITIONS_ATTACHMENT_SUBFOLDER,
     CLOSING_LEVELS,
+    INPUT_MODULE_ACTIONS,
     MAX_DAILY_TICKET_PER_USER,
     NEW_TICKET_CREATED_ALERT,
     ORGANIZATION_EMPLOYEE_LABEL,
@@ -62,10 +63,12 @@
 
 _protocollo_titolario_list = settings.TITOLARIO_DICT
 _protocollo_uo_list = settings.UO_DICT
+_input_module_rules_action_list = INPUT_MODULE_ACTIONS
 if 'makemigrations' in sys.argv or 'migrate' in sys.argv: # pragma: no cover
     _protocollo_titolario_list = [('', '-')]
     _protocollo_uo_list = [('', '-')]
-
+    _input_module_rules_action_list = [('', '-')]
+    
 
 def _attachment_upload(instance, filename):
     """
@@ -416,14 +419,59 @@ def get_form(
         show_conditions=False,
         **kwargs
     ):
-        ticket_input_list = self.ticketcategoryinputlist_set.all().order_by(
-            "ordinamento"
-        )
+
+        copy_data = data.copy() if data else None
+        print(data)
+        # after post
+        if copy_data:
+            fields_to_exclude = list(self.ticketcategoryinputlist_set.filter(visible=False).values_list('pk', flat=True))
+            labels_to_exclude = []
+            
+            rules = self.ticketcategorymodulerule_set.filter(is_active=True)
+            # check rules on fields
+            for rule in rules:
+                source = rule.source_field
+                target = rule.target_field
+                source_label = format_field_name(source.name)
+                target_label = format_field_name(target.name)
+                if source.pk in fields_to_exclude:
+                    continue
+                if copy_data.get(source_label, None) and copy_data[source_label] in rule.condition:
+                    # show
+                    if rule.action == 2 and target.pk in fields_to_exclude:
+                        fields_to_exclude.remove(target.pk)
+                        if target_label in labels_to_exclude:
+                            labels_to_exclude.remove(target_label)
+                    # hide
+                    if rule.action == 1 and not target.pk in fields_to_exclude:
+                        fields_to_exclude.append(target.pk)
+                        if target_label not in labels_to_exclude:
+                            labels_to_exclude.append(target_label)
+
+            for lte in labels_to_exclude:
+                copy_data.pop(lte, None)
+            print("labels_to_exclude", labels_to_exclude)
+            print("fields_to_exclude", fields_to_exclude)
+            print("data", data)
+            print("copy_data", copy_data)
+
+            ticket_input_list = self.ticketcategoryinputlist_set.all().exclude(pk__in=fields_to_exclude).order_by(
+                "ordinamento"
+            )
+            print("ticket_input_list", ticket_input_list)
+            print()
+            print()
+            print()
+        # blank form
+        else:
+            ticket_input_list = self.ticketcategoryinputlist_set.filter(visible=True).order_by(
+                "ordinamento"
+            )
 
         # Static method of BaseDynamicForm
         constructor_dict = BaseDynamicForm.build_constructor_dict(
             ticket_input_list)
-
+        
         custom_params = {}
         custom_params["show_conditions"] = show_conditions
         custom_params["category_owner"] = self.ticket_category
@@ -438,7 +486,7 @@ def get_form(
             class_obj=DynamicForm,
             constructor_dict=constructor_dict,
             custom_params=custom_params,
-            data=data,
+            data=copy_data,
             files=files,
             remove_filefields=remove_filefields,
             remove_datafields=remove_datafields,
@@ -459,7 +507,8 @@ class TicketCategoryInputList(DynamicFieldMap):
 
     category_module = models.ForeignKey(
         TicketCategoryModule, on_delete=models.CASCADE)
-
+    visible = models.BooleanField(default=True)
+    
     class Meta:
         verbose_name = _("Modulo di inserimento")
         verbose_name_plural = _("Moduli di inserimento")
@@ -475,6 +524,28 @@ def field_exist(module, field_name):
         return False
 
 
+class TicketCategoryModuleRule(models.Model):
+    category_module = models.ForeignKey(
+        TicketCategoryModule, on_delete=models.CASCADE)
+    source_field = models.ForeignKey(
+        TicketCategoryInputList,
+        on_delete=models.PROTECT,
+        related_name="source_rules"
+    )
+    target_field = models.ForeignKey(
+        TicketCategoryInputList,
+        on_delete=models.PROTECT,
+        related_name="target_rules"
+    )
+    condition = models.CharField(max_length=255)
+    action = models.IntegerField(
+        choices=INPUT_MODULE_ACTIONS
+    )
+    value = models.TextField(max_length=20000, default="", blank=True)
+    created = models.DateTimeField(auto_now_add=True)
+    is_active = models.BooleanField(default=False)
+    
+
 class Ticket(SavedFormContent):
     """
     Ticket

uniticket/uni_ticket/settings.py

@@ -146,6 +146,17 @@
     ),
 )
 
+# input modules rules actions
+INPUT_MODULE_ACTIONS = getattr(
+    settings,
+    "INPUT_MODULE_ACTIONS",
+    (
+        (1, _("Nascondi il campo")),
+        (2, _("Mostra il campo")),
+        # ~ (3, _("Modifica il valore")),
+    ),
+)
+
 # If 0 = Unlimited
 MAX_DAILY_TICKET_PER_USER = getattr(settings, "MAX_DAILY_TICKET_PER_USER", 10)