test/crm
1
0
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #1010 from frappe/mergify/bp/main-hotfix/pr-1009

Browse Source 
fix: grid field is not getting set (backport #1009)
This commit is contained in:
Shariq Ansari 2025-07-02 17:52:45 +05:30 committed by GitHub
commit b6b5d47168
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 13 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
frontend/src/data/document.js
View File

@ -136,8 +136,14 @@ export function useDocument(doctype, docname) {
} }
async function triggerOnChange(fieldname, value, row) { async function triggerOnChange(fieldname, value, row) {
const oldValue = documentsCache[doctype][docname || ''].doc[fieldname] let oldValue = null
documentsCache[doctype][docname || ''].doc[fieldname] = value if (row) {
oldValue = row[fieldname]
row[fieldname] = value
} else {
oldValue = documentsCache[doctype][docname || ''].doc[fieldname]
documentsCache[doctype][docname || ''].doc[fieldname] = value
}
const handler = async function () { const handler = async function () {
this.value = value this.value = value
@ -151,7 +157,11 @@ export function useDocument(doctype, docname) {
try { try {
await trigger(handler, row) await trigger(handler, row)
} catch (error) { } catch (error) {
documentsCache[doctype][docname || ''].doc[fieldname] = oldValue if (row) {
row[fieldname] = oldValue
} else {
documentsCache[doctype][docname || ''].doc[fieldname] = oldValue
}
console.error(handler) console.error(handler)
throw error throw error
} }