From fc0689ad0439818f882ad7fa6ce0a03fd2fa0c9e Mon Sep 17 00:00:00 2001 From: Horilla Date: Wed, 26 Jun 2024 14:19:56 +0530 Subject: [PATCH] [UPDT] ATTENDANCE: Notification redirect method updated to reverse method --- attendance/views.py | 11 ++++++----- attendance/views/requests.py | 12 +++++++----- attendance/views/views.py | 23 ++++++++++++++--------- 3 files changed, 27 insertions(+), 19 deletions(-) diff --git a/attendance/views.py b/attendance/views.py index fcc0e8061..016ba9f96 100644 --- a/attendance/views.py +++ b/attendance/views.py @@ -20,6 +20,7 @@ from django.core.paginator import Paginator from django.db.models import Q from django.http import HttpResponse, HttpResponseRedirect, JsonResponse from django.shortcuts import redirect, render +from django.urls import reverse from django.utils.translation import gettext_lazy as _ from django.views.decorators.http import require_http_methods @@ -1170,7 +1171,7 @@ def validate_bulk_attendance(request): verb_de=f"Ihre Anwesenheit für das Datum {attendance.attendance_date} wurde bestätigt", verb_es=f"Se ha validado su asistencia para la fecha {attendance.attendance_date}", verb_fr=f"Votre présence pour la date {attendance.attendance_date} est validée", - redirect="/attendance/view-my-attendance", + redirect=reverse("view-my-attendance"), icon="checkmark", ) return JsonResponse({"message": f"{attendance.employee_id} success"}) @@ -1199,7 +1200,7 @@ def validate_this_attendance(request, obj_id): verb_de=f"Deine Anwesenheit für das Datum {attendance.attendance_date} ist bestätigt.", verb_es=f"Se valida tu asistencia para la fecha {attendance.attendance_date}.", verb_fr=f"Votre présence pour la date {attendance.attendance_date} est validée.", - redirect="/attendance/view-my-attendance", + redirect=reverse("view-my-attendance"), icon="checkmark", ) return HttpResponseRedirect(request.META.get("HTTP_REFERER", "/")) @@ -1232,7 +1233,7 @@ def revalidate_this_attendance(request, obj_id): verb_de=f"{attendance.employee_id} beantragte eine Neubewertung der Teilnahme am {attendance.attendance_date}", verb_es=f"{attendance.employee_id} solicitó la validación nuevamente para la asistencia del {attendance.attendance_date}", verb_fr=f"{attendance.employee_id} a demandé une revalidation pour la présence du {attendance.attendance_date}", - redirect="/attendance/view-my-attendance", + redirect=reverse("view-my-attendance"), icon="refresh", ) return HttpResponseRedirect(request.META.get("HTTP_REFERER", "/")) @@ -1259,7 +1260,7 @@ def approve_overtime(request, obj_id): verb_de=f"Die Überstunden für den {attendance.attendance_date} wurden genehmigt.", verb_es=f"Se ha aprobado el tiempo extra de asistencia para el {attendance.attendance_date}.", verb_fr=f"Les heures supplémentaires pour la date {attendance.attendance_date} ont été approuvées.", - redirect="/attendance/attendance-overtime-view", + redirect=reverse("attendance-overtime-view"), icon="checkmark", ) return HttpResponseRedirect(request.META.get("HTTP_REFERER", "/")) @@ -1286,7 +1287,7 @@ def approve_bulk_overtime(request): verb_de=f"Überstunden für die Anwesenheit am {attendance.attendance_date} genehmigt", verb_es=f"Horas extra aprobadas para la asistencia del {attendance.attendance_date}", verb_fr=f"Heures supplémentaires approuvées pour la présence du {attendance.attendance_date}", - redirect="/attendance/attendance-overtime-view", + redirect=reverse("attendance-overtime-view"), icon="checkmark", ) diff --git a/attendance/views/requests.py b/attendance/views/requests.py index 451d80f8a..57493057f 100644 --- a/attendance/views/requests.py +++ b/attendance/views/requests.py @@ -13,6 +13,7 @@ from django.contrib import messages from django.http import HttpResponse, HttpResponseRedirect, JsonResponse from django.shortcuts import render from django.template.loader import render_to_string +from django.urls import reverse from django.utils.translation import gettext_lazy as _ from attendance.filters import AttendanceFilters, AttendanceRequestReGroup @@ -275,7 +276,8 @@ def attendance_request_changes(request, attendance_id): {user_last_name} el {attendance.attendance_date}", verb_fr=f"La demande de mise à jour de présence de {employee.employee_first_name}\ {user_last_name} pour le {attendance.attendance_date} a été créée", - redirect=f"/attendance/request-attendance-view?id={attendance.id}", + redirect=reverse("request-attendance-view") + + f"?id={attendance.id}", icon="checkmark-circle-outline", ) return HttpResponse( @@ -420,7 +422,7 @@ def approve_validate_attendance_request(request, attendance_id): para la fecha {attendance.attendance_date}", verb_fr=f"Votre demande de présence pour la date \ {attendance.attendance_date} est validée", - redirect=f"/attendance/request-attendance-view?id={attendance.id}", + redirect=reverse("request-attendance-view") + f"?id={attendance.id}", icon="checkmark-circle-outline", ) if attendance.employee_id.employee_work_info.reporting_manager_id: @@ -441,7 +443,7 @@ def approve_validate_attendance_request(request, attendance_id): {employee.employee_first_name} {user_last_name} para el {attendance.attendance_date}", verb_fr=f"La demande de présence de {employee.employee_first_name} \ {user_last_name} pour le {attendance.attendance_date} a été validée", - redirect=f"/attendance/request-attendance-view?id={attendance.id}", + redirect=reverse("request-attendance-view") + f"?id={attendance.id}", icon="checkmark-circle-outline", ) return HttpResponseRedirect(request.META.get("HTTP_REFERER", "/")) @@ -617,7 +619,7 @@ def bulk_approve_attendance_request(request): para la fecha {attendance.attendance_date}", verb_fr=f"Votre demande de présence pour la date \ {attendance.attendance_date} est validée", - redirect=f"/attendance/request-attendance-view?id={attendance.id}", + redirect=reverse("request-attendance-view") + f"?id={attendance.id}", icon="checkmark-circle-outline", ) if attendance.employee_id.employee_work_info.reporting_manager_id: @@ -638,7 +640,7 @@ def bulk_approve_attendance_request(request): {employee.employee_first_name} {user_last_name} para el {attendance.attendance_date}", verb_fr=f"La demande de présence de {employee.employee_first_name} \ {user_last_name} pour le {attendance.attendance_date} a été validée", - redirect=f"/attendance/request-attendance-view?id={attendance.id}", + redirect=reverse("request-attendance-view") + f"?id={attendance.id}", icon="checkmark-circle-outline", ) return HttpResponse("success") diff --git a/attendance/views/views.py b/attendance/views/views.py index 66045fdd1..edac86b56 100644 --- a/attendance/views/views.py +++ b/attendance/views/views.py @@ -25,6 +25,7 @@ from django.core.paginator import Paginator from django.db.models import ProtectedError from django.http import HttpResponse, HttpResponseRedirect, JsonResponse from django.shortcuts import redirect, render +from django.urls import reverse from django.utils.translation import gettext as __ from django.utils.translation import gettext_lazy as _ from django.views.decorators.http import require_http_methods @@ -1227,7 +1228,7 @@ def validate_bulk_attendance(request): verb_de=f"Ihre Anwesenheit für das Datum {attendance.attendance_date} wurde bestätigt", verb_es=f"Se ha validado su asistencia para la fecha {attendance.attendance_date}", verb_fr=f"Votre présence pour la date {attendance.attendance_date} est validée", - redirect=f"/attendance/view-my-attendance?id={attendance.id}", + redirect=reverse("view-my-attendance") + f"?id={attendance.id}", icon="checkmark", ) except (Attendance.DoesNotExist, OverflowError, ValueError): @@ -1264,7 +1265,7 @@ def validate_this_attendance(request, obj_id): verb_de=f"Deine Anwesenheit für das Datum {attendance.attendance_date} ist bestätigt.", verb_es=f"Se valida tu asistencia para la fecha {attendance.attendance_date}.", verb_fr=f"Votre présence pour la date {attendance.attendance_date} est validée.", - redirect=f"/attendance/view-my-attendance?id={attendance.id}", + redirect=reverse("view-my-attendance") + f"?id={attendance.id}", icon="checkmark", ) except (Attendance.DoesNotExist, ValueError): @@ -1303,7 +1304,7 @@ def revalidate_this_attendance(request, obj_id): para la asistencia del {attendance.attendance_date}", verb_fr=f"{attendance.employee_id} a demandé une revalidation pour la \ présence du {attendance.attendance_date}", - redirect=f"/attendance/view-my-attendance?id={attendance.id}", + redirect=reverse("view-my-attendance") + f"?id={attendance.id}", icon="refresh", ) return HttpResponseRedirect(request.META.get("HTTP_REFERER", "/")) @@ -1342,7 +1343,7 @@ def approve_overtime(request, obj_id): {attendance.attendance_date}.", verb_fr=f"Les heures supplémentaires pour la date\ {attendance.attendance_date} ont été approuvées.", - redirect=f"/attendance/attendance-overtime-view?id={attendance.id}", + redirect=reverse("attendance-overtime-view") + f"?id={attendance.id}", icon="checkmark", ) except (Attendance.DoesNotExist, OverflowError): @@ -1377,7 +1378,7 @@ def approve_bulk_overtime(request): {attendance.attendance_date}", verb_fr=f"Heures supplémentaires approuvées pour la présence du \ {attendance.attendance_date}", - redirect=f"/attendance/attendance-overtime-view?id={attendance.id}", + redirect=reverse("attendance-overtime-view") + f"?id={attendance.id}", icon="checkmark", ) except (Attendance.DoesNotExist, OverflowError, ValueError): @@ -1841,7 +1842,8 @@ def create_attendancerequest_comment(request, attendance_id): verb_de=f"{attendance.employee_id}s Anfrage zur Anwesenheit hat einen Kommentar erhalten.", verb_es=f"La solicitud de asistencia de {attendance.employee_id} ha recibido un comentario.", verb_fr=f"La demande de présence de {attendance.employee_id} a reçu un commentaire.", - redirect=f"/attendance/request-attendance-view?id={attendance.id}", + redirect=reverse("request-attendance-view") + + f"?id={attendance.id}", icon="chatbox-ellipses", ) elif ( @@ -1857,7 +1859,8 @@ def create_attendancerequest_comment(request, attendance_id): verb_de="Ihr Antrag auf Anwesenheit hat einen Kommentar erhalten.", verb_es="Tu solicitud de asistencia ha recibido un comentario.", verb_fr="Votre demande de présence a reçu un commentaire.", - redirect=f"/attendance/request-attendance-view?id={attendance.id}", + redirect=reverse("request-attendance-view") + + f"?id={attendance.id}", icon="chatbox-ellipses", ) else: @@ -1873,7 +1876,8 @@ def create_attendancerequest_comment(request, attendance_id): verb_de=f"{attendance.employee_id}s Anfrage zur Anwesenheit hat einen Kommentar erhalten.", verb_es=f"La solicitud de asistencia de {attendance.employee_id} ha recibido un comentario.", verb_fr=f"La demande de présence de {attendance.employee_id} a reçu un commentaire.", - redirect=f"/attendance/request-attendance-view?id={attendance.id}", + redirect=reverse("request-attendance-view") + + f"?id={attendance.id}", icon="chatbox-ellipses", ) else: @@ -1886,7 +1890,8 @@ def create_attendancerequest_comment(request, attendance_id): verb_de="Ihr Antrag auf Anwesenheit hat einen Kommentar erhalten.", verb_es="Tu solicitud de asistencia ha recibido un comentario.", verb_fr="Votre demande de présence a reçu un commentaire.", - redirect=f"/attendance/request-attendance-view?id={attendance.id}", + redirect=reverse("request-attendance-view") + + f"?id={attendance.id}", icon="chatbox-ellipses", ) return render(