Merge pull request #110 from lklong709/PIM-80

fixes #19 - Bulk action - Missing message notification when user use bulk action but not choose item
aesirxio · Jan 16, 2023 · bad0b8c · bad0b8c
2 parents 0231def + 5677fa1
commit bad0b8c
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 8 deletions.
diff --git a/src/containers/CategoriesPage/Component/ListCategories.jsx b/src/containers/CategoriesPage/Component/ListCategories.jsx
@@ -8,6 +8,7 @@ import Table from 'components/Table';
 import Spinner from 'components/Spinner';
 import SelectComponent from 'components/Select';
 import history from 'routes/history';
+import { notify } from 'components/Toast';
 
 const ListCategories = observer((props) => {
   const { t } = props;
@@ -213,8 +214,12 @@ const ListCategories = observer((props) => {
   };
 
   const deleteCategories = () => {
-    viewModel.isLoading();
-    viewModel.deleteCategories(listSelected);
+    if (listSelected.length < 1) {
+      notify(t('txt_row_select_error'), 'error');
+    } else {
+      viewModel.isLoading();
+      viewModel.deleteCategories(listSelected);
+    }
   };
 
   const selectCategoryHandler = (value) => {

diff --git a/src/containers/FieldsPage/Component/ListFields.jsx b/src/containers/FieldsPage/Component/ListFields.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 ListFields = observer((props) => {
   const { t } = props;
@@ -124,8 +125,12 @@ const ListFields = observer((props) => {
   };
 
   const deleteFields = () => {
-    viewModel.isLoading();
-    viewModel.deleteFields(listSelected);
+    if (listSelected.length < 1) {
+      notify(t('txt_row_select_error'), 'error');
+    } else {
+      viewModel.isLoading();
+      viewModel.deleteFields(listSelected);
+    }
   };
 
   return (

diff --git a/src/containers/ProductPricesPage/Component/ListProductPrice.jsx b/src/containers/ProductPricesPage/Component/ListProductPrice.jsx
@@ -9,6 +9,7 @@ import Table from 'components/Table';
 import '../index.scss';
 import ActionsBar from 'components/ActionsBar';
 import history from 'routes/history';
+import { notify } from 'components/Toast';
 
 const ListProductPrice = observer((props) => {
   const { t } = props;
@@ -57,8 +58,12 @@ const ListProductPrice = observer((props) => {
   };
 
   const deleteProductPrices = () => {
-    viewModel.isLoading();
-    viewModel.deleteProductPrices(listSelected);
+    if (listSelected.length < 1) {
+      notify(t('txt_row_select_error'), 'error');
+    } else {
+      viewModel.isLoading();
+      viewModel.deleteProductPrices(listSelected);
+    }
   };
 
   const columnsTable = [

diff --git a/src/containers/ProductsPage/Component/ListProducts.jsx b/src/containers/ProductsPage/Component/ListProducts.jsx
@@ -80,8 +80,12 @@ const ListProducts = observer((props) => {
   };
 
   const deleteProducts = () => {
-    viewModel.isLoading();
-    viewModel.deleteProducts(listSelected);
+    if (listSelected.length < 1) {
+      notify(t('txt_row_select_error'), 'error');
+    } else {
+      viewModel.isLoading();
+      viewModel.deleteProducts(listSelected);
+    }
   };
 
   return (