[FIX] base_cancel_confirm: split method to update value in docs

Author: Saran440

base_cancel_confirm/model/base_cancel_confirm.py

@@ -54,15 +54,16 @@ def open_cancel_confirm_wizard(self):
         }
         return action
 
+    def _get_value_clear_cancel(self):
+        return {
+            "cancel_confirm": False,
+            "cancel_reason": False,
+            "cancel_by": False,
+            "cancel_date": False,
+        }
+
     def clear_cancel_confirm_data(self):
-        self.write(
-            {
-                "cancel_confirm": False,
-                "cancel_reason": False,
-                "cancel_by": False,
-                "cancel_date": False,
-            }
-        )
+        self.write(self._get_value_clear_cancel())
 
     def get_view(self, view_id=None, view_type="form", **options):
         res = super().get_view(view_id=view_id, view_type=view_type, **options)

base_cancel_confirm/wizard/cancel_confirm.py

@@ -18,21 +18,27 @@ class CancelConfirm(models.TransientModel):
         required=True,
     )
 
-    def confirm_cancel(self):
-        self.ensure_one()
-        res_model = self._context.get("cancel_res_model")
-        res_ids = self._context.get("cancel_res_ids")
-        cancel_method = self._context.get("cancel_method")
-        docs = self.env[res_model].browse(res_ids)
-        docs.write({"cancel_confirm": True})
+    def _get_dict_udpate(self):
+        """Hooks this method to update value in docs"""
+        dict_update = {"cancel_confirm": True}
         # Cancel Reason
         if self.has_cancel_reason in ["optional", "required"]:
-            docs.write(
+            dict_update.update(
                 {
                     "cancel_reason": self.cancel_reason,
                     "cancel_by": self.env.user,
                     "cancel_date": fields.Date.context_today(self),
                 }
             )
+        return dict_update
+
+    def confirm_cancel(self):
+        self.ensure_one()
+        res_model = self._context.get("cancel_res_model")
+        res_ids = self._context.get("cancel_res_ids")
+        cancel_method = self._context.get("cancel_method")
+        docs = self.env[res_model].browse(res_ids)
+        dict_update = self._get_dict_udpate()
+        docs.write(dict_update)
         res = getattr(docs, cancel_method)()
         return res