Merge pull request #463 from pateljannat/user-singles-issue

lms/hooks.py

@@ -287,6 +287,4 @@ profile_url_prefix = "/users/"
 
 signup_form_template = "lms.plugins.show_custom_signup"
 
-on_login = "lms.overrides.user.on_login"
-
 on_session_creation = "lms.overrides.user.on_session_creation"

lms/overrides/user.py

@@ -273,7 +273,6 @@ def sign_up(email, full_name, verify_terms, user_category):
 def set_country_from_ip(login_manager=None, user=None):
 	if not user and login_manager:
 		user = login_manager.user
-
 	user_country = frappe.db.get_value("User", user, "country")
 	# if user_country:
 	#    return
@@ -294,10 +293,6 @@ def get_country_code():
 	return
 
 
-def on_login(login_manager):
-	set_country_from_ip()
-
-
 def on_session_creation(login_manager):
 	if frappe.db.get_single_value("System Settings", "setup_complete"):
 		frappe.local.response["home_page"] = "/courses"

lms/patches.txt

@@ -44,3 +44,4 @@ lms.patches.v0_0.rename_instructor_role
 lms.patches.v0_0.change_course_creation_settings #12-12-2022
 lms.patches.v0_0.check_onboarding_status #21-12-2022
 lms.patches.v0_0.assignment_file_type
+lms.patches.v0_0.user_singles_issue #23-11-2022

lms/patches/v0_0/user_singles_issue.py

@@ -0,0 +1,10 @@
+import frappe
+
+
+def execute():
+	table = frappe.qb.DocType("Singles")
+	q = frappe.qb.from_(table).select(table.field).where(table.doctype == "User")
+	rows = q.run()
+
+	if len(rows):
+		frappe.db.delete("Singles", {"doctype": "User"})