nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-05-28 17:34:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: incorrect number of gl entries issue (#20351)

Browse Source
This commit is contained in:
rohitwaghchaure
2020-01-20 17:53:44 +05:30
committed by Nabin Hait
parent 8b943af911
commit c7f0ab8ed6
2 changed files with 14 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
erpnext/manufacturing/doctype/bom/bom.py
View File

@@ -782,11 +782,12 @@ def add_non_stock_items_cost(stock_entry, work_order, expense_account):
for name in non_stock_items: for name in non_stock_items:
non_stock_items_cost += flt(items.get(name[0])) * flt(stock_entry.fg_completed_qty) / flt(bom.quantity) non_stock_items_cost += flt(items.get(name[0])) * flt(stock_entry.fg_completed_qty) / flt(bom.quantity)
stock_entry.append('additional_costs', { if non_stock_items_cost:
'expense_account': expense_account, stock_entry.append('additional_costs', {
'description': _("Non stock items"), 'expense_account': expense_account,
'amount': non_stock_items_cost 'description': _("Non stock items"),
}) 'amount': non_stock_items_cost
})
def add_operations_cost(stock_entry, work_order=None, expense_account=None): def add_operations_cost(stock_entry, work_order=None, expense_account=None):
from erpnext.stock.doctype.stock_entry.stock_entry import get_operating_cost_per_unit from erpnext.stock.doctype.stock_entry.stock_entry import get_operating_cost_per_unit
@@ -803,11 +804,12 @@ def add_operations_cost(stock_entry, work_order=None, expense_account=None):
additional_operating_cost_per_unit = \ additional_operating_cost_per_unit = \
flt(work_order.additional_operating_cost) / flt(work_order.qty) flt(work_order.additional_operating_cost) / flt(work_order.qty)
stock_entry.append('additional_costs', { if additional_operating_cost_per_unit:
"expense_account": expense_account, stock_entry.append('additional_costs', {
"description": "Additional Operating Cost", "expense_account": expense_account,
"amount": additional_operating_cost_per_unit * flt(stock_entry.fg_completed_qty) "description": "Additional Operating Cost",
}) "amount": additional_operating_cost_per_unit * flt(stock_entry.fg_completed_qty)
})
@frappe.whitelist() @frappe.whitelist()
def get_bom_diff(bom1, bom2): def get_bom_diff(bom1, bom2):

2
erpnext/stock/doctype/stock_entry/stock_entry.py
View File

@@ -682,6 +682,8 @@ class StockEntry(StockController):
if item_account_wise_additional_cost: if item_account_wise_additional_cost:
for d in self.get("items"): for d in self.get("items"):
for account, amount in iteritems(item_account_wise_additional_cost.get((d.item_code, d.name), {})): for account, amount in iteritems(item_account_wise_additional_cost.get((d.item_code, d.name), {})):
if not amount: continue
gl_entries.append(self.get_gl_dict({ gl_entries.append(self.get_gl_dict({
"account": account, "account": account,
"against": d.expense_account, "against": d.expense_account,