From 745ed9a30f999f519dce1c9591d865a79e487ecf Mon Sep 17 00:00:00 2001
From: Jamie <tidusjar@gmail.com>
Date: Tue, 25 Sep 2018 19:51:37 +0100
Subject: [PATCH 1/4] Fixed #2535

---
 .../usermanagement-user.component.html               | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/Ombi/ClientApp/app/usermanagement/usermanagement-user.component.html b/src/Ombi/ClientApp/app/usermanagement/usermanagement-user.component.html
index 0d13b17c3..6f3ae63ee 100644
--- a/src/Ombi/ClientApp/app/usermanagement/usermanagement-user.component.html
+++ b/src/Ombi/ClientApp/app/usermanagement/usermanagement-user.component.html
@@ -144,7 +144,7 @@
                             <div class="panel panel-default a">
                                 <div class="panel-body">
 
-                                    <div class="form-group"  *ngIf="user.userQualityProfiles.sonarrQualityProfile">
+                                    <div class="form-group" *ngIf="sonarrQualities">
                                         <label for="sonarrQualities" class="control-label">Sonarr Quality Profile</label>
                                         <div id="sonarrQualities">
                                             <select class="form-control form-control-custom" [(ngModel)]="user.userQualityProfiles.sonarrQualityProfile">
@@ -153,7 +153,7 @@
                                         </div>
                                     </div>
 
-                                    <div class="form-group"  *ngIf="user.userQualityProfiles.sonarrQualityProfileAnime">
+                                    <div class="form-group" *ngIf="sonarrQualities">
                                         <label for="sonarrQualityProfileAnime" class="control-label">Sonarr Quality Profile (Anime)</label>
                                         <div id="sonarrQualityProfileAnime">
                                             <select class="form-control form-control-custom" [(ngModel)]="user.userQualityProfiles.sonarrQualityProfileAnime">
@@ -162,7 +162,7 @@
                                         </div>
                                     </div>
                              
-                                    <div class="form-group"  *ngIf="user.userQualityProfiles.sonarrRootPath">
+                                    <div class="form-group" *ngIf="sonarrRootFolders">
                                         <label for="sonarrRootPath" class="control-label">Sonarr Root Folder</label>
                                         <div id="sonarrRootPath">
                                             <select class="form-control form-control-custom" [(ngModel)]="user.userQualityProfiles.sonarrRootPath">
@@ -171,7 +171,7 @@
                                         </div>
                                     </div>
 
-                                    <div class="form-group" *ngIf="user.userQualityProfiles.sonarrRootPathAnime">
+                                    <div class="form-group" *ngIf="sonarrRootFolders">
                                         <label for="sonarrRootPathAnime" class="control-label">Sonarr Root Folder (Anime)</label>
                                         <div id="sonarrRootPathAnime">
                                             <select class="form-control form-control-custom" [(ngModel)]="user.userQualityProfiles.sonarrRootPathAnime">
@@ -179,7 +179,7 @@
                                             </select>
                                         </div>
                                     </div>
-                                    <div class="form-group" *ngIf="user.userQualityProfiles.radarrQualityProfile">
+                                    <div class="form-group" *ngIf="radarrQualities">
                                         <label for="radarrQualityProfile" class="control-label">Radarr Quality Profiles</label>
                                         <div id="radarrQualityProfile">
                                             <select class="form-control form-control-custom" [(ngModel)]="user.userQualityProfiles.radarrQualityProfile">
@@ -187,7 +187,7 @@
                                             </select>
                                         </div>
                                     </div>
-                                    <div class="form-group" *ngIf="user.userQualityProfiles.radarrRootPath">
+                                    <div class="form-group" *ngIf="radarrRootFolders">
                                         <label for="radarrRootPath" class="control-label">Radarr Root Folder</label>
                                         <div id="radarrRootPath">
                                             <select class="form-control form-control-custom" [(ngModel)]="user.userQualityProfiles.radarrRootPath">

From fdb985dd08e8e9bdb0723cd22fbc968923229f00 Mon Sep 17 00:00:00 2001
From: Jamie <tidusjar@gmail.com>
Date: Tue, 25 Sep 2018 20:04:11 +0100
Subject: [PATCH 2/4] Truncate the request title

