From a1fb4018b56188798c1697a3641a712b29b53b87 Mon Sep 17 00:00:00 2001
From: "Jamie.Rees" <Jamie.Rees>
Date: Wed, 25 Jan 2017 14:41:04 +0000
Subject: [PATCH] Fixed the login issue and pass Radarr the year #990

---
 Ombi.Api.Interfaces/IRadarrApi.cs        |  2 +-
 Ombi.Api.Models/Radarr/RadarrAddMovie.cs |  1 +
 Ombi.Api/RadarrApi.cs                    |  5 +++--
 Ombi.Core/MovieSender.cs                 |  2 +-
 Ombi.UI/Modules/BaseModule.cs            | 11 +++++++----
 5 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/Ombi.Api.Interfaces/IRadarrApi.cs b/Ombi.Api.Interfaces/IRadarrApi.cs
index 8d6af90ce..88e6d3028 100644
--- a/Ombi.Api.Interfaces/IRadarrApi.cs
+++ b/Ombi.Api.Interfaces/IRadarrApi.cs
@@ -7,7 +7,7 @@ namespace Ombi.Api.Interfaces
 {
     public interface IRadarrApi
     {
-        RadarrAddMovie AddMovie(int tmdbId, string title, int qualityId, string rootPath, string apiKey, Uri baseUrl, bool searchNow = false);
+        RadarrAddMovie AddMovie(int tmdbId, string title, int year, int qualityId, string rootPath, string apiKey, Uri baseUrl, bool searchNow = false);
         List<RadarrMovieResponse> GetMovies(string apiKey, Uri baseUrl);
         List<SonarrProfile> GetProfiles(string apiKey, Uri baseUrl);
         SystemStatus SystemStatus(string apiKey, Uri baseUrl);
diff --git a/Ombi.Api.Models/Radarr/RadarrAddMovie.cs b/Ombi.Api.Models/Radarr/RadarrAddMovie.cs
index 5c589bd12..4b8aeb452 100644
--- a/Ombi.Api.Models/Radarr/RadarrAddMovie.cs
+++ b/Ombi.Api.Models/Radarr/RadarrAddMovie.cs
@@ -50,6 +50,7 @@ namespace Ombi.Api.Models.Radarr
         public string imdbId { get; set; }
         public string titleSlug { get; set; }
         public int id { get; set; }
+        public int year { get; set; }
 
     }
 }
\ No newline at end of file
diff --git a/Ombi.Api/RadarrApi.cs b/Ombi.Api/RadarrApi.cs
index 648ff87b3..af01f6e40 100644
--- a/Ombi.Api/RadarrApi.cs
+++ b/Ombi.Api/RadarrApi.cs
@@ -63,7 +63,7 @@ namespace Ombi.Api
             return obj;
         }
 
-        public RadarrAddMovie AddMovie(int tmdbId, string title, int qualityId, string rootPath, string apiKey, Uri baseUrl, bool searchNow = false)
+        public RadarrAddMovie AddMovie(int tmdbId, string title, int year, int qualityId, string rootPath, string apiKey, Uri baseUrl, bool searchNow = false)
         {
             var request = new RestRequest
             {
@@ -78,7 +78,8 @@ namespace Ombi.Api
                 qualityProfileId = qualityId,
                 rootFolderPath = rootPath,
                 titleSlug = title,
-                monitored = true
+                monitored = true,
+                year = year
             };
 
             if (searchNow)
diff --git a/Ombi.Core/MovieSender.cs b/Ombi.Core/MovieSender.cs
index cf5b4fda3..97d45fcd5 100644
--- a/Ombi.Core/MovieSender.cs
+++ b/Ombi.Core/MovieSender.cs
@@ -106,7 +106,7 @@ namespace Ombi.Core
         {
             var qualityProfile = 0;
             int.TryParse(settings.QualityProfile, out qualityProfile);
-            var result = RadarrApi.AddMovie(model.ProviderId, model.Title, qualityProfile, settings.RootPath, settings.ApiKey, settings.FullUri, true);
+            var result = RadarrApi.AddMovie(model.ProviderId, model.Title, model.ReleaseDate.Year, qualityProfile, settings.RootPath, settings.ApiKey, settings.FullUri, true);
 
             if (!string.IsNullOrEmpty(result.Error?.message))
             {
diff --git a/Ombi.UI/Modules/BaseModule.cs b/Ombi.UI/Modules/BaseModule.cs
index 5e6ca5d4a..f5f8010be 100644
--- a/Ombi.UI/Modules/BaseModule.cs
+++ b/Ombi.UI/Modules/BaseModule.cs
@@ -31,6 +31,7 @@ using System.Linq;
 using System.Threading;
 using Nancy;
 using Nancy.Security;
+using NLog;
 using Ombi.Core;
 using Ombi.Core.SettingModels;
 using Ombi.Helpers;
@@ -100,6 +101,7 @@ namespace Ombi.UI.Modules
         }
 
 
+        private static Logger Log = LogManager.GetCurrentClassLogger();
         private string _username;
         /// <summary>
         /// Returns the Username or UserAlias
@@ -112,16 +114,17 @@ namespace Ombi.UI.Modules
                 {
                     try
                     {
-                        var username = Security.GetUsername(User.UserName, Session);
+                        var username = Security.GetUsername(User?.UserName, Session);
                         if (string.IsNullOrEmpty(username))
                         {
-                            return "Unknown User";
+                            return string.Empty;
                         }
                         _username = username;
                     }
-                    catch (Exception)
+                    catch (Exception e)
                     {
-                        return "Unknown User Error";
+                        Log.Info(e);
+                        return string.Empty;
                     }
                 }
                 return _username;