From c3e6d3dc48bb15b6c8a7a4ffbc6ca19d453d6e51 Mon Sep 17 00:00:00 2001
From: Dave Richer <dave@imexsystems.ca>
Date: Wed, 21 Aug 2024 15:09:13 -0400
Subject: [PATCH 1/7] - Checkpoint

Signed-off-by: Dave Richer <dave@imexsystems.ca>
---
 ...oduction-list-config-manager.component.jsx | 105 ++++++++++--------
 1 file changed, 56 insertions(+), 49 deletions(-)

diff --git a/client/src/components/production-list-table/production-list-config-manager.component.jsx b/client/src/components/production-list-table/production-list-config-manager.component.jsx
index c33b164ff..95f4ceccf 100644
--- a/client/src/components/production-list-table/production-list-config-manager.component.jsx
+++ b/client/src/components/production-list-table/production-list-config-manager.component.jsx
@@ -376,7 +376,14 @@ export function ProductionListConfigManager({
           >
             {t("general.actions.save")}
           </Button>
-          <Button onClick={() => setOpen(false)}>{t("general.actions.close")}</Button>
+          <Button
+            onClick={() => {
+              setIsAddingNewProfile(false);
+              setOpen(false);
+            }}
+          >
+            {t("general.actions.close")}
+          </Button>
         </Space>
       </Form>
     </div>
@@ -384,55 +391,55 @@ export function ProductionListConfigManager({
 
   return (
     <Space>
-      <Popover open={open} content={popMenu}>
-        <Button loading={loading} onClick={() => setOpen(true)} disabled={!isAddingNewProfile && !hasUnsavedChanges}>
-          {t("production.actions.saveconfig")}
-        </Button>
+      <Button loading={loading} onClick={() => setOpen(true)} disabled={isAddingNewProfile || !hasUnsavedChanges}>
+        {t("production.actions.saveconfig")}
+      </Button>
+      <Popover open={open} content={popMenu} placement="bottom">
+        <Select
+          style={{
+            minWidth: "150px"
+          }}
+          onSelect={handleSelect}
+          placeholder={t("production.labels.selectview")}
+          optionLabelProp="label"
+          popupMatchSelectWidth={false}
+          value={activeView}
+        >
+          {bodyshop.production_config
+            .slice()
+            .sort((a, b) =>
+              a.name === t("production.constants.main_profile")
+                ? -1
+                : b.name === t("production.constants.main_profile")
+                  ? 1
+                  : 0
+            ) //
+            .map((config) => (
+              <Select.Option key={config.name} label={config.name}>
+                <div style={{ display: "flex", justifyContent: "space-between", alignItems: "center" }}>
+                  <span style={{ flex: 1, maxWidth: "80%", marginRight: "1rem", textOverflow: "ellipsis" }}>
+                    {config.name}
+                  </span>
+                  {config.name !== t("production.constants.main_profile") && (
+                    <Popconfirm
+                      placement="right"
+                      title={t("general.labels.areyousure")}
+                      onConfirm={() => handleTrash(config.name)}
+                    >
+                      <DeleteOutlined onClick={(e) => e.stopPropagation()} />
+                    </Popconfirm>
+                  )}
+                </div>
+              </Select.Option>
+            ))}
+          <Select.Option key="add_new" label={t("production.labels.addnewprofile")}>
+            <div style={{ display: "flex", alignItems: "center" }}>
+              <PlusOutlined style={{ marginRight: "0.5rem" }} />
+              {t("production.labels.addnewprofile")}
+            </div>
+          </Select.Option>
+        </Select>
       </Popover>
-      <Select
-        style={{
-          minWidth: "150px"
-        }}
-        onSelect={handleSelect}
-        placeholder={t("production.labels.selectview")}
-        optionLabelProp="label"
-        popupMatchSelectWidth={false}
-        value={activeView} // Ensure this only changes when appropriate
-      >
-        {bodyshop.production_config
-          .slice()
-          .sort((a, b) =>
-            a.name === t("production.constants.main_profile")
-              ? -1
-              : b.name === t("production.constants.main_profile")
-                ? 1
-                : 0
-          ) //
-          .map((config) => (
-            <Select.Option key={config.name} label={config.name}>
-              <div style={{ display: "flex", justifyContent: "space-between", alignItems: "center" }}>
-                <span style={{ flex: 1, maxWidth: "80%", marginRight: "1rem", textOverflow: "ellipsis" }}>
-                  {config.name}
-                </span>
-                {config.name !== t("production.constants.main_profile") && (
-                  <Popconfirm
-                    placement="right"
-                    title={t("general.labels.areyousure")}
-                    onConfirm={() => handleTrash(config.name)}
-                  >
-                    <DeleteOutlined onClick={(e) => e.stopPropagation()} />
-                  </Popconfirm>
-                )}
-              </div>
-            </Select.Option>
-          ))}
-        <Select.Option key="add_new" label={t("production.labels.addnewprofile")}>
-          <div style={{ display: "flex", alignItems: "center" }}>
-            <PlusOutlined style={{ marginRight: "0.5rem" }} />
-            {t("production.labels.addnewprofile")}
-          </div>
-        </Select.Option>
-      </Select>
     </Space>
   );
 }

From c89e4f1b414e2f8f41263f144aff90186c5ed619 Mon Sep 17 00:00:00 2001
From: Dave Richer <dave@imexsystems.ca>
Date: Wed, 21 Aug 2024 15:14:33 -0400
Subject: [PATCH 2/7] - Checkpoint

Signed-off-by: Dave Richer <dave@imexsystems.ca>
---
 client/src/translations/en_us/common.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/client/src/translations/en_us/common.json b/client/src/translations/en_us/common.json
index ff93baff8..1304eff5f 100644
--- a/client/src/translations/en_us/common.json
+++ b/client/src/translations/en_us/common.json
@@ -1180,7 +1180,7 @@
         "vehicle": "Vehicle"
       },
       "labels": {
-        "unsavedchanges": "Unsaved change.",
+        "unsavedchanges": "Unsaved changes.",
         "actions": "Actions",
         "areyousure": "Are you sure?",
         "barcode": "Barcode",

From 90532427b67f3833f54ad8df71190a8801533206 Mon Sep 17 00:00:00 2001
From: Dave Richer <dave@imexsystems.ca>
Date: Wed, 21 Aug 2024 15:32:22 -0400
Subject: [PATCH 3/7] - Checkpoint

Signed-off-by: Dave Richer <dave@imexsystems.ca>
---
 ...oduction-list-config-manager.component.jsx | 40 +++++++++++++++++--
 client/src/translations/en_us/common.json     |  7 +++-
 client/src/translations/es/common.json        |  7 +++-
 client/src/translations/fr/common.json        |  7 +++-
 4 files changed, 52 insertions(+), 9 deletions(-)

diff --git a/client/src/components/production-list-table/production-list-config-manager.component.jsx b/client/src/components/production-list-table/production-list-config-manager.component.jsx
index 95f4ceccf..80c32985f 100644
--- a/client/src/components/production-list-table/production-list-config-manager.component.jsx
+++ b/client/src/components/production-list-table/production-list-config-manager.component.jsx
@@ -362,7 +362,25 @@ export function ProductionListConfigManager({
     <div>
       <Form layout="vertical" form={form} onFinish={handleSaveConfig}>
         {isAddingNewProfile && (
-          <Form.Item label={t("production.labels.viewname")} name="name" rules={[{ required: true }]}>
+          <Form.Item
+            label={t("production.labels.viewname")}
+            name="name"
+            rules={[
+              { required: true, message: t("production.errors.name_required") },
+              {
+                validator: (_, value) => {
+                  if (!value) {
+                    return Promise.resolve();
+                  }
+                  const nameExists = bodyshop.production_config.some((pc) => pc.name === value);
+                  if (nameExists) {
+                    return Promise.reject(new Error(t("production.errors.name_exists")));
+                  }
+                  return Promise.resolve();
+                }
+              }
+            ]}
+          >
             <Input />
           </Form.Item>
         )}
@@ -376,13 +394,22 @@ export function ProductionListConfigManager({
           >
             {t("general.actions.save")}
           </Button>
+          <Button
+            type="default"
+            onClick={() => {
+              setIsAddingNewProfile(true);
+              setOpen(true);
+            }}
+          >
+            {t("general.actions.saveas")}
+          </Button>
           <Button
             onClick={() => {
               setIsAddingNewProfile(false);
               setOpen(false);
             }}
           >
-            {t("general.actions.close")}
+            {t("general.actions.cancel")}
           </Button>
         </Space>
       </Form>
@@ -417,7 +444,14 @@ export function ProductionListConfigManager({
             .map((config) => (
               <Select.Option key={config.name} label={config.name}>
                 <div style={{ display: "flex", justifyContent: "space-between", alignItems: "center" }}>
-                  <span style={{ flex: 1, maxWidth: "80%", marginRight: "1rem", textOverflow: "ellipsis" }}>
+                  <span
+                    style={{
+                      flex: 1,
+                      maxWidth: "80%",
+                      marginRight: "1rem",
+                      textOverflow: "ellipsis"
+                    }}
+                  >
                     {config.name}
                   </span>
                   {config.name !== t("production.constants.main_profile") && (
diff --git a/client/src/translations/en_us/common.json b/client/src/translations/en_us/common.json
index 1304eff5f..98d9418ec 100644
--- a/client/src/translations/en_us/common.json
+++ b/client/src/translations/en_us/common.json
@@ -1165,7 +1165,8 @@
         "tryagain": "Try Again",
         "view": "View",
         "viewreleasenotes": "See What's Changed",
-        "remove_alert": "Are you sure you want to dismiss the alert?"
+        "remove_alert": "Are you sure you want to dismiss the alert?",
+        "saveas": "Save As"
       },
       "errors": {
         "fcm": "You must allow notification permissions to have real time messaging.  Click to try again.",
@@ -2788,7 +2789,9 @@
       "errors": {
         "boardupdate": "Error encountered updating Job. {{message}}",
         "removing": "Error removing from production board. {{error}}",
-        "settings": "Error saving board settings: {{error}}"
+        "settings": "Error saving board settings: {{error}}",
+        "name_exists": "A Profile with this name already exists. Please choose a different name.",
+        "name_required": "Profile name is required."
       },
       "labels": {
         "kiosk_mode": "Kiosk Mode",
diff --git a/client/src/translations/es/common.json b/client/src/translations/es/common.json
index 76f537e61..18edcbeeb 100644
--- a/client/src/translations/es/common.json
+++ b/client/src/translations/es/common.json
@@ -1165,7 +1165,8 @@
         "tryagain": "",
         "view": "",
         "viewreleasenotes": "",
-        "remove_alert": ""
+        "remove_alert": "",
+        "saveas": ""
       },
       "errors": {
         "fcm": "",
@@ -2788,7 +2789,9 @@
       "errors": {
         "boardupdate": "",
         "removing": "",
-        "settings": ""
+        "settings": "",
+        "name_exists": "",
+        "name_required": ""
       },
       "labels": {
         "kiosk_mode": "",
diff --git a/client/src/translations/fr/common.json b/client/src/translations/fr/common.json
index fccb1ce65..adc23a1a0 100644
--- a/client/src/translations/fr/common.json
+++ b/client/src/translations/fr/common.json
@@ -1165,7 +1165,8 @@
         "tryagain": "",
         "view": "",
         "viewreleasenotes": "",
-        "remove_alert": ""
+        "remove_alert": "",
+        "saveas": ""
       },
       "errors": {
         "fcm": "",
@@ -2788,7 +2789,9 @@
       "errors": {
         "boardupdate": "",
         "removing": "",
-        "settings": ""
+        "settings": "",
+        "name_exists": "",
+        "name_required": ""
       },
       "labels": {
         "kiosk_mode": "",

From 11785f3b862af6a3145fc1c46f92049e6d2b0715 Mon Sep 17 00:00:00 2001
From: Dave Richer <dave@imexsystems.ca>
Date: Wed, 21 Aug 2024 15:39:16 -0400
Subject: [PATCH 4/7] - Checkpoint

Signed-off-by: Dave Richer <dave@imexsystems.ca>
---
 ...oduction-list-config-manager.component.jsx | 21 +++++++++++--------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/client/src/components/production-list-table/production-list-config-manager.component.jsx b/client/src/components/production-list-table/production-list-config-manager.component.jsx
index 80c32985f..26fc45511 100644
--- a/client/src/components/production-list-table/production-list-config-manager.component.jsx
+++ b/client/src/components/production-list-table/production-list-config-manager.component.jsx
@@ -394,15 +394,17 @@ export function ProductionListConfigManager({
           >
             {t("general.actions.save")}
           </Button>
-          <Button
-            type="default"
-            onClick={() => {
-              setIsAddingNewProfile(true);
-              setOpen(true);
-            }}
-          >
-            {t("general.actions.saveas")}
-          </Button>
+          {!isAddingNewProfile && (
+            <Button
+              type="default"
+              onClick={() => {
+                setIsAddingNewProfile(true);
+                setOpen(true);
+              }}
+            >
+              {t("general.actions.saveas")}
+            </Button>
+          )}
           <Button
             onClick={() => {
               setIsAddingNewProfile(false);
@@ -431,6 +433,7 @@ export function ProductionListConfigManager({
           optionLabelProp="label"
           popupMatchSelectWidth={false}
           value={activeView}
+          disabled={open || isAddingNewProfile} // Disable the Select box when the popover is open or adding a new profile
         >
           {bodyshop.production_config
             .slice()

From 02f4677aefc640dbd2b9569750dd2775d4b91577 Mon Sep 17 00:00:00 2001
From: Dave Richer <dave@imexsystems.ca>
Date: Wed, 21 Aug 2024 16:22:56 -0400
Subject: [PATCH 5/7] - Checkpoint

Signed-off-by: Dave Richer <dave@imexsystems.ca>
---
 ...oduction-list-config-manager.component.jsx | 32 ++++++++++++++++---
 1 file changed, 27 insertions(+), 5 deletions(-)

diff --git a/client/src/components/production-list-table/production-list-config-manager.component.jsx b/client/src/components/production-list-table/production-list-config-manager.component.jsx
index 26fc45511..973d8d925 100644
--- a/client/src/components/production-list-table/production-list-config-manager.component.jsx
+++ b/client/src/components/production-list-table/production-list-config-manager.component.jsx
@@ -23,7 +23,6 @@ export function ProductionListConfigManager({
   setColumns,
   setState,
   onSave,
-  defaultView,
   hasUnsavedChanges,
   setHasUnsavedChanges
 }) {
@@ -34,6 +33,7 @@ export function ProductionListConfigManager({
   const [open, setOpen] = useState(false);
   const [isAddingNewProfile, setIsAddingNewProfile] = useState(false);
   const [form] = Form.useForm();
+
   const [activeView, setActiveView] = useState(() => {
     const assoc = bodyshop.associations.find((a) => a.useremail === currentUser.email);
     return assoc && assoc.default_prod_list_view;
@@ -257,7 +257,29 @@ export function ProductionListConfigManager({
       } else {
         await updateActiveProdView(null);
         setColumns([]);
-        setState(defaultState); // Reset to default state if no configs are left
+        setState(defaultState);
+      }
+    } else {
+      // Revert back to the active view and load its columns and state
+      const activeConfig = bodyshop.production_config.find((pc) => pc.name === activeView);
+      if (activeConfig) {
+        await updateActiveProdView(activeView);
+        setColumns(
+          activeConfig.columns.columnKeys.map((k) => {
+            return {
+              ...ProductionListColumns({
+                technician,
+                state: ensureDefaultState(state),
+                refetch,
+                data: data,
+                activeStatuses: bodyshop.md_ro_statuses.active_statuses,
+                treatments: { Enhanced_Payroll }
+              }).find((e) => e.key === k.key),
+              width: k.width
+            };
+          })
+        );
+        setState(ensureDefaultState(activeConfig.columns.tableState));
       }
     }
   };
@@ -311,7 +333,7 @@ export function ProductionListConfigManager({
 
   useEffect(() => {
     const validateAndSetDefaultView = () => {
-      const configExists = bodyshop.production_config.some((pc) => pc.name === defaultView);
+      const configExists = bodyshop.production_config.some((pc) => pc.name === activeView);
 
       if (!configExists) {
         // If the default view doesn't exist, revert to the main profile
@@ -344,7 +366,7 @@ export function ProductionListConfigManager({
         }
       } else {
         // If the default view exists, set it as active
-        setActiveView(defaultView);
+        setActiveView(activeView);
       }
     };
 
@@ -356,7 +378,7 @@ export function ProductionListConfigManager({
       validateAndSetDefaultView();
     }
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [defaultView, bodyshop.production_config]);
+  }, [activeView, bodyshop.production_config]);
 
   const popMenu = (
     <div>

From 8af33646608156abbcd59b619f26b9722c6493c2 Mon Sep 17 00:00:00 2001
From: Dave Richer <dave@imexsystems.ca>
Date: Wed, 21 Aug 2024 16:31:42 -0400
Subject: [PATCH 6/7] - Checkpoint

Signed-off-by: Dave Richer <dave@imexsystems.ca>
---
 .../production-list-table/production-list-table.component.jsx | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/client/src/components/production-list-table/production-list-table.component.jsx b/client/src/components/production-list-table/production-list-table.component.jsx
index 728330ece..279e26b4a 100644
--- a/client/src/components/production-list-table/production-list-table.component.jsx
+++ b/client/src/components/production-list-table/production-list-table.component.jsx
@@ -269,17 +269,17 @@ export function ProductionListTable({ loading, data, refetch, bodyshop, technici
               data={data}
               onColumnAdd={addColumn}
             />
+
             <ProductionListConfigManager
               columns={columns}
-              tableState={state}
               setColumns={setColumns}
+              state={state}
               setState={setState}
               refetch={refetch}
               data={data}
               bodyshop={bodyshop}
               technician={technician}
               currentUser={currentUser}
-              defaultView={defaultView}
               setHasUnsavedChanges={setHasUnsavedChanges}
               hasUnsavedChanges={hasUnsavedChanges}
               onSave={() => {

From 7ff1051d3c41ff58d082903b139bed4385918fe0 Mon Sep 17 00:00:00 2001
From: Dave Richer <dave@imexsystems.ca>
Date: Wed, 21 Aug 2024 16:49:29 -0400
Subject: [PATCH 7/7] - Checkpoint

Signed-off-by: Dave Richer <dave@imexsystems.ca>
---
 .../production-list-config-manager.component.jsx                 | 1 +
 1 file changed, 1 insertion(+)

diff --git a/client/src/components/production-list-table/production-list-config-manager.component.jsx b/client/src/components/production-list-table/production-list-config-manager.component.jsx
index 973d8d925..b34fb92d7 100644
--- a/client/src/components/production-list-table/production-list-config-manager.component.jsx
+++ b/client/src/components/production-list-table/production-list-config-manager.component.jsx
@@ -484,6 +484,7 @@ export function ProductionListConfigManager({
                       placement="right"
                       title={t("general.labels.areyousure")}
                       onConfirm={() => handleTrash(config.name)}
+                      onCancel={(e) => e.stopPropagation()}
                     >
                       <DeleteOutlined onClick={(e) => e.stopPropagation()} />
                     </Popconfirm>