Browse Source

fix: avoid saving if from expanded record

Signed-off-by: mertmit <mertmit99@gmail.com>
pull/8371/head
mertmit 2 months ago
parent
commit
dd573ac84d
  1. 2
      packages/nc-gui/components/smartsheet/grid/Table.vue
  2. 2
      packages/nc-gui/composables/useExpandedFormStore.ts
  3. 1
      packages/nc-gui/lib/types.ts

2
packages/nc-gui/components/smartsheet/grid/Table.vue

@ -1070,6 +1070,8 @@ const saveOrUpdateRecords = async (
args: { metaValue?: TableType; viewMetaValue?: ViewType; data?: any; keepNewRecords?: boolean } = {},
) => {
for (const currentRow of args.data || dataRef.value) {
if (currentRow.rowMeta.fromExpandedForm) continue
/** if new record save row and save the LTAR cells */
if (currentRow.rowMeta.new) {
const beforeSave = clone(currentRow)

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

@ -56,6 +56,8 @@ const [useProvideExpandedFormStore, useExpandedFormStore] = useInjectionState((m
: ({ row: {}, oldRow: {}, rowMeta: {} } as Row),
)
row.value.rowMeta.fromExpandedForm = true
const rowStore = useProvideSmartsheetRowStore(row)
const activeView = inject(ActiveViewInj, ref())

1
packages/nc-gui/lib/types.ts

@ -65,6 +65,7 @@ interface Row {
changed?: boolean
saving?: boolean
ltarState?: Record<string, Record<string, any> | Record<string, any>[] | null>
fromExpandedForm?: boolean
// use in datetime picker component
isUpdatedFromCopyNPaste?: Record<string, boolean>
// Used in Calendar view

Loading…
Cancel
Save