Merge branch 'kay.one' of github.com:NzbDrone/NzbDrone into markus

.gitignore

@@ -43,3 +43,4 @@ NzbDrone.zip
 NzbDrone.sln.DotSettings.user*
 config.xml
 UpdateLogs/
+NzbDrone.Web/MediaCover

NzbDrone.Common/PathExtentions.cs

@@ -85,9 +85,19 @@ namespace NzbDrone.Common
             return Path.Combine(enviromentProvider.GetAppDataPath(), LOG_DB_FILE);
         }
 
+        public static string GetMediaCoverPath(this EnviromentProvider enviromentProvider)
+        {
+            return Path.Combine(enviromentProvider.GetWebRoot(), "MediaCover");
+        }
+
         public static string GetBannerPath(this EnviromentProvider enviromentProvider)
         {
-            return Path.Combine(enviromentProvider.GetWebRoot(), "Content", "Images", "Banners");
+            return Path.Combine(enviromentProvider.GetMediaCoverPath(), "Banners");
+        }
+
+        public static string GetFanArthPath(this EnviromentProvider enviromentProvider)
+        {
+            return Path.Combine(enviromentProvider.GetMediaCoverPath(), "Fanarts");
         }
 
         public static string GetCacheFolder(this EnviromentProvider enviromentProvider)

NzbDrone.Core.Test/ProviderTests/JobProviderTests/JobProviderFixture.cs

@@ -301,8 +301,29 @@ namespace NzbDrone.Core.Test.ProviderTests.JobProviderTests
             //Assert
             var registeredJobs = Db.Fetch<JobDefinition>();
             registeredJobs.Should().HaveCount(1);
-            registeredJobs.Should().NotContain(c => c.Name == deletedJob.Name);
+            registeredJobs.Should().NotContain(c => c.TypeName == deletedJob.TypeName);
+        }
 
+        [Test]
+        public void inti_should_removed_jobs_that_no_longer_exist_even_with_same_name()
+        {
+            IList<IJob> fakeJobs = new List<IJob> { fakeJob };
+            Mocker.SetConstant(fakeJobs);
+
+            WithRealDb();
+            var deletedJob = Builder<JobDefinition>.CreateNew()
+                .With(c => c.Name = fakeJob.Name).Build();
+            
+            Db.Insert(deletedJob);
+            var jobProvider = Mocker.Resolve<JobProvider>();
+
+            //Act
+            jobProvider.Initialize();
+
+            //Assert
+            var registeredJobs = Db.Fetch<JobDefinition>();
+            registeredJobs.Should().HaveCount(1);
+            registeredJobs.Should().NotContain(c => c.TypeName == deletedJob.TypeName);
         }
 
         [Test]

NzbDrone.Core/Datastore/Connection.cs

@@ -1,9 +1,7 @@
 using System;
 using System.Configuration;
 using System.Data.Common;
-using System.Data.EntityClient;
 using System.Data.SqlServerCe;
-using MvcMiniProfiler;
 using NzbDrone.Common;
 using NzbDrone.Core.Instrumentation;
 using PetaPoco;
@@ -14,9 +12,9 @@ namespace NzbDrone.Core.Datastore
     {
         private readonly EnviromentProvider _enviromentProvider;
 
-
-        public static void InitiFacotry()
+        static Connection()
         {
+            Database.Mapper = new CustomeMapper();
 
             var dataSet = ConfigurationManager.GetSection("system.data") as System.Data.DataSet;
             dataSet.Tables[0].Rows.Add("Microsoft SQL Server Compact Data Provider 4.0"
@@ -30,12 +28,6 @@ namespace NzbDrone.Core.Datastore
             _enviromentProvider = enviromentProvider;
         }
         
-        static Connection()
-        {
-            Database.Mapper = new CustomeMapper();
-            InitiFacotry();
-        }
-
         public String MainConnectionString
         {
             get
@@ -54,7 +46,6 @@ namespace NzbDrone.Core.Datastore
 
         public static string GetConnectionString(string path)
         {
-            //return String.Format("Data Source={0};Version=3;Cache Size=30000;Pooling=true;Default Timeout=2", path);
             return String.Format("Data Source={0}", path);
         }
 
@@ -73,8 +64,6 @@ namespace NzbDrone.Core.Datastore
             return GetLogDbContext(LogConnectionString);
         }
 
-
-
         public static IDatabase GetPetaPocoDb(string connectionString, Boolean profiled = true)
         {
             MigrationsHelper.Run(connectionString, true);

NzbDrone.Core/Jobs/JobProvider.cs

@@ -76,7 +76,7 @@ namespace NzbDrone.Core.Jobs
 
             foreach (var currentJob in currentJobs)
             {
-                if (!_jobs.Any(c => c.Name == currentJob.Name))
+                if (!_jobs.Any(c => c.GetType().ToString() == currentJob.TypeName))
                 {
                     logger.Debug("Removing job from database '{0}'", currentJob.Name);
                     _database.Delete(currentJob);

NzbDrone.Web/Views/Series/Details.cshtml

@@ -78,7 +78,7 @@
             
             if(Model.HasBanner)
             {
-                bannerUrl = "../../Content/Images/Banners/" + Model.SeriesId + ".jpg";
+                bannerUrl = "../../MediaCover/Banners/" + Model.SeriesId + ".jpg";
             }
         }
         <img src="@bannerUrl" alt="Banner"/>