Remove OMG indexer

Closes #7527

Co-Authored-By: Mark McDowall <markus101@users.noreply.github.com>

src/NzbDrone.Core.Test/IndexerTests/IndexerServiceFixture.cs

@@ -1,10 +1,10 @@
-using System.Collections.Generic;
+using System.Collections.Generic;
 using FizzWare.NBuilder;
 using FluentAssertions;
 using NUnit.Framework;
 using NzbDrone.Core.Indexers;
+using NzbDrone.Core.Indexers.FileList;
 using NzbDrone.Core.Indexers.Newznab;
-using NzbDrone.Core.Indexers.Omgwtfnzbs;
 using NzbDrone.Core.Lifecycle;
 using NzbDrone.Core.Test.Framework;
 
@@ -20,7 +20,7 @@ namespace NzbDrone.Core.Test.IndexerTests
             _indexers = new List<IIndexer>();
 
             _indexers.Add(Mocker.Resolve<Newznab>());
-            _indexers.Add(Mocker.Resolve<Omgwtfnzbs>());
+            _indexers.Add(Mocker.Resolve<FileList>());
 
             Mocker.SetConstant<IEnumerable<IIndexer>>(_indexers);
         }

src/NzbDrone.Core.Test/IndexerTests/OmgwtfnzbsTests/OmgwtfnzbsFixture.cs

@@ -1,56 +0,0 @@
-using System;
-using System.Linq;
-using System.Net.Http;
-using FluentAssertions;
-using Moq;
-using NUnit.Framework;
-using NzbDrone.Common.Http;
-using NzbDrone.Core.Indexers;
-using NzbDrone.Core.Indexers.Omgwtfnzbs;
-using NzbDrone.Core.Test.Framework;
-
-namespace NzbDrone.Core.Test.IndexerTests.OmgwtfnzbsTests
-{
-    [TestFixture]
-    public class OmgwtfnzbsFixture : CoreTest<Omgwtfnzbs>
-    {
-        [SetUp]
-        public void Setup()
-        {
-            Subject.Definition = new IndexerDefinition()
-            {
-                Name = "Omgwtfnzbs",
-                Settings = new OmgwtfnzbsSettings()
-                {
-                    ApiKey = "xxx",
-                    Username = "me@my.domain"
-                }
-            };
-        }
-
-        [Test]
-        public void should_parse_recent_feed_from_omgwtfnzbs()
-        {
-            var recentFeed = ReadAllText(@"Files/Indexers/Omgwtfnzbs/Omgwtfnzbs.xml");
-
-            Mocker.GetMock<IHttpClient>()
-                .Setup(o => o.Execute(It.Is<HttpRequest>(v => v.Method == HttpMethod.Get)))
-                .Returns<HttpRequest>(r => new HttpResponse(r, new HttpHeader(), recentFeed));
-
-            var releases = Subject.FetchRecent();
-
-            releases.Should().HaveCount(100);
-
-            var releaseInfo = releases.First();
-
-            releaseInfo.Title.Should().Be("Un.Petit.Boulot.2016.FRENCH.720p.BluRay.DTS.x264-LOST");
-            releaseInfo.DownloadProtocol.Should().Be(DownloadProtocol.Usenet);
-            releaseInfo.DownloadUrl.Should().Be("https://api.omgwtfnzbs.me/nzb/?id=8a2Bw&user=nzbdrone&api=nzbdrone");
-            releaseInfo.InfoUrl.Should().Be("https://omgwtfnzbs.me/details.php?id=8a2Bw");
-            releaseInfo.CommentUrl.Should().BeNullOrEmpty();
-            releaseInfo.Indexer.Should().Be(Subject.Definition.Name);
-            releaseInfo.PublishDate.Should().Be(DateTime.Parse("2017/01/09 00:16:54"));
-            releaseInfo.Size.Should().Be(5354909355);
-        }
-    }
-}

src/NzbDrone.Core/Datastore/Migration/217_remove_omg.cs

@@ -0,0 +1,14 @@
+using FluentMigrator;
+using NzbDrone.Core.Datastore.Migration.Framework;
+
+namespace NzbDrone.Core.Datastore.Migration
+{
+    [Migration(217)]
+    public class remove_omg : NzbDroneMigrationBase
+    {
+        protected override void MainDbUpgrade()
+        {
+            Delete.FromTable("Indexers").Row(new { Implementation = "Omgwtfnzbs" });
+        }
+    }
+}

