From 282b07dffcde0987fe1eb6e80dbd3acb79718a07 Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Thu, 19 May 2022 16:49:53 +0530
Subject: [PATCH] fix: format amount on course cards

---
 lms/hooks.py                    | 3 ++-
 lms/lms/utils.py                | 9 ++++++++-
 lms/lms/widgets/CourseCard.html | 2 +-
 3 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/lms/hooks.py b/lms/hooks.py
index 81783b67..99d428cf 100644
--- a/lms/hooks.py
+++ b/lms/hooks.py
@@ -189,7 +189,8 @@ jinja = {
         "lms.lms.utils.is_instructor",
         "lms.lms.utils.convert_number_to_character",
         "lms.lms.utils.get_signup_optin_checks",
-        "lms.lms.utils.get_popular_courses"
+        "lms.lms.utils.get_popular_courses",
+        "lms.lms.utils.format_amount"
     ],
     "filters": []
 }
diff --git a/lms/lms/utils.py b/lms/lms/utils.py
index 6e1120e7..7b550d51 100644
--- a/lms/lms/utils.py
+++ b/lms/lms/utils.py
@@ -1,6 +1,6 @@
 import re
 import frappe
-from frappe.utils import flt, cint, cstr, getdate, add_months
+from frappe.utils import flt, cint, cstr, getdate, add_months, fmt_money
 from lms.lms.md import markdown_to_html, find_macros
 import string
 from frappe import _
@@ -366,3 +366,10 @@ def get_evaluation_details(course, member=None):
         "request": request,
         "no_of_attempts": no_of_attempts
     })
+
+def format_amount(amount, currency):
+    amount_reduced = amount / 1000
+    if amount_reduced < 1:
+        return amount
+    precision = 0 if amount % 1000 == 0 else 1
+    return _("{0}K").format(fmt_money(amount_reduced, precision, currency))
diff --git a/lms/lms/widgets/CourseCard.html b/lms/lms/widgets/CourseCard.html
index 75c20ca6..05908ca7 100644
--- a/lms/lms/widgets/CourseCard.html
+++ b/lms/lms/widgets/CourseCard.html
@@ -68,7 +68,7 @@
 
     {% if course.paid_certificate %}
     <div class="certificate-price small mt-3" data-price="{{ course.price_certificate }}">
-        {{ _("Certificate Price: ") }} {{ frappe.utils.fmt_money(course.price_certificate, 2, course.currency) }}
+        {{ _("Certificate Price: {0}").format(format_amount(course.price_certificate, course.currency)) }}
     </div>
     {% endif %}