fix: fixed the issue of duplicate join requests
Someone was spamming mon.school by generating thousands of join requests. Added to fix to avoid creating duplicate requests.
This commit is contained in:
@@ -65,11 +65,16 @@ def join_cohort(course, cohort, subgroup, invite_code):
|
|||||||
"doctype": "Cohort Join Request",
|
"doctype": "Cohort Join Request",
|
||||||
"cohort": cohort_doc.name,
|
"cohort": cohort_doc.name,
|
||||||
"subgroup": subgroup_doc.name,
|
"subgroup": subgroup_doc.name,
|
||||||
"email": frappe.session.user
|
"email": frappe.session.user,
|
||||||
|
"status": "Pending"
|
||||||
}
|
}
|
||||||
doc = frappe.get_doc(data)
|
# Don't insert duplicate records
|
||||||
doc.insert(ignore_permissions=True)
|
if frappe.db.exists(data):
|
||||||
return {"ok": True}
|
return {"ok": True, "status": "record found"}
|
||||||
|
else:
|
||||||
|
doc = frappe.get_doc(data)
|
||||||
|
doc.insert(ignore_permissions=True)
|
||||||
|
return {"ok": True, "status": "record created"}
|
||||||
|
|
||||||
@frappe.whitelist()
|
@frappe.whitelist()
|
||||||
def approve_cohort_join_request(join_request):
|
def approve_cohort_join_request(join_request):
|
||||||
|
|||||||
Reference in New Issue
Block a user