fix: fixed masked report logic on backend cr, temp hard code of 2024 …

…new reports in dropdown, test fix
bcgov · Feb 26, 2025 · f7f271c · f7f271c
1 parent 9bf2fa0
commit f7f271c
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 31 deletions.
diff --git a/backend/lcfs/web/api/compliance_report/services.py b/backend/lcfs/web/api/compliance_report/services.py
@@ -207,34 +207,31 @@ async def get_compliance_reports_paginated(
         )
 
     def _mask_report_status(self, reports: List) -> List:
-        recommended_statuses = {
-            ComplianceReportStatusEnum.Recommended_by_analyst.underscore_value(),
-            ComplianceReportStatusEnum.Recommended_by_manager.underscore_value(),
-        }
-
-        masked_reports = []
         for report in reports:
-            if (
-                isinstance(report, ComplianceReportViewSchema)
-                and report.report_status in recommended_statuses
-            ):
-                report.report_status = ComplianceReportStatusEnum.Submitted.value
-                report.report_status_id = None
-                masked_reports.append(report)
-            elif (
-                isinstance(report, ComplianceReportBaseSchema)
-                and report.current_status.status in recommended_statuses
-            ):
-                report.current_status.status = (
-                    ComplianceReportStatusEnum.Submitted.value
-                )
-                report.current_status.compliance_report_status_id = None
-
-                masked_reports.append(report)
-            else:
-                masked_reports.append(report)
-
-        return masked_reports
+            if isinstance(report, ComplianceReportViewSchema):
+                statuses = {
+                    ComplianceReportStatusEnum.Recommended_by_analyst.underscore_value(),
+                    ComplianceReportStatusEnum.Recommended_by_manager.underscore_value(),
+                }
+                if report.report_status in statuses:
+                    report.report_status, report.report_status_id = (
+                        ComplianceReportStatusEnum.Submitted.value,
+                        None,
+                    )
+            elif isinstance(report, ComplianceReportBaseSchema):
+                statuses = {
+                    ComplianceReportStatusEnum.Recommended_by_analyst.value,
+                    ComplianceReportStatusEnum.Recommended_by_manager.value,
+                }
+                if report.current_status.status in statuses:
+                    (
+                        report.current_status.status,
+                        report.current_status.compliance_report_status_id,
+                    ) = (
+                        ComplianceReportStatusEnum.Submitted.value,
+                        None,
+                    )
+        return reports
 
     @service_handler
     async def get_compliance_report_by_id(

diff --git a/frontend/src/views/ComplianceReports/components/NewComplianceReportButton.jsx b/frontend/src/views/ComplianceReports/components/NewComplianceReportButton.jsx
@@ -40,12 +40,20 @@ export const NewComplianceReportButton = forwardRef((props, ref) => {
 
   const isMenuOpen = Boolean(anchorEl)
 
+  // Temporarily disable future year compliance reports
+  // const filteredDates = () => {
+  //   const currentYear = new Date().getFullYear()
+  //   const yearAhead = currentYear + 1
+  //   return periods?.data.filter((item) => {
+  //     const effectiveYear = new Date(item.effectiveDate).getFullYear()
+  //     return effectiveYear <= yearAhead && effectiveYear >= 2024
+  //   })
+  // }
+
   const filteredDates = () => {
-    const currentYear = new Date().getFullYear()
-    const yearAhead = currentYear + 1
     return periods?.data.filter((item) => {
       const effectiveYear = new Date(item.effectiveDate).getFullYear()
-      return effectiveYear <= yearAhead && effectiveYear >= 2024
+      return effectiveYear === 2024
     })
   }
 

diff --git a/frontend/src/views/Users/AddEditUser/AddEditUser.jsx b/frontend/src/views/Users/AddEditUser/AddEditUser.jsx
@@ -72,7 +72,7 @@ export const AddEditUser = ({ userType }) => {
       hasRoles(roles.supplier) || orgName || userType === 'bceid'
         ? bceidTextFields(t)
         : idirTextFields(t),
-    [hasRoles, orgName, t]
+    [hasRoles, orgName, t, userType]
   )
   const status = watch('status')
   const readOnly = watch('readOnly')