Merge pull request #498 from pateljannat/new-class-modal

feat: New Class Dialog

lms/lms/doctype/lms_class/lms_class.py

@@ -157,3 +157,22 @@ def authenticate():
 	}
 	response = requests.request("POST", authenticate_url, headers=headers)
 	return response.json()["access_token"]
+
+
+@frappe.whitelist()
+def create_class(title, start_date, end_date, description=None, name=None):
+	if name:
+		class_details = frappe.get_doc("LMS Class", name)
+	else:
+		class_details = frappe.get_doc({"doctype": "LMS Class"})
+
+	class_details.update(
+		{
+			"title": title,
+			"start_date": start_date,
+			"end_date": end_date,
+			"description": description,
+		}
+	)
+	class_details.save()
+	return class_details

lms/lms/web_form/class/class.js

@@ -1,3 +0,0 @@
-frappe.ready(function () {
-	// bind events here
-});

lms/lms/web_form/class/class.json

@@ -1,87 +0,0 @@
-{
- "accept_payment": 0,
- "allow_comments": 0,
- "allow_delete": 0,
- "allow_edit": 0,
- "allow_incomplete": 0,
- "allow_multiple": 0,
- "allow_print": 0,
- "amount": 0.0,
- "amount_based_on_field": 0,
- "apply_document_permissions": 0,
- "button_label": "Save",
- "creation": "2022-11-11 12:10:29.640675",
- "custom_css": "",
- "doc_type": "LMS Class",
- "docstatus": 0,
- "doctype": "Web Form",
- "idx": 0,
- "is_standard": 1,
- "list_columns": [],
- "login_required": 0,
- "max_attachment_size": 0,
- "modified": "2022-11-21 10:56:01.627821",
- "modified_by": "Administrator",
- "module": "LMS",
- "name": "class",
- "owner": "Administrator",
- "payment_button_label": "Buy Now",
- "published": 1,
- "route": "class",
- "show_attachments": 0,
- "show_list": 0,
- "show_sidebar": 0,
- "success_title": "",
- "success_url": "/classes",
- "title": "Class",
- "web_form_fields": [
-  {
-   "allow_read_on_all_link_options": 0,
-   "fieldname": "title",
-   "fieldtype": "Data",
-   "hidden": 0,
-   "label": "Title",
-   "max_length": 0,
-   "max_value": 0,
-   "read_only": 0,
-   "reqd": 1,
-   "show_in_filter": 0
-  },
-  {
-   "allow_read_on_all_link_options": 0,
-   "fieldname": "start_date",
-   "fieldtype": "Date",
-   "hidden": 0,
-   "label": "Start Date",
-   "max_length": 0,
-   "max_value": 0,
-   "read_only": 0,
-   "reqd": 1,
-   "show_in_filter": 0
-  },
-  {
-   "allow_read_on_all_link_options": 0,
-   "fieldname": "end_date",
-   "fieldtype": "Date",
-   "hidden": 0,
-   "label": "End Date",
-   "max_length": 0,
-   "max_value": 0,
-   "read_only": 0,
-   "reqd": 1,
-   "show_in_filter": 0
-  },
-  {
-   "allow_read_on_all_link_options": 0,
-   "fieldname": "description",
-   "fieldtype": "Small Text",
-   "hidden": 0,
-   "label": "Description",
-   "max_length": 0,
-   "max_value": 0,
-   "read_only": 0,
-   "reqd": 0,
-   "show_in_filter": 0
-  }
- ]
-}

lms/lms/web_form/class/class.py

@@ -1,6 +0,0 @@
-import frappe
-
-
-def get_context(context):
-	# do your magic here
-	pass

lms/patches.txt

@@ -52,3 +52,4 @@ lms.patches.v0_0.add_question_type #09-04-2023
 lms.patches.v0_0.add_evaluator_to_assignment #09-04-2023
 lms.patches.v0_0.convert_lesson_markdown_to_html #05-04-2023
 lms.patches.v0_0.convert_course_description_to_html
+execute:frappe.delete_doc("Web Form", "class", ignore_missing=True, force=True)

lms/public/css/style.css

@@ -1976,7 +1976,7 @@ select {
 	font-weight: 500;
 }
 
