fix: .format instead of f strings for translations

(cherry picked from commit 69eac61)
frappe · Dec 12, 2023 · aa21c5e · aa21c5e
1 parent 48fcac0
commit aa21c5e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/hrms/hr/doctype/job_opening/job_opening.py b/hrms/hr/doctype/job_opening/job_opening.py
@@ -63,10 +63,10 @@ def validate_past_future(self, field, is_past):
 		today = getdate()
 		if is_past:
 			if getdate(date) > today:
-				frappe.throw(_(f"{frappe.bold(label)} cannot be a future date"))
+				frappe.throw(_("{} cannot be a future date").format(frappe.bold(label)))
 		else:
 			if getdate(date) < today:
-				frappe.throw(_(f"{frappe.bold(label)} cannot be a past date"))
+				frappe.throw(_("{} cannot be a past date").format(frappe.bold(label)))
 
 	def validate_current_vacancies(self):
 		if not self.staffing_plan:

diff --git a/hrms/www/jobs/index.html b/hrms/www/jobs/index.html
@@ -87,7 +87,9 @@ <h3 class="mt-0 mb-10">{{ _("Job Openings") }}</h3>
                     </div>
                 </div>
             </div>
-            <p class="text-secondary mb-4 md-text">Showing {{ job_openings|length }} results</p>
+            <p class="text-secondary mb-4 md-text">
+                Showing {{ job_openings|length }} {{ "result" if job_openings|length == 1 else "results" }}
+            </p>
             {% for jo in job_openings %}
                 <div class="mb-8 card border p-6">
                     <div class="flex">