From 0919dd07432b982eb7737b7090a77bdbc3362a86 Mon Sep 17 00:00:00 2001
From: Soybean <soybeanjs@outlook.com>
Date: Mon, 27 Oct 2025 18:18:33 +0800
Subject: [PATCH] optimize(manage): replace Object.assign with jsonClone for
 safer data handling in model initialization

---
 .../manage/role/modules/role-operate-drawer.vue  |  3 ++-
 src/views/manage/role/modules/role-search.vue    | 12 +++++-------
 .../manage/user/modules/user-operate-drawer.vue  |  3 ++-
 src/views/manage/user/modules/user-search.vue    | 16 +++++-----------
 4 files changed, 14 insertions(+), 20 deletions(-)

diff --git a/src/views/manage/role/modules/role-operate-drawer.vue b/src/views/manage/role/modules/role-operate-drawer.vue
index 2d958403..d1b8092e 100644
--- a/src/views/manage/role/modules/role-operate-drawer.vue
+++ b/src/views/manage/role/modules/role-operate-drawer.vue
@@ -1,5 +1,6 @@
 <script setup lang="ts">
 import { computed, ref, watch } from 'vue';
+import { jsonClone } from '@sa/utils';
 import { useBoolean } from '@sa/hooks';
 import { enableStatusOptions } from '@/constants/business';
 import { useFormRules, useNaiveForm } from '@/hooks/common/form';
@@ -72,7 +73,7 @@ function handleInitModel() {
   model.value = createDefaultModel();
 
   if (props.operateType === 'edit' && props.rowData) {
-    Object.assign(model.value, props.rowData);
+    Object.assign(model.value, jsonClone(props.rowData));
   }
 }
 
diff --git a/src/views/manage/role/modules/role-search.vue b/src/views/manage/role/modules/role-search.vue
index b464992f..8cd270ec 100644
--- a/src/views/manage/role/modules/role-search.vue
+++ b/src/views/manage/role/modules/role-search.vue
@@ -1,4 +1,6 @@
 <script setup lang="ts">
+import { toRaw } from 'vue';
+import { jsonClone } from '@sa/utils';
 import { enableStatusOptions } from '@/constants/business';
 import { translateOptions } from '@/utils/common';
 import { $t } from '@/locales';
@@ -15,14 +17,10 @@ const emit = defineEmits<Emits>();
 
 const model = defineModel<Api.SystemManage.RoleSearchParams>('model', { required: true });
 
+const defaultModel = jsonClone(toRaw(model.value));
+
 function resetModel() {
-  model.value = {
-    current: 1,
-    size: 10,
-    roleName: null,
-    roleCode: null,
-    status: null
-  };
+  Object.assign(model.value, defaultModel);
 }
 
 function search() {
diff --git a/src/views/manage/user/modules/user-operate-drawer.vue b/src/views/manage/user/modules/user-operate-drawer.vue
index e4a195be..c2cfa288 100644
--- a/src/views/manage/user/modules/user-operate-drawer.vue
+++ b/src/views/manage/user/modules/user-operate-drawer.vue
@@ -1,5 +1,6 @@
 <script setup lang="ts">
 import { computed, ref, watch } from 'vue';
+import { jsonClone } from '@sa/utils';
 import { enableStatusOptions, userGenderOptions } from '@/constants/business';
 import { fetchGetAllRoles } from '@/service/api';
 import { useFormRules, useNaiveForm } from '@/hooks/common/form';
@@ -93,7 +94,7 @@ function handleInitModel() {
   model.value = createDefaultModel();
 
   if (props.operateType === 'edit' && props.rowData) {
-    Object.assign(model.value, props.rowData);
+    Object.assign(model.value, jsonClone(props.rowData));
   }
 }
 
diff --git a/src/views/manage/user/modules/user-search.vue b/src/views/manage/user/modules/user-search.vue
index 439ec665..158fdf52 100644
--- a/src/views/manage/user/modules/user-search.vue
+++ b/src/views/manage/user/modules/user-search.vue
@@ -1,5 +1,6 @@
 <script setup lang="ts">
-import { computed } from 'vue';
+import { computed, toRaw } from 'vue';
+import { jsonClone } from '@sa/utils';
 import { enableStatusOptions, userGenderOptions } from '@/constants/business';
 import { useFormRules, useNaiveForm } from '@/hooks/common/form';
 import { translateOptions } from '@/utils/common';
@@ -30,17 +31,10 @@ const rules = computed<Record<RuleKey, App.Global.FormRule>>(() => {
   };
 });
 
+const defaultModel = jsonClone(toRaw(model.value));
+
 function resetModel() {
-  model.value = {
-    current: 1,
-    size: 10,
-    status: null,
-    userName: null,
-    userGender: null,
-    nickName: null,
-    userPhone: null,
-    userEmail: null
-  };
+  Object.assign(model.value, defaultModel);
 }
 
 async function reset() {