From acf30e00ce11276bcef7fdce3abe5efe834c1fed Mon Sep 17 00:00:00 2001
From: Bond_009 <bond.009@outlook.com>
Date: Mon, 28 Feb 2022 00:46:34 +0100
Subject: [PATCH] Remove obsolete code for premium plugins

---
 Jellyfin.Api/Controllers/PluginsController.cs | 71 -------------------
 .../Models/PluginDtos/MBRegistrationRecord.cs | 40 -----------
 .../Models/PluginDtos/PluginSecurityInfo.cs   | 18 -----
 3 files changed, 129 deletions(-)
 delete mode 100644 Jellyfin.Api/Models/PluginDtos/MBRegistrationRecord.cs
 delete mode 100644 Jellyfin.Api/Models/PluginDtos/PluginSecurityInfo.cs

diff --git a/Jellyfin.Api/Controllers/PluginsController.cs b/Jellyfin.Api/Controllers/PluginsController.cs
index b41df1abb1..b227dba2df 100644
--- a/Jellyfin.Api/Controllers/PluginsController.cs
+++ b/Jellyfin.Api/Controllers/PluginsController.cs
@@ -7,7 +7,6 @@ using System.Text.Json;
 using System.Threading.Tasks;
 using Jellyfin.Api.Attributes;
 using Jellyfin.Api.Constants;
-using Jellyfin.Api.Models.PluginDtos;
 using Jellyfin.Extensions.Json;
 using MediaBrowser.Common.Plugins;
 using MediaBrowser.Common.Updates;
@@ -43,61 +42,6 @@ namespace Jellyfin.Api.Controllers
             _serializerOptions = JsonDefaults.Options;
         }
 
-        /// <summary>
-        /// Get plugin security info.
-        /// </summary>
-        /// <response code="200">Plugin security info returned.</response>
-        /// <returns>Plugin security info.</returns>
-        [Obsolete("This endpoint should not be used.")]
-        [HttpGet("SecurityInfo")]
-        [ProducesResponseType(StatusCodes.Status200OK)]
-        public static ActionResult<PluginSecurityInfo> GetPluginSecurityInfo()
-        {
-            return new PluginSecurityInfo
-            {
-                IsMbSupporter = true,
-                SupporterKey = "IAmTotallyLegit"
-            };
-        }
-
-        /// <summary>
-        /// Gets registration status for a feature.
-        /// </summary>
-        /// <param name="name">Feature name.</param>
-        /// <response code="200">Registration status returned.</response>
-        /// <returns>Mb registration record.</returns>
-        [Obsolete("This endpoint should not be used.")]
-        [HttpPost("RegistrationRecords/{name}")]
-        [ProducesResponseType(StatusCodes.Status200OK)]
-        public static ActionResult<MBRegistrationRecord> GetRegistrationStatus([FromRoute, Required] string name)
-        {
-            return new MBRegistrationRecord
-            {
-                IsRegistered = true,
-                RegChecked = true,
-                TrialVersion = false,
-                IsValid = true,
-                RegError = false
-            };
-        }
-
-        /// <summary>
-        /// Gets registration status for a feature.
-        /// </summary>
-        /// <param name="name">Feature name.</param>
-        /// <response code="501">Not implemented.</response>
-        /// <returns>Not Implemented.</returns>
-        /// <exception cref="NotImplementedException">This endpoint is not implemented.</exception>
-        [Obsolete("Paid plugins are not supported")]
-        [HttpGet("Registrations/{name}")]
-        [ProducesResponseType(StatusCodes.Status501NotImplemented)]
-        public static ActionResult GetRegistration([FromRoute, Required] string name)
-        {
-            // TODO Once we have proper apps and plugins and decide to break compatibility with paid plugins,
-            // delete all these registration endpoints. They are only kept for compatibility.
-            throw new NotImplementedException();
-        }
-
         /// <summary>
         /// Gets a list of currently installed plugins.
         /// </summary>
@@ -317,20 +261,5 @@ namespace Jellyfin.Api.Controllers
 
             return NotFound();
         }
-
-        /// <summary>
-        /// Updates plugin security info.
-        /// </summary>
-        /// <param name="pluginSecurityInfo">Plugin security info.</param>
-        /// <response code="204">Plugin security info updated.</response>
-        /// <returns>An <see cref="NoContentResult"/>.</returns>
-        [Obsolete("This endpoint should not be used.")]
-        [HttpPost("SecurityInfo")]
-        [Authorize(Policy = Policies.RequiresElevation)]
-        [ProducesResponseType(StatusCodes.Status204NoContent)]
-        public ActionResult UpdatePluginSecurityInfo([FromBody, Required] PluginSecurityInfo pluginSecurityInfo)
-        {
-            return NoContent();
-        }
     }
 }
diff --git a/Jellyfin.Api/Models/PluginDtos/MBRegistrationRecord.cs b/Jellyfin.Api/Models/PluginDtos/MBRegistrationRecord.cs
deleted file mode 100644
index 7f1255f4b6..0000000000
--- a/Jellyfin.Api/Models/PluginDtos/MBRegistrationRecord.cs
+++ /dev/null
@@ -1,40 +0,0 @@
-using System;
-
-namespace Jellyfin.Api.Models.PluginDtos
-{
-    /// <summary>
-    /// MB Registration Record.
-    /// </summary>
-    public class MBRegistrationRecord
-    {
-        /// <summary>
-        /// Gets or sets expiration date.
-        /// </summary>
-        public DateTime ExpirationDate { get; set; }
-
-        /// <summary>
-        /// Gets or sets a value indicating whether is registered.
-        /// </summary>
-        public bool IsRegistered { get; set; }
-
-        /// <summary>
-        /// Gets or sets a value indicating whether reg checked.
-        /// </summary>
-        public bool RegChecked { get; set; }
-
-        /// <summary>
-        /// Gets or sets a value indicating whether reg error.
-        /// </summary>
-        public bool RegError { get; set; }
-
-        /// <summary>
-        /// Gets or sets a value indicating whether trial version.
-        /// </summary>
-        public bool TrialVersion { get; set; }
-
-        /// <summary>
-        /// Gets or sets a value indicating whether is valid.
-        /// </summary>
-        public bool IsValid { get; set; }
-    }
-}
diff --git a/Jellyfin.Api/Models/PluginDtos/PluginSecurityInfo.cs b/Jellyfin.Api/Models/PluginDtos/PluginSecurityInfo.cs
deleted file mode 100644
index a90398425a..0000000000
--- a/Jellyfin.Api/Models/PluginDtos/PluginSecurityInfo.cs
+++ /dev/null
@@ -1,18 +0,0 @@
-namespace Jellyfin.Api.Models.PluginDtos
-{
-    /// <summary>
-    /// Plugin security info.
-    /// </summary>
-    public class PluginSecurityInfo
-    {
-        /// <summary>
-        /// Gets or sets the supporter key.
-        /// </summary>
-        public string? SupporterKey { get; set; }
-
-        /// <summary>
-        /// Gets or sets a value indicating whether is mb supporter.
-        /// </summary>
-        public bool IsMbSupporter { get; set; }
-    }
-}