Merge pull request #1491 from pateljannat/issues-103

fix: only assign lms roles to admin

lms/install.py

@@ -11,7 +11,7 @@ def after_install():
 def after_sync():
 	create_lms_roles()
 	set_default_certificate_print_format()
-	add_all_roles_to("Administrator")
+	give_lms_roles_to_admin()
 
 
 def before_uninstall():
@@ -172,3 +172,15 @@ def create_batch_source():
 			doc = frappe.new_doc("LMS Source")
 			doc.source = source
 			doc.save()
+
+
+def give_lms_roles_to_admin():
+	roles = ["Course Creator", "Moderator", "Batch Evaluator"]
+	for role in roles:
+		if not frappe.db.exists("Has Role", {"parent": "Administrator", "role": role}):
+			doc = frappe.new_doc("Has Role")
+			doc.parent = "Administrator"
+			doc.parenttype = "User"
+			doc.parentfield = "roles"
+			doc.role = role
+			doc.save()