From a7d2dd7bdb3441f5c5770275ec1a3b74851cc6e6 Mon Sep 17 00:00:00 2001
From: Artem Baskal <a.baskal@adguard.com>
Date: Fri, 2 Oct 2020 13:30:19 +0300
Subject: [PATCH] - client: 2144 Fix DNS allowlists tab crash after opening
 "Choose blocklists" modal

Close #2144

Squashed commit of the following:

commit 435f17df9ce41e0f56b78cf8b7a390b73f16243b
Merge: 8899d65e 69a74071
Author: ArtemBaskal <a.baskal@adguard.com>
Date:   Fri Oct 2 13:02:36 2020 +0300

    Merge branch 'master' into fix/2144

commit 8899d65e9cf320fa07561f1f5a35cd3424a49a32
Author: ArtemBaskal <a.baskal@adguard.com>
Date:   Thu Oct 1 23:00:06 2020 +0300

    - client: 2144 Fix DNS allowlists tab crash after opening "Choose blocklists" modal
---
 client/src/components/Filters/DnsBlocklist.js | 2 +-
 client/src/components/Filters/Form.js         | 4 ++--
 client/src/components/Filters/Modal.js        | 2 +-
 client/src/reducers/filtering.js              | 1 +
 4 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/client/src/components/Filters/DnsBlocklist.js b/client/src/components/Filters/DnsBlocklist.js
index 05418d1c..4059b48f 100644
--- a/client/src/components/Filters/DnsBlocklist.js
+++ b/client/src/components/Filters/DnsBlocklist.js
@@ -23,7 +23,7 @@ class DnsBlocklist extends Component {
     }
 
     handleSubmit = (values, _, { initialValues }) => {
-        const { filtering: { modalFilterUrl, modalType } } = this.props;
+        const { modalFilterUrl, modalType } = this.props.filtering;
 
         switch (modalType) {
             case MODAL_TYPE.EDIT_FILTERS:
diff --git a/client/src/components/Filters/Form.js b/client/src/components/Filters/Form.js
index a994c1ec..9b2787d2 100644
--- a/client/src/components/Filters/Form.js
+++ b/client/src/components/Filters/Form.js
@@ -8,6 +8,8 @@ import { validatePath, validateRequiredValue } from '../../helpers/validators';
 import { renderCheckboxField, renderInputField } from '../../helpers/form';
 import { MODAL_OPEN_TIMEOUT, MODAL_TYPE, FORM_NAME } from '../../helpers/constants';
 
+const filtersCatalog = require('../../helpers/filters/filters.json');
+
 const getIconsData = (homepage, source) => ([
     {
         iconName: 'dashboard',
@@ -82,7 +84,6 @@ const Form = (props) => {
         modalType,
         toggleFilteringModal,
         selectedSources,
-        filtersCatalog,
     } = props;
 
     const openModal = (modalType, timeout = MODAL_OPEN_TIMEOUT) => {
@@ -168,7 +169,6 @@ Form.propTypes = {
     whitelist: PropTypes.bool,
     modalType: PropTypes.string.isRequired,
     toggleFilteringModal: PropTypes.func.isRequired,
-    filtersCatalog: PropTypes.object,
     selectedSources: PropTypes.object,
 };
 
diff --git a/client/src/components/Filters/Modal.js b/client/src/components/Filters/Modal.js
index 0421f548..aba53b30 100644
--- a/client/src/components/Filters/Modal.js
+++ b/client/src/components/Filters/Modal.js
@@ -78,6 +78,7 @@ class Modal extends Component {
                 break;
             }
             default:
+                break;
         }
 
         const title = t(getTitle(modalType, whitelist));
@@ -99,7 +100,6 @@ class Modal extends Component {
                     <Form
                         selectedSources={selectedSources}
                         initialValues={initialValues}
-                        filtersCatalog={filtersCatalog}
                         modalType={modalType}
                         onSubmit={handleSubmit}
                         processingAddFilter={processingAddFilter}
diff --git a/client/src/reducers/filtering.js b/client/src/reducers/filtering.js
index a8e5a766..649fd532 100644
--- a/client/src/reducers/filtering.js
+++ b/client/src/reducers/filtering.js
@@ -56,6 +56,7 @@ const filtering = handleActions(
                 ...state,
                 isModalOpen: !state.isModalOpen,
                 isFilterAdded: false,
+                modalType: '',
             };
             return newState;
         },