nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-06-04 04:39:11 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: Accounts mandatory depending on share transfer type (#19523)

Browse Source
This commit is contained in:
Nabin Hait
2019-11-07 18:05:40 +05:30
committed by GitHub
parent bac50bc295
commit 628701f1a5
2 changed files with 209 additions and 845 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
erpnext/accounts/doctype/share_transfer/share_transfer.js
View File

@@ -21,6 +21,8 @@ frappe.ui.form.on('Share Transfer', {
erpnext.share_transfer.make_jv(frm); erpnext.share_transfer.make_jv(frm);
}); });
} }
frm.toggle_reqd("asset_account", frm.doc.transfer_type != "Transfer");
}, },
no_of_shares: (frm) => { no_of_shares: (frm) => {
if (frm.doc.rate != undefined || frm.doc.rate != null){ if (frm.doc.rate != undefined || frm.doc.rate != null){
@@ -56,6 +58,10 @@ frappe.ui.form.on('Share Transfer', {
}; };
}); });
} }
},
transfer_type: function(frm) {
frm.toggle_reqd("asset_account", frm.doc.transfer_type != "Transfer");
} }
}); });

1048
erpnext/accounts/doctype/share_transfer/share_transfer.json
View File

File diff suppressed because it is too large Load Diff