From 4d6f067cb79446b2035759ebe5a347c8e9590bbb Mon Sep 17 00:00:00 2001 From: Saurabh Date: Tue, 11 Apr 2017 16:12:58 +0530 Subject: [PATCH 1/2] [fix] set member only once --- foundation/erpnext_foundation/doctype/membership/membership.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/foundation/erpnext_foundation/doctype/membership/membership.py b/foundation/erpnext_foundation/doctype/membership/membership.py index 5ea494cd..4da73a60 100644 --- a/foundation/erpnext_foundation/doctype/membership/membership.py +++ b/foundation/erpnext_foundation/doctype/membership/membership.py @@ -22,7 +22,8 @@ class Membership(Document): )).insert(ignore_permissions=True) member_name = member.name - self.member = member_name + if self.get("__islocal"): + self.member = member_name # get last membership (if active) last_membership = foundation.get_last_membership() From b0ba24d72ca9b9dd9dd763f17e1fe4235edf6834 Mon Sep 17 00:00:00 2001 From: Saurabh Date: Tue, 11 Apr 2017 18:04:28 +0530 Subject: [PATCH 2/2] [fix] link service providers and members on membership subscription --- .../erpnext_foundation/doctype/membership/membership.py | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/foundation/erpnext_foundation/doctype/membership/membership.py b/foundation/erpnext_foundation/doctype/membership/membership.py index 4da73a60..6418802d 100644 --- a/foundation/erpnext_foundation/doctype/membership/membership.py +++ b/foundation/erpnext_foundation/doctype/membership/membership.py @@ -44,3 +44,12 @@ class Membership(Document): self.load_from_db() self.db_set('paid', 1) + self.link_member_with_service_provider() + + def link_member_with_service_provider(self): + service_provider = frappe.db.get_value("Service Provider", dict(owner=frappe.session.user)) + + if service_provider: + frappe.db.set_value("Service Provider", service_provider, "member", frappe.session.user, + update_modified=False) +