---
 .../ClientApp/app/requests/music/musicrequests.component.html   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Ombi/ClientApp/app/requests/music/musicrequests.component.html b/src/Ombi/ClientApp/app/requests/music/musicrequests.component.html
index c89c2be0a..f6ad4bb2a 100644
--- a/src/Ombi/ClientApp/app/requests/music/musicrequests.component.html
+++ b/src/Ombi/ClientApp/app/requests/music/musicrequests.component.html
@@ -59,7 +59,7 @@
                 <div>
                     <h4>
                         <a href="" target="_blank">
-                            {{request.title}}
+                            {{request.title  | truncate: 36}}
                         </a>
         
                     </h4>

From c14be4eaf95e80271df540750ee65ed31c210db2 Mon Sep 17 00:00:00 2001
From: Jamie <tidusjar@gmail.com>
Date: Tue, 25 Sep 2018 20:19:42 +0100
Subject: [PATCH 3/4] The album in Lidarr does not need to be marked as
 monitored for us to pick up it's available. Fixes #2536

---
 src/Ombi.Schedule/Jobs/Lidarr/LidarrAvailabilityChecker.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Ombi.Schedule/Jobs/Lidarr/LidarrAvailabilityChecker.cs b/src/Ombi.Schedule/Jobs/Lidarr/LidarrAvailabilityChecker.cs
index d5ba14a6d..9db24784d 100644
--- a/src/Ombi.Schedule/Jobs/Lidarr/LidarrAvailabilityChecker.cs
+++ b/src/Ombi.Schedule/Jobs/Lidarr/LidarrAvailabilityChecker.cs
@@ -43,7 +43,7 @@ namespace Ombi.Schedule.Jobs.Lidarr
                 var cachedAlbum = await _cachedAlbums.FirstOrDefaultAsync(x => x.ForeignAlbumId.Equals(request.ForeignAlbumId));
                 if (cachedAlbum != null)
                 {
-                    if (cachedAlbum.Monitored && cachedAlbum.FullyAvailable)
+                    if (cachedAlbum.FullyAvailable)
                     {
                         request.Available = true;
                         request.MarkedAsAvailable = DateTime.Now;

From dffd27d8044e6e4cb64d080b3d02b596b1d4ce98 Mon Sep 17 00:00:00 2001
From: TidusJar <tidusjar@gmail.com>
Date: Thu, 27 Sep 2018 13:17:39 +0100
Subject: [PATCH 4/4] Added the ability to purge/remove issues

---
 src/Ombi.DependencyInjection/IocExtensions.cs |  1 +
 src/Ombi.Schedule.Tests/IssuesPurgeTests.cs   | 85 +++++++++++++++++++
 src/Ombi.Schedule/Jobs/Ombi/IIssuesPurge.cs   |  9 ++
 src/Ombi.Schedule/Jobs/Ombi/IssuesPurge.cs    | 63 ++++++++++++++
 .../Settings/Models/IssueSettings.cs          |  3 +
 .../Settings/Models/JobSettings.cs            |  1 +
 .../Settings/Models/JobSettingsHelper.cs      |  5 ++
 src/Ombi.Store/Entities/Requests/Issues.cs    |  1 +
 .../ClientApp/app/interfaces/ISettings.ts     |  3 +
 .../app/settings/issues/issues.component.html | 14 +++
 .../app/settings/issues/issues.component.ts   |  6 +-
 .../app/settings/jobs/jobs.component.html     |  9 +-
 .../app/settings/jobs/jobs.component.ts       |  1 +
 src/Ombi/Controllers/SettingsController.cs    |  3 +-
 14 files changed, 200 insertions(+), 4 deletions(-)
 create mode 100644 src/Ombi.Schedule.Tests/IssuesPurgeTests.cs
 create mode 100644 src/Ombi.Schedule/Jobs/Ombi/IIssuesPurge.cs
 create mode 100644 src/Ombi.Schedule/Jobs/Ombi/IssuesPurge.cs

diff --git a/src/Ombi.DependencyInjection/IocExtensions.cs b/src/Ombi.DependencyInjection/IocExtensions.cs
index 334b94675..3af06b476 100644
--- a/src/Ombi.DependencyInjection/IocExtensions.cs
+++ b/src/Ombi.DependencyInjection/IocExtensions.cs
@@ -191,6 +191,7 @@ namespace Ombi.DependencyInjection
             services.AddTransient<ILidarrAlbumSync, LidarrAlbumSync>();
             services.AddTransient<ILidarrArtistSync, LidarrArtistSync>();
             services.AddTransient<ILidarrAvailabilityChecker, LidarrAvailabilityChecker>();
+            services.AddTransient<IIssuesPurge, IssuesPurge>();
         }
     }
 }
