[fix] [minor] perpetual inventory: account for each warehouse
diff --git a/accounts/doctype/journal_voucher/test_journal_voucher.py b/accounts/doctype/journal_voucher/test_journal_voucher.py
index 6ade931..f80b945 100644
--- a/accounts/doctype/journal_voucher/test_journal_voucher.py
+++ b/accounts/doctype/journal_voucher/test_journal_voucher.py
@@ -34,16 +34,17 @@
where against_jv=%s""", jv_invoice.doc.name))
def test_jv_against_stock_account(self):
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 1)
+ from stock.doctype.purchase_receipt.test_purchase_receipt import set_perpetual_inventory
+ set_perpetual_inventory()
jv = webnotes.bean(copy=test_records[0])
- jv.doclist[1].account = "_Test Account Stock in Hand - _TC"
+ jv.doclist[1].account = "_Test Warehouse - _TC"
jv.insert()
from accounts.general_ledger import StockAccountInvalidTransaction
self.assertRaises(StockAccountInvalidTransaction, jv.submit)
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 0)
+ set_perpetual_inventory(0)
def test_monthly_budget_crossed_ignore(self):
webnotes.conn.set_value("Company", "_Test Company", "monthly_bgt_flag", "Ignore")
diff --git a/accounts/doctype/purchase_invoice/test_purchase_invoice.py b/accounts/doctype/purchase_invoice/test_purchase_invoice.py
index 8826b3f..9e9e2f6 100644
--- a/accounts/doctype/purchase_invoice/test_purchase_invoice.py
+++ b/accounts/doctype/purchase_invoice/test_purchase_invoice.py
@@ -9,13 +9,14 @@
import json
from webnotes.utils import cint
import webnotes.defaults
+from stock.doctype.purchase_receipt.test_purchase_receipt import set_perpetual_inventory
test_dependencies = ["Item", "Cost Center"]
test_ignore = ["Serial No"]
class TestPurchaseInvoice(unittest.TestCase):
def test_gl_entries_without_auto_accounting_for_stock(self):
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 0)
+ set_perpetual_inventory(0)
self.assertTrue(not cint(webnotes.defaults.get_global_default("auto_accounting_for_stock")))
wrapper = webnotes.bean(copy=test_records[0])
@@ -42,7 +43,7 @@
self.assertEqual([d.debit, d.credit], expected_gl_entries.get(d.account))
def test_gl_entries_with_auto_accounting_for_stock(self):
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 1)
+ set_perpetual_inventory(1)
self.assertEqual(cint(webnotes.defaults.get_global_default("auto_accounting_for_stock")), 1)
pi = webnotes.bean(copy=test_records[1])
@@ -68,10 +69,10 @@
self.assertEquals(expected_values[i][1], gle.debit)
self.assertEquals(expected_values[i][2], gle.credit)
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 0)
+ set_perpetual_inventory(0)
def test_gl_entries_with_aia_for_non_stock_items(self):
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 1)
+ set_perpetual_inventory()
self.assertEqual(cint(webnotes.defaults.get_global_default("auto_accounting_for_stock")), 1)
pi = webnotes.bean(copy=test_records[1])
@@ -98,8 +99,7 @@
self.assertEquals(expected_values[i][0], gle.account)
self.assertEquals(expected_values[i][1], gle.debit)
self.assertEquals(expected_values[i][2], gle.credit)
-
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 0)
+ set_perpetual_inventory(0)
def test_purchase_invoice_calculation(self):
wrapper = webnotes.bean(copy=test_records[0])
diff --git a/accounts/general_ledger.py b/accounts/general_ledger.py
index acb1694..05c52db 100644
--- a/accounts/general_ledger.py
+++ b/accounts/general_ledger.py
@@ -91,8 +91,8 @@
def validate_account_for_auto_accounting_for_stock(gl_map):
if gl_map[0].voucher_type=="Journal Voucher":
- aii_accounts = [d[0] for d in webnotes.conn.sql("""select account from tabWarehouse
- where ifnull(account, '')!=''""")]
+ aii_accounts = [d[0] for d in webnotes.conn.sql("""select name from tabAccount
+ where account_type = 'Warehouse' and ifnull(master_name, '')!=''""")]
for entry in gl_map:
if entry.account in aii_accounts:
diff --git a/controllers/accounts_controller.py b/controllers/accounts_controller.py
index c918c20..a75fb62 100644
--- a/controllers/accounts_controller.py
+++ b/controllers/accounts_controller.py
@@ -62,7 +62,7 @@
if self.meta.get_field(fieldname) and self.doc.fields.get(fieldname):
if not self.doc.price_list_currency:
self.doc.price_list_currency = webnotes.conn.get_value("Price List",
- self.doc.fields.get(fieldame), "currency")
+ self.doc.fields.get(fieldname), "currency")
if self.doc.price_list_currency == company_currency:
self.doc.plc_conversion_rate = 1.0
diff --git a/controllers/stock_controller.py b/controllers/stock_controller.py
index 4734dea..d4c92a9 100644
--- a/controllers/stock_controller.py
+++ b/controllers/stock_controller.py
@@ -14,18 +14,23 @@
def make_gl_entries(self):
if not cint(webnotes.defaults.get_global_default("auto_accounting_for_stock")):
return
-
+
+ warehouse_account = self.get_warehouse_account()
+
if self.doc.docstatus==1:
- gl_entries = self.get_gl_entries_for_stock()
+ gl_entries = self.get_gl_entries_for_stock(warehouse_account)
make_gl_entries(gl_entries)
else:
delete_gl_entries(voucher_type=self.doc.doctype, voucher_no=self.doc.name)
- self.update_gl_entries_after()
+ self.update_gl_entries_after(warehouse_account)
- def get_gl_entries_for_stock(self, default_expense_account=None, default_cost_center=None):
+ def get_gl_entries_for_stock(self, warehouse_account=None, default_expense_account=None,
+ default_cost_center=None):
from accounts.general_ledger import process_gl_map
- warehouse_account = self.get_warehouse_account()
+ if not warehouse_account:
+ warehouse_account = self.get_warehouse_account()
+
stock_ledger = self.get_stock_ledger_details()
voucher_details = self.get_voucher_details(stock_ledger, default_expense_account,
default_cost_center)
@@ -93,14 +98,17 @@
where account_type = 'Warehouse' and ifnull(master_name, '') != ''"""))
return warehouse_account
- def update_gl_entries_after(self):
+ def update_gl_entries_after(self, warehouse_account=None):
from accounts.utils import get_stock_and_account_difference
future_stock_vouchers = self.get_future_stock_vouchers()
gle = self.get_voucherwise_gl_entries(future_stock_vouchers)
+ if not warehouse_account:
+ warehouse_account = self.get_warehouse_account()
+
for voucher_type, voucher_no in future_stock_vouchers:
existing_gle = gle.get((voucher_type, voucher_no), [])
voucher_obj = webnotes.get_obj(voucher_type, voucher_no)
- expected_gle = voucher_obj.get_gl_entries_for_stock()
+ expected_gle = voucher_obj.get_gl_entries_for_stock(warehouse_account)
if expected_gle:
matched = True
diff --git a/docs/docs.user.stock.accounting_for_stock.md b/docs/docs.user.stock.accounting_for_stock.md
index 0088105..22fa7bd 100644
--- a/docs/docs.user.stock.accounting_for_stock.md
+++ b/docs/docs.user.stock.accounting_for_stock.md
@@ -4,7 +4,7 @@
}
---
-The value of available inventory is treated as an Asset in company's Chart of Accounts. Depending on the type of item, it can be treated as Fixed Asset or Current Asset. To prepare Balance Sheet, you should make the accounting entries for those assets.
+The value of available inventory is treated as an Asset in company's Chart of Accounts. Depending on the type of items, it can be treated as Fixed Asset or Current Asset. To prepare Balance Sheet, you should make the accounting entries for those assets.
There are generally two different methods of accounting for inventory:
@@ -12,7 +12,7 @@
In this process, for each stock transactions system posts relevant accounting entries to sync stock balance and accounting balance. This is the default settings in ERPNext for new accounts.
-When you buy and receive items, those items are booked as the company’s assets (stock-in-hand / fixed-assets). When you sell and deliver those items, an expense (cost-of-goods-sold) equal to the buying cost of the items is booked. General Ledger entries are made after every transaction. This improves accuracy of Balance Sheet and Profit and Loss statement. And the value as per Stock Ledger always remains same with the relevant account balance.
+When you buy and receive items, those items are booked as the company’s assets (stock-in-hand / fixed-assets). When you sell and deliver those items, an expense (cost-of-goods-sold) equal to the buying cost of the items is booked. General Ledger entries are made after every stock transaction. This improves accuracy of Balance Sheet and Profit and Loss statement. And the value as per Stock Ledger always remains same with the relevant account balance.
To check accounting entries for a particular stock transaction, please check [**examples**](docs.user.stock.perpetual_inventory.html)
diff --git a/docs/docs.user.stock.perpetual_inventory.md b/docs/docs.user.stock.perpetual_inventory.md
index 0ff5517..2b6adc4 100644
--- a/docs/docs.user.stock.perpetual_inventory.md
+++ b/docs/docs.user.stock.perpetual_inventory.md
@@ -4,7 +4,7 @@
}
---
-In perpetual accounting, system creates accounting entries for each stock transactions. Hence, stock balance are always remains same as relevant account balance.
+In perpetual accounting, system creates accounting entries for each stock transactions. Hence, stock balance will always remains same as relevant account balance.
## **Activation**
@@ -14,22 +14,21 @@
- Expenses Included In Valuation
- Cost Center
-2. Go to Setup > Accounts Settings > check "Make Accounting Entry For Every Stock Entry"
-
+2. The system will create an account head for each warehouse. Enter "Create Account Under" (account group under which account will be created) in warehouse master, based on type of items it stores (Stores, Fixed Asset Warehouse, etc).
-3. Enter Asset / Expense account for each warehouse depending upon type of warehouse (Stores, Fixed Asset Warehouse etc)
+3. Go to Setup > Accounts Settings > check "Make Accounting Entry For Every Stock Movement"
+
## **Migration from Periodic Inventory**
-Migration from Periodic Inventory is not a one click settings, it involves some speacial steps. As Perpetual Inventory always maintain a sync between stock and account balance, it is not possible to enable it with existing Warehouse setup. You have to create a whole new set of Warehouses, each linked to relevant account.
+Migration from Periodic Inventory is not a one click settings, it involves some special steps. As Perpetual Inventory always maintain a sync between stock and account balance, it is not possible to enable it with existing Warehouse setup. You have to create a whole new set of Warehouses, each linked to relevant account.
Steps to be followed:
- Nullify current stock-in-hand / fixed-asset account balance through Journal Voucher.
-- Create new warehouse for each existing warehouse.
-- Assign Asset / Expense account while creating new warehouse.
-- Follow Activation Step 1 & 2
+- Create new warehouse for each existing warehouse
+- Follow Activation Step 1, 2 & 3
- Create Stock Entry (Material Transfer) to transfer available stock from existing warehouse to new warehouse
>Note: System will not post any accounting entries for existing stock transactions submitted prior to the activation of Perpetual Inventory as those old warehouses will not be linked to account.
@@ -47,11 +46,13 @@
> - Accounts Receivable
> - Jane Doe
> - Stock Assets
-> - Stock In Hand
+> - Stores
+> - Finished Goods
+> - Work In Progress
> - Tax Assets
> - VAT
> - Fixed Assets
-> - Office Equipments
+> - Fixed Asset Warehouse
>- Liabilities (Cr)
> - Current Liabilities
> - Accounts Payable
@@ -74,10 +75,10 @@
#### Warehouse - Account Configuration
->- Stores - Stock In Hand
->- Work In Progress - Stock In Hand
->- Finished Goods - Stock In Hand
->- Fixed Asset Warehouse - Office Equipments
+>- Stores
+>- Work In Progress
+>- Finished Goods
+>- Fixed Asset Warehouse
### **Purchase Receipt**
@@ -125,7 +126,7 @@
-As stock balance increases through Purchase Receipt, "Stock In Hand" account has been debited and a temporary account "Stock Receipt But Not Billed" account has been credited, to maintain double entry accounting system.
+As stock balance increases through Purchase Receipt, "Store" and "Fixed Asset Warehouse" accounts have been debited and a temporary account "Stock Receipt But Not Billed" account has been credited, to maintain double entry accounting system.
--
@@ -180,7 +181,7 @@
-As item has delivered from "Stores" warehouse, "Stock In Hand" account has been credited and equal amount will be debited to the expense account "Cost of Goods Sold". The debit/credit amount is equal to the total buying cost of the selling items. And buying cost is calculated based on valuation method (FIFO / Moving Average) or serial no cost for serialized items.
+As item has delivered from "Stores" warehouse, "Stores" account has been credited and equal amount will be debited to the expense account "Cost of Goods Sold". The debit/credit amount is equal to the total buying cost of the selling items. And buying cost is calculated based on valuation method (FIFO / Moving Average) or serial no cost for serialized items.
In this eample, Buying cost of RM0001 = (2200/10)*5 = 1100
@@ -198,7 +199,7 @@
-Here apart from normal account entries for invoice, "Stock In Hand" and "Cost of Goods Sold" accounts are also affected based on buying cost.
+Here apart from normal account entries for invoice, "Stores" and "Cost of Goods Sold" accounts are also affected based on buying cost.
--
diff --git a/patches/august_2013/p01_auto_accounting_for_stock_patch.py b/patches/august_2013/p01_auto_accounting_for_stock_patch.py
index 212e5ae..c6a22dd 100644
--- a/patches/august_2013/p01_auto_accounting_for_stock_patch.py
+++ b/patches/august_2013/p01_auto_accounting_for_stock_patch.py
@@ -1,9 +1,6 @@
import webnotes
from webnotes.utils import cint
-def execute():
- import patches.september_2012.repost_stock
- patches.september_2012.repost_stock.execute()
-
+def execute():
import patches.march_2013.p08_create_aii_accounts
patches.march_2013.p08_create_aii_accounts.execute()
\ No newline at end of file
diff --git a/patches/march_2013/p08_create_aii_accounts.py b/patches/march_2013/p08_create_aii_accounts.py
index 8f4fa4a..b7d1c0b 100644
--- a/patches/march_2013/p08_create_aii_accounts.py
+++ b/patches/march_2013/p08_create_aii_accounts.py
@@ -44,7 +44,6 @@
def add_ledger_accounts():
accounts_to_add = [
- ["Stock In Hand", "Stock Assets", "Ledger", ""],
["Cost of Goods Sold", "Stock Expenses", "Ledger", "Expense Account"],
["Stock Adjustment", "Stock Expenses", "Ledger", "Expense Account"],
["Expenses Included In Valuation", "Stock Expenses", "Ledger", "Expense Account"],
diff --git a/setup/doctype/setup_control/setup_control.py b/setup/doctype/setup_control/setup_control.py
index 9659173..3f09ff1 100644
--- a/setup/doctype/setup_control/setup_control.py
+++ b/setup/doctype/setup_control/setup_control.py
@@ -106,8 +106,9 @@
})
global_defaults.save()
- webnotes.conn.set_value("Accounts Settings", None, "auto_accounting_for_stock", 1)
- webnotes.conn.set_default("auto_accounting_for_stock", 1)
+ accounts_settings = webnotes.bean("Accounts Settings")
+ accounts_settings.doc.auto_accounting_for_stock = 1
+ accounts_settings.save()
stock_settings = webnotes.bean("Stock Settings")
stock_settings.doc.item_naming_by = "Item Code"
diff --git a/stock/doctype/purchase_receipt/purchase_receipt.py b/stock/doctype/purchase_receipt/purchase_receipt.py
index b9aa233..4d17db4 100644
--- a/stock/doctype/purchase_receipt/purchase_receipt.py
+++ b/stock/doctype/purchase_receipt/purchase_receipt.py
@@ -305,10 +305,11 @@
def get_rate(self,arg):
return get_obj('Purchase Common').get_rate(arg,self)
- def get_gl_entries_for_stock(self):
+ def get_gl_entries_for_stock(self, warehouse_account=None):
against_stock_account = self.get_company_default("stock_received_but_not_billed")
- gl_entries = super(DocType, self).get_gl_entries_for_stock(against_stock_account, None)
+ gl_entries = super(DocType, self).get_gl_entries_for_stock(warehouse_account,
+ against_stock_account)
return gl_entries
diff --git a/stock/doctype/stock_entry/test_stock_entry.py b/stock/doctype/stock_entry/test_stock_entry.py
index b6626f2..e2358eb 100644
--- a/stock/doctype/stock_entry/test_stock_entry.py
+++ b/stock/doctype/stock_entry/test_stock_entry.py
@@ -8,10 +8,12 @@
import webnotes, unittest
from webnotes.utils import flt
from stock.doctype.stock_ledger_entry.stock_ledger_entry import *
+from stock.doctype.purchase_receipt.test_purchase_receipt import set_perpetual_inventory
+
class TestStockEntry(unittest.TestCase):
def tearDown(self):
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 0)
+ set_perpetual_inventory(0)
if hasattr(self, "old_default_company"):
webnotes.conn.set_default("company", self.old_default_company)
@@ -81,14 +83,14 @@
def test_material_receipt_gl_entry(self):
self._clear_stock_account_balance()
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 1)
+ set_perpetual_inventory()
mr = webnotes.bean(copy=test_records[0])
mr.insert()
mr.submit()
- stock_in_hand_account = webnotes.conn.get_value("Warehouse", mr.doclist[1].t_warehouse,
- "account")
+ stock_in_hand_account = webnotes.conn.get_value("Account", {"account_type": "Warehouse",
+ "master_name": mr.doclist[1].t_warehouse})
self.check_stock_ledger_entries("Stock Entry", mr.doc.name,
[["_Test Item", "_Test Warehouse - _TC", 50.0]])
@@ -111,7 +113,7 @@
def test_material_issue_gl_entry(self):
self._clear_stock_account_balance()
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 1)
+ set_perpetual_inventory()
self._insert_material_receipt()
@@ -122,8 +124,9 @@
self.check_stock_ledger_entries("Stock Entry", mi.doc.name,
[["_Test Item", "_Test Warehouse - _TC", -40.0]])
- stock_in_hand_account = webnotes.conn.get_value("Warehouse", mi.doclist[1].s_warehouse,
- "account")
+ stock_in_hand_account = webnotes.conn.get_value("Account", {"account_type": "Warehouse",
+ "master_name": mi.doclist[1].s_warehouse})
+
self.check_gl_entries("Stock Entry", mi.doc.name,
sorted([
[stock_in_hand_account, 0.0, 4000.0],
@@ -146,7 +149,7 @@
def test_material_transfer_gl_entry(self):
self._clear_stock_account_balance()
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 1)
+ set_perpetual_inventory()
self._insert_material_receipt()
@@ -157,10 +160,12 @@
self.check_stock_ledger_entries("Stock Entry", mtn.doc.name,
[["_Test Item", "_Test Warehouse - _TC", -45.0], ["_Test Item", "_Test Warehouse 1 - _TC", 45.0]])
- stock_in_hand_account = webnotes.conn.get_value("Warehouse", mtn.doclist[1].s_warehouse,
- "account")
- fixed_asset_account = webnotes.conn.get_value("Warehouse", mtn.doclist[1].t_warehouse,
- "account")
+ stock_in_hand_account = webnotes.conn.get_value("Account", {"account_type": "Warehouse",
+ "master_name": mtn.doclist[1].s_warehouse})
+
+ fixed_asset_account = webnotes.conn.get_value("Account", {"account_type": "Warehouse",
+ "master_name": mtn.doclist[1].t_warehouse})
+
self.check_gl_entries("Stock Entry", mtn.doc.name,
sorted([
@@ -180,7 +185,7 @@
def test_repack_no_change_in_valuation(self):
self._clear_stock_account_balance()
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 1)
+ set_perpetual_inventory()
self._insert_material_receipt()
@@ -197,11 +202,11 @@
order by account desc""", repack.doc.name, as_dict=1)
self.assertFalse(gl_entries)
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 0)
+ set_perpetual_inventory(0)
def test_repack_with_change_in_valuation(self):
self._clear_stock_account_balance()
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 1)
+ set_perpetual_inventory()
self._insert_material_receipt()
@@ -210,8 +215,8 @@
repack.insert()
repack.submit()
- stock_in_hand_account = webnotes.conn.get_value("Warehouse",
- repack.doclist[2].t_warehouse, "account")
+ stock_in_hand_account = webnotes.conn.get_value("Account", {"account_type": "Warehouse",
+ "master_name": repack.doclist[2].t_warehouse})
self.check_gl_entries("Stock Entry", repack.doc.name,
sorted([
@@ -219,7 +224,7 @@
["Stock Adjustment - _TC", 0.0, 1000.0],
])
)
- webnotes.defaults.set_global_default("auto_accounting_for_stock", 0)
+ set_perpetual_inventory(0)
def check_stock_ledger_entries(self, voucher_type, voucher_no, expected_sle):
expected_sle.sort(key=lambda x: x[0])
diff --git a/stock/doctype/stock_ledger_entry/stock_ledger_entry.py b/stock/doctype/stock_ledger_entry/stock_ledger_entry.py
index 4e3e790..f0bffe9 100644
--- a/stock/doctype/stock_ledger_entry/stock_ledger_entry.py
+++ b/stock/doctype/stock_ledger_entry/stock_ledger_entry.py
@@ -88,7 +88,10 @@
if not webnotes.conn.sql("""select name from `tabBatch`
where item='%s' and name ='%s' and docstatus != 2""" % (self.doc.item_code, self.doc.batch_no)):
webnotes.throw("'%s' is not a valid Batch Number for Item '%s'" % (self.doc.batch_no, self.doc.item_code))
-
+
+ if not self.doc.stock_uom:
+ self.doc.stock_uom = item_det.stock_uom
+
def get_item_details(self):
return webnotes.conn.sql("""select name, has_batch_no, docstatus,
is_stock_item, has_serial_no, serial_no_series
@@ -97,11 +100,6 @@
def validate_serial_no(self):
item_det = self.get_item_details()
-
- if not self.doc.stock_uom:
- self.doc.stock_uom = item_det.stock_uom
-
- self.validate_serial_no(item_det)
if item_det.has_serial_no=="No":
if self.doc.serial_no:
diff --git a/stock/doctype/stock_reconciliation/stock_reconciliation.py b/stock/doctype/stock_reconciliation/stock_reconciliation.py
index 8bba2d9..38247bb 100644
--- a/stock/doctype/stock_reconciliation/stock_reconciliation.py
+++ b/stock/doctype/stock_reconciliation/stock_reconciliation.py
@@ -295,12 +295,12 @@
webnotes.conn.set(self.doc, "stock_value_difference", json.dumps(stock_value_difference))
- def get_gl_entries_for_stock(self):
+ def get_gl_entries_for_stock(self, warehouse_account=None):
if not self.doc.cost_center:
msgprint(_("Please enter Cost Center"), raise_exception=1)
- return super(DocType, self).get_gl_entries_for_stock(self.doc.expense_account,
- self.doc.cost_center)
+ return super(DocType, self).get_gl_entries_for_stock(warehouse_account,
+ self.doc.expense_account, self.doc.cost_center)
def validate_expense_account(self):
diff --git a/utilities/demo/make_demo.py b/utilities/demo/make_demo.py
index 4383d32..9527b48 100644
--- a/utilities/demo/make_demo.py
+++ b/utilities/demo/make_demo.py
@@ -270,7 +270,9 @@
st = webnotes.bean(make_stock_entry(pro_id, purpose))
st.doc.posting_date = current_date
st.doc.fiscal_year = "2013"
- st.doc.expense_adjustment_account = "Stock in Hand - WP"
+ for d in st.doclist.get({"parentfield": "mtn_details"}):
+ d.expense_account = "Stock Adjustment - " + company_abbr
+ d.cost_center = "Main - " + company_abbr
st.insert()
webnotes.conn.commit()
st.submit()