Merge pull request #16 from lemonde/marius-fix-memoize-youtube

fix(youtube): retire le memoize qui ne fonctionne pas correctement (v3)
lemonde · Jul 4, 2018 · 08ee696 · 08ee696
2 parents 797545f + 10c9b81
commit 08ee696
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/lib/providers/youtube.js b/lib/providers/youtube.js
@@ -33,11 +33,11 @@ var searchUrl = function searchUrl(query, token) {
   return url;
 };
 
-var fetchVideo = _.memoize(function (videoId, part) {
+var fetchVideo = function fetchVideo(videoId, part) {
   return fetch(fetchUrl(videoId, part), { headers: provider.headers }).then(function (res) {
     return res.json();
   });
-});
+};
 
 /**
  * Convert duration ISO 8601 to seconds

diff --git a/src/providers/youtube.js b/src/providers/youtube.js
@@ -29,9 +29,8 @@ const searchUrl = (query, token) => {
   return url;
 };
 
-const fetchVideo = _.memoize((videoId, part) =>
-  fetch(fetchUrl(videoId, part), { headers: provider.headers }).then(res => res.json())
-);
+const fetchVideo = (videoId, part) => fetch(fetchUrl(videoId, part), { headers: provider.headers })
+  .then(res => res.json());
 
 /**
  * Convert duration ISO 8601 to seconds