From 488a92e5d16333967e7e1e1385df8bfc958d0978 Mon Sep 17 00:00:00 2001
From: Pramod Mahajan <pramodmahajan1411200@gmail.com>
Date: Tue, 10 Jun 2025 16:08:30 +0530
Subject: [PATCH] handled local project selection independently to avoid
 mutating global state

---
 src/pages/employee/EmployeeList.jsx | 70 +++++++++++++++--------------
 1 file changed, 37 insertions(+), 33 deletions(-)

diff --git a/src/pages/employee/EmployeeList.jsx b/src/pages/employee/EmployeeList.jsx
index e719d0f3..65eb6269 100644
--- a/src/pages/employee/EmployeeList.jsx
+++ b/src/pages/employee/EmployeeList.jsx
@@ -22,10 +22,11 @@ import {
 import EmployeeRepository from "../../repositories/EmployeeRepository";
 import ManageEmployee from "../../components/Employee/ManageEmployee";
 import ConfirmModal from "../../components/common/ConfirmModal";
+import {useSelector} from "react-redux";
 
 const EmployeeList = () => {
-  const { profile: loginUser } = useProfile();
-  const [selectedProject, setSelectedProject] = useState("");
+const selectedProjectId = useSelector((store) => store.localVariables.projectId);
+  const [ selectedProject, setSelectedProject ] = useState( () => selectedProjectId || "" );
   const { projects, loading: projectLoading } = useProjects();
   const [showInactive, setShowInactive] = useState(false);
   const Manage_Employee = useHasUserPermission(MANAGE_EMPLOYEES);
@@ -164,7 +165,7 @@ const EmployeeList = () => {
         exportToExcel(currentItems, "employees");
         break;
       case "pdf":
-        exportToPDF(currentItems, "employees"); // Pass the employeeList directly
+        exportToPDF(currentItems, "employees");
         break;
       case "print":
         printTable(tableRef.current);
@@ -190,6 +191,14 @@ const EmployeeList = () => {
     setIsDeleteModalOpen(true);
   };
 
+  const handleProjectSelection = (e) => {
+  setSelectedProject(e.target.value);
+  };
+  useEffect(() => {
+  setSelectedProject(selectedProjectId || "");
+}, [selectedProjectId]);
+
+
   return (
     <>
       {isCreateModalOpen && (
@@ -262,36 +271,31 @@ const EmployeeList = () => {
                         className="dataTables_length text-start"
                         id="DataTables_Table_0_length"
                       >
-                        <label>
-                          <select
-                            id="project-select"
-                            onChange={(e) => setSelectedProject(e.target.value)}
-                            name="DataTables_Table_0_length"
-                            aria-controls="DataTables_Table_0"
-                            className="form-select form-select-sm"
-                            value={selectedProject || ""}
-                          >
-                            {projectLoading ? (
-                              <option value="Loading">Loading...</option>
-                            ) : (
-                              <>
-                                <option value="">All Employees</option>
-                                {Array.isArray(projects) &&
-                                  projects
-                                    .filter((item) =>
-                                      loginUser?.projects?.includes(
-                                        String(item.id)
-                                      )
-                                    )
-                                    .map((item) => (
-                                      <option key={item.id} value={item.id}>
-                                        {item.name}
-                                      </option>
-                                    ))}
-                              </>
-                            )}
-                          </select>
-                        </label>
+                     <label>
+  <select
+    id="project-select"
+    onChange={handleProjectSelection}
+    name="DataTables_Table_0_length"
+    aria-controls="DataTables_Table_0"
+    className="form-select form-select-sm"
+    value={selectedProject}
+  >
+    {projectLoading ? (
+      <option value="Loading">Loading...</option>
+    ) : (
+      <>
+        <option value="">All Employees</option>
+        {Array.isArray(projects) &&
+          projects.map((item) => (
+            <option key={item.id} value={item.id}>
+              {item.name}
+            </option>
+          ))}
+      </>
+    )}
+  </select>
+</label>
+
                       </div>
                     </div>
                   </div>