From 59e853a012968ed237b22efc7eb4dfc5b03bb03a Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Mon, 25 Oct 2021 11:52:08 +0530
Subject: [PATCH] fix: removed unnecessary lines

---
 school/hooks.py                    |   2 +-
 school/lms/widgets/CourseCard.html | 166 +++++++++--------------------
 2 files changed, 49 insertions(+), 119 deletions(-)

diff --git a/school/hooks.py b/school/hooks.py
index 3494ae78..92966398 100644
--- a/school/hooks.py
+++ b/school/hooks.py
@@ -164,7 +164,7 @@ jinja = {
 
 ## Specify the extension to be used to control what scripts and stylesheets
 ## to be included in lesson pages. The specified value must be be a
-## subclass of school.v.plugins.PageExtension
+## subclass of school.plugins.PageExtension
 # school_lesson_page_extension = None
 
 #school_lesson_page_extensions = [
diff --git a/school/lms/widgets/CourseCard.html b/school/lms/widgets/CourseCard.html
index aab7ad52..614c7f6e 100644
--- a/school/lms/widgets/CourseCard.html
+++ b/school/lms/widgets/CourseCard.html
@@ -2,22 +2,19 @@
 {% set progress = frappe.utils.cint(membership.progress) %}
 <div class="common-card-style course-card">
 
-  <div class="course-image
-    {% if not course.image %}default-image{% endif %}"
-    {% if course.image %}style="background-image: url( {{ course.image }} );" {% endif %}>
-  <div class="course-tags">
-    {% for tag in course.get_tags() %}
-    <div class="course-card-pills">{{ tag }}</div>
-    {% endfor %}
-    {% if membership and not read_only %}
-    {% if progress < 100 %}
-    <div class="course-card-pills dark-pills ml-auto">{
-      { frappe.utils.rounded(progress) }}% {{ _("Completed") }}
+  <div class="course-image {% if not course.image %}default-image{% endif %}" {% if course.image %}
+    style="background-image: url( {{ course.image }} );" {% endif %}>
+    <div class="course-tags">
+      {% for tag in course.get_tags() %}
+      <div class="course-card-pills">{{ tag }}</div>
+      {% endfor %}
+      {% if membership and not read_only %}
+      {% if progress < 100 %}  <div class="course-card-pills dark-pills ml-auto">{{ frappe.utils.rounded(progress) }}%
+        {{ _("Completed") }}
     </div>
     {% else %}
-    <div class="course-card-pills dark-pills ml-auto">
-      <img src="/assets/community/icons/check.svg"> {{ _("Completed")}}
-    </div>
+    <div class="course-card-pills dark-pills ml-auto"> <img src="/assets/community/icons/check.svg"> {{ _("Completed")
+      }}</div>
     {% endif %}
     {% endif %}
   </div>
@@ -65,123 +62,56 @@
         {{ frappe.utils.flt(avg_rating, frappe.get_system_settings("float_precision") or 3) }}
       </span>
       {% endif %}
-      <<<<<<< HEAD:school/lms/widgets/CourseCard.html </div>
-        <div class="course-card-title">{{ course.title }}</div>
-        <div {% if not read_only %} class="mb-4" {% endif %}>
-          <span class="zindex">
-            {{ widgets.Avatar(member=course.get_instructor(), avatar_class="avatar-small") }}
-            <a class="button-links" href="{{ get_profile_url(course.get_instructor().username) }}">
-              <span class="course-instructor">
-                {{ course.get_instructor().full_name }}
-              </span>
-            </a>
-          </span>
-          <span class="course-student-count">
-            {% if course.get_students() | length %}
-            <span class="mr-4">
-              <img class="icon-background" src="/assets/school/icons/user.svg" />
-              {{ course.get_students() | length }}
-            </span> {% endif %}
-            {% set avg_rating = course.get_average_rating() %}
-            {% if avg_rating %}
-            <span class="">
-              <img class="icon-background" src="/assets/school/icons/rating.svg" />
-              {{ frappe.utils.flt(avg_rating, frappe.get_system_settings("float_precision") or 3) }}
-            </span>
-            {% endif %}
-          </span>
-        </div>
-
-        {% if read_only %}
-        <a class="stretched-link" href="/courses/{{ course.name }}"></a>
-        {% else %}
-        =======
     </span>
   </div>
-  >>>>>>> 866829af97608c49779499d9a2eba3999b24eed7:community/lms/widgets/CourseCard.html
 
   {% if read_only %}
   <a class="stretched-link" href="/courses/{{ course.name }}"></a>
   {% else %}
 
-  <<<<<<< HEAD:school/lms/widgets/CourseCard.html {% if certificate %} <div class="view-course-link is-default">
-    _("Get Certificate") <img class="ml-3" src="/assets/school/icons/black-arrow.svg" />
-</div>
-<a class="stretched-link" href="/courses/{{ course.name }}/{{ certificate }}"></a>
+  {% set lesson_index = course.get_lesson_index(membership.current_lesson) if membership and
+  membership.current_lesson else '1.1' %}
+  {% set query_parameter = "?batch=" + membership.batch if membership and
+  membership.batch else "" %}
+  {% set certificate = course.is_certified() %}
 
-{% elif course.enable_certification and progress == 100 %}
-<div class="view-course-link is-default" id="certification" data-course="{{ course.name }}">
-  {{ _("Get Certificate") }} <img class="ml-3" src="/assets/school/icons/black-arrow.svg" />
-</div>
+  {% if certificate %}
+  <div class="view-course-link is-default">
+    _("Get Certificate") <img class="ml-3" src="/assets/community/icons/black-arrow.svg" />
+  </div>
+  <a class="stretched-link" href="/courses/{{ course.name }}/{{ certificate }}"></a>
 
-{% elif progress == 100 %}
-<div class="view-course-link is-default">
-  {{ _("Course Completed") }} <img class="ml-3" src="/assets/school/icons/black-arrow.svg" />
-</div>
-<a class="stretched-link" href="/courses/{{ course.name }}"></a>
+  {% elif course.enable_certification and progress == 100 %}
+  <div class="view-course-link is-default" id="certification" data-course="{{ course.name }}">
+    {{ _("Get Certificate") }} <img class="ml-3" src="/assets/community/icons/black-arrow.svg" />
+  </div>
 
-{% elif course.upcoming %}
-<div class="view-course-link is-secondary border">
-  {{ _("Upcoming Course") }} <img class="ml-3" src="/assets/school/icons/black-arrow.svg" />
-</div>
-<a class="stretched-link" href="/courses/{{ course.name }}"></a>
+  {% elif progress == 100 %}
+  <div class="view-course-link is-default">
+    {{ _("Course Completed") }} <img class="ml-3" src="/assets/community/icons/black-arrow.svg" />
+  </div>
+  <a class="stretched-link" href="/courses/{{ course.name }}"></a>
 
-{% elif membership %}
-<div class="view-course-link is-primary">
-  {{ _("Continue Course") }} <img class="ml-3" src="/assets/school/icons/white-arrow.svg" />
-</div>
-<a class="stretched-link" href="{{ course.get_learn_url(lesson_index) }}{{ query_parameter }}"></a>
+  {% elif course.upcoming %}
+  <div class="view-course-link is-secondary border">
+    {{ _("Upcoming Course") }} <img class="ml-3" src="/assets/community/icons/black-arrow.svg" />
+  </div>
+  <a class="stretched-link" href="/courses/{{ course.name }}"></a>
 
-{% else %}
-<div class="view-course-link is-default">
-  {{ _("View Course") }} <img class="ml-3" src="/assets/school/icons/black-arrow.svg" />
-</div>
-<a class="stretched-link" href="/courses/{{ course.name }}"></a>
-=======
-{% set lesson_index = course.get_lesson_index(membership.current_lesson) if membership and
-membership.current_lesson else '1.1' %}
-{% set query_parameter = "?batch=" + membership.batch if membership and
-membership.batch else "" %}
-{% set certificate = course.is_certified() %}
+  {% elif membership %}
+  <div class="view-course-link is-primary">
+    {{ _("Continue Course") }} <img class="ml-3" src="/assets/community/icons/white-arrow.svg" />
+  </div>
+  <a class="stretched-link" href="{{ course.get_learn_url(lesson_index) }}{{ query_parameter }}"></a>
 
-{% if certificate %}
-<div class="view-course-link is-default">
-  _("Get Certificate") <img class="ml-3" src="/assets/community/icons/black-arrow.svg" />
-</div>
-<a class="stretched-link" href="/courses/{{ course.name }}/{{ certificate }}"></a>
+  {% else %}
+  <div class="view-course-link is-default">
+    {{ _("View Course") }} <img class="ml-3" src="/assets/community/icons/black-arrow.svg" />
+  </div>
+  <a class="stretched-link" href="/courses/{{ course.name }}"></a>
 
-{% elif course.enable_certification and progress == 100 %}
-<div class="view-course-link is-default" id="certification" data-course="{{ course.name }}">
-  {{ _("Get Certificate") }} <img class="ml-3" src="/assets/community/icons/black-arrow.svg" />
-</div>
-
-{% elif progress == 100 %}
-<div class="view-course-link is-default">
-  {{ _("Course Completed") }} <img class="ml-3" src="/assets/community/icons/black-arrow.svg" />
-</div>
-<a class="stretched-link" href="/courses/{{ course.name }}"></a>
-
-{% elif course.upcoming %}
-<div class="view-course-link is-secondary border">
-  {{ _("Upcoming Course") }} <img class="ml-3" src="/assets/community/icons/black-arrow.svg" />
-</div>
-<a class="stretched-link" href="/courses/{{ course.name }}"></a>
-
-{% elif membership %}
-<div class="view-course-link is-primary">
-  {{ _("Continue Course") }} <img class="ml-3" src="/assets/community/icons/white-arrow.svg" />
-</div>
-<a class="stretched-link" href="{{ course.get_learn_url(lesson_index) }}{{ query_parameter }}"></a>
->>>>>>> 866829af97608c49779499d9a2eba3999b24eed7:community/lms/widgets/CourseCard.html
-
-{% else %}
-<div class="view-course-link is-default">
-  {{ _("View Course") }} <img class="ml-3" src="/assets/community/icons/black-arrow.svg" />
-</div>
-<a class="stretched-link" href="/courses/{{ course.name }}"></a>
-
-{% endif %}
-{% endif %}
+  {% endif %}
+  {% endif %}
 </div>
 </div>
 
@@ -207,7 +137,7 @@ membership.batch else "" %}
     e.preventDefault();
     course = $(e.currentTarget).attr("data-course");
     frappe.call({
-      method: "school.lms.doctype.lms_certification.lms_certification.create_certificate",
+      method: "community.lms.doctype.lms_certification.lms_certification.create_certificate",
       args: {
         "course": course
       },