diff --git a/accounts/doctype/account/account.py b/accounts/doctype/account/account.py index 410994fed20..eb65604e026 100644 --- a/accounts/doctype/account/account.py +++ b/accounts/doctype/account/account.py @@ -69,11 +69,13 @@ class DocType: raise_exception=1) def validate_duplicate_account(self): - if (self.doc.fields.get('__islocal') or not self.doc.name) and \ - sql("""select name from tabAccount where account_name=%s and company=%s""", - (self.doc.account_name, self.doc.company)): - msgprint("Account Name: %s already exists, please rename" - % self.doc.account_name, raise_exception=1) + + if self.doc.fields.get('__islocal') or not self.doc.name: + company_abbr = webnotes.conn.get_value("Company", self.doc.company, "abbr") + if sql("""select name from tabAccount where name=%s""", + (self.doc.account_name + " - " + company_abbr)): + msgprint("Account Name: %s already exists, please rename" + % self.doc.account_name, raise_exception=1) def validate_root_details(self): #does not exists parent diff --git a/patches/april_2013/p05_fixes_in_reverse_modules.py b/patches/april_2013/p05_fixes_in_reverse_modules.py index 8f5b2f773ec..41e2e5ef2f9 100644 --- a/patches/april_2013/p05_fixes_in_reverse_modules.py +++ b/patches/april_2013/p05_fixes_in_reverse_modules.py @@ -4,7 +4,7 @@ import webnotes.utils def execute(): modules = webnotes.get_config().modules - ml = json.loads(webnotes.conn.get_global("hidden_modules")) + ml = json.loads(webnotes.conn.get_global("hidden_modules") or "[]") if len(ml) == len(modules.keys()): webnotes.conn.set_global("hidden_modules", json.dumps([])) \ 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 ac563b52fae..00230d8c666 100644 --- a/patches/march_2013/p08_create_aii_accounts.py +++ b/patches/march_2013/p08_create_aii_accounts.py @@ -61,7 +61,8 @@ def add_accounts(accounts_to_add, check_fn=None): continue for account_name, parent_account_name, group_or_ledger, account_type in accounts_to_add: - if not webnotes.conn.exists("Account", "%s - %s" % (account_name, abbr)): + if not webnotes.conn.sql("""select name from `tabAccount` where account_name = %s + and company = %s""", (account_name, company)): parent_account = "%s - %s" % (parent_account_name, abbr) if check_fn: parent_account = check_fn(parent_account, company) @@ -77,7 +78,8 @@ def add_accounts(accounts_to_add, check_fn=None): def add_aii_cost_center(): for company, abbr in webnotes.conn.sql("""select name, abbr from `tabCompany`"""): - if not webnotes.conn.exists("Cost Center", "Auto Inventory Accounting - %s" % abbr): + if not webnotes.conn.sql("""select name from `tabCost Center` where cost_center_name = + 'Auto Inventory Accounting' and company_name = %s""", company): parent_cost_center = webnotes.conn.get_value("Cost Center", {"parent_cost_center['']": '', "company_name": company}) diff --git a/public/js/queries.js b/public/js/queries.js index e15e6867b4a..9809cd980d0 100644 --- a/public/js/queries.js +++ b/public/js/queries.js @@ -133,10 +133,8 @@ erpnext.queries.item = function(opts) { WHERE tabItem.docstatus!=2 \ AND (ifnull(`tabItem`.`end_of_life`,"") in ("", "0000-00-00") \ OR `tabItem`.`end_of_life` > NOW()) \ - AND tabItem.%(key)s LIKE "%s" ' + (conditions - ? (" AND " + conditions.join(" AND ")) - : "") - + " LIMIT 50" + AND (tabItem.%(key)s LIKE \"%s\" OR tabItem.item_name LIKE \"%%%s\")' + + (conditions ? (" AND " + conditions.join(" AND ")) : "") + " LIMIT 50" } erpnext.queries.item_std = function() { diff --git a/stock/doctype/item/item.py b/stock/doctype/item/item.py index 55a0ae75ab5..2b2dbfe815e 100644 --- a/stock/doctype/item/item.py +++ b/stock/doctype/item/item.py @@ -239,8 +239,8 @@ class DocType(DocListController): vals = webnotes.conn.get_value("Item", self.doc.name, ["has_serial_no", "is_stock_item", "valuation_method"], as_dict=True) - if vals and (vals.has_serial_no != self.doc.has_serial_no or - vals.is_stock_item != self.doc.is_stock_item or + if vals and ((self.doc.is_stock_item == "No" and vals.is_stock_item == "Yes") or + vals.has_serial_no != self.doc.has_serial_no or vals.valuation_method != self.doc.valuation_method): if self.check_if_sle_exists(): webnotes.msgprint(_("As there are existing stock transactions for this \ diff --git a/stock/doctype/packing_slip/packing_slip.js b/stock/doctype/packing_slip/packing_slip.js index 4c20289af9b..5ece164c688 100644 --- a/stock/doctype/packing_slip/packing_slip.js +++ b/stock/doctype/packing_slip/packing_slip.js @@ -138,3 +138,17 @@ cur_frm.cscript.calc_net_total_pkg = function(doc, ps_detail) { refresh_many(['net_weight_pkg', 'net_weight_uom', 'gross_weight_uom', 'gross_weight_pkg']); } +var make_row = function(title,val,bold){ + var bstart = ''; var bend = ''; + return '