From 06a922c3140e34207b2f293af4c94d61d43b5205 Mon Sep 17 00:00:00 2001
From: Horilla <teamhorilla@gmail.com>
Date: Mon, 19 Feb 2024 15:28:10 +0530
Subject: [PATCH] [ADD] EMPLOYEE: Dynamic Group by on employee

---
 employee/filters.py                           | 14 ++---
 .../employee_personal_info/group_by.html      | 62 ++++++++++++++++++-
 employee/views.py                             | 25 ++++----
 3 files changed, 81 insertions(+), 20 deletions(-)

diff --git a/employee/filters.py b/employee/filters.py
index f17c4efc1..db1e56f83 100644
--- a/employee/filters.py
+++ b/employee/filters.py
@@ -271,13 +271,13 @@ class EmployeeReGroup:
 
     fields = [
         ("", "select"),
-        ("employee_work_info.job_position_id", "Job Position"),
-        ("employee_work_info.department_id", "Department"),
-        ("employee_work_info.shift_id", "Shift"),
-        ("employee_work_info.work_type_id", "Work Type"),
-        ("employee_work_info.job_role_id", "Job Role"),
-        ("employee_work_info.reporting_manager_id", "Reporting Manager"),
-        ("employee_work_info.company_id", "Company"),
+        ("employee_work_info__job_position_id", "Job Position"),
+        ("employee_work_info__department_id", "Department"),
+        ("employee_work_info__shift_id", "Shift"),
+        ("employee_work_info__work_type_id", "Work Type"),
+        ("employee_work_info__job_role_id", "Job Role"),
+        ("employee_work_info__reporting_manager_id", "Reporting Manager"),
+        ("employee_work_info__company_id", "Company"),
     ]
 
 
diff --git a/employee/templates/employee_personal_info/group_by.html b/employee/templates/employee_personal_info/group_by.html
index 9182a3cc2..1c56bd279 100644
--- a/employee/templates/employee_personal_info/group_by.html
+++ b/employee/templates/employee_personal_info/group_by.html
@@ -23,8 +23,7 @@
 </div>
 <div class="oh-checkpoint-badge text-danger mb-2" id="selectedShow"></div>
 <div class="oh-card">
-    {% dynamic_regroup data by field as employee_grouper %}
-    {% for employee_list in employee_grouper %}
+    {% for employee_list in data %}
     <div class="oh-accordion-meta">
       <div class="oh-accordion-meta__item">
         <div class="oh-accordion-meta__header" onclick='$(this).toggleClass("oh-accordion-meta__header--show");'>
@@ -235,6 +234,65 @@
                 </div>
               </div>
           </div>
+          <div class="oh-pagination">
+            <span class="oh-pagination__page">
+              {% trans "Page" %} {{ employee_list.list.number }} {% trans "of" %} {{ employee_list.list.paginator.num_pages }}.
+            </span>
+            <nav class="oh-pagination__nav">
+              <div class="oh-pagination__input-container me-3">
+              <span class="oh-pagination__label me-1">{% trans "Page" %}</span>
+              <input
+                type="number"
+                name="{{employee_list.dynamic_name}}"
+                class="oh-pagination__input"
+                value="{{employee_list.list.number}}"
+                hx-get="{% url 'employee-filter-view' %}?{{pd}}"
+                hx-target="#view-container"
+                min="1"
+              />
+              <span class="oh-pagination__label"
+                >{% trans "of" %} {{employee_list.list.paginator.num_pages}}</span
+              >
+              </div>
+              <ul class="oh-pagination__items">
+              {% if employee_list.list.has_previous %}
+              <li class="oh-pagination__item oh-pagination__item--wide">
+                <a
+                hx-target="#view-container"
+                hx-get="{% url 'employee-filter-view' %}?{{pd}}&{{employee_list.dynamic_name}}=1"
+                class="oh-pagination__link"
+                >{% trans "First" %}</a
+                >
+              </li>
+              <li class="oh-pagination__item oh-pagination__item--wide">
+                <a
+                hx-target="#view-container"
+                hx-get="{% url 'employee-filter-view' %}?{{pd}}&{{employee_list.dynamic_name}}={{ employee_list.list.previous_page_number }}"
+                class="oh-pagination__link"
+                >{% trans "Previous" %}</a
+                >
+              </li>
+              {% endif %} {% if employee_list.list.has_next %}
+              <li class="oh-pagination__item oh-pagination__item--wide">
+                <a
+                hx-target="#view-container"
+                hx-get="{% url 'employee-filter-view' %}?{{pd}}&{{employee_list.dynamic_name}}={{ employee_list.list.next_page_number }}"
+                class="oh-pagination__link"
+                >{% trans "Next" %}</a
+                >
+              </li>
+              <li class="oh-pagination__item oh-pagination__item--wide">
+                <a
+                hx-target="#view-container"
+                hx-get="{% url 'employee-filter-view' %}?{{pd}}&{{employee_list.dynamic_name}}={{ employee_list.list.paginator.num_pages }}"
+                class="oh-pagination__link"
+                >{% trans "Last" %}</a
+                >
+              </li>
+              {% endif %}
+              </ul>
+            </nav>
+          </div>
         </div>
       </div>
     </div>
