From 1a7e77c8146dd8f6c269e2a81bc32c7b5ba6ebc0 Mon Sep 17 00:00:00 2001
From: pramod mahajan <pramod.mahajan@marcoaiot.com>
Date: Wed, 10 Sep 2025 19:09:55 +0530
Subject: [PATCH] structured code

---
 src/pages/master/MasterPage.jsx  | 15 ++++--
 src/pages/master/MasterTable.jsx | 88 ++++++++++++++++++--------------
 2 files changed, 61 insertions(+), 42 deletions(-)

diff --git a/src/pages/master/MasterPage.jsx b/src/pages/master/MasterPage.jsx
index 86640587..0991b7f5 100644
--- a/src/pages/master/MasterPage.jsx
+++ b/src/pages/master/MasterPage.jsx
@@ -111,11 +111,16 @@ const MasterPage = () => {
   return (
     <>
       {isCreateModalOpen && (
-        <GlobalModel  size={
-    modalConfig?.masterType === "Application Role" || modalConfig?.masterType === "Edit-Application Role" 
-      ? "lg" 
-      : "md"
-  } isOpen={isCreateModalOpen} closeModal={()=>closeModal()}>
+        <GlobalModel
+          size={
+            modalConfig?.masterType === "Application Role" ||
+            modalConfig?.masterType === "Edit-Application Role"
+              ? "lg"
+              : "md"
+          }
+          isOpen={isCreateModalOpen}
+          closeModal={() => closeModal()}
+        >
           <MasterModal modaldata={modalConfig} closeModal={closeModal} />
         </GlobalModel>
       )}
diff --git a/src/pages/master/MasterTable.jsx b/src/pages/master/MasterTable.jsx
index a677588b..b2285842 100644
--- a/src/pages/master/MasterTable.jsx
+++ b/src/pages/master/MasterTable.jsx
@@ -9,7 +9,7 @@ const MasterTable = ({ data, columns, loading, handleModalData }) => {
   const selectedMaster = useSelector(
     (store) => store.localVariables.selectedMaster
   );
-   const hiddenColumns = [
+  const hiddenColumns = [
     "id",
     "featurePermission",
     "tenant",
@@ -27,7 +27,7 @@ const MasterTable = ({ data, columns, loading, handleModalData }) => {
     "maxFilesAllowed",
     "maxSizeAllowedInMB",
     "isValidationRequired",
-    "documentCategory"
+    "documentCategory",
   ];
 
   const safeData = Array.isArray(data) ? data : [];
@@ -71,11 +71,13 @@ const MasterTable = ({ data, columns, loading, handleModalData }) => {
           : col.label,
     }));
 
-    const handleSystemDefined = (message) =>{
-      if(message){
-      showToast(`The system-defined item ${selectedMaster} cannot be ${message}.`)
-      }
+  const handleSystemDefined = (message) => {
+    if (message) {
+      showToast(
+        `The system-defined item ${selectedMaster} cannot be ${message}.`
+      );
     }
+  };
 
   return (
     <div className="table-responsive">
@@ -86,12 +88,18 @@ const MasterTable = ({ data, columns, loading, handleModalData }) => {
           <thead className="shadow-sm">
             <tr>
               <th></th>
-              <th className="text-start"> {selectedMaster === "Activity" ? "Activity" : "Name"}</th>
-              <th className="text-start">  {selectedMaster === "Activity"
-    ? "Unit"
-    : selectedMaster === "Document Type"
-    ? "Content Type"
-    : "Description"}</th>
+              <th className="text-start">
+                {" "}
+                {selectedMaster === "Activity" ? "Activity" : "Name"}
+              </th>
+              <th className="text-start">
+                {" "}
+                {selectedMaster === "Activity"
+                  ? "Unit"
+                  : selectedMaster === "Document Type"
+                  ? "Content Type"
+                  : "Description"}
+              </th>
               <th className={` ${!hasMasterPermission && "d-none"}`}>
                 Actions
               </th>
@@ -126,28 +134,28 @@ const MasterTable = ({ data, columns, loading, handleModalData }) => {
                     </td>
                   ))}
                   <td className={!hasMasterPermission ? "d-none" : ""}>
-                  {(selectedMaster === "Application Role" || selectedMaster === "Work Category") && item?.isSystem ? ( 
+                    {(selectedMaster === "Application Role" ||
+                      selectedMaster === "Work Category") &&
+                    item?.isSystem ? (
                       <>
-                      <button
-                        aria-label="Modify"
-                        type="button"
-                        className="btn p-0 dropdown-toggle hide-arrow"
-                        onClick={() =>
-                          handleSystemDefined("updated")
-                        }
-                      >
-                        <i className="bx bxs-edit me-2 text-primary"></i>
-                      </button>
+                        <button
+                          aria-label="Modify"
+                          type="button"
+                          className="btn p-0 dropdown-toggle hide-arrow"
+                          onClick={() => handleSystemDefined("updated")}
+                        >
+                          <i className="bx bxs-edit me-2 text-primary"></i>
+                        </button>
 
-                      <button
-                        aria-label="Delete"
-                        type="button"
-                        className="btn p-0 dropdown-toggle hide-arrow"
-                        onClick={() => handleSystemDefined("deleted")}
-                      >
-                        <i className="bx bx-trash me-1 text-danger"></i>
-                      </button>
-                    </>
+                        <button
+                          aria-label="Delete"
+                          type="button"
+                          className="btn p-0 dropdown-toggle hide-arrow"
+                          onClick={() => handleSystemDefined("deleted")}
+                        >
+                          <i className="bx bx-trash me-1 text-danger"></i>
+                        </button>
+                      </>
                     ) : (
                       <>
                         <button
@@ -157,17 +165,23 @@ const MasterTable = ({ data, columns, loading, handleModalData }) => {
                           data-bs-target="#master-modal"
                           className="btn p-0 dropdown-toggle hide-arrow"
                           onClick={() =>
-                            handleModalData(`Edit-${selectedMaster}`, item, selectedMaster)
+                            handleModalData(
+                              `Edit-${selectedMaster}`,
+                              item,
+                              selectedMaster
+                            )
                           }
                         >
                           <i className="bx bxs-edit me-2 text-primary"></i>
                         </button>
 
                         <button
-          aria-label="Delete"
-  type="button"
-  className="btn p-0 dropdown-toggle hide-arrow"
-  onClick={() => handleModalData("delete", item, selectedMaster)}
+                          aria-label="Delete"
+                          type="button"
+                          className="btn p-0 dropdown-toggle hide-arrow"
+                          onClick={() =>
+                            handleModalData("delete", item, selectedMaster)
+                          }
                         >
                           <i className="bx bx-trash me-1 text-danger"></i>
                         </button>