Browse Source

fix: reload only virtual column value in expanded form when triggered by links

pull/7355/head
Pranav C 10 months ago
parent
commit
ddc3686a59
  1. 2
      packages/nc-gui/components/smartsheet/expanded-form/index.vue
  2. 19
      packages/nc-gui/composables/useExpandedFormStore.ts

2
packages/nc-gui/components/smartsheet/expanded-form/index.vue

@ -278,7 +278,7 @@ const reloadHook = createEventHook()
reloadHook.on(() => {
reloadParentRowHook?.trigger(false)
if (isNew.value) return
_loadRow()
_loadRow(null, true)
})
provide(ReloadRowDataHookInj, reloadHook)

19
packages/nc-gui/composables/useExpandedFormStore.ts

@ -1,4 +1,4 @@
import { UITypes, ViewTypes } from 'nocodb-sdk'
import { UITypes, ViewTypes, isVirtualCol } from 'nocodb-sdk'
import type { AuditType, ColumnType, TableType } from 'nocodb-sdk'
import type { Ref } from 'vue'
import dayjs from 'dayjs'
@ -295,8 +295,8 @@ const [useProvideExpandedFormStore, useExpandedFormStore] = useInjectionState((m
changedColumns.value = new Set()
}
const loadRow = async (rowId?: string) => {
const record = await $api.dbTableRow.read(
const loadRow = async (rowId?: string, onlyVirtual = false) => {
let record = await $api.dbTableRow.read(
NOCO,
// todo: base_id missing on view type
(base?.value?.id || (sharedView.value?.view as any)?.base_id) as string,
@ -307,6 +307,19 @@ const [useProvideExpandedFormStore, useExpandedFormStore] = useInjectionState((m
},
)
// update only virtual columns value if `onlyVirtual` is true
if (onlyVirtual) {
record = {
...row.value.row,
...meta.value.columns.reduce((partialRecord, col) => {
if (isVirtualCol(col) && col.title in record) {
partialRecord[col.title] = record[col.title]
}
return partialRecord
}, {} as Record<string, any>),
}
}
Object.assign(row.value, {
row: record,
oldRow: { ...record },

Loading…
Cancel
Save