diff --git a/employee/views.py b/employee/views.py
index c85aaddb9..d1d600590 100755
--- a/employee/views.py
+++ b/employee/views.py
@@ -38,6 +38,7 @@ from attendance.methods.group_by import group_by_queryset
 from attendance.models import Attendance, AttendanceOverTime
 from employee.methods.methods import get_ordered_badge_ids
 from horilla.filters import HorillaPaginator
+from horilla_audit.models import AccountBlockUnblock
 from leave.models import LeaveRequest
 from notifications.signals import notify
 from horilla.decorators import (
@@ -46,6 +47,7 @@ from horilla.decorators import (
     login_required,
     hx_request_required,
     manager_can_enter,
+    logger,
 )
 from base.models import (
     Department,
@@ -236,7 +238,10 @@ def employee_view_individual(request, obj_id, **kwargs):
     instances = LeaveRequest.objects.filter(employee_id=employee)
     leave_request_ids = json.dumps([instance.id for instance in instances])
     employee_leaves = employee.available_leave.all()
-
+    enabled_block_unblock = (
+        AccountBlockUnblock.objects.exists()
+        and AccountBlockUnblock.objects.first().is_enabled
+    )
     return render(
         request,
         "employee/view/individual.html",
@@ -244,6 +249,7 @@ def employee_view_individual(request, obj_id, **kwargs):
             "employee": employee,
             "employee_leaves": employee_leaves,
             "leave_request_ids": leave_request_ids,
+            "enabled_block_unblock":enabled_block_unblock,
         },
     )
 
@@ -1615,16 +1621,16 @@ def employee_filter_view(request):
     if view == "list":
         template = "employee_personal_info/employee_list.html"
     if field != "" and field is not None:
-        field_copy = field.replace(".", "__")
-        employees = employees.order_by(field_copy)
-        employees = employees.exclude(employee_work_info__isnull=True)
+        employees = group_by_queryset(employees,field,page_number,"page")
         template = "employee_personal_info/group_by.html"
-    employees = sortby(request, employees, "orderby")
+    else:
+        employees = sortby(request, employees, "orderby")
+        employees = paginator_qry(employees, page_number)
     return render(
         request,
         template,
         {
-            "data": paginator_qry(employees, page_number),
+            "data": employees,
             "f": EmployeeFilter(request.GET),
             "pd": previous_data,
             "field": field,
@@ -1844,7 +1850,7 @@ def employee_archive(request, obj_id):
     if save:
         employee.save()
         messages.success(request, message)
-        return HttpResponse("<script>window.location.reload();</script>")
+        return HttpResponseRedirect(request.META.get("HTTP_REFERER", "/"))
     else:
         return render(
             request,
@@ -2280,9 +2286,7 @@ def work_info_import(request):
                     employee_work_info.basic_salary = basic_salary
                     employee_work_info.save()
             except Exception as e:
-                print("++++++++++++++++++++++++++++++++++++")
-                print(e)
-                print("++++++++++++++++++++++++++++++++++++")
+                logger.error(e)
                 error_lists.append(work_info)
         if error_lists:
             res = defaultdict(list)
@@ -2295,7 +2299,6 @@ def work_info_import(request):
             response = HttpResponse(content_type="application/ms-excel")
             response["Content-Disposition"] = 'attachment; filename="ImportError.xlsx"'
             data_frame.to_excel(response, index=False)
-            print("HEREEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEEE")
             return response
         return HttpResponse("Imported successfully")
     return response