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

Merge pull request #53730 from khushi8112/asset-location-overwritten-by-accounting-dimension

Browse Source 
fix: skip overwriting existing asset fields with accounting dimensions
This commit is contained in:
ruthra kumar
2026-03-24 11:04:59 +05:30
committed by GitHub
parent bd9b0185f4 2859a143f2
commit 107fecd4b0
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
erpnext/controllers/buying_controller.py
View File

@@ -1094,7 +1094,8 @@ class BuyingController(SubcontractingController):
for dimension in accounting_dimensions[0]:
fieldname = dimension["fieldname"]
default_dimension = accounting_dimensions[1].get(self.company, {}).get(fieldname)
asset.update({fieldname: row.get(fieldname) or self.get(fieldname) or default_dimension})
if not asset.get(fieldname):
asset.update({fieldname: row.get(fieldname) or self.get(fieldname) or default_dimension})
asset.flags.ignore_validate = True
asset.flags.ignore_mandatory = True