diff --git a/src/Ombi.Schedule.Tests/IssuesPurgeTests.cs b/src/Ombi.Schedule.Tests/IssuesPurgeTests.cs
new file mode 100644
index 000000000..932022cd8
--- /dev/null
+++ b/src/Ombi.Schedule.Tests/IssuesPurgeTests.cs
@@ -0,0 +1,85 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using Moq;
+using NUnit.Framework;
+using Ombi.Core.Settings;
+using Ombi.Schedule.Jobs.Ombi;
+using Ombi.Settings.Settings.Models;
+using Ombi.Store.Entities.Requests;
+using Ombi.Store.Repository;
+using System.Threading.Tasks;
+
+namespace Ombi.Schedule.Tests
+{
+    [TestFixture]
+    public class IssuesPurgeTests
+    {
+
+        [SetUp]
+        public void Setup()
+        {
+            Repo = new Mock<IRepository<Issues>>();
+            Settings = new Mock<ISettingsService<IssueSettings>>();
+            Settings.Setup(x => x.GetSettingsAsync()).ReturnsAsync(new IssueSettings());
+            Job = new IssuesPurge(Repo.Object, Settings.Object);
+        }
+
+        public Mock<IRepository<Issues>> Repo { get; set; }
+        public Mock<ISettingsService<IssueSettings>> Settings { get; set; }
+        public IssuesPurge Job { get; set; }
+
+        [Test]
+        public async Task DoesNotRun_WhenDisabled()
+        {
+            await Job.Start();
+            Repo.Verify(x => x.GetAll(),Times.Never);
+        }
+
+        [Test]
+        public async Task Deletes_Correct_Issue()
+        {
+            var issues = new List<Issues>()
+            {
+                new Issues
+                {
+                    Status = IssueStatus.Resolved,
+                    ResovledDate = DateTime.Now.AddDays(-5).AddHours(-1)
+                }
+            };
+
+            Settings.Setup(x => x.GetSettingsAsync()).ReturnsAsync(new IssueSettings { DeleteIssues = true, DaysAfterResolvedToDelete = 5 });
+            Repo.Setup(x => x.GetAll()).Returns(new EnumerableQuery<Issues>(issues));
+            await Job.Start();
+
+            Assert.That(issues.First().Status, Is.EqualTo(IssueStatus.Deleted));
+            Repo.Verify(x => x.SaveChangesAsync(), Times.Once);
+        }
+
+        [Test]
+        public async Task DoesNot_Delete_AnyIssues()
+        {
+            var issues = new List<Issues>()
+            {
+                new Issues
+                {
+                    Status = IssueStatus.Resolved,
+                    ResovledDate = DateTime.Now.AddDays(-2)
+                },
+                new Issues
+                {
+                    Status = IssueStatus.Resolved,
+                    ResovledDate = DateTime.Now.AddDays(-6)
+                }
+            };
+
+            Settings.Setup(x => x.GetSettingsAsync()).ReturnsAsync(new IssueSettings { DeleteIssues = true, DaysAfterResolvedToDelete = 5 });
+            Repo.Setup(x => x.GetAll()).Returns(new EnumerableQuery<Issues>(issues));
+            await Job.Start();
+
+            Assert.That(issues[0].Status, Is.Not.EqualTo(IssueStatus.Deleted));
+            Assert.That(issues[1].Status, Is.EqualTo(IssueStatus.Deleted));
+            Repo.Verify(x => x.SaveChangesAsync(), Times.Once);
+        }
+    }
+}
\ No newline at end of file
diff --git a/src/Ombi.Schedule/Jobs/Ombi/IIssuesPurge.cs b/src/Ombi.Schedule/Jobs/Ombi/IIssuesPurge.cs
new file mode 100644
index 000000000..fbd1e3aaf
--- /dev/null
+++ b/src/Ombi.Schedule/Jobs/Ombi/IIssuesPurge.cs
@@ -0,0 +1,9 @@
+using System.Threading.Tasks;
+
+namespace Ombi.Schedule.Jobs.Ombi
+{
+    public interface IIssuesPurge : IBaseJob
+    {
+        Task Start();
+    }
+}
\ No newline at end of file
diff --git a/src/Ombi.Schedule/Jobs/Ombi/IssuesPurge.cs b/src/Ombi.Schedule/Jobs/Ombi/IssuesPurge.cs
new file mode 100644
index 000000000..92ca31071
--- /dev/null
+++ b/src/Ombi.Schedule/Jobs/Ombi/IssuesPurge.cs
@@ -0,0 +1,63 @@
+using System;
+using System.Linq;
+using System.Threading.Tasks;
+using Ombi.Core.Settings;
+using Ombi.Settings.Settings.Models;
+using Ombi.Store.Entities.Requests;
+using Ombi.Store.Repository;
+
+namespace Ombi.Schedule.Jobs.Ombi
+{
+    public class IssuesPurge : IIssuesPurge
+    {
+        public IssuesPurge(IRepository<Issues> issuesRepo, ISettingsService<IssueSettings> issueSettings)
+        {
+            _issuesRepository = issuesRepo;
+            _settings = issueSettings;
+            _settings.ClearCache();
+        }
+
+        private readonly IRepository<Issues> _issuesRepository;
+        private readonly ISettingsService<IssueSettings> _settings;
+
+        public async Task Start()
+        {
+            var settings = await _settings.GetSettingsAsync();
+            if (!settings.DeleteIssues)
+            {
+                return;
+            }
+
+            var now = DateTime.Now.AddDays(-settings.DaysAfterResolvedToDelete).Date;
+            var resolved = _issuesRepository.GetAll().Where(x => x.Status == IssueStatus.Resolved);
+            var toDelete = resolved.Where(x => x.ResovledDate.HasValue && x.ResovledDate.Value.Date <= now);
+
+            foreach (var d in toDelete)
+            {
+                d.Status = IssueStatus.Deleted;
+            }
+
+            await _issuesRepository.SaveChangesAsync();
+        }
+
+        private bool _disposed;
+        protected virtual void Dispose(bool disposing)
+        {
+            if (_disposed)
+                return;
+
+            if (disposing)
+            {
+                _issuesRepository?.Dispose();
+                _settings?.Dispose();
+            }
+            _disposed = true;
+        }
+
+        public void Dispose()
+        {
+            Dispose(true);
+            GC.SuppressFinalize(this);
+        }
+    }
+}
\ No newline at end of file
diff --git a/src/Ombi.Settings/Settings/Models/IssueSettings.cs b/src/Ombi.Settings/Settings/Models/IssueSettings.cs
index e025c82d1..d7a35c0d9 100644
--- a/src/Ombi.Settings/Settings/Models/IssueSettings.cs
+++ b/src/Ombi.Settings/Settings/Models/IssueSettings.cs
@@ -4,5 +4,8 @@
     {
         public bool Enabled { get; set; }
         public bool EnableInProgress { get; set; }
+
+        public bool DeleteIssues { get; set; }
+        public int DaysAfterResolvedToDelete { get; set; }
     }
 }
