feat(cr): lock operation after proceeding and hide disabled radio options

- Add is_operation_locked boolean field set when proceeding to Documents/Review
- Show computed operation_display char field when locked instead of radio
- Change filterable_radio widget to hide disabled options (d-none) instead of graying them out
- Reset lock when operation is changed via onchange

Author: emjay0921

spp_base_common/static/src/js/filterable_radio_field.js

@@ -5,7 +5,7 @@ import {registry} from "@web/core/registry";
 import {_t} from "@web/core/l10n/translation";
 
 /**
- * A radio widget that supports disabling individual options based on
+ * A radio widget that supports hiding individual options based on
  * boolean fields on the record.
  *
  * Usage in XML views:
@@ -22,7 +22,7 @@ import {_t} from "@web/core/l10n/translation";
  *
  * The `disabled_map` option maps selection values to boolean field names.
  * When the boolean field is falsy, the corresponding radio option is
- * rendered as disabled (grayed out, not clickable).
+ * hidden from the user.
  */
 export class FilterableRadioField extends RadioField {
     static template = "spp_base_common.FilterableRadioField";

spp_base_common/static/src/xml/filterable_radio_field.xml

@@ -8,7 +8,10 @@
             t-att-aria-label="props.label"
         >
             <t t-foreach="items" t-as="item" t-key="item[0]">
-                <div class="form-check o_radio_item" aria-atomic="true">
+                <div
+                    t-attf-class="form-check o_radio_item #{isItemDisabled(item[0]) ? 'd-none' : ''}"
+                    aria-atomic="true"
+                >
                     <input
                         type="radio"
                         class="form-check-input o_radio_input"

spp_change_request_v2/details/update_id.py

@@ -55,6 +55,15 @@ class SPPCRDetailUpdateID(models.Model):
         help="Upload scanned copies or photos of the ID document",
     )
     remarks = fields.Text(string="Remarks", tracking=True)
+    is_operation_locked = fields.Boolean(
+        string="Operation Locked",
+        default=False,
+        help="Set to True when user proceeds to Documents or Review stage, locking the operation selection.",
+    )
+    operation_display = fields.Char(
+        string="Operation",
+        compute="_compute_operation_display",
+    )
 
     # ══════════════════════════════════════════════════════════════════════════
     # COMPUTED FIELDS
@@ -84,6 +93,22 @@ class SPPCRDetailUpdateID(models.Model):
         readonly=True,
     )
 
+    @api.depends("operation")
+    def _compute_operation_display(self):
+        labels = dict(self._fields["operation"].selection)
+        for rec in self:
+            rec.operation_display = labels.get(rec.operation, "")
+
+    def action_next_documents(self):
+        """Lock operation before proceeding to documents stage."""
+        self.write({"is_operation_locked": True})
+        return super().action_next_documents()
+
+    def action_skip_to_review(self):
+        """Lock operation before proceeding to review stage."""
+        self.write({"is_operation_locked": True})
+        return super().action_skip_to_review()
+
     @api.constrains("operation")
     def _check_operation_allowed(self):
         """Validate that the chosen operation is allowed by the CR type config."""
@@ -122,6 +147,9 @@ def _onchange_operation(self):
                 "message": _("Remove ID is disabled for this change request type."),
             }
 
+        # Unlock operation when changed (user came back to edit)
+        self.is_operation_locked = False
+
         if self.operation == "add":
             self.existing_id_record_id = False
             self.id_type_id = False

spp_change_request_v2/views/detail_update_id_views.xml

@@ -60,11 +60,18 @@
                             <field name="allow_id_edit" invisible="1" />
                             <field name="allow_id_remove" invisible="1" />
                             <field name="approval_state" invisible="1" />
+                            <field name="is_operation_locked" invisible="1" />
                             <field
                                 name="operation"
                                 widget="filterable_radio"
                                 options="{'disabled_map': {'update': 'allow_id_edit', 'remove': 'allow_id_remove'}}"
                                 readonly="not is_cr_manager or approval_state not in ('draft', 'revision')"
+                                invisible="is_operation_locked"
+                            />
+                            <field
+                                name="operation_display"
+                                readonly="1"
+                                invisible="not is_operation_locked"
                             />
                         </group>
                     </group>