Shiloh/brotherton-erpnext
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: Copy previous balance dict object instead of assigning (#26942)

Browse Source 
- Due to plain assignment, dict mutation gave wrong monthly values

(cherry picked from commit fe2a34f17197a2877356bcdf5d0bb6c46312ed33)
This commit is contained in:
Marica 2021-08-13 15:37:45 +05:30 committed by frappe-pr-bot
parent f4b2f4aaf7
commit 2e6899fbe4
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
erpnext/stock/report/stock_analytics/stock_analytics.py
View File

@ -144,7 +144,8 @@ def get_periodic_data(entry, filters):
# if period against item does not exist yet, instantiate it
# insert existing balance dict against period, and add/subtract to it
if periodic_data.get(d.item_code) and not periodic_data.get(d.item_code).get(period):
periodic_data[d.item_code][period] = periodic_data[d.item_code]['balance']
previous_balance = periodic_data[d.item_code]['balance'].copy()
periodic_data[d.item_code][period] = previous_balance
if d.voucher_type == "Stock Reconciliation":
if periodic_data.get(d.item_code) and periodic_data.get(d.item_code).get('balance').get(d.warehouse):