vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Merge pull request #31100 from akhilnarang/fix-attach-child-table

Browse source 
fix: move child table checks to within `set_input()`
This commit is contained in:
Akhil Narang 2025-02-04 15:24:09 +05:30 • committed by GitHub
commit c273c52092
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 5 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
frappe/public/js/frappe/form/controls/attach.js
View file

@ -32,12 +32,6 @@ frappe.ui.form.ControlAttach = class ControlAttach extends frappe.ui.form.Contro
frappe.utils.bind_actions_with_object(this.$value, this);
this.toggle_reload_button();
// Don't allow attaching to child tables for new documents
if (this.doc?.__islocal && this.doc.parent) {
this.$input.prop("disabled", true);
this.$input.attr("title", __("Save to enable file upload"));
}
}
clear_attachment() {
let me = this;
@ -125,7 +119,11 @@ frappe.ui.form.ControlAttach = class ControlAttach extends frappe.ui.form.Contro
`);
}
} else {
this.$input.toggle(true);
// Don't allow attaching to child tables for new documents
if (this.doc?.__islocal && this.doc.parent) {
this.$input.prop("disabled", true);
this.$input.attr("title", __("Save to enable file upload"));
}
this.$value.toggle(false);
}
}