Make sure NULL users don't crash the email

rism-digital · Jan 20, 2021 · 6bc66b9 · 6bc66b9
1 parent cdb4d11
commit 6bc66b9
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/app/mailers/comment_notifications.rb b/app/mailers/comment_notifications.rb
@@ -28,7 +28,7 @@ def new_comment(comment)
     users.compact.uniq! # Users can be duplicated
     users -= [comment.author_id] # Don't send the comment to myself!
 
-    addresses = users.each.map do |u|
+    addresses = users.compact.each.map do |u|
       next if u == 0 # Don't send to user 0 (autogenerated items during import)
       next if u == 1 # Don't sent to admin
       email = User.find(u).email