Refactor: Refactor payroll toggle to use same payroll permissions as …

…page (#571)
uktrade · Dec 9, 2024 · 2e62ef1 · 2e62ef1
1 parent 0fe4dea
commit 2e62ef1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/forecast/templates/forecast/edit/edit.html b/forecast/templates/forecast/edit/edit.html
@@ -1,4 +1,5 @@
 {% extends "forecast/edit/forecast_base.html" %}
+{% load payroll_permissions %}
 {% load util vite %}
 {% load breadcrumbs %}
 
@@ -41,14 +42,15 @@
     {% endblock %}
     {% block scripts %}
     {{ view.get_payroll_forecast_report|json_script:'payroll_forecast_data' }}
+    {% can_edit_payroll user as can_edit_payroll %}
     <script>
         window.actuals = {{ actuals|safe }};
         window.period_display = {{ period_display|safe }};
         window.table_data = {{ forecast_dump|safe }};
         window.cost_centre = {{ view.cost_centre_details.cost_centre_code|safe }};
         window.financial_year = {{ view.financial_year|safe }};
         window.payroll_forecast_data = JSON.parse(document.getElementById("payroll_forecast_data").textContent);
-        window.can_toggle_payroll = "{{ can_toggle_payroll }}";
+        window.can_edit_payroll = "{{ can_edit_payroll }}";
     </script>
     {% vite_dev_client %}
     {% vite_js 'src/index.jsx' %}

diff --git a/forecast/views/edit_forecast.py b/forecast/views/edit_forecast.py
@@ -528,7 +528,6 @@ def get_context_data(self, **kwargs):
         context["forecast_dump"] = forecast_dump
         context["actuals"] = actual_data
         context["period_display"] = period_display
-        context["can_toggle_payroll"] = self.request.user.is_superuser
 
         return context
 

diff --git a/front_end/src/Components/EditForecast/index.jsx b/front_end/src/Components/EditForecast/index.jsx
@@ -325,7 +325,7 @@ function EditForecast() {
 
     return (
         <Fragment>
-          {window.can_toggle_payroll === "True" && <ToggleCheckbox toggle={isPayrollEnabled} handler={handleIsPayrollEnabled} id="payroll-forecast" value="payroll" label="Toggle payroll forecast rows" />}
+          {window.can_edit_payroll === "True" && <ToggleCheckbox toggle={isPayrollEnabled} handler={handleIsPayrollEnabled} id="payroll-forecast" value="payroll" label="Toggle payroll forecast rows" />}
             {errorMessage != null &&
                 <div className="govuk-error-summary" aria-labelledby="error-summary-title" role="alert" tabIndex="-1" data-module="govuk-error-summary">
                   <h2 className="govuk-error-summary__title" id="error-summary-title">