frappe/frappe#478
diff --git a/erpnext/controllers/selling_controller.py b/erpnext/controllers/selling_controller.py
index 5e0ea65..a3d140b 100644
--- a/erpnext/controllers/selling_controller.py
+++ b/erpnext/controllers/selling_controller.py
@@ -59,9 +59,8 @@
shipping_amount = condition.shipping_amount
break
- self.doclist.append({
+ self.append("other_charges", {
"doctype": "Sales Taxes and Charges",
- "parentfield": "other_charges",
"charge_type": "Actual",
"account_head": shipping_rule.doc.account,
"cost_center": shipping_rule.doc.cost_center,