(fix) Fix undefined errors thrown when clearing/deselecting values fr…

…om selectors (#40) thanks @makombe
openmrs · Dec 12, 2023 · ae0bb8a · ae0bb8a
1 parent f6e0be6
commit ae0bb8a
Show file tree

Hide file tree

Showing 9 changed files with 16 additions and 16 deletions.
diff --git a/...ck-items/add-stock-item/preferred-vendor-selector/preferred-vendor-selector.component.tsx b/...ck-items/add-stock-item/preferred-vendor-selector/preferred-vendor-selector.component.tsx
@@ -47,7 +47,7 @@ const PreferredVendorSelector = <T,>(
           items={sourcesList || []}
           onChange={(data: { selectedItem: StockSource }) => {
             props.onPreferredVendorChange?.(data.selectedItem);
-            onChange(data.selectedItem.uuid);
+            onChange(data.selectedItem?.uuid);
           }}
           initialSelectedItem={
             sourcesList?.find((p) => p.uuid === props.preferredVendorUuid) || {}

diff --git a/...ms/add-stock-item/stock-item-category-selector/stock-item-category-selector.component.tsx b/...ms/add-stock-item/stock-item-category-selector/stock-item-category-selector.component.tsx
@@ -44,7 +44,7 @@ const StockItemCategorySelector = <T,>(
           items={categories || []}
           onChange={(data: { selectedItem: Concept }) => {
             props.onCategoryUuidChange?.(data.selectedItem);
-            onChange(data.selectedItem.uuid);
+            onChange(data.selectedItem?.uuid);
           }}
           initialSelectedItem={
             categories?.find((p) => p.uuid === props.categoryUuid) || {}

diff --git a/src/stock-operations/add-stock-operation/base-operation-details.component.tsx b/src/stock-operations/add-stock-operation/base-operation-details.component.tsx
@@ -266,7 +266,7 @@ const BaseOperationDetails: React.FC<BaseOperationDetailsProps> = ({
             errors?.responsiblePersonUuid?.message
           }
           onUserChanged={(user) => {
-            if (user.uuid === otherUser.uuid) {
+            if (user?.uuid === otherUser.uuid) {
               setIsOtherUser(true);
             }
           }}

diff --git a/src/stock-operations/add-stock-operation/stock-items-addition-row.component.tsx b/src/stock-operations/add-stock-operation/stock-items-addition-row.component.tsx
@@ -151,9 +151,9 @@ const StockItemsAdditionRow: React.FC<StockItemsAdditionRowProps> = ({
                     handleStockItemChange(index, item);
                     setValue(
                       `stockItems.${index}.packagingUnits`,
-                      item.packagingUnits
+                      item?.packagingUnits
                     );
-                    setStockItemUuid(item.uuid);
+                    setStockItemUuid(item?.uuid);
                   }}
                 />
               )}
@@ -192,12 +192,12 @@ const StockItemsAdditionRow: React.FC<StockItemsAdditionRowProps> = ({
                     <BatchNoSelector
                       batchUuid={row?.stockBatchUuid}
                       onBatchNoChanged={(item) => {
-                        setValue(`stockItems.${index}.batchNo`, item.batchNo);
+                        setValue(`stockItems.${index}.batchNo`, item?.batchNo);
                         setValue(
                           `stockItems.${index}.expiration`,
-                          item.expiration
+                          item?.expiration
                         );
-                        setStockItemExpiy(item.expiration);
+                        setStockItemExpiy(item?.expiration);
                       }}
                       placeholder={"Filter..."}
                       invalid={!!errors?.stockItems?.[index]?.stockBatchUuid}
@@ -288,7 +288,7 @@ const StockItemsAdditionRow: React.FC<StockItemsAdditionRowProps> = ({
                   onStockPackageChanged={(selectedItem) => {
                     setValue(
                       `stockItems.${index}.stockItemPackagingUOMUuid`,
-                      selectedItem.uuid
+                      selectedItem?.uuid
                     );
                   }}
                   placeholder={"Filter..."}

diff --git a/src/stock-operations/batch-no-selector/batch-no-selector.component.tsx b/src/stock-operations/batch-no-selector/batch-no-selector.component.tsx
@@ -52,7 +52,7 @@ const BatchNoSelector = <T,>(props: BatchNoSelectorProps<T>) => {
             items={stockItemBatchNos || []}
             onChange={(data: { selectedItem?: StockBatchDTO }) => {
               props.onBatchNoChanged?.(data.selectedItem);
-              onChange(data.selectedItem.uuid);
+              onChange(data.selectedItem?.uuid);
             }}
             initialSelectedItem={
               stockItemBatchNos?.find((p) => p.uuid === props.batchUuid) ?? ""

diff --git a/src/stock-operations/party-selector/party-selector.component.tsx b/src/stock-operations/party-selector/party-selector.component.tsx
@@ -36,7 +36,7 @@ const PartySelector = <T,>(props: PartySelectorProps<T>) => {
           items={props.parties}
           onChange={(data: { selectedItem: Party }) => {
             props.onPartyChange?.(data.selectedItem);
-            onChange(data.selectedItem.uuid);
+            onChange(data.selectedItem?.uuid);
           }}
           initialSelectedItem={
             props.parties?.find((p) => p.uuid === props.partyUuid) || {}

diff --git a/src/stock-operations/qty-uom-selector/qty-uom-selector.component.tsx b/src/stock-operations/qty-uom-selector/qty-uom-selector.component.tsx
@@ -37,13 +37,13 @@ const QtyUomSelector = <T,>(props: QtyUomSelectorProps<T>) => {
             controllerName={props.controllerName}
             id={props.name}
             size={"sm"}
-            items={item.packagingUnits || []}
+            items={item?.packagingUnits || []}
             onChange={(data: { selectedItem?: StockItemPackagingUOMDTO }) => {
               props.onStockPackageChanged?.(data.selectedItem);
-              onChange(data.selectedItem.uuid);
+              onChange(data.selectedItem?.uuid);
             }}
             initialSelectedItem={
-              item.packagingUnits?.find(
+              item?.packagingUnits?.find(
                 (p) => p.uuid === props.stockItemUuid
               ) ?? ""
             }

diff --git a/src/stock-operations/stock-item-selector/stock-item-selector.component.tsx b/src/stock-operations/stock-item-selector/stock-item-selector.component.tsx
@@ -42,7 +42,7 @@ const StockItemSelector = <T,>(props: StockItemSelectorProps<T>) => {
             items={stockItemsList || []}
             onChange={(data: { selectedItem: StockItemDTO }) => {
               props.onStockItemChanged?.(data.selectedItem);
-              onChange(data.selectedItem.uuid);
+              onChange(data.selectedItem?.uuid);
             }}
             initialSelectedItem={
               stockItemsList?.find((p) => p.uuid === props.stockItemUuid) ?? ""

diff --git a/src/stock-operations/users-selector/users-selector.component.tsx b/src/stock-operations/users-selector/users-selector.component.tsx
@@ -45,7 +45,7 @@ const UsersSelector = <T,>(props: UsersSelectorProps<T>) => {
             items={userList || []}
             onChange={(data: { selectedItem: User }) => {
               props.onUserChanged?.(data.selectedItem);
-              onChange(data.selectedItem.uuid);
+              onChange(data.selectedItem?.uuid);
             }}
             initialSelectedItem={
               userList?.find((p) => p.uuid === props.userUuid) ?? ""