From 64b27ea52c6356ac0196bfb7f4bc299f801ff11c Mon Sep 17 00:00:00 2001
From: Faheed <faheed@tenhal.sa>
Date: Mon, 15 Dec 2025 17:28:09 +0300
Subject: [PATCH] interview rersult implementation fixed'

---
 recruitment/views.py                       |  6 +--
 templates/interviews/interview_detail.html | 25 +++++++++-
 templates/recruitment/settings_list.html   | 55 +---------------------
 3 files changed, 28 insertions(+), 58 deletions(-)

diff --git a/recruitment/views.py b/recruitment/views.py
index bfa2d13..d133771 100644
--- a/recruitment/views.py
+++ b/recruitment/views.py
@@ -4290,12 +4290,10 @@ def update_interview_result(request,slug):
     form = InterviewResultForm(request.POST, instance=interview)
     
     if form.is_valid():
-       
-        interview.save(update_fields=['interview_result', 'result_comments'])
-
+        
         form.save()  # Saves form data
 
-        messages.success(request, _("Interview cancelled successfully."))
+        messages.success(request, _(f"Interview result updated successfully to {interview.interview_result}."))
         return redirect("interview_detail", slug=schedule.slug)
     else:
         error_list = [
diff --git a/templates/interviews/interview_detail.html b/templates/interviews/interview_detail.html
index 00f14dc..7383fb3 100644
--- a/templates/interviews/interview_detail.html
+++ b/templates/interviews/interview_detail.html
@@ -523,11 +523,34 @@
                     </button>
 
                     {% if schedule.status == 'completed' %}
-                        <button type="button" class="btn btn-outline-success btn-sm"
+                        <button type="button" class="btn btn-outline-success btn-sm w-100"
                                 data-bs-toggle="modal"
                                 data-bs-target="#resultModal">
                             <i class="fas fa-check-circle me-1"></i> {% trans "Update Result" %}
                         </button>
+                    <div class="w-100 text-center">
+                        {% if interview.interview_result %}
+                            {% trans 'Interview Result : ' %}
+                            {% if interview.interview_result == 'passed' %}
+                                <span class="badge  bg-success text-white p-1">
+                                    <i class="fas fa-check-circle me-1"></i> {{ interview.interview_result }}
+                                </span>
+                            {% elif interview.interview_result == 'failed' %}
+                                <span class="badge  bg-danger text-white p-1 fs-5">
+                                    <i class="fas fa-times-circle me-1"></i> {{ interview.interview_result }}
+                                </span>
+                            {% else %}
+                                <span class="badge  bg-info text-dark p-1">
+                                    <i class="fas fa-info-circle me-1"></i> {{ interview.interview_result }}
+                                </span>
+                            {% endif %}
+                        {% else %}
+                            <span class="badge rounded-pill bg-secondary text-white">
+                                {% trans "No Result Yet" %}
+                            </span>
+                        {% endif %}
+                        </div>
+
                     {% endif %}
                 </div>
             </div>
diff --git a/templates/recruitment/settings_list.html b/templates/recruitment/settings_list.html
index 3eef67a..527daf5 100644
--- a/templates/recruitment/settings_list.html
+++ b/templates/recruitment/settings_list.html
@@ -109,49 +109,7 @@
                         </div>
 
                         <!-- Pagination -->
-                        {% if page_obj.has_other_pages %}
-                            <nav aria-label="Settings pagination">
-                                <ul class="pagination justify-content-center">
-                                    {% if page_obj.has_previous %}
-                                        <li class="page-item">
-                                            <a class="page-link" href="?page=1{% if search_query %}&q={{ search_query }}{% endif %}">
-                                                <i class="fas fa-angle-double-left"></i>
-                                            </a>
-                                        </li>
-                                        <li class="page-item">
-                                            <a class="page-link" href="?page={{ page_obj.previous_page_number }}{% if search_query %}&q={{ search_query }}{% endif %}">
-                                                <i class="fas fa-angle-left"></i>
-                                            </a>
-                                        </li>
-                                    {% endif %}
-
-                                    {% for num in page_obj.paginator.page_range %}
-                                        {% if page_obj.number == num %}
-                                            <li class="page-item active">
-                                                <span class="page-link">{{ num }}</span>
-                                            </li>
-                                        {% elif num > page_obj.number|add:'-3' and num < page_obj.number|add:'3' %}
-                                            <li class="page-item">
-                                                <a class="page-link" href="?page={{ num }}{% if search_query %}&q={{ search_query }}{% endif %}">{{ num }}</a>
-                                            </li>
-                                        {% endif %}
-                                    {% endfor %}
-
-                                    {% if page_obj.has_next %}
-                                        <li class="page-item">
-                                            <a class="page-link" href="?page={{ page_obj.next_page_number }}{% if search_query %}&q={{ search_query }}{% endif %}">
-                                                <i class="fas fa-angle-right"></i>
-                                            </a>
-                                        </li>
-                                        <li class="page-item">
-                                            <a class="page-link" href="?page={{ page_obj.paginator.num_pages }}{% if search_query %}&q={{ search_query }}{% endif %}">
-                                                <i class="fas fa-angle-double-right"></i>
-                                            </a>
-                                        </li>
-                                    {% endif %}
-                                </ul>
-                            </nav>
-                        {% endif %}
+                        {% include "includes/paginator.html" %}
                     {% else %}
                         <div class="text-center py-5">
                             <i class="fas fa-cog fa-3x text-muted mb-3"></i>
@@ -169,16 +127,7 @@
                         </div>
                     {% endif %}
 
-                    <div class="text-center mt-3">
-                        <small class="text-muted">
-                            {% if page_obj %}
-                                Showing {{ page_obj.start_index }}-{{ page_obj.end_index }} of {{ page_obj.paginator.count }} settings
-                                {% if search_query %}
-                                    (filtered by: "{{ search_query }}")
-                                {% endif %}
-                            {% endif %}
-                        </small>
-                    </div>
+                    
                 </div>
             </div>
             </div>