a membership with no user is always notifiable

since this is for user invites and the like.

1 files changed, 10 insertions, 2 deletions

diff --git a/app/models/member.rb b/app/models/member.rb
index b5f75c9bff0..57f85a9adaf 100644
--- a/app/models/member.rb
+++ b/app/models/member.rb
@@ -276,6 +276,14 @@ class Member < ActiveRecord::Base
     @notification_setting ||= user.notification_settings_for(source)
   end
 
+  def notifiable?(type, opts={})
+    # always notify when there isn't a user yet
+    return true if user.blank?
+
+    NotificationRecipientService.notifiable?(user, type, notifiable_options.merge(opts))
+  end
+
+
   private
 
   def send_invite
@@ -333,7 +341,7 @@ class Member < ActiveRecord::Base
     NotificationService.new
   end
 
-  def notifiable?(type, opts={})
-    raise 'abstract'
+  def notifiable_options
+    {}
   end
 end