From 2006ce42d0e935225ee6d678b40c2b66dbed187b Mon Sep 17 00:00:00 2001
From: Qstick <qstick@gmail.com>
Date: Fri, 23 Oct 2020 14:57:54 -0400
Subject: [PATCH] Fix UI issues with no indexers exist

---
 frontend/src/Indexer/Index/IndexerIndex.js    |  5 ++-
 frontend/src/Indexer/NoIndexer.js             | 10 +++--
 frontend/src/Search/SearchFooter.js           |  6 ++-
 frontend/src/Search/SearchIndex.js            | 37 ++++++++++++++++++-
 frontend/src/Search/SearchIndexConnector.js   |  3 +-
 .../Indexers/Indexers/AddIndexerItem.js       |  2 -
 6 files changed, 52 insertions(+), 11 deletions(-)

diff --git a/frontend/src/Indexer/Index/IndexerIndex.js b/frontend/src/Indexer/Index/IndexerIndex.js
index a10e1f74d..cb4b6ac39 100644
--- a/frontend/src/Indexer/Index/IndexerIndex.js
+++ b/frontend/src/Indexer/Index/IndexerIndex.js
@@ -429,7 +429,10 @@ class IndexerIndex extends Component {
 
             {
               !error && isPopulated && !items.length &&
-                <NoIndexer totalItems={totalItems} />
+                <NoIndexer
+                  totalItems={totalItems}
+                  onAddIndexerPress={this.onAddIndexerPress}
+                />
             }
           </PageContentBody>
 
diff --git a/frontend/src/Indexer/NoIndexer.js b/frontend/src/Indexer/NoIndexer.js
index f135b4d10..f94df7902 100644
--- a/frontend/src/Indexer/NoIndexer.js
+++ b/frontend/src/Indexer/NoIndexer.js
@@ -6,7 +6,10 @@ import translate from 'Utilities/String/translate';
 import styles from './NoIndexer.css';
 
 function NoIndexer(props) {
-  const { totalItems } = props;
+  const {
+    totalItems,
+    onAddIndexerPress
+  } = props;
 
   if (totalItems > 0) {
     return (
@@ -26,7 +29,7 @@ function NoIndexer(props) {
 
       <div className={styles.buttonContainer}>
         <Button
-          to="/add/new"
+          onPress={onAddIndexerPress}
           kind={kinds.PRIMARY}
         >
           {translate('AddNewIndexer')}
@@ -37,7 +40,8 @@ function NoIndexer(props) {
 }
 
 NoIndexer.propTypes = {
-  totalItems: PropTypes.number.isRequired
+  totalItems: PropTypes.number.isRequired,
+  onAddIndexerPress: PropTypes.func.isRequired
 };
 
 export default NoIndexer;
diff --git a/frontend/src/Search/SearchFooter.js b/frontend/src/Search/SearchFooter.js
index 976afc8ab..2409ea6aa 100644
--- a/frontend/src/Search/SearchFooter.js
+++ b/frontend/src/Search/SearchFooter.js
@@ -51,7 +51,8 @@ class SearchFooter extends Component {
 
   render() {
     const {
-      isFetching
+      isFetching,
+      hasIndexers
     } = this.props;
 
     const {
@@ -97,7 +98,7 @@ class SearchFooter extends Component {
               <SpinnerButton
                 className={styles.deleteSelectedButton}
                 isSpinning={isFetching}
-                isDisabled={isFetching}
+                isDisabled={isFetching || !hasIndexers}
                 onPress={this.onSearchPress}
               >
                 Search
@@ -113,6 +114,7 @@ class SearchFooter extends Component {
 SearchFooter.propTypes = {
   isFetching: PropTypes.bool.isRequired,
   onSearchPress: PropTypes.func.isRequired,
+  hasIndexers: PropTypes.bool.isRequired,
   searchError: PropTypes.object
 };
 
diff --git a/frontend/src/Search/SearchIndex.js b/frontend/src/Search/SearchIndex.js
index 238775259..3775b46fa 100644
--- a/frontend/src/Search/SearchIndex.js
+++ b/frontend/src/Search/SearchIndex.js
@@ -12,6 +12,8 @@ import PageToolbarSeparator from 'Components/Page/Toolbar/PageToolbarSeparator';
 import TableOptionsModalWrapper from 'Components/Table/TableOptions/TableOptionsModalWrapper';
 import { align, icons, sortDirections } from 'Helpers/Props';
 import NoIndexer from 'Indexer/NoIndexer';
+import AddIndexerModal from 'Settings/Indexers/Indexers/AddIndexerModal';
+import EditIndexerModalConnector from 'Settings/Indexers/Indexers/EditIndexerModalConnector';
 import * as keyCodes from 'Utilities/Constants/keyCodes';
 import getErrorMessage from 'Utilities/Object/getErrorMessage';
 import hasDifferentItemsOrOrder from 'Utilities/Object/hasDifferentItemsOrOrder';
@@ -39,6 +41,8 @@ class SearchIndex extends Component {
       scroller: null,
       jumpBarItems: { order: [] },
       jumpToCharacter: null,
+      isAddIndexerModalOpen: false,
+      isEditIndexerModalOpen: false,
       searchType: null,
       lastToggled: null
     };
@@ -123,6 +127,21 @@ class SearchIndex extends Component {
   //
   // Listeners
 
+  onAddIndexerPress = () => {
+    this.setState({ isAddIndexerModalOpen: true });
+  }
+
+  onAddIndexerModalClose = ({ indexerSelected = false } = {}) => {
+    this.setState({
+      isAddIndexerModalOpen: false,
+      isEditIndexerModalOpen: indexerSelected
+    });
+  }
+
+  onEditIndexerModalClose = () => {
+    this.setState({ isEditIndexerModalOpen: false });
+  }
+
   onJumpBarItemPress = (jumpToCharacter) => {
     this.setState({ jumpToCharacter });
   }
@@ -169,6 +188,8 @@ class SearchIndex extends Component {
     const {
       scroller,
       jumpBarItems,
+      isAddIndexerModalOpen,
+      isEditIndexerModalOpen,
       jumpToCharacter
     } = this.state;
 
@@ -248,7 +269,10 @@ class SearchIndex extends Component {
 
             {
               !error && !isFetching && !hasIndexers &&
-                <NoIndexer />
+                <NoIndexer
+                  totalItems={0}
+                  onAddIndexerPress={this.onAddIndexerPress}
+                />
             }
 
             {
@@ -268,8 +292,19 @@ class SearchIndex extends Component {
 
         <SearchFooter
           isFetching={isFetching}
+          hasIndexers={hasIndexers}
           onSearchPress={this.onSearchPress}
         />
+
+        <AddIndexerModal
+          isOpen={isAddIndexerModalOpen}
+          onModalClose={this.onAddIndexerModalClose}
+        />
+
+        <EditIndexerModalConnector
+          isOpen={isEditIndexerModalOpen}
+          onModalClose={this.onEditIndexerModalClose}
+        />
       </PageContent>
     );
   }
diff --git a/frontend/src/Search/SearchIndexConnector.js b/frontend/src/Search/SearchIndexConnector.js
index 26f59a022..571cec5ad 100644
--- a/frontend/src/Search/SearchIndexConnector.js
+++ b/frontend/src/Search/SearchIndexConnector.js
@@ -6,13 +6,12 @@ import withScrollPosition from 'Components/withScrollPosition';
 import { fetchReleases, setReleasesFilter, setReleasesSort, setReleasesTableOption } from 'Store/Actions/releaseActions';
 import scrollPositions from 'Store/scrollPositions';
 import createDimensionsSelector from 'Store/Selectors/createDimensionsSelector';
-import createIndexerClientSideCollectionItemsSelector from 'Store/Selectors/createIndexerClientSideCollectionItemsSelector';
 import createReleaseClientSideCollectionItemsSelector from 'Store/Selectors/createReleaseClientSideCollectionItemsSelector';
 import SearchIndex from './SearchIndex';
 
 function createMapStateToProps() {
   return createSelector(
-    createIndexerClientSideCollectionItemsSelector('indexerIndex'),
+    (state) => state.indexers,
     createReleaseClientSideCollectionItemsSelector('releases'),
     createDimensionsSelector(),
     (
diff --git a/frontend/src/Settings/Indexers/Indexers/AddIndexerItem.js b/frontend/src/Settings/Indexers/Indexers/AddIndexerItem.js
index 48efe2b6b..2dadcffb2 100644
--- a/frontend/src/Settings/Indexers/Indexers/AddIndexerItem.js
+++ b/frontend/src/Settings/Indexers/Indexers/AddIndexerItem.js
@@ -30,7 +30,6 @@ class AddIndexerItem extends Component {
     const {
       name,
       implementation,
-      implementationName,
       infoLink,
       presets,
       onIndexerSelect
@@ -105,7 +104,6 @@ class AddIndexerItem extends Component {
 AddIndexerItem.propTypes = {
   name: PropTypes.string.isRequired,
   implementation: PropTypes.string.isRequired,
-  implementationName: PropTypes.string.isRequired,
   infoLink: PropTypes.string.isRequired,
   presets: PropTypes.arrayOf(PropTypes.object),
   onIndexerSelect: PropTypes.func.isRequired