From 3b99500d4dd959e48f9ff68ced325a2e70959e7c Mon Sep 17 00:00:00 2001
From: Alex Beauchemin <alexbeauchemin01@gmail.com>
Date: Fri, 12 Feb 2016 22:42:12 -0500
Subject: [PATCH] Fix undefined images Pluck removed from lodash

---
 app/client/templates/components/playlist-item.html |  2 +-
 app/client/templates/components/playlist-item.js   |  5 +++++
 app/collections/radios/methods.js                  | 12 ++++++++++--
 3 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/app/client/templates/components/playlist-item.html b/app/client/templates/components/playlist-item.html
index 57888e7..cb1830b 100644
--- a/app/client/templates/components/playlist-item.html
+++ b/app/client/templates/components/playlist-item.html
@@ -7,7 +7,7 @@
             <a href="#" data-action="downvote" class="{{hasDownvote currentUser}}"><i class="material-icons">thumb_down</i></a>
         </div>
         {{/if}}
-        <img src="{{song.data.thumbnails.default.url}}" alt="" class="circle">
+        <img src="{{getImage song.data.thumbnails.default.url}}" alt="" class="circle">
         <div class="info">
             <p><a href="{{getLink}}" target="_blank">{{song.data.title}}</a></p>
             {{#if song.user}}
diff --git a/app/client/templates/components/playlist-item.js b/app/client/templates/components/playlist-item.js
index 30f4c7e..7473b85 100644
--- a/app/client/templates/components/playlist-item.js
+++ b/app/client/templates/components/playlist-item.js
@@ -6,6 +6,11 @@ Template.PlaylistItem.rendered = function() {
 };
 
 Template.PlaylistItem.helpers({
+  getImage(img) {
+    if (img) return img;
+    return '/album-default.jpg';
+  },
+
   getLink() {
     const id = _.get(this, 'song.id');
     const link = _.get(this, 'song.data.link');
diff --git a/app/collections/radios/methods.js b/app/collections/radios/methods.js
index 891dc6d..6720f2b 100644
--- a/app/collections/radios/methods.js
+++ b/app/collections/radios/methods.js
@@ -31,6 +31,7 @@ if (Meteor.isClient) {
 }
 
 if (Meteor.isServer) {
+  const PLAYLIST_SONG_LIMIT = 300;
   let Helpers = App.collectionHelpers.radio;
 
   Meteor.methods({
@@ -146,7 +147,7 @@ if (Meteor.isServer) {
       if (Helpers.isSongBlocked(radioId, song.id)) throw new Meteor.Error(500, 'This song is blocked');
       if (Helpers.isUserBlocked(radioId, song.user)) throw new Meteor.Error(500, 'Sorry, you cannot add songs to this radio');
 
-      if (radio.playlist.length >= 100) throw new Meteor.Error(500, 'Sorry, the playlist is full');
+      if (radio.playlist.length >= PLAYLIST_SONG_LIMIT) throw new Meteor.Error(500, 'Sorry, the playlist is full');
 
       if (!Helpers.isOwner(radioId)) {
         if (Helpers.hasUserReachedLimit(radio)) throw new Meteor.Error(500, 'You have reached your limit for this radio. Please try again later');
@@ -177,6 +178,7 @@ if (Meteor.isServer) {
       let guestRadios = [];
       let guestUsers = [];
       let toDelete;
+      let toDeleteArr = [];
 
       Radios.find({isGuest: true}).forEach(function(radio) {
         guestRadios.push({id: radio._id, user: radio.users[0]});
@@ -190,8 +192,14 @@ if (Meteor.isServer) {
         return guestUsers.indexOf(radio.user) !== -1;
       });
 
+      _.map(toDelete, (item) => {
+        toDeleteArr.push(item.id);
+      });
+
+      //TODO: test this and clean up the toDeleteArr (pluck not working anymore)
+
       if (!_.isEmpty(toDelete)) {
-        Radios.remove({'_id': {'$in': _.pluck(toDelete, 'id')}});
+        Radios.remove({'_id': {'$in': toDeleteArr}});
       }
     },