Create a pending task when a user is mentioned on a note

8 files changed, 109 insertions, 20 deletions

diff --git a/app/assets/stylesheets/pages/tasks.scss b/app/assets/stylesheets/pages/tasks.scss
index f0e13f4d786..a3dffeed4ab 100644
--- a/app/assets/stylesheets/pages/tasks.scss
+++ b/app/assets/stylesheets/pages/tasks.scss
@@ -59,6 +59,15 @@
     .task-note {
       word-wrap: break-word;
 
+      .md {
+        color: #7f8fa4;
+        font-size: $gl-font-size;
+
+        p {
+          color: #5c5d5e;
+        }
+      }
+
       pre {
         border: none;
         background: #f9f9f9;
@@ -68,10 +77,25 @@
         overflow: hidden;
       }
 
+      .note-image-attach {
+        margin-top: 4px;
+        margin-left: 0px;
+        max-width: 200px;
+        float: none;
+      }
+
       p:last-child {
         margin-bottom: 0;
       }
     }
+
+    .task-note-icon {
+      color: #777;
+      float: left;
+      font-size: $gl-font-size;
+      line-height: 16px;
+      margin-right: 5px;
+    }
   }
 
   &:last-child { border:none }
diff --git a/app/helpers/tasks_helper.rb b/app/helpers/tasks_helper.rb
index cf4eab0ef94..6975c1d1604 100644
--- a/app/helpers/tasks_helper.rb
+++ b/app/helpers/tasks_helper.rb
@@ -22,4 +22,20 @@ module TasksHelper
 
     [task.action_name, target].join(" ")
   end
+
+  def task_note_link_html(task)
+    link_to task_note_target_path(task) do
+      "##{task.target_iid}"
+    end
+  end
+
+  def task_note_target_path(task)
+    polymorphic_path([task.project.namespace.becomes(Namespace),
+      task.project, task.target], anchor: dom_id(task.note))
+  end
+
+  def task_note(text, options = {})
+    text = first_line_in_markdown(text, 150, options)
+    sanitize(text, tags: %w(a img b pre code p span))
+  end
 end
diff --git a/app/models/note.rb b/app/models/note.rb
index b3809ad81e0..73412024f4e 100644
--- a/app/models/note.rb
+++ b/app/models/note.rb
@@ -37,6 +37,8 @@ class Note < ActiveRecord::Base
   belongs_to :author, class_name: "User"
   belongs_to :updated_by, class_name: "User"
 
+  has_many :tasks, dependent: :delete_all
+
   delegate :name, to: :project, prefix: true
   delegate :name, :email, to: :author, prefix: true
 
diff --git a/app/models/task.rb b/app/models/task.rb
index c9881991e38..38c6637e456 100644
--- a/app/models/task.rb
+++ b/app/models/task.rb
@@ -8,6 +8,7 @@
 #  target_id   :integer          not null
 #  target_type :string           not null
 #  author_id   :integer
+#  note_id     :integer
 #  action      :integer
 #  state       :string           not null
 #  created_at  :datetime
@@ -19,6 +20,7 @@ class Task < ActiveRecord::Base
   MENTIONED = 2
 
   belongs_to :author, class_name: "User"
+  belongs_to :note
   belongs_to :project
   belongs_to :target, polymorphic: true, touch: true
   belongs_to :user
@@ -52,6 +54,10 @@ class Task < ActiveRecord::Base
     target.respond_to? :title
   end
 
+  def note_text
+    note.try(:note)
+  end
+
   def target_iid
     target.respond_to?(:iid) ? target.iid : target_id
   end
diff --git a/app/services/task_service.rb b/app/services/task_service.rb
index fa615f4cfcf..d97fa146f5c 100644
--- a/app/services/task_service.rb
+++ b/app/services/task_service.rb
@@ -77,7 +77,17 @@ class TaskService
   def new_note(note)
     # Skip system notes, like status changes and cross-references
     unless note.system
