From 3ca2883efb9f209694dc630da1702330b1626e7e Mon Sep 17 00:00:00 2001
From: Shariq Ansari <sharique.rik@gmail.com>
Date: Tue, 29 Jul 2025 14:08:11 +0530
Subject: [PATCH] fix: reload doc after sending email

(cherry picked from commit ea0011771b268ee60dce56cd9262e24a921a45d4)
---
 .../src/components/Activities/Activities.vue  |  4 +
 frontend/src/components/SidePanelLayout.vue   | 79 ++++++++-----------
 2 files changed, 35 insertions(+), 48 deletions(-)

diff --git a/frontend/src/components/Activities/Activities.vue b/frontend/src/components/Activities/Activities.vue
index e0b6b2f0..1ef91dc1 100644
--- a/frontend/src/components/Activities/Activities.vue
+++ b/frontend/src/components/Activities/Activities.vue
@@ -490,6 +490,7 @@ import { timeAgo, formatDate, startCase } from '@/utils'
 import { globalStore } from '@/stores/global'
 import { usersStore } from '@/stores/users'
 import { whatsappEnabled, callEnabled } from '@/composables/settings'
+import { useDocument } from '@/data/document'
 import { capture } from '@/telemetry'
 import { Button, Tooltip, createResource } from 'frappe-ui'
 import { useElementVisibility } from '@vueuse/core'
@@ -527,6 +528,8 @@ const doc = defineModel()
 const reload = defineModel('reload')
 const tabIndex = defineModel('tabIndex')
 
+const { document: _document } = useDocument(props.doctype, doc.value.data.name)
+
 const reload_email = ref(false)
 const modalRef = ref(null)
 const showFilesUploader = ref(false)
