[fix] tests
diff --git a/erpnext/shopping_cart/test_shopping_cart.py b/erpnext/shopping_cart/test_shopping_cart.py
index 17fc73e..a8308b2 100644
--- a/erpnext/shopping_cart/test_shopping_cart.py
+++ b/erpnext/shopping_cart/test_shopping_cart.py
@@ -197,8 +197,7 @@
def remove_all_items_from_cart(self):
quotation = _get_cart_quotation()
- quotation.set("items", [])
- quotation.save(ignore_permissions=True)
+ quotation.delete()
def create_user_if_not_exists(self, email, first_name = None):
if frappe.db.exists("User", email):
diff --git a/erpnext/utilities/doctype/contact/contact.py b/erpnext/utilities/doctype/contact/contact.py
index ae2fdfb..312b0b1 100644
--- a/erpnext/utilities/doctype/contact/contact.py
+++ b/erpnext/utilities/doctype/contact/contact.py
@@ -98,7 +98,7 @@
contact_name = frappe.db.get_value("Contact", {"email_id": doc.name})
if contact_name:
contact = frappe.get_doc("Contact", contact_name)
- contact.first_name = doc.first_name
- contact.last_name = doc.last_name
- contact.phone = doc.phone
+ for key in ("first_name", "last_name", "phone"):
+ if doc.get(key):
+ contact.set(key, doc.get(key))
contact.save(ignore_permissions=True)