Changes in search employee when double error message is shown.

This commit is contained in:
Kartik sharma 2025-06-23 17:27:22 +05:30
parent 3283eb389a
commit 066678caba

View File

@ -6,13 +6,13 @@ import Avatar from "../../components/common/Avatar";
import Breadcrumb from "../../components/common/Breadcrumb"; import Breadcrumb from "../../components/common/Breadcrumb";
import ManageEmp from "../../components/Employee/ManageRole"; import ManageEmp from "../../components/Employee/ManageRole";
import { useEmployeesAllOrByProjectId } from "../../hooks/useEmployees"; import { useEmployeesAllOrByProjectId } from "../../hooks/useEmployees";
import { useProjects } from "../../hooks/useProjects"; import { useProjects } from "../../hooks/useProjects"; // Keep if you use projects elsewhere
import { useProfile } from "../../hooks/useProfile"; import { useProfile } from "../../hooks/useProfile"; // Keep if you use profile elsewhere
import { hasUserPermission } from "../../utils/authUtils"; import { hasUserPermission } from "../../utils/authUtils"; // Keep if you use this elsewhere
import { ITEMS_PER_PAGE, MANAGE_EMPLOYEES } from "../../utils/constants"; import { ITEMS_PER_PAGE, MANAGE_EMPLOYEES } from "../../utils/constants";
import { clearCacheKey } from "../../slices/apiDataManager"; import { clearCacheKey } from "../../slices/apiDataManager";
import { useHasUserPermission } from "../../hooks/useHasUserPermission"; import { useHasUserPermission } from "../../hooks/useHasUserPermission";
import SuspendEmp from "../../components/Employee/SuspendEmp"; import SuspendEmp from "../../components/Employee/SuspendEmp"; // Keep if you use SuspendEmp
import { import {
exportToCSV, exportToCSV,
exportToExcel, exportToExcel,
@ -24,13 +24,9 @@ import ManageEmployee from "../../components/Employee/ManageEmployee";
import ConfirmModal from "../../components/common/ConfirmModal"; import ConfirmModal from "../../components/common/ConfirmModal";
import { useSelector } from "react-redux"; import { useSelector } from "react-redux";
import eventBus from "../../services/eventBus"; import eventBus from "../../services/eventBus";
import { newlineChars } from "pdf-lib"; import { newlineChars } from "pdf-lib"; // Unused import, consider removing
const EmployeeList = () => { const EmployeeList = () => {
// const selectedProjectId = useSelector((store) => store.localVariables.projectId);
// const [selectedProject, setSelectedProject] = useState(() => selectedProjectId || "");
// const { projects, loading: projectLoading } = useProjects();
const selectedProjectId = useSelector( const selectedProjectId = useSelector(
(store) => store.localVariables.projectId (store) => store.localVariables.projectId
); );
@ -40,9 +36,8 @@ const EmployeeList = () => {
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( useEmployeesAllOrByProjectId(
showAllEmployees ? null : selectedProjectId, showAllEmployees ? null : selectedProjectId, // Use selectedProjectId here
showInactive showInactive
); );
@ -105,13 +100,12 @@ const EmployeeList = () => {
setEmployeeList(sorted); setEmployeeList(sorted);
const results = applySearchFilter(sorted, searchText); const results = applySearchFilter(sorted, searchText);
setFilteredData(results); setFilteredData(results);
} else if (!loading && !employees) { } else if (!loading && !employees) {
setEmployeeList([]); setEmployeeList([]);
setFilteredData([]); setFilteredData([]);
} }
}, [loading, employees, showAllEmployees, searchText, selectedProjectId]); }, [loading, employees, showAllEmployees, searchText, selectedProjectId]); // Add selectedProjectId to dependencies
const displayData = filteredData; const displayData = filteredData;
const indexOfLastItem = currentPage * itemsPerPage; const indexOfLastItem = currentPage * itemsPerPage;
const indexOfFirstItem = indexOfLastItem - itemsPerPage; const indexOfFirstItem = indexOfLastItem - itemsPerPage;
@ -140,7 +134,7 @@ const EmployeeList = () => {
} }
setShowModal(false); setShowModal(false);
clearCacheKey("employeeProfile"); clearCacheKey("employeeProfile");
recallEmployeeData(showInactive, showAllEmployees ? null : selectedProject); recallEmployeeData(showInactive, showAllEmployees ? null : selectedProjectId); // Use selectedProjectId here
}; };
const handleShow = () => setShowModal(true); const handleShow = () => setShowModal(true);
const handleClose = () => setShowModal(false); const handleClose = () => setShowModal(false);
@ -155,7 +149,7 @@ const EmployeeList = () => {
clearCacheKey("allInactiveEmployeeList"); clearCacheKey("allInactiveEmployeeList");
clearCacheKey("employeeProfile"); clearCacheKey("employeeProfile");
// Recall data based on current filter states after deletion to refresh the table // Recall data based on current filter states after deletion to refresh the table
recallEmployeeData(showInactive, showAllEmployees ? null : selectedProject); recallEmployeeData(showInactive, showAllEmployees ? null : selectedProjectId); // Use selectedProjectId here
setemployeeLodaing(false); setemployeeLodaing(false);
setIsDeleteModalOpen(false); setIsDeleteModalOpen(false);
}) })
@ -204,7 +198,7 @@ const EmployeeList = () => {
const handleToggle = (e) => { const handleToggle = (e) => {
setShowInactive(e.target.checked); setShowInactive(e.target.checked);
recallEmployeeData(e.target.checked, showAllEmployees ? null : selectedProject); recallEmployeeData(e.target.checked, showAllEmployees ? null : selectedProjectId); // Use selectedProjectId here
}; };
const handleAllEmployeesToggle = (e) => { const handleAllEmployeesToggle = (e) => {
@ -217,7 +211,7 @@ const EmployeeList = () => {
recallEmployeeData(false, selectedProjectId); recallEmployeeData(false, selectedProjectId);
} }
// recallEmployeeData(false, isChecked ? null : selectedProject); // recallEmployeeData(false, isChecked ? null : selectedProject); // This line also used selectedProject
}; };
const handleEmployeeModel = (id) => { const handleEmployeeModel = (id) => {
@ -230,10 +224,6 @@ const EmployeeList = () => {
setIsDeleteModalOpen(true); setIsDeleteModalOpen(true);
}; };
// useEffect(() => {
// setSelectedProject(selectedProjectId || "");
// }, [selectedProjectId]);
useEffect(() => { useEffect(() => {
if (!showAllEmployees) { if (!showAllEmployees) {
recallEmployeeData(showInactive, selectedProjectId); recallEmployeeData(showInactive, selectedProjectId);
@ -244,9 +234,9 @@ const EmployeeList = () => {
(msg) => { (msg) => {
if(employees.some((item) => item.id == msg.employeeId)){ if(employees.some((item) => item.id == msg.employeeId)){
setEmployeeList([]); setEmployeeList([]);
recallEmployeeData(showInactive); recallEmployeeData(showInactive, showAllEmployees ? null : selectedProjectId); // Use selectedProjectId here
} }
},[employees] },[employees, showInactive, showAllEmployees, selectedProjectId] // Add all relevant dependencies
); );
useEffect(() => { useEffect(() => {
@ -370,8 +360,6 @@ const EmployeeList = () => {
className="form-control form-control-sm" className="form-control form-control-sm"
placeholder="Search User" placeholder="Search User"
aria-controls="DataTables_Table_0" aria-controls="DataTables_Table_0"
// The 'disabled' attribute is intentionally removed here
// to keep the search box always active as requested.
/> />
</label> </label>
</div> </div>
@ -727,4 +715,4 @@ const EmployeeList = () => {
); );
}; };
export default EmployeeList; export default EmployeeList;