From 19f5251736ecb97c0913f33008270d93614d00de Mon Sep 17 00:00:00 2001
From: Jannat Patel <pateljannat2308@gmail.com>
Date: Tue, 11 Jan 2022 19:27:12 +0530
Subject: [PATCH] fix: login button before job application

---
 school/job/doctype/job_opportunity/job_opportunity.py | 2 +-
 school/www/jobs/job.html                              | 8 ++++----
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/school/job/doctype/job_opportunity/job_opportunity.py b/school/job/doctype/job_opportunity/job_opportunity.py
index c47a06d0..db9d4475 100644
--- a/school/job/doctype/job_opportunity/job_opportunity.py
+++ b/school/job/doctype/job_opportunity/job_opportunity.py
@@ -13,8 +13,8 @@ class JobOpportunity(Document):
 @frappe.whitelist()
 def report(job, reason):
     system_managers = get_system_managers(only_name=True)
-    subject = _("User {0} has reported the job post {1}").format(user, job)
     user = frappe.db.get_value("User", frappe.session.user, "full_name")
+    subject = _("User {0} has reported the job post {1}").format(user, job)
     args = {
         "job": job,
         "job_url": get_link_to_form("Job Opportunity", job),
diff --git a/school/www/jobs/job.html b/school/www/jobs/job.html
index bf5a8331..42e37e49 100644
--- a/school/www/jobs/job.html
+++ b/school/www/jobs/job.html
@@ -11,14 +11,14 @@
       {% include "school/templates/job_card.html" %}
       <div class="p-4">
         <div>{{ _(job.description) }}</div>
-        {% if frappe.session.user != "Guest" %}
         <div class="d-flex justify-content-between mt-5">
+          {% if frappe.session.user != "Guest" %}
           <a class="button is-primary" href="{{ job.application_link }}">{{ _("Apply Now") }}</a>
           <div class="button is-secondary" id="report">{{ _("Report this post") }}</div>
+          {% else %}
+          <a class="button is-secondary dark-links" href="/login?redirect-to=/jobs/{{ job.name }}">Login to apply for this job.</a>
+          {% endif %}
         </div>
-        {% else %}
-        <p class="small alert alert-secondary text-center">Login to apply for this job.</p>
-        {% endif %}
       </div>
     </div>
   </div>