-.common-page-style .tooltip-content {
+.tooltip-content {
 	display: none;
 }
 

lms/public/js/common_functions.js

@@ -51,6 +51,10 @@ frappe.ready(() => {
 	$(".chapter-dropzone").each((i, el) => {
 		setSortable(el);
 	});
+
+	$("#create-class").click((e) => {
+		open_class_dialog(e);
+	});
 });
 
 const setSortable = (el) => {
@@ -385,3 +389,68 @@ const reorder_chapter = (e) => {
 		},
 	});
 };
+
+const open_class_dialog = (e) => {
+	this.class_dialog = new frappe.ui.Dialog({
+		title: __("New Class"),
+		fields: [
+			{
+				fieldtype: "Data",
+				label: __("Title"),
+				fieldname: "title",
+				reqd: 1,
+				default: class_info && class_info.title,
+			},
+			{
+				fieldtype: "Date",
+				label: __("Start Date"),
+				fieldname: "start_date",
+				reqd: 1,
+				default: class_info && class_info.start_date,
+			},
+			{
+				fieldtype: "Date",
+				label: __("End Date"),
+				fieldname: "end_date",
+				reqd: 1,
+				default: class_info && class_info.end_date,
+			},
+			{
+				fieldtype: "Small Text",
+				label: __("Description"),
+				fieldname: "description",
+				default: class_info && class_info.description,
+			},
+		],
+		primary_action_label: __("Save"),
+		primary_action: (values) => {
+			create_class(values);
+		},
+	});
+	this.class_dialog.show();
+};
+
+const create_class = (values) => {
+	frappe.call({
+		method: "lms.lms.doctype.lms_class.lms_class.create_class",
+		args: {
+			title: values.title,
+			start_date: values.start_date,
+			end_date: values.end_date,
+			description: values.description,
+			name: class_info && class_info.name,
+		},
+		callback: (r) => {
+			if (r.message) {
+				frappe.show_alert({
+					message: class_info
+						? __("Class Updated")
+						: __("Class Created"),
+					indicator: "green",
+				});
+				this.class_dialog.hide();
+				window.location.href = `/classes/${r.message.name}`;
+			}
+		},
+	});
+};

lms/www/classes/class.html

@@ -61,6 +61,11 @@
 <!-- Class Sections -->
 {% macro ClassSections(class_info, class_courses, class_students, published_courses) %}
 <div class="mt-4">
+
+	<button class="btn btn-secondary btn-sm pull-right" id="create-class">
+		{{ _("Edit") }}
+	</button>
+
 	<ul class="nav lms-nav" id="classes-tab">
 		<li class="nav-item">
 			<a class="nav-link active" data-toggle="tab" href="#courses">
@@ -280,7 +285,6 @@
 
 {%- block script %}
     {{ super() }}
-
 	{% if is_moderator %}
 	<script>
 		frappe.boot.user = {
@@ -294,6 +298,7 @@
 				return name.toLowerCase().replace(/ /g, "-");
 			}
 		}
+		let class_info = {{ class_info | json }};
 	</script>
 	{% endif %}
 

lms/www/classes/index.html

@@ -6,10 +6,10 @@
 {% block page_content %}
 <div class="common-page-style lms-page-style">
 	<div class="container">
-		{% if has_course_moderator_role() %}
+		{% if is_moderator %}
-		<a class="btn btn-secondary btn-sm pull-right" href="/class/new">
+		<button class="btn btn-secondary btn-sm pull-right" id="create-class">
 			{{ _("Create Class") }}
-		</a>
+		</button>
 		{% endif %}
 		<div class="course-home-headings"> {{ _("All Classes") }} </div>
 		{% if classes %}
@@ -27,6 +27,15 @@
 </div>
 {% endblock %}
 
+{%- block script %}
+    {{ super() }}
+    {{ include_script('controls.bundle.js') }}
+	{% if is_moderator %}
+	<script>
+		let class_info = null;
+	</script>
+	{% endif %}
+{% endblock %}
 
 {% macro ClassCards(classes) %}
 <div class="lms-card-parent">

lms/www/classes/index.py

@@ -1,10 +1,11 @@
 import frappe
 from frappe.utils import getdate
+from lms.lms.utils import has_course_moderator_role
 
 
 def get_context(context):
 	context.no_cache = 1
-
+	context.is_moderator = has_course_moderator_role()
 	context.classes = frappe.get_all(
 		"LMS Class",
 		{"end_date": [">=", getdate()]},