show sales invoice data in gross profit report
diff --git a/accounts/doctype/sales_invoice/sales_invoice.py b/accounts/doctype/sales_invoice/sales_invoice.py
index 620859e..9b5c80c 100644
--- a/accounts/doctype/sales_invoice/sales_invoice.py
+++ b/accounts/doctype/sales_invoice/sales_invoice.py
@@ -27,8 +27,6 @@
from webnotes.model.code import get_obj
from webnotes import _, msgprint
-from stock.utils import get_buying_amount, get_sales_bom
-
session = webnotes.session
month_map = {'Monthly': 1, 'Quarterly': 3, 'Half-yearly': 6, 'Yearly': 12}
@@ -701,19 +699,9 @@
})
)
- def make_item_gl_entries(self, gl_entries):
- # item gl entries
- auto_inventory_accounting = \
- cint(webnotes.defaults.get_global_default("auto_inventory_accounting"))
-
- if auto_inventory_accounting:
- if cint(self.doc.is_pos) and cint(self.doc.update_stock):
- stock_account = self.get_default_account("stock_in_hand_account")
- else:
- stock_account = self.get_default_account("stock_delivered_but_not_billed")
-
+ def make_item_gl_entries(self, gl_entries):
+ # income account gl entries
for item in self.doclist.get({"parentfield": "entries"}):
- # income account gl entries
if flt(item.amount):
gl_entries.append(
self.get_gl_dict({
@@ -725,28 +713,16 @@
})
)
- # expense account gl entries
- if auto_inventory_accounting and flt(item.buying_amount):
+ # expense account gl entries
+ if cint(webnotes.defaults.get_global_default("auto_inventory_accounting")) \
+ and cint(self.doc.is_pos) and cint(self.doc.update_stock):
+
+ for item in self.doclist.get({"parentfield": "entries"}):
self.check_expense_account(item)
+
+ gl_entries += self.get_gl_entries_for_stock(item.expense_account,
+ -1*item.buying_amount, cost_center=item.cost_center)
- gl_entries.append(
- self.get_gl_dict({
- "account": item.expense_account,
- "against": stock_account,
- "debit": item.buying_amount,
- "remarks": self.doc.remarks or "Accounting Entry for Stock",
- "cost_center": item.cost_center
- })
- )
- gl_entries.append(
- self.get_gl_dict({
- "account": stock_account,
- "against": item.expense_account,
- "credit": item.buying_amount,
- "remarks": self.doc.remarks or "Accounting Entry for Stock"
- })
- )
-
def make_pos_gl_entries(self, gl_entries):
if cint(self.doc.is_pos) and self.doc.cash_bank_account and self.doc.paid_amount:
# POS, make payment entries
@@ -789,42 +765,6 @@
"cost_center": self.doc.write_off_cost_center
})
)
-
- def set_buying_amount(self):
- if cint(self.doc.is_pos) and cint(self.doc.update_stock):
- stock_ledger_entries = self.get_stock_ledger_entries()
- item_sales_bom = get_sales_bom(self.doc.doctype, self.doc.name)
- else:
- stock_ledger_entries = item_sales_bom = None
-
- for item in self.doclist.get({"parentfield": "entries"}):
- if item.item_code in self.stock_items or \
- (item_sales_bom and item_sales_bom.get(item.item_code)):
- item.buying_amount = self.get_item_buying_amount(item, stock_ledger_entries,
- item_sales_bom)
- webnotes.conn.set_value("Sales Invoice Item", item.name,
- "buying_amount", item.buying_amount)
-
- def get_item_buying_amount(self, item, stock_ledger_entries, item_sales_bom):
- item_buying_amount = 0
- if stock_ledger_entries:
- # is pos and update stock
- item_buying_amount = get_buying_amount(item.item_code, item.warehouse, -1*item.qty,
- self.doc.doctype, self.doc.name, item.name, stock_ledger_entries, item_sales_bom)
- item.buying_amount = item_buying_amount > 0 and item_buying_amount or 0
- elif item.delivery_note and item.dn_detail:
- # against delivery note
- dn_item = webnotes.conn.get_value("Delivery Note Item", item.dn_detail,
- ["buying_amount", "qty"], as_dict=1)
- item_buying_rate = flt(dn_item.buying_amount) / flt(dn_item.qty)
- item_buying_amount = item_buying_rate * flt(item.qty)
-
- return item_buying_amount
-
- def check_expense_account(self, item):
- if not item.expense_account:
- msgprint(_("""Expense account is mandatory for item: """) + item.item_code,
- raise_exception=1)
def update_c_form(self):
"""Update amended id in C-form"""
diff --git a/accounts/doctype/sales_invoice/test_sales_invoice.py b/accounts/doctype/sales_invoice/test_sales_invoice.py
index 92feae8..7aa0c27 100644
--- a/accounts/doctype/sales_invoice/test_sales_invoice.py
+++ b/accounts/doctype/sales_invoice/test_sales_invoice.py
@@ -86,68 +86,6 @@
self.assertEquals(gle_count[0][0], 8)
- def test_sales_invoice_gl_entry_with_aii_delivery_note(self):
- webnotes.conn.sql("delete from `tabStock Ledger Entry`")
-
- webnotes.defaults.set_global_default("auto_inventory_accounting", 1)
-
- self._insert_purchase_receipt()
- dn = self._insert_delivery_note()
-
- si_against_dn = webnotes.copy_doclist(test_records[1])
- si_against_dn[1]["delivery_note"] = dn.doc.name
- si_against_dn[1]["dn_detail"] = dn.doclist[1].name
- si = webnotes.bean(si_against_dn)
- si.insert()
-
- si.submit()
-
- gl_entries = webnotes.conn.sql("""select account, debit, credit
- from `tabGL Entry` where voucher_type='Sales Invoice' and voucher_no=%s
- order by account asc""", si.doc.name, as_dict=1)
- self.assertTrue(gl_entries)
-
- expected_values = sorted([
- [si.doc.debit_to, 630.0, 0.0],
- [test_records[1][1]["income_account"], 0.0, 500.0],
- [test_records[1][2]["account_head"], 0.0, 80.0],
- [test_records[1][3]["account_head"], 0.0, 50.0],
- ["Stock Delivered But Not Billed - _TC", 0.0, 375.0],
- [test_records[1][1]["expense_account"], 375.0, 0.0]
- ])
- for i, gle in enumerate(gl_entries):
- self.assertEquals(expected_values[i][0], gle.account)
- self.assertEquals(expected_values[i][1], gle.debit)
- self.assertEquals(expected_values[i][2], gle.credit)
-
- si.cancel()
- gl_entries = webnotes.conn.sql("""select account, debit, credit
- from `tabGL Entry` where voucher_type='Sales Invoice' and voucher_no=%s
- and ifnull(is_cancelled, 'No') = 'No'
- order by account asc, name asc""", si.doc.name, as_dict=1)
-
- expected_values = sorted([
- [si.doc.debit_to, 630.0, 0.0],
- [si.doc.debit_to, 0.0, 630.0],
- [test_records[1][1]["income_account"], 0.0, 500.0],
- [test_records[1][1]["income_account"], 500.0, 0.0],
- [test_records[1][2]["account_head"], 0.0, 80.0],
- [test_records[1][2]["account_head"], 80.0, 0.0],
- [test_records[1][3]["account_head"], 0.0, 50.0],
- [test_records[1][3]["account_head"], 50.0, 0.0],
- ["Stock Delivered But Not Billed - _TC", 0.0, 375.0],
- ["Stock Delivered But Not Billed - _TC", 375.0, 0.0],
- [test_records[1][1]["expense_account"], 375.0, 0.0],
- [test_records[1][1]["expense_account"], 0.0, 375.0]
-
- ])
- for i, gle in enumerate(gl_entries):
- 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_inventory_accounting", 0)
-
def test_pos_gl_entry_with_aii(self):
webnotes.conn.sql("delete from `tabStock Ledger Entry`")
webnotes.defaults.set_global_default("auto_inventory_accounting", 1)
@@ -262,8 +200,6 @@
webnotes.defaults.set_global_default("auto_inventory_accounting", 0)
-
-
def _insert_purchase_receipt(self):
from stock.doctype.purchase_receipt.test_purchase_receipt import test_records \
as pr_test_records
diff --git a/accounts/doctype/sales_invoice_item/sales_invoice_item.txt b/accounts/doctype/sales_invoice_item/sales_invoice_item.txt
index 6f6ad39..bc51198 100644
--- a/accounts/doctype/sales_invoice_item/sales_invoice_item.txt
+++ b/accounts/doctype/sales_invoice_item/sales_invoice_item.txt
@@ -2,7 +2,7 @@
{
"creation": "2013-03-07 11:42:55",
"docstatus": 0,
- "modified": "2013-03-18 15:41:19",
+ "modified": "2013-03-21 18:35:47",
"modified_by": "Administrator",
"owner": "Administrator"
},
@@ -207,11 +207,10 @@
"width": "120px"
},
{
- "depends_on": "eval:sys_defaults.auto_inventory_accounting",
"doctype": "DocField",
"fieldname": "expense_account",
"fieldtype": "Link",
- "hidden": 0,
+ "hidden": 1,
"in_filter": 1,
"label": "Expense Account",
"options": "Account",
diff --git a/accounts/report/gross_profit/gross_profit.js b/accounts/report/gross_profit/gross_profit.js
index 77ac581..1f07df9 100644
--- a/accounts/report/gross_profit/gross_profit.js
+++ b/accounts/report/gross_profit/gross_profit.js
@@ -6,6 +6,6 @@
"fieldtype": "Link",
"options": "Company",
"default": wn.defaults.get_user_default("company")
- }
+ },
]
}
\ No newline at end of file
diff --git a/accounts/report/gross_profit/gross_profit.py b/accounts/report/gross_profit/gross_profit.py
index a871bab..a479161 100644
--- a/accounts/report/gross_profit/gross_profit.py
+++ b/accounts/report/gross_profit/gross_profit.py
@@ -1,33 +1,28 @@
from __future__ import unicode_literals
import webnotes
from webnotes.utils import flt
-from stock.utils import get_buying_amount, get_sales_bom
+from stock.utils import get_buying_amount
def execute(filters=None):
if not filters: filters = {}
stock_ledger_entries = get_stock_ledger_entries(filters)
- item_sales_bom = get_sales_bom("Delivery Note")
+ item_sales_bom = get_item_sales_bom()
+
+ source = get_source_data(filters)
- delivery_note_items = webnotes.conn.sql("""select dn.name, dn.posting_date, dn.posting_time,
- dn.project_name, item.item_code, item.item_name, item.description, item.warehouse,
- item.qty, item.basic_rate, item.amount, item.name as "item_row"
- from `tabDelivery Note` dn, `tabDelivery Note Item` item
- where item.parent = dn.name and dn.docstatus = 1
- order by dn.posting_date desc, dn.posting_time desc""", as_dict=1)
-
- columns = ["Delivery Note:Link/Delivery Note", "Posting Date:Date", "Posting Time",
+ columns = ["Delivery Note/Sales Invoice::120", "Posting Date:Date", "Posting Time",
"Item Code:Link/Item", "Item Name", "Description", "Warehouse:Link/Warehouse",
"Qty:Float", "Selling Rate:Currency", "Selling Amount:Currency", "Buying Amount:Currency",
"Gross Profit:Currency", "Gross Profit %:Percent", "Project:Link/Project"]
-
+
data = []
- for row in delivery_note_items:
+ for row in source:
selling_amount = flt(row.amount)
buying_amount = get_buying_amount(row.item_code, row.warehouse, -1*row.qty,
- "Delivery Note", row.name, row.item_row, stock_ledger_entries,
- item_sales_bom.get(row.name, webnotes._dict()))
+ row.parenttype, row.name, row.item_row, stock_ledger_entries,
+ item_sales_bom.get(row.parenttype, {}).get(row.name, webnotes._dict()))
buying_amount = buying_amount > 0 and buying_amount or 0
@@ -37,7 +32,8 @@
else:
gross_profit = gross_profit_percent = 0.0
- data.append([row.name, row.posting_date, row.posting_time, row.item_code, row.item_name,
+ name = """<a href="%s">%s</a>""" % ("/".join(["#Form", row.parenttype, row.name]), row.name)
+ data.append([name, row.posting_date, row.posting_time, row.item_code, row.item_name,
row.description, row.warehouse, row.qty, row.basic_rate, row.amount, buying_amount,
gross_profit, gross_profit_percent, row.project])
@@ -55,3 +51,44 @@
query += " order by item_code desc, warehouse desc, posting_date desc, posting_time desc, name desc"
return webnotes.conn.sql(query, filters, as_dict=True)
+
+def get_item_sales_bom():
+ item_sales_bom = {}
+
+ for d in webnotes.conn.sql("""select parenttype, parent, parent_item,
+ item_code, warehouse, -1*qty as total_qty
+ from `tabDelivery Note Packing Item` where docstatus=1""", as_dict=1):
+ item_sales_bom.setdefault(d.parenttype, webnotes._dict()).setdefault(d.parent,
+ webnotes._dict()).setdefault(d.parent_item, []).append(d)
+
+ return item_sales_bom
+
+def get_source_data(filters):
+ conditions = ""
+ if filters.get("company"):
+ conditions += " and company=%(company)s"
+
+ delivery_note_items = webnotes.conn.sql("""select item.parenttype, dn.name,
+ dn.posting_date, dn.posting_time, dn.project_name,
+ item.item_code, item.item_name, item.description, item.warehouse,
+ item.qty, item.basic_rate, item.amount, item.name as "item_row",
+ timestamp(dn.posting_date, dn.posting_time) as posting_datetime
+ from `tabDelivery Note` dn, `tabDelivery Note Item` item
+ where item.parent = dn.name and dn.docstatus = 1 %s
+ order by dn.posting_date desc, dn.posting_time desc""" % (conditions,), filters, as_dict=1)
+
+ sales_invoice_items = webnotes.conn.sql("""select item.parenttype, si.name,
+ si.posting_date, si.posting_time, si.project_name,
+ item.item_code, item.item_name, item.description, item.warehouse,
+ item.qty, item.basic_rate, item.amount, item.name as "item_row",
+ timestamp(si.posting_date, si.posting_time) as posting_datetime
+ from `tabSales Invoice` si, `tabSales Invoice Item` item
+ where item.parent = si.name and si.docstatus = 1 %s
+ and si.is_pos = 1 and si.update_stock = 1
+ order by si.posting_date desc, si.posting_time desc""" % (conditions,), filters, as_dict=1)
+
+ source = delivery_note_items + sales_invoice_items
+ if len(source) > len(delivery_note_items):
+ source.sort(key=lambda d: d.posting_datetime, reverse=True)
+
+ return source
\ No newline at end of file
diff --git a/controllers/selling_controller.py b/controllers/selling_controller.py
index cd81e18..538c5c1 100644
--- a/controllers/selling_controller.py
+++ b/controllers/selling_controller.py
@@ -39,3 +39,24 @@
if self.meta.get_field("in_words_export"):
self.doc.in_words_export = money_in_words(disable_rounded_total and
self.doc.grand_total_export or self.doc.rounded_total_export, self.doc.currency)
+
+ def set_buying_amount(self):
+ from stock.utils import get_buying_amount
+ stock_ledger_entries = self.get_stock_ledger_entries()
+
+ item_sales_bom = {}
+ for d in self.doclist.get({"parentfield": "packing_details"}):
+ new_d = webnotes._dict(d.fields.copy())
+ new_d.total_qty = -1 * d.qty
+ item_sales_bom.setdefault(d.parent_item, []).append(new_d)
+
+ if stock_ledger_entries:
+ for item in self.doclist.get({"parentfield": self.fname}):
+ if item.item_code in self.stock_items or \
+ (item_sales_bom and item_sales_bom.get(item.item_code)):
+ buying_amount = get_buying_amount(item.item_code, item.warehouse, -1*item.qty,
+ self.doc.doctype, self.doc.name, item.name, stock_ledger_entries,
+ item_sales_bom)
+ item.buying_amount = buying_amount > 0 and buying_amount or 0
+ webnotes.conn.set_value(self.tname, item.name, "buying_amount",
+ item.buying_amount)
\ No newline at end of file
diff --git a/controllers/stock_controller.py b/controllers/stock_controller.py
index eec7352..c76865d 100644
--- a/controllers/stock_controller.py
+++ b/controllers/stock_controller.py
@@ -16,11 +16,14 @@
from __future__ import unicode_literals
import webnotes
+from webnotes import msgprint, _
from controllers.accounts_controller import AccountsController
class StockController(AccountsController):
- def make_gl_entries(self, against_stock_account, amount, cost_center=None):
- stock_in_hand_account = self.get_default_account("stock_in_hand_account")
+ def get_gl_entries_for_stock(self, against_stock_account, amount,
+ stock_in_hand_account=None, cost_center=None):
+ if not stock_in_hand_account:
+ stock_in_hand_account = self.get_default_account("stock_in_hand_account")
if amount:
gl_entries = [
@@ -41,9 +44,14 @@
"remarks": self.doc.remarks or "Accounting Entry for Stock",
}, self.doc.docstatus == 2),
]
- from accounts.general_ledger import make_gl_entries
- make_gl_entries(gl_entries, cancel=self.doc.docstatus == 2)
+ return gl_entries
+
+
+ def check_expense_account(self, item):
+ if not item.expense_account:
+ msgprint(_("""Expense account is mandatory for item: """) + item.item_code,
+ raise_exception=1)
def get_stock_ledger_entries(self, item_list=None, warehouse_list=None):
if not (item_list and warehouse_list):
diff --git a/stock/doctype/delivery_note/delivery_note.js b/stock/doctype/delivery_note/delivery_note.js
index a8af107..7e6031c 100644
--- a/stock/doctype/delivery_note/delivery_note.js
+++ b/stock/doctype/delivery_note/delivery_note.js
@@ -308,4 +308,24 @@
if(cint(wn.boot.notification_settings.delivery_note)) {
cur_frm.email_doc(wn.boot.notification_settings.delivery_note_message);
}
+}
+
+// expense account
+cur_frm.fields_dict['delivery_note_details'].grid.get_field('expense_account').get_query = function(doc) {
+ return {
+ "query": "accounts.utils.get_account_list",
+ "filters": {
+ "is_pl_account": "Yes",
+ "debit_or_credit": "Debit",
+ "company": doc.company
+ }
+ }
+}
+
+// cost center
+cur_frm.fields_dict["delivery_note_details"].grid.get_field("cost_center").get_query = function(doc) {
+ return {
+ query: "accounts.utils.get_cost_center_list",
+ filters: { company_name: doc.company}
+ }
}
\ No newline at end of file
diff --git a/stock/doctype/delivery_note/delivery_note.py b/stock/doctype/delivery_note/delivery_note.py
index 82881c5..5e3c9e9 100644
--- a/stock/doctype/delivery_note/delivery_note.py
+++ b/stock/doctype/delivery_note/delivery_note.py
@@ -144,6 +144,8 @@
self.validate_mandatory()
self.validate_reference_value()
self.validate_for_items()
+ self.validate_warehouse()
+
sales_com_obj.validate_max_discount(self, 'delivery_note_details')
sales_com_obj.get_allocated_sum(self)
sales_com_obj.check_conversion_rate(self)
@@ -203,6 +205,12 @@
else:
chk_dupl_itm.append(f)
+ def validate_warehouse(self):
+ for d in self.get_item_list():
+ if webnotes.conn.get_value("Item", d['item_code'], "is_stock_item") == "Yes":
+ if not d['warehouse']:
+ msgprint("Please enter Warehouse for item %s as it is stock item"
+ % d['item_code'], raise_exception=1)
def validate_items_with_prevdoc(self, d):
"""check if same item, warehouse present in prevdoc"""
@@ -393,41 +401,17 @@
total = (amount/self.doc.net_total)*self.doc.grand_total
get_obj('Sales Common').check_credit(self, total)
- def set_buying_amount(self):
- from stock.utils import get_buying_amount, get_sales_bom
- stock_ledger_entries = self.get_stock_ledger_entries()
- item_sales_bom = get_sales_bom(self.doc.doctype, self.doc.name)
-
- if stock_ledger_entries:
- for item in self.doclist.get({"parentfield": "delivery_note_details"}):
- if item.item_code in self.stock_items or \
- (item_sales_bom and item_sales_bom.get(item.item_code)):
- buying_amount = get_buying_amount(item.item_code, item.warehouse, -1*item.qty,
- self.doc.doctype, self.doc.name, item.name, stock_ledger_entries,
- item_sales_bom)
- item.buying_amount = buying_amount > 0 and buying_amount or 0
- webnotes.conn.set_value("Delivery Note Item", item.name, "buying_amount",
- item.buying_amount)
-
- self.validate_warehouse()
-
- def validate_warehouse(self):
- for d in self.get_item_list():
- if webnotes.conn.get_value("Item", d['item_code'], "is_stock_item") == "Yes":
- if not d['warehouse']:
- msgprint("Please enter Warehouse for item %s as it is stock item"
- % d['item_code'], raise_exception=1)
-
def make_gl_entries(self):
if not cint(webnotes.defaults.get_global_default("auto_inventory_accounting")):
return
-
- against_stock_account = self.get_default_account("stock_delivered_but_not_billed")
- total_buying_amount = self.get_total_buying_amount()
-
- super(DocType, self).make_gl_entries(against_stock_account, -1*total_buying_amount)
-
- def get_total_buying_amount(self):
- total_buying_amount = sum([item.buying_amount for item in
- self.doclist.get({"parentfield": "delivery_note_details"})])
- return total_buying_amount
+
+ gl_entries = []
+ for item in self.doclist.get({"parentfield": "delivery_note_details"}):
+ self.check_expense_account(item)
+
+ gl_entries += self.get_gl_entries_for_stock(item.expense_account, -1*item.buying_amount,
+ cost_center=item.cost_center)
+
+ if gl_entries:
+ from accounts.general_ledger import make_gl_entries
+ make_gl_entries(gl_entries)
diff --git a/stock/doctype/delivery_note/test_delivery_note.py b/stock/doctype/delivery_note/test_delivery_note.py
index d0b440c..d0ec631 100644
--- a/stock/doctype/delivery_note/test_delivery_note.py
+++ b/stock/doctype/delivery_note/test_delivery_note.py
@@ -56,15 +56,19 @@
self._insert_purchase_receipt()
dn = webnotes.bean(copy=test_records[0])
+ dn.doclist[1].expense_account = "Cost of Goods Sold - _TC"
+ dn.doclist[1].cost_center = "Auto Inventory Accounting - _TC"
+
stock_in_hand_account = webnotes.conn.get_value("Company", dn.doc.company,
"stock_in_hand_account")
from accounts.utils import get_balance_on
prev_bal = get_balance_on(stock_in_hand_account, dn.doc.posting_date)
-
+
dn.insert()
dn.submit()
+
gl_entries = webnotes.conn.sql("""select account, debit, credit
from `tabGL Entry` where voucher_type='Delivery Note' and voucher_no=%s
order by account asc""", dn.doc.name, as_dict=1)
@@ -72,9 +76,8 @@
expected_values = sorted([
[stock_in_hand_account, 0.0, 375.0],
- ["Stock Delivered But Not Billed - _TC", 375.0, 0.0]
+ ["Cost of Goods Sold - _TC", 375.0, 0.0]
])
-
for i, gle in enumerate(gl_entries):
self.assertEquals(expected_values[i][0], gle.account)
self.assertEquals(expected_values[i][1], gle.debit)
diff --git a/stock/doctype/delivery_note_item/delivery_note_item.txt b/stock/doctype/delivery_note_item/delivery_note_item.txt
index 47b0d33..94c6541 100644
--- a/stock/doctype/delivery_note_item/delivery_note_item.txt
+++ b/stock/doctype/delivery_note_item/delivery_note_item.txt
@@ -2,7 +2,7 @@
{
"creation": "2013-03-07 11:42:59",
"docstatus": 0,
- "modified": "2013-03-07 15:46:43",
+ "modified": "2013-03-21 18:36:22",
"modified_by": "Administrator",
"owner": "Administrator"
},
@@ -236,6 +236,7 @@
"doctype": "DocField",
"fieldname": "batch_no",
"fieldtype": "Link",
+ "hidden": 0,
"label": "Batch No",
"oldfieldname": "batch_no",
"oldfieldtype": "Link",
@@ -245,6 +246,28 @@
},
{
"doctype": "DocField",
+ "fieldname": "expense_account",
+ "fieldtype": "Link",
+ "hidden": 1,
+ "label": "Expense Account",
+ "no_copy": 1,
+ "options": "Account",
+ "print_hide": 1,
+ "width": "120px"
+ },
+ {
+ "doctype": "DocField",
+ "fieldname": "cost_center",
+ "fieldtype": "Link",
+ "hidden": 1,
+ "label": "Cost Center",
+ "no_copy": 1,
+ "options": "Cost Center",
+ "print_hide": 1,
+ "width": "120px"
+ },
+ {
+ "doctype": "DocField",
"fieldname": "item_group",
"fieldtype": "Link",
"hidden": 1,
diff --git a/stock/doctype/purchase_receipt/purchase_receipt.py b/stock/doctype/purchase_receipt/purchase_receipt.py
index e7d030d..7949a1c 100644
--- a/stock/doctype/purchase_receipt/purchase_receipt.py
+++ b/stock/doctype/purchase_receipt/purchase_receipt.py
@@ -318,10 +318,14 @@
if not cint(webnotes.defaults.get_global_default("auto_inventory_accounting")):
return
+ from accounts.general_ledger import make_gl_entries
+
against_stock_account = self.get_default_account("stock_received_but_not_billed")
total_valuation_amount = self.get_total_valuation_amount()
+ gl_entries = self.get_gl_entries_for_stock(against_stock_account, total_valuation_amount)
- super(DocType, self).make_gl_entries(against_stock_account, total_valuation_amount)
+ if gl_entries:
+ make_gl_entries(gl_entries, cancel=self.doc.docstatus == 2)
def get_total_valuation_amount(self):
total_valuation_amount = 0.0
diff --git a/stock/doctype/stock_entry/stock_entry.py b/stock/doctype/stock_entry/stock_entry.py
index 76d3a42..f54ce60 100644
--- a/stock/doctype/stock_entry/stock_entry.py
+++ b/stock/doctype/stock_entry/stock_entry.py
@@ -173,13 +173,17 @@
if not self.doc.expense_adjustment_account:
webnotes.msgprint(_("Please enter Expense/Adjustment Account"), raise_exception=1)
-
+
+ from accounts.general_ledger import make_gl_entries
+
cost_center = "Auto Inventory Accounting - %s" % (self.company_abbr,)
total_valuation_amount = self.get_total_valuation_amount()
- super(DocType, self).make_gl_entries(self.doc.expense_adjustment_account,
- total_valuation_amount, cost_center)
-
+ gl_entries = self.get_gl_entries_for_stock(self.doc.expense_adjustment_account,
+ total_valuation_amount, cost_center=cost_center)
+ if gl_entries:
+ make_gl_entries(gl_entries, cancel=self.doc.docstatus == 2)
+
def get_total_valuation_amount(self):
total_valuation_amount = 0
for item in self.doclist.get({"parentfield": "mtn_details"}):
diff --git a/stock/doctype/stock_entry/test_stock_entry.py b/stock/doctype/stock_entry/test_stock_entry.py
index 1bd068a..ded71db 100644
--- a/stock/doctype/stock_entry/test_stock_entry.py
+++ b/stock/doctype/stock_entry/test_stock_entry.py
@@ -26,7 +26,7 @@
self.assertTrue(mr_name)
- def atest_material_receipt_gl_entry(self):
+ def test_material_receipt_gl_entry(self):
webnotes.conn.sql("delete from `tabStock Ledger Entry`")
webnotes.defaults.set_global_default("auto_inventory_accounting", 1)
@@ -63,7 +63,7 @@
webnotes.defaults.set_global_default("auto_inventory_accounting", 0)
- def atest_material_issue_gl_entry(self):
+ def test_material_issue_gl_entry(self):
webnotes.conn.sql("delete from `tabStock Ledger Entry`")
webnotes.defaults.set_global_default("auto_inventory_accounting", 1)
@@ -151,7 +151,7 @@
self.assertEquals(expected_sle[i][1], sle.warehouse)
self.assertEquals(expected_sle[i][2], sle.actual_qty)
- def acheck_gl_entries(self, voucher_type, voucher_no, expected_gl_entries):
+ def check_gl_entries(self, voucher_type, voucher_no, expected_gl_entries):
# check gl entries
gl_entries = webnotes.conn.sql("""select account, debit, credit
diff --git a/stock/doctype/stock_reconciliation/stock_reconciliation.py b/stock/doctype/stock_reconciliation/stock_reconciliation.py
index c2f5a94..ac0ab98 100644
--- a/stock/doctype/stock_reconciliation/stock_reconciliation.py
+++ b/stock/doctype/stock_reconciliation/stock_reconciliation.py
@@ -309,10 +309,14 @@
if not self.doc.expense_account:
msgprint(_("Please enter Expense Account"), raise_exception=1)
+ from accounts.general_ledger import make_gl_entries
+
cost_center = "Auto Inventory Accounting - %s" % (self.company_abbr,)
- super(DocType, self).make_gl_entries(self.doc.expense_account,
- self.doc.stock_value_difference, cost_center)
+ gl_entries = self.get_gl_entries_for_stock(self.doc.expense_account,
+ self.doc.stock_value_difference, cost_center=cost_center)
+ if gl_entries:
+ make_gl_entries(gl_entries, cancel=self.doc.docstatus == 2)
@webnotes.whitelist()
def upload():
diff --git a/stock/utils.py b/stock/utils.py
index 047ff33..1406af6 100644
--- a/stock/utils.py
+++ b/stock/utils.py
@@ -191,31 +191,4 @@
buying_amount = previous_stock_value - flt(sle.stock_value)
return buying_amount
- return 0.0
-
-def get_sales_bom(doctype=None, docname=None):
- item_sales_bom = webnotes._dict()
-
- query = """select parenttype, parent, parent_item,
- item_code, warehouse, -1*qty as total_qty
- from `tabDelivery Note Packing Item` where docstatus=1"""
-
- args = {}
- if doctype:
- query += " and parenttype=%(parenttype)s"
- args["parenttype"] = doctype
-
- if docname:
- query += " and parent=%(parent)s"
- args["parent"] = docname
-
- for d in webnotes.conn.sql(query, args, as_dict=1):
- item_sales_bom.setdefault(d.parenttype, webnotes._dict()).setdefault(d.parent,
- webnotes._dict()).setdefault(d.parent_item, []).append(d)
-
- if doctype and docname:
- return item_sales_bom[doctype].get(docname, webnotes._dict())
- elif doctype:
- return item_sales_bom[doctype]
- else:
- return item_sales_bom
+ return 0.0
\ No newline at end of file