From 78441730a7cf89beaa1a85bb995090922451ec07 Mon Sep 17 00:00:00 2001
From: Claus Vium <clausvium@gmail.com>
Date: Sun, 18 Aug 2019 13:38:49 +0200
Subject: [PATCH] Add Writer and Producer to crew list

---
 .../Tmdb/Movies/GenericTmdbMovieInfo.cs           | 12 ++++--------
 .../Tmdb/TV/TmdbEpisodeProvider.cs                | 15 +++++----------
 2 files changed, 9 insertions(+), 18 deletions(-)

diff --git a/MediaBrowser.Providers/Tmdb/Movies/GenericTmdbMovieInfo.cs b/MediaBrowser.Providers/Tmdb/Movies/GenericTmdbMovieInfo.cs
index 1a6e076bb4..09c51f816b 100644
--- a/MediaBrowser.Providers/Tmdb/Movies/GenericTmdbMovieInfo.cs
+++ b/MediaBrowser.Providers/Tmdb/Movies/GenericTmdbMovieInfo.cs
@@ -255,20 +255,16 @@ namespace MediaBrowser.Providers.Tmdb.Movies
                 var keepTypes = new[]
                 {
                     PersonType.Director,
-                    //PersonType.Writer,
-                    //PersonType.Producer
+                    PersonType.Writer,
+                    PersonType.Producer
                 };
 
                 foreach (var person in movieData.casts.crew)
                 {
                     // Normalize this
-                    var type = person.department;
-                    if (string.Equals(type, "writing", StringComparison.OrdinalIgnoreCase))
-                    {
-                        type = PersonType.Writer;
-                    }
+                    var type = TmdbUtils.MapCrewToPersonType(person);
 
-                    if (!keepTypes.Contains(type ?? string.Empty, StringComparer.OrdinalIgnoreCase) &&
+                    if (!keepTypes.Contains(type, StringComparer.OrdinalIgnoreCase) &&
                         !keepTypes.Contains(person.job ?? string.Empty, StringComparer.OrdinalIgnoreCase))
                     {
                         continue;
diff --git a/MediaBrowser.Providers/Tmdb/TV/TmdbEpisodeProvider.cs b/MediaBrowser.Providers/Tmdb/TV/TmdbEpisodeProvider.cs
index 58f06a8a0a..c950963902 100644
--- a/MediaBrowser.Providers/Tmdb/TV/TmdbEpisodeProvider.cs
+++ b/MediaBrowser.Providers/Tmdb/TV/TmdbEpisodeProvider.cs
@@ -119,9 +119,8 @@ namespace MediaBrowser.Providers.Tmdb.TV
                 item.Overview = response.overview;
 
                 item.CommunityRating = (float)response.vote_average;
-                //item.VoteCount = response.vote_count;
 
-                if (response.videos != null && response.videos.results != null)
+                if (response.videos?.results != null)
                 {
                     foreach (var video in response.videos.results)
                     {
@@ -167,20 +166,16 @@ namespace MediaBrowser.Providers.Tmdb.TV
                         var keepTypes = new[]
                         {
                             PersonType.Director,
-                            //PersonType.Writer,
-                            //PersonType.Producer
+                            PersonType.Writer,
+                            PersonType.Producer
                         };
 
                         foreach (var person in credits.crew)
                         {
                             // Normalize this
-                            var type = person.department;
-                            if (string.Equals(type, "writing", StringComparison.OrdinalIgnoreCase))
-                            {
-                                type = PersonType.Writer;
-                            }
+                            var type = TmdbUtils.MapCrewToPersonType(person);
 
-                            if (!keepTypes.Contains(type ?? string.Empty, StringComparer.OrdinalIgnoreCase) &&
+                            if (!keepTypes.Contains(type, StringComparer.OrdinalIgnoreCase) &&
                                 !keepTypes.Contains(person.job ?? string.Empty, StringComparer.OrdinalIgnoreCase))
                             {
                                 continue;