diff --git a/base/templates/base/action_type/action_type.html b/base/templates/base/action_type/action_type.html index 382540257..c818c40f6 100644 --- a/base/templates/base/action_type/action_type.html +++ b/base/templates/base/action_type/action_type.html @@ -17,7 +17,9 @@ {% endif %} {% if action_types %} - {% include 'base/action_type/action_type_view.html' %} +
+ {% include 'base/action_type/action_type_view.html' %} +
{% else %}
Page not found. 404. diff --git a/base/templates/base/action_type/action_type_form.html b/base/templates/base/action_type/action_type_form.html index eb1033466..7bdc5c4a1 100644 --- a/base/templates/base/action_type/action_type_form.html +++ b/base/templates/base/action_type/action_type_form.html @@ -1,4 +1,9 @@ {% load i18n %} +{% if messages and not hx_vals %} + + +{% endif %}

{% if act_id %} diff --git a/base/templates/base/action_type/action_type_view.html b/base/templates/base/action_type/action_type_view.html index 6b557ac6a..a57acd751 100644 --- a/base/templates/base/action_type/action_type_view.html +++ b/base/templates/base/action_type/action_type_view.html @@ -29,14 +29,14 @@ {% if perms.employee.change_actiontype %} + class="oh-btn oh-btn--light-bkg w-100" title="{% trans 'Edit' %}"> {% endif %} {% if perms.employee.delete_actiontype %}
+ hx-swap="outerHTML" class="w-100"> {% csrf_token %} +

+ {% if tag_id %} {% trans "Update Employee Tag" %} {% else %} {% trans "Create Employee Tag" %} {% endif %} +

+

- {% if form.errors %} - -
-
- {% for error in form.non_field_errors %} -
- {{ error }} -
- {% endfor %} -
-
- {% endif %} - +
+
+ {% for error in form.non_field_errors %} +
+ {{ error }} +
+ {% endfor %} +
+
{% endif %} - hx-encoding="multipart/form-data" - class="oh-profile-section" - > - {% csrf_token %} {{form.as_p}} - - +
+ {% csrf_token %} {{form.as_p}} + +
diff --git a/base/templates/base/employee_tag/employee_tag_view.html b/base/templates/base/employee_tag/employee_tag_view.html index 7ef1df6f0..1c473afca 100644 --- a/base/templates/base/employee_tag/employee_tag_view.html +++ b/base/templates/base/employee_tag/employee_tag_view.html @@ -5,7 +5,7 @@
{% trans "Title" %}
{% trans "Color" %}
- {% if perms.employee.delete_employeetag or perms.employee.delete_employeetag %} + {% if perms.employee.change_employeetag or perms.employee.delete_employeetag %}
{% trans "Actions" %}
{% endif %}
@@ -21,23 +21,23 @@ border-radius: 50%; display: inline-block;">
- {% if perms.employee.delete_employeetag or perms.employee.delete_employeetag %} + {% if perms.employee.change_employeetag or perms.employee.delete_employeetag %}
{% if perms.employee.change_employeetag %} + class="oh-btn oh-btn--light-bkg w-100" title="{% trans 'Edit' %}"> {% endif %} {% if perms.employee.delete_employeetag %} -
{% csrf_token %}
diff --git a/base/templates/base/tags/employee_tags.html b/base/templates/base/tags/employee_tags.html index e0cbe0e1d..e62e4d74b 100644 --- a/base/templates/base/tags/employee_tags.html +++ b/base/templates/base/tags/employee_tags.html @@ -22,7 +22,9 @@ {% endif %}
{% if employeetags %} - {% include 'base/employee_tag/employee_tag_view.html' %} +
+ {% include 'base/employee_tag/employee_tag_view.html' %} +
{% else %}
Page not found. 404. diff --git a/employee/models.py b/employee/models.py index e90227677..5d6a394b5 100644 --- a/employee/models.py +++ b/employee/models.py @@ -853,6 +853,10 @@ class Actiontype(HorillaModel): def __str__(self) -> str: return f"{self.title}" + class Meta: + verbose_name = _("Action Type") + verbose_name_plural = _("Action Types") + class DisciplinaryAction(HorillaModel): """ diff --git a/employee/policies.py b/employee/policies.py index e4b3b0d75..56ef1e789 100644 --- a/employee/policies.py +++ b/employee/policies.py @@ -122,7 +122,7 @@ def delete_policies(request): @login_required -@permission_required("employee.change_policy") +@permission_required("employee.add_policymultiplefile") def add_attachment(request): """ This method is used to add attachment to policy @@ -142,7 +142,7 @@ def add_attachment(request): @login_required -@permission_required("employee.delete_policy") +@permission_required("employee.delete_policymultiplefile") def remove_attachment(request): """ This method is used to remove the attachments @@ -170,12 +170,16 @@ def disciplinary_actions(request): This method is used to view all Disciplinaryaction """ employee = Employee.objects.filter(employee_user_id=request.user).first() - dis_actions = filtersubordinates( - request, DisciplinaryAction.objects.all(), "base.add_disciplinaryaction" - ).distinct() - dis_actions = ( - dis_actions | DisciplinaryAction.objects.filter(employee_id=employee).distinct() - ) + if request.user.has_perm("employee.view_disciplinaryaction"): + dis_actions = DisciplinaryAction.objects.all() + else: + dis_actions = filtersubordinates( + request, DisciplinaryAction.objects.all(), "base.add_disciplinaryaction" + ).distinct() + dis_actions = ( + dis_actions + | DisciplinaryAction.objects.filter(employee_id=employee).distinct() + ) form = DisciplinaryActionFilter(request.GET, queryset=dis_actions) page_number = request.GET.get("page") diff --git a/employee/templates/disciplinary_actions/disciplinary_nav.html b/employee/templates/disciplinary_actions/disciplinary_nav.html index 0281ae2c0..10bb65770 100644 --- a/employee/templates/disciplinary_actions/disciplinary_nav.html +++ b/employee/templates/disciplinary_actions/disciplinary_nav.html @@ -10,100 +10,101 @@ onsubmit="event.preventDefault()">
- {% if perms.employee.add_disciplinaryaction %} + {% if data %} -
-
- - -
-
- -