src/components/Activities/AttendcesLogs.jsx

@@ -6,11 +6,7 @@ import RenderAttendanceStatus from "./RenderAttendanceStatus";
 import { useSelector, useDispatch } from "react-redux";
 import { fetchAttendanceData, setAttendanceData } from "../../slices/apiSlice/attedanceLogsSlice"; // Make sure setAttendanceData is correctly imported
 import DateRangePicker from "../common/DateRangePicker";
-import { clearCacheKey, getCachedData } from "../../slices/apiDataManager";
 import eventBus from "../../services/eventBus";
-import AttendanceRepository from "../../repositories/AttendanceRepository";
-import { useAttendancesLogs } from "../../hooks/useAttendance";
-import { queryClient } from "../../layouts/AuthLayout";
 
 // Custom hook for pagination
 const usePagination = (data, itemsPerPage) => {
@@ -20,6 +16,9 @@ const usePagination = (data, itemsPerPage) => {
   const maxPage = Math.ceil(totalItems / itemsPerPage);
 
   const currentItems = useMemo(() => {
+    if (!Array.isArray(data) || data.length === 0) {
+      return [];
+    }
     const startIndex = (currentPage - 1) * itemsPerPage;
     const endIndex = startIndex + itemsPerPage;
     return data.slice(startIndex, endIndex);
@@ -61,8 +60,6 @@ const AttendanceLog = ({
     endDate: defaultEndDate 
   });
   const dispatch = useDispatch();
-  const [loading, setLoading] = useState(false);
-  const [showPending,setShowPending] = useState(false)
 
   const { data: attendanceLogsData, loading: logsLoading, isFetching: logsFetching } = useSelector(
     (state) => state.attendanceLogs
@@ -70,6 +67,7 @@ const AttendanceLog = ({
 
   const [isRefreshing, setIsRefreshing] = useState(false); // Local state for refresh spinner
 
+  // Memoize today and yesterday dates to prevent re-creation on every render
   const today = useMemo(() => {
     const d = new Date();
     d.setHours(0, 0, 0, 0);
@@ -82,22 +80,20 @@ const AttendanceLog = ({
     d.setHours(0, 0, 0, 0); // Set to start of day for accurate comparison
     return d;
   }, []);
-  const yesterday = new Date();
-  yesterday.setDate(yesterday.getDate() - 1);
 
-  const isSameDay = (dateStr) => {
+  const isSameDay = useCallback((dateStr) => {
     if (!dateStr) return false;
     const d = new Date(dateStr);
     d.setHours(0, 0, 0, 0);
     return d.getTime() === today.getTime();
-  };
+  }, [today]);
 
-  const isBeforeToday = (dateStr) => {
+  const isBeforeToday = useCallback((dateStr) => {
     if (!dateStr) return false;
     const d = new Date(dateStr);
     d.setHours(0, 0, 0, 0);
     return d.getTime() < today.getTime();
-  };
+  }, [today]);
 
   const sortByName = useCallback((a, b) => {
     const nameA = `${a.firstName || ""} ${a.lastName || ""}`.toLowerCase();
@@ -105,7 +101,7 @@ const AttendanceLog = ({
     return nameA.localeCompare(nameB);
   }, []);
 
-  // Effect to fetch attendance data when dateRange or projectId changes
+  // Effect to fetch attendance data when dateRange or projectId changes, or when refreshed
   useEffect(() => {
     const { startDate, endDate } = dateRange;
     dispatch(
@@ -115,21 +111,26 @@ const AttendanceLog = ({
         toDate: endDate,
       })
     );
-    setIsRefreshing(false); // Reset refreshing state after fetch attempt
-  }, [dateRange, projectId, dispatch, isRefreshing]); // isRefreshing is a dependency because it triggers a re-fetch
+    // Reset refreshing state only after the dispatch, assuming fetchAttendanceData
+    // will eventually update logsLoading/logsFetching
+    const timer = setTimeout(() => { // Give Redux time to update
+      setIsRefreshing(false);
+    }, 500); // Small delay to show spinner for a bit longer
+
+    return () => clearTimeout(timer);
+  }, [dateRange, projectId, dispatch, isRefreshing]);
 
   const processedData = useMemo(() => {
-    let filteredData = showOnlyCheckout // Use the prop directly
-      ? attendanceLogsData.filter((item) => item.checkOutTime === null) // Use attendanceLogsData
-      : attendanceLogsData; // Use attendanceLogsData
+    let filteredData = showOnlyCheckout
+      ? (attendanceLogsData || []).filter((item) => item.checkOutTime === null) // Ensure attendanceLogsData is an array
+      : (attendanceLogsData || []); // Ensure attendanceLogsData is an array
 
     // Apply search query filter
     if (searchQuery) {
       const lowerCaseSearchQuery = searchQuery.toLowerCase();
       filteredData = filteredData.filter((att) => {
-        // Construct a full name from available parts, filtering out null/undefined
         const fullName = [att.firstName, att.middleName, att.lastName]
-          .filter(Boolean) // This removes null, undefined, or empty string parts
+          .filter(Boolean)
           .join(" ")
           .toLowerCase();
 
@@ -242,19 +243,15 @@ const AttendanceLog = ({
   const employeeHandler = useCallback(
     (msg) => {
       const { startDate, endDate } = dateRange;
-      if (data.some((item) => item.employeeId == msg.employeeId)) {
-        // dispatch(
-        //   fetchAttendanceData({
-        //     ,
-        //     fromDate: startDate,
-        //     toDate: endDate,
-        //   })
-        // );
-
-        refetch()
-      }
+      dispatch(
+        fetchAttendanceData({
+          projectId,
+          fromDate: startDate,
+          toDate: endDate,
+        })
+      );
     },
-    [selectedProject, dateRange, data]
+    [projectId, dateRange, dispatch]
   );
 
   useEffect(() => {
@@ -420,8 +417,7 @@ const AttendanceLog = ({
               (pageNumber) => (
                 <li
                   key={pageNumber}
-                  className={`page-item ${
-                    currentPage === pageNumber ? "active" : ""
+                  className={`page-item ${currentPage === pageNumber ? "active" : ""
                     }`}
                 >
                   <button
@@ -434,8 +430,7 @@ const AttendanceLog = ({
               )
             )}
             <li
-              className={`page-item ${
-                currentPage === totalPages ? "disabled" : ""
+              className={`page-item ${currentPage === totalPages ? "disabled" : ""
                 }`}
             >
               <button

src/components/Activities/Regularization.jsx

@@ -7,19 +7,16 @@ import { useRegularizationRequests } from "../../hooks/useAttendance";
 import moment from "moment";
 import usePagination from "../../hooks/usePagination";
 import eventBus from "../../services/eventBus";
-import { cacheData, clearCacheKey } from "../../slices/apiDataManager";
-import { useQueryClient } from "@tanstack/react-query";
+import { cacheData } from "../../slices/apiDataManager";
 
-const Regularization = ({ handleRequest }) => {
-  const queryClient = useQueryClient();
-  var selectedProject = useSelector((store) => store.localVariables.projectId);
-  const [regularizesList, setregularizedList] = useState([]);
-  const { regularizes, loading, error, refetch } =
-    useRegularizationRequests(selectedProject);
+const Regularization = ({ handleRequest, searchQuery }) => {
+  const selectedProject = useSelector((store) => store.localVariables.projectId);
+  const [regularizesList, setRegularizedList] = useState([]);
+  const { regularizes, loading, refetch } = useRegularizationRequests(selectedProject);
 
   // Update regularizesList when regularizes data changes
   useEffect(() => {
-    setregularizedList(regularizes);
+    setRegularizedList(regularizes);
   }, [regularizes]);
 
   const sortByName = useCallback((a, b) => {
@@ -86,11 +83,6 @@ const Regularization = ({ handleRequest }) => {
 
   return (
     <div className="table-responsive text-nowrap pb-4">
-      {loading ? (
-        <div className="my-2">
-          <p className="text-secondary">Loading...</p>
-        </div>
-      ) : currentItems?.length > 0 ? (
       <table className="table mb-0">
         <thead>
           <tr>
@@ -106,14 +98,12 @@ const Regularization = ({ handleRequest }) => {
           </tr>
         </thead>
         <tbody>
-            {currentItems?.map((att, index) => (
+          {!loading && currentItems?.length > 0 ? (
+            currentItems.map((att, index) => (
               <tr key={index}>
                 <td colSpan={2}>
                   <div className="d-flex justify-content-start align-items-center">
-                    <Avatar
-                      firstName={att.firstName}
-                      lastName={att.lastName}
-                    ></Avatar>
+                    <Avatar firstName={att.firstName} lastName={att.lastName} />
                     <div className="d-flex flex-column">
                       <a href="#" className="text-heading text-truncate">
                         <span className="fw-normal">
@@ -134,18 +124,27 @@ const Regularization = ({ handleRequest }) => {
                     handleRequest={handleRequest}
                     refresh={refetch}
                   />
-                  {/* </div> */}
                 </td>
               </tr>
-            ))}
+            ))
+          ) : (
+            <tr>
+              <td
+                colSpan={6}
+                className="text-center"
+                style={{
+                  height: "200px",
+                  verticalAlign: "middle",
+                  borderBottom: "none",
+                }}
+              >
+                {loading ? "Loading..." : "No Record Found"}
+              </td>
+            </tr>
+          )}
         </tbody>
       </table>
-      ) : (
-        <div className="my-4">
-          {" "}
-          <span className="text-secondary">No Requests Found !</span>
-        </div>
-      )}
+
       {!loading && totalPages > 1 && (
         <nav aria-label="Page ">
           <ul className="pagination pagination-sm justify-content-end py-1 mt-3">
@@ -160,22 +159,15 @@ const Regularization = ({ handleRequest }) => {
             {[...Array(totalPages)].map((_, index) => (
               <li
                 key={index}
-                className={`page-item ${
-                  currentPage === index + 1 ? "active" : ""
-                }`}
-              >
-                <button
-                  className="page-link "
-                  onClick={() => paginate(index + 1)}
+                className={`page-item ${currentPage === index + 1 ? "active" : ""}`}
               >
+                <button className="page-link" onClick={() => paginate(index + 1)}>
                   {index + 1}
                 </button>
               </li>
             ))}
             <li
-              className={`page-item ${
-                currentPage === totalPages ? "disabled" : ""
-              }`}
+              className={`page-item ${currentPage === totalPages ? "disabled" : ""}`}
             >
               <button
                 className="page-link"

src/components/Project/AssignTask.jsx

@@ -129,6 +129,8 @@ const AssignTask = ({ assignData, onClose, setAssigned }) => {
     dispatch(changeMaster("Job Role"));
     // Initial state should reflect "All Roles" selected
     setSelectedRoles(["all"]);
+    // Initial state should reflect "All Roles" selected
+    setSelectedRoles(["all"]);
   }, [dispatch]);
 
   // Modified handleRoleChange to handle multiple selections
@@ -649,5 +651,4 @@ const AssignTask = ({ assignData, onClose, setAssigned }) => {
   </div>
 </div>  );
 };
-
 export default AssignTask;

src/pages/Activities/AttendancePage.jsx

@@ -6,25 +6,18 @@ import {
 import Breadcrumb from "../../components/common/Breadcrumb";
 import AttendanceLog from "../../components/Activities/AttendcesLogs";
 import Attendance from "../../components/Activities/Attendance";
-// import AttendanceModel from "../../components/Activities/AttendanceModel";
+import AttendanceModel from "../../components/Activities/AttendanceModel";
 import showToast from "../../services/toastService";
-// import { useProjects } from "../../hooks/useProjects";
 import Regularization from "../../components/Activities/Regularization";
 import { useAttendance } from "../../hooks/useAttendance";
 import { useDispatch, useSelector } from "react-redux";
 import { setProjectId } from "../../slices/localVariablesSlice";
-// import { markCurrentAttendance } from "../../slices/apiSlice/attendanceAllSlice";
-import { hasUserPermission } from "../../utils/authUtils";
+import { markCurrentAttendance } from "../../slices/apiSlice/attendanceAllSlice";
 import { useHasUserPermission } from "../../hooks/useHasUserPermission";
 import { REGULARIZE_ATTENDANCE } from "../../utils/constants";
 import eventBus from "../../services/eventBus";
-// import AttendanceRepository from "../../repositories/AttendanceRepository";
+import AttendanceRepository from "../../repositories/AttendanceRepository";
 import { useProjectName } from "../../hooks/useProjects";
-import GlobalModel from "../../components/common/GlobalModel";
-import CheckCheckOutmodel from "../../components/Activities/CheckCheckOutForm";
-import AttendLogs from "../../components/Activities/AttendLogs";
-// import Confirmation from "../../components/Activities/Confirmation";
-import { useQueryClient } from "@tanstack/react-query";
 
 const AttendancePage = () => {
   const [activeTab, setActiveTab] = useState("all");
@@ -218,11 +211,11 @@ const AttendancePage = () => {
 
   return (
     <>
-      {/* {isCreateModalOpen && modelConfig && (
+      {isCreateModalOpen && modelConfig && (
         <div
           className="modal fade show"
           style={{ display: "block" }}
-          id="check-Out-modalg"
+          id="check-Out-modal"
           tabIndex="-1"
           aria-hidden="true"
         >
@@ -232,29 +225,6 @@ const AttendancePage = () => {
             handleSubmitForm={handleSubmit}
           />
         </div>
-      )} */}
-      {isCreateModalOpen && modelConfig && (
-        <GlobalModel
-          isOpen={isCreateModalOpen}
-          size={modelConfig?.action === 6 && "lg"}
-          closeModal={closeModal}
-        >
-          {(modelConfig?.action === 0 ||
-            modelConfig?.action === 1 ||
-            modelConfig?.action === 2) && (
-            <CheckCheckOutmodel
-              modeldata={modelConfig}
-              closeModal={closeModal}
-            />
-          )}
-          {/* For view logs */}
-          {modelConfig?.action === 6 && (
-            <AttendLogs Id={modelConfig?.id} closeModal={closeModal} />
-          )}
-          {modelConfig?.action === 7 && (
-            <Confirmation closeModal={closeModal} />
-          )}
-        </GlobalModel>
       )}
 
       <div className="container-fluid">
@@ -316,8 +286,11 @@ const AttendancePage = () => {
           </ul>
           <div className="tab-content attedanceTabs py-0 px-1 px-sm-3" >
             {activeTab === "all" && (
+              <>
+                {!attLoading && (
                   <div className="tab-pane fade show active py-0">
                     <Attendance
+                      attendance={filteredAndSearchedTodayAttendance()}
                       handleModalData={handleModalData}
                       getRole={getRole}
                       setshowOnlyCheckout={setShowPending}
@@ -328,13 +301,12 @@ const AttendancePage = () => {
                 {!attLoading && filteredAndSearchedTodayAttendance()?.length === 0 && (
                   <p>
                     {" "}
-                    {showPending
                     {showPending
                       ? "No Pending Available"
                       : "No Employee assigned yet."}{" "}
                   </p>
                 )}
-              </div>
+              </>
             )}
             {activeTab === "logs" && (
               <div className="tab-pane fade show active py-0">
@@ -355,6 +327,8 @@ const AttendancePage = () => {
                 />
               </div>
             )}
+
+            {!attLoading && !attendances && <span>Not Found</span>}
           </div>
         </div>
       </div>