From f4fd908f8d7ffcdea6acaf75928f6c2960ed6338 Mon Sep 17 00:00:00 2001
From: 1hitsong <3330318+1hitsong@users.noreply.github.com>
Date: Thu, 15 Sep 2022 20:49:25 -0400
Subject: Create ILyricManager

---
 MediaBrowser.Controller/Lyrics/ILyricManager.cs   | 37 +++++++++++++++++
 MediaBrowser.Controller/Lyrics/ILyricProvider.cs  | 29 +++++++++++++
 MediaBrowser.Controller/Lyrics/ILyricsProvider.cs | 24 -----------
 MediaBrowser.Controller/Lyrics/LyricInfo.cs       | 50 +----------------------
 4 files changed, 68 insertions(+), 72 deletions(-)
 create mode 100644 MediaBrowser.Controller/Lyrics/ILyricManager.cs
 create mode 100644 MediaBrowser.Controller/Lyrics/ILyricProvider.cs
 delete mode 100644 MediaBrowser.Controller/Lyrics/ILyricsProvider.cs

(limited to 'MediaBrowser.Controller/Lyrics')

diff --git a/MediaBrowser.Controller/Lyrics/ILyricManager.cs b/MediaBrowser.Controller/Lyrics/ILyricManager.cs
new file mode 100644
index 0000000000..4fd11b9e02
--- /dev/null
+++ b/MediaBrowser.Controller/Lyrics/ILyricManager.cs
@@ -0,0 +1,37 @@
+#nullable disable
+
+#pragma warning disable CS1591
+
+using System;
+using System.Collections.Generic;
+using System.Threading;
+using System.Threading.Tasks;
+using MediaBrowser.Controller.Entities;
+using MediaBrowser.Model.Configuration;
+using MediaBrowser.Model.Providers;
+
+namespace MediaBrowser.Controller.Lyrics
+{
+    public interface ILyricManager
+    {
+        /// <summary>
+        /// Adds the parts.
+        /// </summary>
+        /// <param name="lyricProviders">The lyric providers.</param>
+        void AddParts(IEnumerable<ILyricProvider> lyricProviders);
+
+        /// <summary>
+        /// Gets the lyrics.
+        /// </summary>
+        /// <param name="item">The media item.</param>
+        /// <returns>Lyrics for passed item.</returns>
+        LyricResponse GetLyric(BaseItem item);
+
+        /// <summary>
+        /// Checks if requested item has a matching local lyric file.
+        /// </summary>
+        /// <param name="item">The media item.</param>
+        /// <returns>True if item has a matching lyrics file; otherwise false.</returns>
+        bool HasLyricFile(BaseItem item);
+    }
+}
diff --git a/MediaBrowser.Controller/Lyrics/ILyricProvider.cs b/MediaBrowser.Controller/Lyrics/ILyricProvider.cs
new file mode 100644
index 0000000000..691fed1fd2
--- /dev/null
+++ b/MediaBrowser.Controller/Lyrics/ILyricProvider.cs
@@ -0,0 +1,29 @@
+using System.Collections.Generic;
+using MediaBrowser.Controller.Entities;
+
+namespace MediaBrowser.Controller.Lyrics
+{
+    /// <summary>
+    /// Interface ILyricsProvider.
+    /// </summary>
+    public interface ILyricProvider
+    {
+        /// <summary>
+        /// Gets a value indicating the provider name.
+        /// </summary>
+        string Name { get; }
+
+        /// <summary>
+        /// Gets the supported media types for this provider.
+        /// </summary>
+        /// <value>The supported media types.</value>
+        IEnumerable<string> SupportedMediaTypes { get; }
+
+        /// <summary>
+        /// Gets the lyrics.
+        /// </summary>
+        /// <param name="item">The item to to process.</param>
+        /// <returns>Task{LyricResponse}.</returns>
+        LyricResponse? GetLyrics(BaseItem item);
+    }
+}
diff --git a/MediaBrowser.Controller/Lyrics/ILyricsProvider.cs b/MediaBrowser.Controller/Lyrics/ILyricsProvider.cs
deleted file mode 100644
index bac32a398a..0000000000
--- a/MediaBrowser.Controller/Lyrics/ILyricsProvider.cs
+++ /dev/null
@@ -1,24 +0,0 @@
-using System.Collections.Generic;
-using MediaBrowser.Controller.Entities;
-
-namespace MediaBrowser.Controller.Lyrics
-{
-    /// <summary>
-    /// Interface ILyricsProvider.
-    /// </summary>
-    public interface ILyricsProvider
-    {
-        /// <summary>
-        /// Gets the supported media types for this provider.
-        /// </summary>
-        /// <value>The supported media types.</value>
-        IEnumerable<string> SupportedMediaTypes { get; }
-
-        /// <summary>
-        /// Gets the lyrics.
-        /// </summary>
-        /// <param name="item">The item to to process.</param>
-        /// <returns>Task{LyricResponse}.</returns>
-        LyricResponse? GetLyrics(BaseItem item);
-    }
-}
diff --git a/MediaBrowser.Controller/Lyrics/LyricInfo.cs b/MediaBrowser.Controller/Lyrics/LyricInfo.cs
index 83a10701a2..d44e14237a 100644
--- a/MediaBrowser.Controller/Lyrics/LyricInfo.cs
+++ b/MediaBrowser.Controller/Lyrics/LyricInfo.cs
@@ -13,42 +13,15 @@ namespace MediaBrowser.Controller.Lyrics
     /// <summary>
     /// Item helper.
     /// </summary>