\ No newline at end of file
diff --git a/src/Ombi.Settings/Settings/Models/JobSettings.cs b/src/Ombi.Settings/Settings/Models/JobSettings.cs
index 48c721e29..8b283cdf7 100644
--- a/src/Ombi.Settings/Settings/Models/JobSettings.cs
+++ b/src/Ombi.Settings/Settings/Models/JobSettings.cs
@@ -14,5 +14,6 @@
         public string RefreshMetadata { get; set; }
         public string Newsletter { get; set; }
         public string LidarrArtistSync { get; set; }
+        public string IssuesPurge { get; set; }
     }
 }
\ No newline at end of file
diff --git a/src/Ombi.Settings/Settings/Models/JobSettingsHelper.cs b/src/Ombi.Settings/Settings/Models/JobSettingsHelper.cs
index 0f8fec5fd..4491ca27a 100644
--- a/src/Ombi.Settings/Settings/Models/JobSettingsHelper.cs
+++ b/src/Ombi.Settings/Settings/Models/JobSettingsHelper.cs
@@ -57,6 +57,11 @@ namespace Ombi.Settings.Settings.Models
             return Get(s.LidarrArtistSync, Cron.Hourly(40));
         }
 
+        public static string IssuePurge(JobSettings s)
+        {
+            return Get(s.IssuesPurge, Cron.Daily());
+        }
+
         private static string Get(string settings, string defaultCron)
         {
             return settings.HasValue() ? settings : defaultCron;
diff --git a/src/Ombi.Store/Entities/Requests/Issues.cs b/src/Ombi.Store/Entities/Requests/Issues.cs
index b1021e362..9fbc6a83e 100644
--- a/src/Ombi.Store/Entities/Requests/Issues.cs
+++ b/src/Ombi.Store/Entities/Requests/Issues.cs
@@ -29,5 +29,6 @@ namespace Ombi.Store.Entities.Requests
         Pending = 0,
         InProgress = 1,
         Resolved = 2,
+        Deleted = 3,
     }
 }