@@ -767,6 +770,7 @@ const whatsappBox = ref(null)
 watch([reload, reload_email], ([reload_value, reload_email_value]) => {
   if (reload_value || reload_email_value) {
     all_activities.reload()
+    _document.reload()
     reload.value = false
     reload_email.value = false
   }
diff --git a/frontend/src/components/SidePanelLayout.vue b/frontend/src/components/SidePanelLayout.vue
index eac3eec8..a209c1da 100644
--- a/frontend/src/components/SidePanelLayout.vue
+++ b/frontend/src/components/SidePanelLayout.vue
@@ -1,8 +1,5 @@
 <template>
-  <div
-    v-if="!document.get?.loading"
-    class="sections flex flex-col overflow-y-auto"
-  >
+  <div class="sections flex flex-col overflow-y-auto">
     <template v-for="(section, i) in _sections" :key="section.name">
       <div v-if="section.visible" class="section flex flex-col">
         <div
@@ -82,21 +79,21 @@
                           class="flex h-7 cursor-pointer items-center px-2 py-1 text-ink-gray-5"
                         >
                           <Tooltip :text="__(field.tooltip)">
-                            <div>{{ document.doc[field.fieldname] }}</div>
+                            <div>{{ doc[field.fieldname] }}</div>
                           </Tooltip>
                         </div>
                         <div v-else-if="field.fieldtype === 'Dropdown'">
                           <NestedPopover>
                             <template #target="{ open }">
                               <Button
-                                :label="document.doc[field.fieldname]"
+                                :label="doc[field.fieldname]"
                                 class="dropdown-button flex w-full items-center justify-between rounded border border-gray-100 bg-surface-gray-2 px-2 py-1.5 text-base text-ink-gray-8 placeholder-ink-gray-4 transition-colors hover:border-outline-gray-modals hover:bg-surface-gray-3 focus:border-outline-gray-4 focus:bg-surface-white focus:shadow-sm focus:outline-none focus:ring-0 focus-visible:ring-2 focus-visible:ring-outline-gray-3"
                               >
                                 <div
-                                  v-if="document.doc[field.fieldname]"
+                                  v-if="doc[field.fieldname]"
                                   class="truncate"
                                 >
-                                  {{ document.doc[field.fieldname] }}
+                                  {{ doc[field.fieldname] }}
                                 </div>
                                 <div
                                   v-else
@@ -153,7 +150,7 @@
                           v-else-if="field.fieldtype == 'Check'"
                           class="form-control"
                           type="checkbox"
-                          v-model="document.doc[field.fieldname]"
+                          v-model="doc[field.fieldname]"
                           @change.stop="
                             fieldChange($event.target.checked, field)
                           "
@@ -170,7 +167,7 @@
                           "
                           class="form-control"
                           type="textarea"
-                          :value="document.doc[field.fieldname]"
+                          :value="doc[field.fieldname]"
                           :placeholder="field.placeholder"
                           :debounce="500"
                           @change.stop="fieldChange($event.target.value, field)"
@@ -179,7 +176,7 @@
                           v-else-if="field.fieldtype === 'Select'"
                           class="form-control cursor-pointer [&_select]:cursor-pointer truncate"
                           type="select"
-                          v-model="document.doc[field.fieldname]"
+                          v-model="doc[field.fieldname]"
                           :options="field.options"
                           :placeholder="field.placeholder"
                           @change.stop="fieldChange($event.target.value, field)"
@@ -188,8 +185,8 @@
                           v-else-if="field.fieldtype === 'User'"
                           class="form-control"
                           :value="
-                            document.doc[field.fieldname] &&
-                            getUser(document.doc[field.fieldname]).full_name
+                            doc[field.fieldname] &&
+                            getUser(doc[field.fieldname]).full_name
                           "
                           doctype="User"
                           :filters="field.filters"
@@ -197,13 +194,10 @@
                           :placeholder="'Select' + ' ' + field.label + '...'"
                           :hideMe="true"
                         >
-                          <template
-                            v-if="document.doc[field.fieldname]"
-                            #prefix
-                          >
+                          <template v-if="doc[field.fieldname]" #prefix>
                             <UserAvatar
                               class="mr-1.5"
-                              :user="document.doc[field.fieldname]"
+                              :user="doc[field.fieldname]"
                               size="sm"
                             />
                           </template>
@@ -227,11 +221,11 @@
                             ['Link', 'Dynamic Link'].includes(field.fieldtype)
                           "
                           class="form-control select-text"
-                          :value="document.doc[field.fieldname]"
+                          :value="doc[field.fieldname]"
                           :doctype="
                             field.fieldtype == 'Link'
                               ? field.options
-                              : document.doc[field.options]
+                              : doc[field.options]
                           "
                           :filters="field.filters"
                           :placeholder="field.placeholder"
@@ -244,7 +238,7 @@
                         >
                           <DateTimePicker
                             icon-left=""
-                            :value="document.doc[field.fieldname]"
+                            :value="doc[field.fieldname]"
                             :formatter="
                               (date) => getFormat(date, '', true, true)
                             "
@@ -260,7 +254,7 @@
                         >
                           <DatePicker
                             icon-left=""
-                            :value="document.doc[field.fieldname]"
+                            :value="doc[field.fieldname]"
                             :formatter="(date) => getFormat(date, '', true)"
                             :placeholder="field.placeholder"
                             placement="left-start"
@@ -272,9 +266,7 @@
                           v-else-if="field.fieldtype === 'Percent'"
                           class="form-control"
                           type="text"
-                          :value="
-                            getFormattedPercent(field.fieldname, document.doc)
-                          "
+                          :value="getFormattedPercent(field.fieldname, doc)"
                           :placeholder="field.placeholder"
                           :debounce="500"
                           @change.stop="
@@ -285,7 +277,7 @@
                         <Password
                           v-else-if="field.fieldtype === 'Password'"
                           class="form-control"
-                          :value="document.doc[field.fieldname]"
+                          :value="doc[field.fieldname]"
                           :placeholder="field.placeholder"
                           :debounce="500"
                           @change.stop="fieldChange($event.target.value, field)"
@@ -295,7 +287,7 @@
                           v-else-if="field.fieldtype === 'Int'"
                           class="form-control"
                           type="text"
-                          :value="document.doc[field.fieldname] || '0'"
+                          :value="doc[field.fieldname] || '0'"
                           :placeholder="field.placeholder"
                           :debounce="500"
                           @change.stop="fieldChange($event.target.value, field)"
@@ -305,9 +297,7 @@
                           v-else-if="field.fieldtype === 'Float'"
                           class="form-control"
                           type="text"
-                          :value="
-                            getFormattedFloat(field.fieldname, document.doc)
-                          "
+                          :value="getFormattedFloat(field.fieldname, doc)"
                           :placeholder="field.placeholder"
                           :debounce="500"
                           @change.stop="
@@ -319,9 +309,7 @@
                           v-else-if="field.fieldtype === 'Currency'"
                           class="form-control"
                           type="text"
-                          :value="
-                            getFormattedCurrency(field.fieldname, document.doc)
-                          "
+                          :value="getFormattedCurrency(field.fieldname, doc)"
                           :placeholder="field.placeholder"
                           :debounce="500"
                           @change.stop="
@@ -333,7 +321,7 @@
                           v-else
                           class="form-control"
                           type="text"
-                          :value="document.doc[field.fieldname]"
+                          :value="doc[field.fieldname]"
                           :placeholder="field.placeholder"
                           :debounce="500"
                           @change.stop="fieldChange($event.target.value, field)"
@@ -344,23 +332,19 @@
                           v-if="
                             field.fieldtype === 'Link' &&
                             field.link &&
-                            document.doc[field.fieldname]
+                            doc[field.fieldname]
                           "
                           class="h-4 w-4 shrink-0 cursor-pointer text-ink-gray-5 hover:text-ink-gray-8"
-                          @click.stop="
-                            field.link(document.doc[field.fieldname])
-                          "
+                          @click.stop="field.link(doc[field.fieldname])"
                         />
                         <EditIcon
                           v-if="
                             field.fieldtype === 'Link' &&
                             field.edit &&
-                            document.doc[field.fieldname]
+                            doc[field.fieldname]
                           "
                           class="size-3.5 shrink-0 cursor-pointer text-ink-gray-5 hover:text-ink-gray-8"
-                          @click.stop="
-                            field.edit(document.doc[field.fieldname])
-                          "
+                          @click.stop="field.edit(doc[field.fieldname])"
                         />
                       </div>
                     </div>
@@ -442,6 +426,8 @@ if (props.docname) {
   triggerOnChange = d.triggerOnChange
 }
 
+const doc = computed(() => document.doc || {})
+
 const _sections = computed(() => {
   if (!props.sections?.length) return []
   let editButtonAdded = false
@@ -482,13 +468,10 @@ function parsedField(field) {
     ...field,
     filters: field.link_filters && JSON.parse(field.link_filters),
     placeholder: field.placeholder || field.label,
-    display_via_depends_on: evaluateDependsOnValue(
-      field.depends_on,
-      document.doc,
-    ),
+    display_via_depends_on: evaluateDependsOnValue(field.depends_on, doc.value),
     mandatory_via_depends_on: evaluateDependsOnValue(
       field.mandatory_depends_on,
-      document.doc,
+      doc.value,
     ),
   }
 
@@ -535,7 +518,7 @@ function isFieldVisible(field) {
   if (props.preview) return true
   return (
     (field.fieldtype == 'Check' ||
-      (field.read_only && document.doc?.[field.fieldname]) ||
+      (field.read_only && doc.value?.[field.fieldname]) ||
       !field.read_only) &&
     (!field.depends_on || field.display_via_depends_on) &&
     !field.hidden