From ab7083d26334adcfb45f7bffe5ebf64e630a87d5 Mon Sep 17 00:00:00 2001
From: Qstick <qstick@gmail.com>
Date: Fri, 30 Aug 2019 23:37:05 -0400
Subject: [PATCH] Fixed: Spec Logging Statement Causing Sentry Duplicates

---
 .../MediaFiles/MovieImport/ImportDecisionMaker.cs         | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/NzbDrone.Core/MediaFiles/MovieImport/ImportDecisionMaker.cs b/src/NzbDrone.Core/MediaFiles/MovieImport/ImportDecisionMaker.cs
index 563686dbd..7e626b441 100644
--- a/src/NzbDrone.Core/MediaFiles/MovieImport/ImportDecisionMaker.cs
+++ b/src/NzbDrone.Core/MediaFiles/MovieImport/ImportDecisionMaker.cs
@@ -174,12 +174,10 @@ private Rejection EvaluateSpec(IImportDecisionEngineSpecification spec, LocalMov
                 _logger.Warn(e, "Spec " + spec.ToString() + " currently does not implement evaluation for movies.");
                 return null;
             }
-            catch (Exception e)
+            catch (Exception ex)
             {
-                //e.Data.Add("report", remoteEpisode.Report.ToJson());
-                //e.Data.Add("parsed", remoteEpisode.ParsedEpisodeInfo.ToJson());
-                _logger.Error(e, "Couldn't evaluate decision on " + localMovie.Path);
-                return new Rejection(string.Format("{0}: {1}", spec.GetType().Name, e.Message));
+                _logger.Error(ex, "Couldn't evaluate decision on {0}", localMovie.Path);
+                return new Rejection($"{spec.GetType().Name}: {ex.Message}");
             }
 
             return null;