Minor bug fixes:

Handle cases where shows do not have any episode groups
Fix chosen episode group not being selected
Fix error when finding cover filename
Increment config version

Author: jamerst

.vscode/launch.json

@@ -14,7 +14,7 @@
             "args": ["/home/james/Desktop/test", "--extended-tagging"],
             "cwd": "${workspaceFolder}/AutoTag.CLI",
             // For more information about the 'console' field, see https://aka.ms/VSCode-CS-LaunchJson-Console
-            "console": "internalConsole",
+            "console": "externalTerminal",
             "stopAtEntry": false
         },
         {

AutoTag.Core/AutoTagConfig.cs

@@ -2,7 +2,7 @@ namespace AutoTag.Core;
 public class AutoTagConfig
 {
     public enum Modes { TV, Movie };
-    public const int CurrentVer = 9;
+    public const int CurrentVer = 10;
     public int ConfigVer { get; set; } = CurrentVer;
     public Modes Mode { get; set; } = Modes.TV;
     public bool ManualMode { get; set; } = false;

AutoTag.Core/TV/ShowResults.cs

@@ -31,15 +31,15 @@ public ShowResults(SearchTv tvSearchResult)
     public static implicit operator ShowResults(SearchTv tv) => new(tv);
 
     /// <summary>
-    /// Generates <see cref="ShowResults"/> list from any <see cref="SearchTv"/> enumerable 
+    /// Generates <see cref="ShowResults"/> list from any <see cref="SearchTv"/> enumerable
     /// </summary>
     /// <param name="results">Result from tmdb client</param>
     public static List<ShowResults> FromSearchResults(IEnumerable<SearchTv> results)
     {
         return results.Select(result => (ShowResults)result).ToList();
     }
 
-    
+
     /// <summary>
     /// Add optional episode group to tv result
     /// </summary>
@@ -71,7 +71,7 @@ public bool TryGetMapping(int seasonNumber, int episodeNumber, out (int season,
 
         return false;
     }
-    
+
     /// <summary>
     /// Tries to generate mapping table between TMDB standard sorting of show
     /// and the given Episode Group

AutoTag.Core/TV/TVProcessor.cs

@@ -121,34 +121,40 @@ FileWriter writer
             {
                 _shows.Add(episodeData.SeriesName, ShowResults.FromSearchResults(seriesResults));
             }
-            
+
             if (config.EpisodeGroup)
             {
                 var seriesResult = _shows[episodeData.SeriesName][0];
                 var tvShow = await _tmdb.GetTvShowAsync(seriesResult.TvSearchResult.Id, TvShowMethods.EpisodeGroups);
                 var groups = tvShow.EpisodeGroups;
-            
-                var chosenGroup = selectResult(groups.Results.Select(group =>
-                    ($"[{group.Type}] {group.Name}", $"S: {group.GroupCount} E: {group.EpisodeCount}")).ToList());
-            
-                if (chosenGroup.HasValue)
+
+                if (groups.Results.Any())
                 {
-                    var groupInfo = await _tmdb.GetTvEpisodeGroupsAsync(groups.Results[0].Id, config.Language);
-                    if (!seriesResult.AddEpisodeGroup(groupInfo))
+                    var chosenGroup = selectResult(groups.Results.Select(group =>
+                        ($"[{group.Type}] {group.Name}", $"{group.GroupCount} seasons, {group.EpisodeCount} episodes")).ToList());
+
+                    if (chosenGroup.HasValue)
                     {
-                        setStatus($"Error: Episode Group {groupInfo.Name} is not containing Seasons or Volumes! ", MessageType.Error);
-                        result.Success = false;
-                        return false;
+                        var groupInfo = await _tmdb.GetTvEpisodeGroupsAsync(groups.Results[chosenGroup.Value].Id, config.Language);
+                        if (!seriesResult.AddEpisodeGroup(groupInfo))
+                        {
+                            setStatus($"Error: Unable to parse required information from episode group {groupInfo.Name}", MessageType.Error);
+                            result.Success = false;
+                            return false;
+                        }
+                    }
+                    else
+                    {
+                        setStatus("File skipped", MessageType.Warning);
+                        return true;
                     }
                 }
                 else
                 {
-                    setStatus("File skipped", MessageType.Warning);
-                    return true;
+                    setStatus($"No episode groups found for series", MessageType.Warning);
                 }
             }
         }
-        
 
         // try searching for each series search result
         foreach (var show in _shows[episodeData.SeriesName])
@@ -157,8 +163,8 @@ FileWriter writer
 
             var lookupSeason = episodeData.Season;
             var lookupEpisode = episodeData.Episode;
-            
-            if(config.EpisodeGroup)
+
+            if (show.EpisodeGroupMappingTable != null)
             {
                 if (!show.TryGetMapping(episodeData.Season, episodeData.Episode, out var groupNumbering))
                 {
@@ -168,7 +174,7 @@ FileWriter writer
                 lookupSeason = groupNumbering!.Value.season;
                 lookupEpisode = groupNumbering.Value.episode;
             }
-            
+
             result.Id = showData.Id;
             result.SeriesName = showData.Name;
 
@@ -242,7 +248,7 @@ FileWriter writer
                 if (seriesImages.Posters.Any())
                 {
                     var bestVotedImage = seriesImages.Posters.OrderByDescending(p => p.VoteAverage).First();
-                    
+
                     result.CoverURL = $"https://image.tmdb.org/t/p/original/{bestVotedImage.FilePath}";
                     _seasonPosters.Add((result.SeriesName, result.Season), result.CoverURL);
                 }
@@ -255,13 +261,13 @@ FileWriter writer
         }
         #endregion
 
-        result.CoverFilename = result.CoverURL?.Split('/')[-1];
+        result.CoverFilename = result.CoverURL?.Split('/')[^1];
 
         var taggingSuccess = await writer.WriteAsync(file, result, setPath, setStatus, config);
 
         return taggingSuccess && result.Success && result.Complete;
     }
-    
+
     private static double SeriesNameSimilarity(string parsedName, string seriesName)
     {
         if (seriesName.ToLower().Contains(parsedName.ToLower()))