src/NzbDrone.Core/Indexers/Newznab/Newznab.cs

@@ -48,7 +48,6 @@ namespace NzbDrone.Core.Indexers.Newznab
                 yield return GetDefinition("NZBFinder.ws", GetSettings("https://nzbfinder.ws", categories: new[] { 2030, 2040, 2045, 2050, 2060, 2070, 2080, 2090 }));
                 yield return GetDefinition("NZBgeek", GetSettings("https://api.nzbgeek.info"));
                 yield return GetDefinition("nzbplanet.net", GetSettings("https://api.nzbplanet.net"));
-                yield return GetDefinition("omgwtfnzbs", GetSettings("https://api.omgwtfnzbs.me", categories: new[] { 2000, 2020, 2030, 2040, 2045, 2050, 2070 }));
                 yield return GetDefinition("OZnzb.com", GetSettings("https://api.oznzb.com"));
                 yield return GetDefinition("SimplyNZBs", GetSettings("https://simplynzbs.com"));
                 yield return GetDefinition("Tabula Rasa", GetSettings("https://www.tabula-rasa.pw", apiPath: @"/api/v1/api"));

src/NzbDrone.Core/Indexers/Omgwtfnzbs/Omgwtfnzbs.cs

@@ -1,29 +0,0 @@
-using NLog;
-using NzbDrone.Common.Http;
-using NzbDrone.Core.Configuration;
-using NzbDrone.Core.Parser;
-
-namespace NzbDrone.Core.Indexers.Omgwtfnzbs
-{
-    public class Omgwtfnzbs : HttpIndexerBase<OmgwtfnzbsSettings>
-    {
-        public override string Name => "omgwtfnzbs";
-
-        public override DownloadProtocol Protocol => DownloadProtocol.Usenet;
-
-        public Omgwtfnzbs(IHttpClient httpClient, IIndexerStatusService indexerStatusService, IConfigService configService, IParsingService parsingService, Logger logger)
-            : base(httpClient, indexerStatusService, configService, parsingService, logger)
-        {
-        }
-
-        public override IIndexerRequestGenerator GetRequestGenerator()
-        {
-            return new OmgwtfnzbsRequestGenerator() { Settings = Settings };
-        }
-
-        public override IParseIndexerResponse GetParser()
-        {
-            return new OmgwtfnzbsRssParser();
-        }
-    }
-}

src/NzbDrone.Core/Indexers/Omgwtfnzbs/OmgwtfnzbsRequestGenerator.cs

@@ -1,59 +0,0 @@
-using System;
-using System.Collections.Generic;
-using System.Text;
-using NzbDrone.Common.Extensions;
-using NzbDrone.Common.Http;
-using NzbDrone.Core.IndexerSearch.Definitions;
-
-namespace NzbDrone.Core.Indexers.Omgwtfnzbs
-{
-    public class OmgwtfnzbsRequestGenerator : IIndexerRequestGenerator
-    {
-        public string BaseUrl { get; set; }
-        public OmgwtfnzbsSettings Settings { get; set; }
-
-        public OmgwtfnzbsRequestGenerator()
-        {
-            BaseUrl = "https://rss.omgwtfnzbs.me/rss-download.php";
-        }
-
-        public virtual IndexerPageableRequestChain GetRecentRequests()
-        {
-            var pageableRequests = new IndexerPageableRequestChain();
-
-            pageableRequests.Add(GetPagedRequests(null));
-
-            return pageableRequests;
-        }
-
-        public IndexerPageableRequestChain GetSearchRequests(MovieSearchCriteria searchCriteria)
-        {
-            var pageableRequests = new IndexerPageableRequestChain();
-
-            foreach (var queryTitle in searchCriteria.CleanSceneTitles)
-            {
-                pageableRequests.Add(GetPagedRequests(string.Format("{0}",
-                    queryTitle)));
-            }
-
-            return pageableRequests;
-        }
-
-        private IEnumerable<IndexerRequest> GetPagedRequests(string query)
-        {
-            var url = new StringBuilder();
-            url.AppendFormat("{0}?catid=15,16,17,18,31,35&user={1}&api={2}&eng=1&delay={3}", BaseUrl, Settings.Username, Settings.ApiKey, Settings.Delay);
-
-            if (query.IsNotNullOrWhiteSpace())
-            {
-                url = url.Replace("rss-download.php", "rss-search.php");
-                url.AppendFormat("&search={0}", query);
-            }
-
-            yield return new IndexerRequest(url.ToString(), HttpAccept.Rss);
-        }
-
-        public Func<IDictionary<string, string>> GetCookies { get; set; }
-        public Action<IDictionary<string, string>, DateTime?> CookiesUpdater { get; set; }
-    }
-}

