admin/marco.pms.web
1
0
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases Wiki Activity

The search functionality in the Employees section does not work correctly after applying a filter. #208

Merged
vikas.nale merged 2 commits from Kartik_Bug#508 into Issues_Jun_3W 2025-06-20 09:09:18 +00:00
Conversation 1 Commits 2 Files Changed 1 +31 -15
1 changed files with 31 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 96d45f0bd2 - Show all commits

42
src/pages/employee/EmployeeList.jsx
View File

@ -25,14 +25,24 @@ import ConfirmModal from "../../components/common/ConfirmModal";
import { useSelector } from "react-redux"; import { useSelector } from "react-redux";
const EmployeeList = () => { const EmployeeList = () => {
const selectedProjectId = useSelector((store) => store.localVariables.projectId); // const selectedProjectId = useSelector((store) => store.localVariables.projectId);
const [selectedProject, setSelectedProject] = useState(() => selectedProjectId || ""); // const [selectedProject, setSelectedProject] = useState(() => selectedProjectId || "");
const { projects, loading: projectLoading } = useProjects(); // const { projects, loading: projectLoading } = useProjects();
const selectedProjectId = useSelector(
(store) => store.localVariables.projectId
);
const [showInactive, setShowInactive] = useState(false); const [showInactive, setShowInactive] = useState(false);
const [showAllEmployees, setShowAllEmployees] = useState(false); const [showAllEmployees, setShowAllEmployees] = useState(false);
const Manage_Employee = useHasUserPermission(MANAGE_EMPLOYEES); const Manage_Employee = useHasUserPermission(MANAGE_EMPLOYEES);
const { employees, loading, setLoading, error, recallEmployeeData } = const { employees, loading, setLoading, error, recallEmployeeData } =
useEmployeesAllOrByProjectId(showAllEmployees ? null : selectedProject, showInactive); // useEmployeesAllOrByProjectId(showAllEmployees ? null : selectedProject, showInactive);
useEmployeesAllOrByProjectId(
showAllEmployees ? null : selectedProjectId,
showInactive
);
const [employeeList, setEmployeeList] = useState([]); const [employeeList, setEmployeeList] = useState([]);
const [modelConfig, setModelConfig] = useState(); const [modelConfig, setModelConfig] = useState();
@ -98,7 +108,8 @@ const EmployeeList = () => {
setEmployeeList([]); setEmployeeList([]);
setFilteredData([]); setFilteredData([]);
} }
}, [loading, employees, showAllEmployees, searchText, selectedProject]); }, [loading, employees, showAllEmployees, searchText, selectedProjectId]);
const displayData = filteredData; const displayData = filteredData;
const indexOfLastItem = currentPage * itemsPerPage; const indexOfLastItem = currentPage * itemsPerPage;
const indexOfFirstItem = indexOfLastItem - itemsPerPage; const indexOfFirstItem = indexOfLastItem - itemsPerPage;
@ -198,8 +209,13 @@ const EmployeeList = () => {
const isChecked = e.target.checked; const isChecked = e.target.checked;
setShowInactive(false); setShowInactive(false);
setShowAllEmployees(isChecked); setShowAllEmployees(isChecked);
if (isChecked) {
recallEmployeeData(false, null);
} else {
recallEmployeeData(false, selectedProjectId);
}
recallEmployeeData(false, isChecked ? null : selectedProject); // recallEmployeeData(false, isChecked ? null : selectedProject);
}; };
const handleEmployeeModel = (id) => { const handleEmployeeModel = (id) => {
@ -212,15 +228,15 @@ const EmployeeList = () => {
setIsDeleteModalOpen(true); setIsDeleteModalOpen(true);
}; };
const handleProjectSelection = (e) => { // useEffect(() => {
const newProjectId = e.target.value; // setSelectedProject(selectedProjectId || "");
setSelectedProject(newProjectId); // }, [selectedProjectId]);
setShowAllEmployees(false);
};
useEffect(() => { useEffect(() => {
setSelectedProject(selectedProjectId || ""); if (!showAllEmployees) {
}, [selectedProjectId]); recallEmployeeData(showInactive, selectedProjectId);
}
}, [selectedProjectId, showInactive, showAllEmployees, recallEmployeeData]);
return ( return (