Merge pull request #117 from lklong709/PIM-19

fixes #19 - Bulk action - Missing message notification when user use bulk action but not choose item
aesirxio · Jan 17, 2023 · 0644e71 · 0644e71
2 parents 97fc458 + aef1539
commit 0644e71
Show file tree

Hide file tree

Showing 5 changed files with 32 additions and 10 deletions.
diff --git a/src/containers/CategoriesPage/Component/ListCategories.jsx b/src/containers/CategoriesPage/Component/ListCategories.jsx
@@ -172,8 +172,12 @@ const ListCategories = observer((props) => {
   }, []);
 
   const selectBulkActionsHandler = (value) => {
-    viewModel.isLoading();
-    viewModel.updateStatus(listSelected, value.value);
+    if (listSelected.length < 1) {
+      notify(t('txt_row_select_error'), 'error');
+    } else {
+      viewModel.isLoading();
+      viewModel.updateStatus(listSelected, value.value);
+    }
   };
 
   const selectShowItemsHandler = (value) => {

diff --git a/src/containers/DebtorGroupPage/Component/ListDebtorGroup.jsx b/src/containers/DebtorGroupPage/Component/ListDebtorGroup.jsx
@@ -8,6 +8,7 @@ import ActionsBar from 'components/ActionsBar';
 import history from 'routes/history';
 import { Tab, Tabs } from 'react-bootstrap';
 import SelectComponent from 'components/Select';
+import { notify } from 'components/Toast';
 
 const ListDebtorGroup = observer((props) => {
   const { t } = props;
@@ -98,8 +99,12 @@ const ListDebtorGroup = observer((props) => {
   ];
 
   const selectBulkActionsHandler = (value) => {
-    viewModel.isLoading();
-    viewModel.updateStatus(listSelected, value.value);
+    if (listSelected.length < 1) {
+      notify(t('txt_row_select_error'), 'error');
+    } else {
+      viewModel.isLoading();
+      viewModel.updateStatus(listSelected, value.value);
+    }
   };
 
   const currentSelectHandler = (arr) => {

diff --git a/src/containers/FieldsGroupPage/Component/ListFieldsGroup.jsx b/src/containers/FieldsGroupPage/Component/ListFieldsGroup.jsx
@@ -8,6 +8,7 @@ import Spinner from 'components/Spinner';
 import history from 'routes/history';
 import { Tab, Tabs } from 'react-bootstrap';
 import SelectComponent from 'components/Select';
+import { notify } from 'components/Toast';
 
 const ListFieldsGroup = observer((props) => {
   const { t } = props;
@@ -73,8 +74,12 @@ const ListFieldsGroup = observer((props) => {
   ];
 
   const selectBulkActionsHandler = (value) => {
-    viewModel.isLoading();
-    viewModel.updateStatus(listSelected, value.value);
+    if (listSelected.length < 1) {
+      notify(t('txt_row_select_error'), 'error');
+    } else {
+      viewModel.isLoading();
+      viewModel.updateStatus(listSelected, value.value);
+    }
   };
 
   const currentSelectHandler = (arr) => {

diff --git a/src/containers/FieldsPage/Component/ListFields.jsx b/src/containers/FieldsPage/Component/ListFields.jsx
@@ -79,8 +79,12 @@ const ListFields = observer((props) => {
   };
 
   const selectBulkActionsHandler = ({ value }) => {
-    viewModel.isLoading();
-    viewModel.updateStatus(listSelected, value);
+    if (listSelected.length < 1) {
+      notify(t('txt_row_select_error'), 'error');
+    } else {
+      viewModel.isLoading();
+      viewModel.updateStatus(listSelected, value);
+    }
   };
 
   const currentSelectHandler = (arr) => {

diff --git a/src/containers/ProductPricesPage/Component/ListProductPrice.jsx b/src/containers/ProductPricesPage/Component/ListProductPrice.jsx
@@ -53,8 +53,12 @@ const ListProductPrice = observer((props) => {
   };
 
   const selectBulkActionsHandler = (value) => {
-    viewModel.isLoading();
-    viewModel.updateStatus(listSelected, value.value);
+    if (listSelected.length < 1) {
+      notify(t('txt_row_select_error'), 'error');
+    } else {
+      viewModel.isLoading();
+      viewModel.updateStatus(listSelected, value.value);
+    }
   };
 
   const deleteProductPrices = () => {