src/NzbDrone.Core/Indexers/Omgwtfnzbs/OmgwtfnzbsRssParser.cs

@@ -1,50 +0,0 @@
-using System.Linq;
-using System.Text.RegularExpressions;
-using System.Xml.Linq;
-using NzbDrone.Common.Extensions;
-using NzbDrone.Core.Indexers.Exceptions;
-
-namespace NzbDrone.Core.Indexers.Omgwtfnzbs
-{
-    public class OmgwtfnzbsRssParser : RssParser
-    {
-        public OmgwtfnzbsRssParser()
-        {
-            UseEnclosureUrl = true;
-            UseEnclosureLength = true;
-        }
-
-        protected override bool PreProcess(IndexerResponse indexerResponse)
-        {
-            var xdoc = LoadXmlDocument(indexerResponse);
-            var notice = xdoc.Descendants("notice").FirstOrDefault();
-
-            if (notice == null)
-            {
-                return true;
-            }
-
-            if (!notice.Value.ContainsIgnoreCase("api"))
-            {
-                return true;
-            }
-
-            throw new ApiKeyException(notice.Value);
-        }
-
-        protected override string GetInfoUrl(XElement item)
-        {
-            // Todo: Me thinks I need to parse details to get this...
-            var match = Regex.Match(item.Description(),
-                @"(?:\<b\>View NZB\:\<\/b\>\s\<a\shref\=\"")(?<URL>.+?)(?:\"")",
-                RegexOptions.IgnoreCase | RegexOptions.Compiled);
-
-            if (match.Success)
-            {
-                return match.Groups["URL"].Value;
-            }
-
-            return string.Empty;
-        }
-    }
-}

src/NzbDrone.Core/Indexers/Omgwtfnzbs/OmgwtfnzbsSettings.cs

@@ -1,49 +0,0 @@
-using System.Collections.Generic;
-using FluentValidation;
-using NzbDrone.Core.Annotations;
-using NzbDrone.Core.Languages;
-using NzbDrone.Core.Validation;
-
-namespace NzbDrone.Core.Indexers.Omgwtfnzbs
-{
-    public class OmgwtfnzbsSettingsValidator : AbstractValidator<OmgwtfnzbsSettings>
-    {
-        public OmgwtfnzbsSettingsValidator()
-        {
-            RuleFor(c => c.Username).NotEmpty();
-            RuleFor(c => c.ApiKey).NotEmpty();
-            RuleFor(c => c.Delay).GreaterThanOrEqualTo(0);
-        }
-    }
-
-    public class OmgwtfnzbsSettings : IIndexerSettings
-    {
-        private static readonly OmgwtfnzbsSettingsValidator Validator = new OmgwtfnzbsSettingsValidator();
-
-        public OmgwtfnzbsSettings()
-        {
-            Delay = 30;
-            MultiLanguages = new List<int>();
-        }
-
-        // Unused since Omg has a hardcoded url.
-        public string BaseUrl { get; set; }
-
-        [FieldDefinition(0, Label = "Username", Privacy = PrivacyLevel.UserName)]
-        public string Username { get; set; }
-
-        [FieldDefinition(1, Label = "API Key", Privacy = PrivacyLevel.ApiKey)]
-        public string ApiKey { get; set; }
-
-        [FieldDefinition(2, Label = "Delay", HelpText = "Time in minutes to delay new nzbs before they appear on the RSS feed", Advanced = true)]
-        public int Delay { get; set; }
-
-        [FieldDefinition(3, Type = FieldType.Select, SelectOptions = typeof(RealLanguageFieldConverter), Label = "Multi Languages", HelpText = "What languages are normally in a multi release on this indexer?", Advanced = true)]
-        public IEnumerable<int> MultiLanguages { get; set; }
-
-        public NzbDroneValidationResult Validate()
-        {
-            return new NzbDroneValidationResult(Validator.Validate(this));
-        }
-    }
-}