-      mark_pending_tasks_as_done(note.noteable, note.author)
+      project = note.project
+      target  = note.noteable
+      author  = note.author
+
+      mark_pending_tasks_as_done(target, author)
+
+      mentioned_users = build_mentioned_users(project, note, author)
+
+      mentioned_users.each do |user|
+        create_task(project, target, author, user, Task::MENTIONED, note)
+      end
     end
   end
 
@@ -94,14 +104,15 @@ class TaskService
 
   private
 
-  def create_task(project, target, author, user, action)
+  def create_task(project, target, author, user, action, note = nil)
     attributes = {
       project: project,
       user_id: user.id,
       author_id: author.id,
       target_id: target.id,
       target_type: target.class.name,
-      action: action
+      action: action,
+      note: note
     }
 
     Task.create(attributes)
diff --git a/app/views/dashboard/tasks/_common.html.haml b/app/views/dashboard/tasks/_common.html.haml
new file mode 100644
index 00000000000..b6d0c3c03ac
--- /dev/null
+++ b/app/views/dashboard/tasks/_common.html.haml
@@ -0,0 +1,17 @@
+.task-title
+  %span.author_name= link_to_author task
+  %span.task_label{class: task.action_name}
+    = task_action_name(task)
+
+  %strong= link_to "##{task.target_iid}", [task.project.namespace.becomes(Namespace), task.project, task.target]
+
+  &middot; #{time_ago_with_tooltip(task.created_at)}
+
+- if task.pending?
+  .task-actions.pull-right
+    = link_to 'Done', [:dashboard, task], method: :delete, class: 'btn'
+
+- if task.body?
+  .task-body
+    .task-note
+      = task.target.title
diff --git a/app/views/dashboard/tasks/_note.html.haml b/app/views/dashboard/tasks/_note.html.haml
new file mode 100644
index 00000000000..2cfd55afccb
--- /dev/null
+++ b/app/views/dashboard/tasks/_note.html.haml
@@ -0,0 +1,26 @@
+.task-title
+  %span.author_name
+    = link_to_author task
+  %span.task_label{class: task.action_name}
+    = task_action_name(task)
+    = task_note_link_html(task)
+
+  &middot; #{time_ago_with_tooltip(task.created_at)}
+
+- if task.pending?
+  .task-actions.pull-right
+    = link_to 'Done', [:dashboard, task], method: :delete, class: 'btn'
+
+.task-body
+  .task-note
+    .md
+      = task_note(task.note_text, project: task.project)
+    - note = task.note
+    - if note.attachment.url
+      - if note.attachment.image?
+        = link_to note.attachment.url, target: '_blank' do
+          = image_tag note.attachment.url, class: 'note-image-attach'
+      - else
+        = link_to note.attachment.url, target: "_blank", class: 'note-file-attach' do
+          %i.fa.fa-paperclip
+          = note.attachment_identifier
diff --git a/app/views/dashboard/tasks/_task.html.haml b/app/views/dashboard/tasks/_task.html.haml
index b33f3894fd3..2ca8f0dad63 100644
--- a/app/views/dashboard/tasks/_task.html.haml
+++ b/app/views/dashboard/tasks/_task.html.haml
@@ -2,20 +2,7 @@
   .task-item{class: "#{task.body? ? 'task-block' : 'task-inline' }"}
     = image_tag avatar_icon(task.author_email, 40), class: "avatar s40", alt:''
 
-    .task-title
-      %span.author_name= link_to_author task
-      %span.task_label{class: task.action_name}
-        = task_action_name(task)
-
-      %strong= link_to "##{task.target_iid}", [task.project.namespace.becomes(Namespace), task.project, task.target]
-
-      &middot; #{time_ago_with_tooltip(task.created_at)}
-
-    - if task.pending?
-      .task-actions.pull-right
-        = link_to 'Done', [:dashboard, task], method: :delete, class: 'btn'
-
-    - if task.body?
-      .task-body
-        .task-note
-          = task.target.title
+    - if task.note.present?
+      = render 'note', task: task
+    - else
+      = render 'common', task: task