ui-soxo-bootstrap-core 2.4.25-dev.14 → 2.4.25-dev.15

package/core/models/users/components/user-add/user-add.js

@@ -160,6 +160,17 @@ const UserAdd = ({ model, callback, edit, history, formContent, match, additiona
       }
     }
   }, []);
+
+  useEffect(() => {
+    if (!loading && formContent?.role_id && roles.length > 0) {
+      const roleId = Number(formContent.role_id);
+
+      if (!Number.isNaN(roleId)) {
+        form.setFieldsValue({ role_id: roleId });
+      }
+    }
+  }, [loading, formContent, roles]);
+
   /**
    *Define the options dynamically
    */
@@ -203,7 +214,6 @@ const UserAdd = ({ model, callback, edit, history, formContent, match, additiona
         }
       })
       .catch((error) => {
-        console.error('Error fetching designations:', error);
         setDesignations([]);
       })
       .finally(() => setLoading(false));
@@ -215,16 +225,10 @@ const UserAdd = ({ model, callback, edit, history, formContent, match, additiona
   function getRoles() {
     RolesAPI.getRole()
       .then((res) => {
-        // if API returns { statusCode: 200, result: [...] }
-        if (Array.isArray(res.result)) {
-          setRoles(res.result.filter((r) => r.active === 'Y')); // optional: only active roles
-        } else {
-          setRoles([]);
-        }
+        const activeRoles = Array.isArray(res.result) ? res.result.filter((r) => r.active === 'Y') : [];
+        setRoles(activeRoles);
       })
-      .catch((err) => {
-        setRoles([]);
-      });
+      .catch(() => setRoles([]));
   }
 
   /** Get Department List */
@@ -243,7 +247,6 @@ const UserAdd = ({ model, callback, edit, history, formContent, match, additiona
         }
       })
       .catch((error) => {
-        console.error('Error fetching departments:', error);
         setDepartments([]);
       })
       .finally(() => setLoading(false));
@@ -320,25 +323,24 @@ const UserAdd = ({ model, callback, edit, history, formContent, match, additiona
         form.setFieldsValue({ staff_code: formContent.staff_code });
       }
     }
-    if (formContent?.id && formContent?.organization_details) {
-      try {
-        const org = JSON.parse(formContent.organization_details);
-        const branchIds = org.branch_ids?.map((b) => b.id) || [];
-        const defaultBranchObj = org.branch_ids?.find((b) => b.DefaultBranch === 'true');
-        const defaultBr = defaultBranchObj?.id || null;
-
-        setSelectedBranches(branchIds);
-        setDefaultBranch(defaultBr);
-
-        form.setFieldsValue({
-          selectedBranches: branchIds,
-          defaultBranch: defaultBr,
-        });
-      } catch (err) {
-        console.error('Invalid organization_details JSON', err);
-      }
-    }
-  }, [formContent]);
+    if (!formContent) return;
+
+    // normalize branch ids to NUMBER
+    const org =
+      typeof formContent.organization_details === 'string' ? JSON.parse(formContent.organization_details) : formContent.organization_details;
+
+    const branchIds = (org?.branch_ids || []).map(Number);
+    const defaultBr = formContent.defaultBranch ? Number(formContent.defaultBranch) : null;
+
+    // state (for filtering)
+    setSelectedBranches(branchIds);
+
+    // form (source of truth)
+    form.setFieldsValue({
+      selectedBranches: branchIds,
+      defaultBranch: defaultBr,
+    });
+  }, [formContent, form]);
   // Generate branch options for Select component
   const branchOptions = branches.map((branch) => ({
     label: branch.br_desc,
@@ -349,6 +351,8 @@ const UserAdd = ({ model, callback, edit, history, formContent, match, additiona
    * Submit values
    */
   const onSubmit = (values) => {
+    values.defaultBranch = String(values.defaultBranch);
+
     /**If PanelOpen is open and edit mode then password will be existing password else new password*/
     if (!isPasswordVisible && mode === 'Edit') {
       values.password = body.password;
@@ -377,6 +381,7 @@ const UserAdd = ({ model, callback, edit, history, formContent, match, additiona
     if (props?.ldap && selectedOption && selectedOption.value) {
       values = {
         ...values,
+
         addAllBranches: props.ldap.addAllBranches,
         auth_user: selectedOption.value,
         auth_type: 'LDAP',
@@ -387,6 +392,7 @@ const UserAdd = ({ model, callback, edit, history, formContent, match, additiona
     if (values.attributes && typeof values === 'object') {
       values = {
         ...values,
+
         auth_type: 'LDAP',
 
         attributes: JSON.stringify(values.attributes),
@@ -624,22 +630,19 @@ const UserAdd = ({ model, callback, edit, history, formContent, match, additiona
                 <Select
                   mode="multiple"
                   placeholder="Select Branches"
-                  value={selectedBranches}
+                  options={branchOptions}
+                  allowClear
+                  showSearch
+                  optionFilterProp="label"
                   onChange={(value) => {
-                    setSelectedBranches(value);
+                    const normalized = value.map(Number);
+                    setSelectedBranches(normalized);
 
-                    // ⬅NEW: remove defaultBranch if it’s not in selectedBranches
-                    if (!value.includes(defaultBranch)) {
-                      setDefaultBranch(undefined);
+                    const currentDefault = form.getFieldValue('defaultBranch');
+                    if (currentDefault && !normalized.includes(currentDefault)) {
                       form.setFieldsValue({ defaultBranch: undefined });
                     }
                   }}
-                  options={branchOptions}
-                  allowClear
-                  showSearch
-                  optionFilterProp="label"
-                  maxTagCount={5} // Show only 5 tags
-                  maxTagPlaceholder={(omittedValues) => `+${omittedValues.length}`} // Show "+n"
                 />
               </Form.Item>
             </Col>
@@ -648,9 +651,9 @@ const UserAdd = ({ model, callback, edit, history, formContent, match, additiona
               <Form.Item label="Default Branch" name="defaultBranch" rules={[{ required: true, message: 'Please select default branch' }]}>
                 <Select placeholder="Select Default Branch" onChange={setDefaultBranch}>
                   {branchOptions
-                    .filter((opt) => selectedBranches.includes(opt.value))
+                    .filter((opt) => selectedBranches.includes(Number(opt.value))) 
                     .map((opt) => (
-                      <Option key={opt.value} value={opt.value}>
+                      <Option key={opt.value} value={Number(opt.value)}>
                         {opt.label}
                       </Option>
                     ))}
@@ -726,8 +729,8 @@ const UserAdd = ({ model, callback, edit, history, formContent, match, additiona
               </Form.Item>
             </Col>
             <Col span={8}>
-              <Form.Item name="role_id" label="Role" rules={[{ required: true, message: 'Please Input Role' }]}>
-                <Select placeholder="Select Role" style={{ width: '100%' }}>
+              <Form.Item name="role_id" label="Role" rules={[{ required: true, message: 'Please select a Role' }]}>
+                <Select placeholder="Select Role">
                   {roles.map((role) => (
                     <Option key={role.id} value={role.id}>
                       {role.name}

package/core/models/users/components/user-add/user-edit.js

@@ -11,7 +11,6 @@ export default function UserEdit(record) {
   // Handle edit button click
   const handleEditClick = () => {
     if (!record.id) {
-      console.warn('Invalid record: Missing ID');
       return;
     }
 
@@ -29,9 +28,7 @@ export default function UserEdit(record) {
             // Try parsing other_details, handle error if invalid JSON
             try {
               otherDetails = JSON.parse(apiData.other_details);
-            } catch (err) {
-              console.warn('Failed to parse other_details:', apiData.other_details);
-            }
+            } catch (err) {}
           }
           let orgDetails = {};
           try {
@@ -50,8 +47,10 @@ export default function UserEdit(record) {
             designation: apiData.designation_code,
             department: apiData.department_id,
             // Handle selected branches and default branch
+            organization_details: orgDetails,
             selectedBranches: orgDetails.branch_ids || [],
-            defaultBranch: apiData.firm_id || null,
+            defaultBranch: apiData.branch_id || null,
+            role_id: apiData.role_id,
             password: apiData.password,
             user_group: apiData.user_group,
             // Handle doctor codes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ui-soxo-bootstrap-core",
-  "version": "2.4.25-dev.14",
+  "version": "2.4.25-dev.15",
   "description": "All the Core Components for you to start",
   "keywords": [
     "all in one"