\ No newline at end of file
diff --git a/src/Ombi/ClientApp/app/interfaces/ISettings.ts b/src/Ombi/ClientApp/app/interfaces/ISettings.ts
index db4db935e..7a889e722 100644
--- a/src/Ombi/ClientApp/app/interfaces/ISettings.ts
+++ b/src/Ombi/ClientApp/app/interfaces/ISettings.ts
@@ -144,11 +144,14 @@ export interface IJobSettings {
   newsletter: string;
   plexRecentlyAddedSync: string;
   lidarrArtistSync: string;
+  issuesPurge: string;
 }
 
 export interface IIssueSettings extends ISettings {
   enabled: boolean;
   enableInProgress: boolean;
+  deleteIssues: boolean;
+  daysAfterResolvedToDelete: number;
 }
 
 export interface IAuthenticationSettings extends ISettings {
diff --git a/src/Ombi/ClientApp/app/settings/issues/issues.component.html b/src/Ombi/ClientApp/app/settings/issues/issues.component.html
index e30d84e26..ee780a1a1 100644
--- a/src/Ombi/ClientApp/app/settings/issues/issues.component.html
+++ b/src/Ombi/ClientApp/app/settings/issues/issues.component.html
@@ -18,6 +18,20 @@
                 </div>
             </div>
 
+            <div class="form-group">
+                <div class="checkbox">
+                    <input type="checkbox" id="deleteIssues" formControlName="deleteIssues" ng-checked="form.deleteIssues">
+                    <label for="deleteIssues">Delete issues after they have been resolved</label>
+                </div>
+            </div>
+
+            <div class="form-group" *ngIf="form.controls.deleteIssues.value">
+                <div class="checkbox">
+                    <input type="checkbox" id="daysAfterResolvedToDelete" formControlName="daysAfterResolvedToDelete" ng-checked="form.daysAfterResolvedToDelete">
+                    <label for="daysAfterResolvedToDelete">Leave the resolved issue for X amount of days before deletion</label>
+                </div>
+            </div>
+
             <div class="form-group">
                 <div>
                     <button type="submit" [disabled]="form.invalid" class="btn btn-primary-outline ">Submit</button>
diff --git a/src/Ombi/ClientApp/app/settings/issues/issues.component.ts b/src/Ombi/ClientApp/app/settings/issues/issues.component.ts
index c8a85e8e1..37ead870a 100644
--- a/src/Ombi/ClientApp/app/settings/issues/issues.component.ts
+++ b/src/Ombi/ClientApp/app/settings/issues/issues.component.ts
@@ -21,8 +21,10 @@ export class IssuesComponent implements OnInit {
     public ngOnInit() {
         this.settingsService.getIssueSettings().subscribe(x => {
             this.form = this.fb.group({
-                enabled:            [x.enabled],
-                enableInProgress:   [x.enableInProgress],
+                enabled:                    [x.enabled],
+                enableInProgress:           [x.enableInProgress],
+                deleteIssues:               [x.deleteIssues],
+                daysAfterResolvedToDelete:  [x.daysAfterResolvedToDelete],
             });
         });
         this.getCategories();
diff --git a/src/Ombi/ClientApp/app/settings/jobs/jobs.component.html b/src/Ombi/ClientApp/app/settings/jobs/jobs.component.html
index a4dcd6fb3..1365710f0 100644
--- a/src/Ombi/ClientApp/app/settings/jobs/jobs.component.html
+++ b/src/Ombi/ClientApp/app/settings/jobs/jobs.component.html
@@ -85,12 +85,19 @@
                     </div>
 
                     
-                <div class="form-group">
+                    <div class="form-group">
                         <label for="userImporter" class="control-label">Newsletter</label>              
                             <input type="text" class="form-control form-control-custom" [ngClass]="{'form-error': form.get('newsletter').hasError('required')}" id="newsletter" name="newsletter" formControlName="newsletter">
                             <small *ngIf="form.get('newsletter').hasError('required')" class="error-text">The Newsletter is required</small>
                             <button type="button" class="btn btn-sm btn-primary-outline" (click)="testCron(form.get('newsletter')?.value)">Test</button>
                     </div>
+
+                    <div class="form-group">
+                        <label for="userImporter" class="control-label">Issue Purge/Delete</label>              
+                            <input type="text" class="form-control form-control-custom" [ngClass]="{'form-error': form.get('issuesPurge').hasError('required')}" id="issuesPurge" name="issuesPurge" formControlName="issuesPurge">
+                            <small *ngIf="form.get('issuesPurge').hasError('required')" class="error-text">The Issues Purge is required</small>
+                            <button type="button" class="btn btn-sm btn-primary-outline" (click)="testCron(form.get('issuesPurge')?.value)">Test</button>
+                    </div>
             </div>
             <div class="form-group">
                     <div>
diff --git a/src/Ombi/ClientApp/app/settings/jobs/jobs.component.ts b/src/Ombi/ClientApp/app/settings/jobs/jobs.component.ts
index 756d6ba89..d8ce106ae 100644
--- a/src/Ombi/ClientApp/app/settings/jobs/jobs.component.ts
+++ b/src/Ombi/ClientApp/app/settings/jobs/jobs.component.ts
@@ -35,6 +35,7 @@ export class JobsComponent implements OnInit {
                 newsletter:               [x.newsletter, Validators.required],
                 plexRecentlyAddedSync:    [x.plexRecentlyAddedSync, Validators.required],
                 lidarrArtistSync:         [x.lidarrArtistSync, Validators.required],
+                issuesPurge:              [x.issuesPurge, Validators.required],
             });  
         });
     }
diff --git a/src/Ombi/Controllers/SettingsController.cs b/src/Ombi/Controllers/SettingsController.cs
index 474ae8823..9785af553 100644
--- a/src/Ombi/Controllers/SettingsController.cs
+++ b/src/Ombi/Controllers/SettingsController.cs
@@ -519,7 +519,8 @@ namespace Ombi.Controllers
             j.PlexRecentlyAddedSync = j.PlexRecentlyAddedSync.HasValue() ? j.PlexRecentlyAddedSync : JobSettingsHelper.PlexRecentlyAdded(j);
             j.Newsletter = j.Newsletter.HasValue() ? j.Newsletter : JobSettingsHelper.Newsletter(j);
             j.LidarrArtistSync = j.LidarrArtistSync.HasValue() ? j.LidarrArtistSync : JobSettingsHelper.LidarrArtistSync(j);
- 
+            j.IssuesPurge = j.IssuesPurge.HasValue() ? j.IssuesPurge : JobSettingsHelper.IssuePurge(j);
+
             return j;
         }