New: Group Import Lists by Type

frontend/src/Settings/ImportLists/ImportLists/AddImportListModalContent.js

@@ -11,6 +11,7 @@ import ModalBody from 'Components/Modal/ModalBody';
 import ModalFooter from 'Components/Modal/ModalFooter';
 import AddImportListItem from './AddImportListItem';
 import styles from './AddImportListModalContent.css';
+import titleCase from 'Utilities/String/titleCase';
 
 class AddImportListModalContent extends Component {
 
@@ -22,7 +23,7 @@ class AddImportListModalContent extends Component {
       isSchemaFetching,
       isSchemaPopulated,
       schemaError,
-      allLists,
+      listGroups,
       onImportListSelect,
       onModalClose
     } = this.props;
@@ -52,23 +53,28 @@ class AddImportListModalContent extends Component {
                   <div>Lidarr supports multiple lists for importing Albums and Artists into the database.</div>
                   <div>For more information on the individual lists, click on the info buttons.</div>
                 </Alert>
-
-                <FieldSet legend="Import Lists">
-                  <div className={styles.lists}>
-                    {
-                      allLists.map((list) => {
-                        return (
-                          <AddImportListItem
-                            key={list.implementation}
-                            implementation={list.implementation}
-                            {...list}
-                            onImportListSelect={onImportListSelect}
-                          />
-                        );
-                      })
-                    }
-                  </div>
-                </FieldSet>
+                {
+                  Object.keys(listGroups).map((key) => {
+                    return (
+                      <FieldSet legend={`${titleCase(key)} List`} key={key}>
+                        <div className={styles.lists}>
+                          {
+                            listGroups[key].map((list) => {
+                              return (
+                                <AddImportListItem
+                                  key={list.implementation}
+                                  implementation={list.implementation}
+                                  {...list}
+                                  onImportListSelect={onImportListSelect}
+                                />
+                              );
+                            })
+                          }
+                        </div>
+                      </FieldSet>
+                    );
+                  })
+                }
               </div>
           }
         </ModalBody>
@@ -88,7 +94,7 @@ AddImportListModalContent.propTypes = {
   isSchemaFetching: PropTypes.bool.isRequired,
   isSchemaPopulated: PropTypes.bool.isRequired,
   schemaError: PropTypes.object,
-  allLists: PropTypes.arrayOf(PropTypes.object).isRequired,
+  listGroups: PropTypes.object.isRequired,
   onImportListSelect: PropTypes.func.isRequired,
   onModalClose: PropTypes.func.isRequired
 };

frontend/src/Settings/ImportLists/ImportLists/AddImportListModalContentConnector.js

@@ -1,3 +1,4 @@
+import _ from 'lodash';
 import PropTypes from 'prop-types';
 import React, { Component } from 'react';
 import { connect } from 'react-redux';
@@ -16,13 +17,13 @@ function createMapStateToProps() {
         schema
       } = importLists;
 
-      const allLists = schema;
+      const listGroups = _.groupBy(schema, 'listType');
 
       return {
         isSchemaFetching,
         isSchemaPopulated,
         schemaError,
-        allLists
+        listGroups
       };
     }
   );

src/Lidarr.Api.V1/ImportLists/ImportListResource.cs

@@ -10,6 +10,7 @@ namespace Lidarr.Api.V1.ImportLists
         public int QualityProfileId { get; set; }
         public int LanguageProfileId { get; set; }
         public int MetadataProfileId { get; set; }
+        public ImportListType ListType { get; set; }
     }
 
     public class ImportListResourceMapper : ProviderResourceMapper<ImportListResource, ImportListDefinition>
@@ -29,6 +30,7 @@ namespace Lidarr.Api.V1.ImportLists
             resource.QualityProfileId = definition.ProfileId;
             resource.LanguageProfileId = definition.LanguageProfileId;
             resource.MetadataProfileId = definition.MetadataProfileId;
+            resource.ListType = definition.ListType;
 
             return resource;
         }
@@ -48,6 +50,7 @@ namespace Lidarr.Api.V1.ImportLists
             definition.ProfileId = resource.QualityProfileId;
             definition.LanguageProfileId = resource.LanguageProfileId;
             definition.MetadataProfileId = resource.MetadataProfileId;
+            definition.ListType = resource.ListType;
 
             return definition;
         }

src/NzbDrone.Core/Datastore/TableMapping.cs

@@ -68,7 +68,8 @@ namespace NzbDrone.Core.Datastore
                   .Ignore(d => d.Tags);
 
             Mapper.Entity<ImportListDefinition>().RegisterDefinition("ImportLists")
-                .Ignore(i => i.Enable);
+                .Ignore(i => i.Enable)
+                .Ignore(i => i.ListType);
 
             Mapper.Entity<NotificationDefinition>().RegisterDefinition("Notifications")
                   .Ignore(i => i.SupportsOnGrab)

src/NzbDrone.Core/ImportLists/HeadphonesImport/HeadphonesImport.cs