-    public class LyricInfo
+    public static class LyricInfo
     {
-        /// <summary>
-        /// Opens lyrics file, converts to a List of Lyrics, and returns it.
-        /// </summary>
-        /// <param name="lyricProviders">Collection of all registered <see cref="ILyricsProvider"/> interfaces.</param>
-        /// <param name="item">Requested Item.</param>
-        /// <returns>Collection of Lyrics.</returns>
-        public static LyricResponse? GetLyricData(IEnumerable<ILyricsProvider> lyricProviders, BaseItem item)
-        {
-
-            foreach (var provider in lyricProviders)
-            {
-                var result = provider.GetLyrics(item);
-                if (result is not null)
-                {
-                    return result;
-                }
-            }
-
-            return new LyricResponse
-            {
-                Lyrics = new List<Lyric>
-                {
-                    new Lyric { Start = 0, Text = "Test" }
-                }
-            };
-        }
-
         /// <summary>
         /// Checks if requested item has a matching lyric file.
         /// </summary>
         /// <param name="lyricProvider">The current lyricProvider interface.</param>
         /// <param name="itemPath">Path of requested item.</param>
         /// <returns>True if item has a matching lyrics file.</returns>
-        public static string? GetLyricFilePath(ILyricsProvider lyricProvider, string itemPath)
+        public static string? GetLyricFilePath(ILyricProvider lyricProvider, string itemPath)
         {
             if (lyricProvider.SupportedMediaTypes.Any())
             {
@@ -64,24 +37,5 @@ namespace MediaBrowser.Controller.Lyrics
 
             return null;
         }
-
-        /// <summary>
-        /// Checks if requested item has a matching local lyric file.
-        /// </summary>
-        /// <param name="lyricProviders">Collection of all registered <see cref="ILyricsProvider"/> interfaces.</param>
-        /// <param name="itemPath">Path of requested item.</param>
-        /// <returns>True if item has a matching lyrics file; otherwise false.</returns>
-        public static bool HasLyricFile(IEnumerable<ILyricsProvider> lyricProviders, string itemPath)
-        {
-            foreach (var provider in lyricProviders)
-            {
-                if (GetLyricFilePath(provider, itemPath) is not null)
-                {
-                    return true;
-                }
-            }
-
-            return false;
-        }
     }
 }
-- 
cgit v1.2.3