From 36d858bcf98868426256ce51a28e7029b38e0c2d Mon Sep 17 00:00:00 2001
From: Zeger-Jan van de Weg <mail@zjvandeweg.nl>
Date: Sat, 9 Jan 2016 12:47:31 +0100
Subject: [PATCH] Add #can_unsubscribe? to SentNotification

---
 app/models/sent_notification.rb    | 4 ++++
 app/views/layouts/notify.html.haml | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/app/models/sent_notification.rb b/app/models/sent_notification.rb
index fd12615717b..03108da17be 100644
--- a/app/models/sent_notification.rb
+++ b/app/models/sent_notification.rb
@@ -62,6 +62,10 @@ class SentNotification < ActiveRecord::Base
     end
   end
 
+  def can_unsubscribe?
+    !for_commit?
+  end
+
   def for_commit?
     noteable_type == "Commit"
   end
diff --git a/app/views/layouts/notify.html.haml b/app/views/layouts/notify.html.haml
index da7de41abad..07f9be12f99 100644
--- a/app/views/layouts/notify.html.haml
+++ b/app/views/layouts/notify.html.haml
@@ -45,7 +45,7 @@
           -# Don't link the host is the line below, one link in the email is easier to quickly click than two.
           You're receiving this email because of your account on #{Gitlab.config.gitlab.host}.
           If you'd like to receive fewer emails, you can
-          - if @sent_notification && !@sent_notification.for_commit?
+          - if @sent_notification && @sent_notification.can_unsubscribe?
             = link_to "unsubscribe", unsubscribe_sent_notification_url(@sent_notification)
             from this thread or
           adjust your notification settings.
-- 
GitLab