@@ -9,6 +9,8 @@ namespace NzbDrone.Core.ImportLists.HeadphonesImport
     {
         public override string Name => "Headphones";
 
+        public override ImportListType ListType => ImportListType.Other;
+
         public override int PageSize => 1000;
 
         public HeadphonesImport(IHttpClient httpClient, IImportListStatusService importListStatusService, IConfigService configService, IParsingService parsingService, Logger logger)

src/NzbDrone.Core/ImportLists/IImportList.cs

@@ -6,6 +6,7 @@ namespace NzbDrone.Core.ImportLists
 {
     public interface IImportList : IProvider
     {
+        ImportListType ListType { get; }
         IList<ImportListItemInfo> Fetch();
     }
 }

src/NzbDrone.Core/ImportLists/ImportListBase.cs

@@ -21,6 +21,8 @@ namespace NzbDrone.Core.ImportLists
 
         public abstract string Name { get; }
 
+        public abstract ImportListType ListType {get; }
+
         public ImportListBase(IImportListStatusService importListStatusService, IConfigService configService, IParsingService parsingService, Logger logger)
         {
             _importListStatusService = importListStatusService;

src/NzbDrone.Core/ImportLists/ImportListDefinition.cs

@@ -14,6 +14,7 @@ namespace NzbDrone.Core.ImportLists
         public override bool Enable => EnableAutomaticAdd;
 
         public ImportListStatus Status { get; set; }
+        public ImportListType ListType { get; set; }
     }
 
     public enum ImportListMonitorType

src/NzbDrone.Core/ImportLists/ImportListFactory.cs

@@ -35,6 +35,13 @@ namespace NzbDrone.Core.ImportLists
             return base.Active().Where(c => c.Enable).ToList();
         }
 
+        public override void SetProviderCharacteristics(IImportList provider, ImportListDefinition definition)
+        {
+            base.SetProviderCharacteristics(provider, definition);
+
+            definition.ListType = provider.ListType;
+        }
+
         public List<IImportList> AutomaticAddEnabled(bool filterBlockedImportLists = true)
         {
             var enabledImportLists = GetAvailableProviders().Where(n => ((ImportListDefinition)n.Definition).EnableAutomaticAdd);

src/NzbDrone.Core/ImportLists/ImportListType.cs

@@ -0,0 +1,15 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+
+namespace NzbDrone.Core.ImportLists
+{
+    public enum ImportListType
+    {
+        Spotify,
+        LastFm,
+        Other
+    }
+}

src/NzbDrone.Core/ImportLists/LastFm/LastFmTag.cs

@@ -9,6 +9,8 @@ namespace NzbDrone.Core.ImportLists.LastFm
     {
         public override string Name => "Last.fm Tag";
 
+        public override ImportListType ListType => ImportListType.LastFm;
+
         public override int PageSize => 1000;
 
         public LastFmTag(IHttpClient httpClient, IImportListStatusService importListStatusService, IConfigService configService, IParsingService parsingService, Logger logger)

src/NzbDrone.Core/ImportLists/LastFm/LastFmUser.cs

@@ -9,6 +9,8 @@ namespace NzbDrone.Core.ImportLists.LastFm
     {
         public override string Name => "Last.fm User";
 
+        public override ImportListType ListType => ImportListType.LastFm;
+
         public override int PageSize => 1000;
 
         public LastFmUser(IHttpClient httpClient, IImportListStatusService importListStatusService, IConfigService configService, IParsingService parsingService, Logger logger)

src/NzbDrone.Core/ImportLists/LidarrLists/LidarrLists.cs

@@ -12,6 +12,8 @@ namespace NzbDrone.Core.ImportLists.LidarrLists
     {
         public override string Name => "Lidarr Lists";
 
+        public override ImportListType ListType => ImportListType.Other;
+
         public override int PageSize => 10;
 
         private readonly IMetadataRequestBuilder _requestBuilder;

src/NzbDrone.Core/ImportLists/Spotify/SpotifyImportListBase.cs

@@ -32,6 +32,8 @@ namespace NzbDrone.Core.ImportLists.Spotify
             _importListRepository = importListRepository;
         }
 
+        public override ImportListType ListType => ImportListType.Spotify;
+
         private void RefreshToken()
         {
             _logger.Trace("Refreshing Token");

src/NzbDrone.Core/NzbDrone.Core.csproj

@@ -552,6 +552,7 @@
     <Compile Include="ImportLists\ImportListBase.cs" />
     <Compile Include="ImportLists\ImportListPageableRequestChain.cs" />
     <Compile Include="ImportLists\ImportListPageableRequest.cs" />
+    <Compile Include="ImportLists\ImportListType.cs" />
     <Compile Include="ImportLists\ImportListUpdatedHandler.cs" />
     <Compile Include="ImportLists\IProcessImportListResponse.cs" />
     <Compile Include="ImportLists\ImportListSyncService.cs" />