From d35fed8793a723928b9b083cbf84bf0524fe131b Mon Sep 17 00:00:00 2001 From: mrPauwHaan <120259765+mrPauwHaan@users.noreply.github.com> Date: Sat, 2 Sep 2023 09:45:32 +0200 Subject: [PATCH] Deleting forced billing cycle based on not existing setting There is no setting called "billing cycle". Also, it generally makes no sense to force a monthly membership. This should be left to the organisation to fill in. --- non_profit/non_profit/doctype/membership/membership.py | 5 ----- 1 file changed, 5 deletions(-) diff --git a/non_profit/non_profit/doctype/membership/membership.py b/non_profit/non_profit/doctype/membership/membership.py index 8ab86910..9d77d6bb 100644 --- a/non_profit/non_profit/doctype/membership/membership.py +++ b/non_profit/non_profit/doctype/membership/membership.py @@ -54,11 +54,6 @@ def validate_membership_period(self): self.from_date = add_days(last_membership.to_date, 1) - if frappe.db.get_single_value("Non Profit Settings", "billing_cycle") == "Yearly": - self.to_date = add_years(self.from_date, 1) - else: - self.to_date = add_months(self.from_date, 1) - def on_payment_authorized(self, status_changed_to=None): if status_changed_to not in ("Completed", "Authorized"): return