mirror of
https://github.com/frappe/erpnext.git
synced 2026-05-28 17:34:47 +00:00
fix: Error due to commma in Pricing rule name
This commit is contained in:
@@ -247,7 +247,9 @@ def get_pricing_rule_for_item(args, price_list_rate=0, doc=None, for_validate=Fa
|
|||||||
if pricing_rules:
|
if pricing_rules:
|
||||||
rules = []
|
rules = []
|
||||||
|
|
||||||
|
print(pricing_rules, "$$$$$$")
|
||||||
for pricing_rule in pricing_rules:
|
for pricing_rule in pricing_rules:
|
||||||
|
print(pricing_rule)
|
||||||
if not pricing_rule: continue
|
if not pricing_rule: continue
|
||||||
|
|
||||||
if isinstance(pricing_rule, string_types):
|
if isinstance(pricing_rule, string_types):
|
||||||
@@ -280,7 +282,7 @@ def get_pricing_rule_for_item(args, price_list_rate=0, doc=None, for_validate=Fa
|
|||||||
|
|
||||||
item_details.has_pricing_rule = 1
|
item_details.has_pricing_rule = 1
|
||||||
|
|
||||||
item_details.pricing_rules = ','.join([d.pricing_rule for d in rules])
|
item_details.pricing_rules = frappe.as_json([d.pricing_rule for d in rules])
|
||||||
|
|
||||||
if not doc: return item_details
|
if not doc: return item_details
|
||||||
|
|
||||||
@@ -370,7 +372,7 @@ def set_discount_amount(rate, item_details):
|
|||||||
|
|
||||||
def remove_pricing_rule_for_item(pricing_rules, item_details, item_code=None):
|
def remove_pricing_rule_for_item(pricing_rules, item_details, item_code=None):
|
||||||
from erpnext.accounts.doctype.pricing_rule.utils import get_pricing_rule_items
|
from erpnext.accounts.doctype.pricing_rule.utils import get_pricing_rule_items
|
||||||
for d in pricing_rules.split(','):
|
for d in json.loads(pricing_rules):
|
||||||
if not d or not frappe.db.exists("Pricing Rule", d): continue
|
if not d or not frappe.db.exists("Pricing Rule", d): continue
|
||||||
pricing_rule = frappe.get_cached_doc('Pricing Rule', d)
|
pricing_rule = frappe.get_cached_doc('Pricing Rule', d)
|
||||||
|
|
||||||
|
|||||||
@@ -471,7 +471,7 @@ def apply_pricing_rule_on_transaction(doc):
|
|||||||
doc.set_missing_values()
|
doc.set_missing_values()
|
||||||
|
|
||||||
def get_applied_pricing_rules(item_row):
|
def get_applied_pricing_rules(item_row):
|
||||||
return (item_row.get("pricing_rules").split(',')
|
return (json.loads(item_row.get("pricing_rules"))
|
||||||
if item_row.get("pricing_rules") else [])
|
if item_row.get("pricing_rules") else [])
|
||||||
|
|
||||||
def get_product_discount_rule(pricing_rule, item_details, args=None, doc=None):
|
def get_product_discount_rule(pricing_rule, item_details, args=None, doc=None):
|
||||||
|
|||||||
Reference in New Issue
Block a user