schuetz3/schuetz3-erpnext
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

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

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

(cherry picked from commit fe2a34f171)

Co-authored-by: Marica <maricadsouza221197@gmail.com>
This commit is contained in:
Frappe PR Bot
2021-08-13 15:56:26 +05:30
committed by GitHub
parent 7881536e09
commit 434692ad34
1 changed files with 2 additions and 1 deletions
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):