From 69751aac32f505a87a40af638dbf14f69f85315e Mon Sep 17 00:00:00 2001
From: Dmitriy Zaporozhets <dzaporozhets@sphereconsultinginc.com>
Date: Mon, 1 Oct 2012 16:58:13 +0300
Subject: [PATCH] Refactoring event views

---
 app/helpers/events_helper.rb             | 14 ++++++++++++++
 app/views/events/_event.html.haml        |  8 ++++++--
 app/views/events/_event_common.html.haml |  8 --------
 app/views/events/_event_push.html.haml   | 27 ---------------------------
 app/views/events/event/_common.html.haml |  7 +++++++
 app/views/events/event/_push.html.haml   | 22 ++++++++++++++++++++++
 6 files changed, 49 insertions(+), 37 deletions(-)
 delete mode 100644 app/views/events/_event_common.html.haml
 delete mode 100644 app/views/events/_event_push.html.haml
 create mode 100644 app/views/events/event/_common.html.haml
 create mode 100644 app/views/events/event/_push.html.haml

diff --git a/app/helpers/events_helper.rb b/app/helpers/events_helper.rb
index 9ad438d..2c466ab 100644
--- a/app/helpers/events_helper.rb
+++ b/app/helpers/events_helper.rb
@@ -19,4 +19,18 @@ module EventsHelper
 
     [event.action_name, target].join(" ")
   end
+
+  def event_image event
+    event_image_path = if event.push?
+                   "event_push.png"
+                 elsif event.merged?
+                   "event_mr_merged.png"
+                 end
+
+    return nil unless event_image_path
+
+    content_tag :div, class: 'event_icon' do
+      image_tag event_image_path
+    end
+  end
 end
diff --git a/app/views/events/_event.html.haml b/app/views/events/_event.html.haml
index cdc2420..0d91a67 100644
--- a/app/views/events/_event.html.haml
+++ b/app/views/events/_event.html.haml
@@ -1,10 +1,14 @@
 - if event.allowed?
   %div.event-item
+    = event_image(event)
+    = image_tag gravatar_icon(event.author_email), class: "avatar"
+
     - if event.push?
-      = render "events/event_push", event: event
+      = render "events/event/push", event: event
     - else
-      = render "events/event_common", event: event
+      = render "events/event/common", event: event
 
+    .clearfix
     %span.cgray.right
       = time_ago_in_words(event.created_at)
       ago.
diff --git a/app/views/events/_event_common.html.haml b/app/views/events/_event_common.html.haml
deleted file mode 100644
index 0f9a86e..0000000
--- a/app/views/events/_event_common.html.haml
+++ /dev/null
@@ -1,8 +0,0 @@
-= image_tag gravatar_icon(event.author_email), class: "avatar"
-.event-title
-  %span.author_name= link_to_author event
-  %span.event_label{class: event.action_name}= event_action_name(event)
-  = link_to [event.project, event.target] do
-    %strong= truncate event.target_title
-  at
-  = link_to_project event.project
diff --git a/app/views/events/_event_push.html.haml b/app/views/events/_event_push.html.haml
deleted file mode 100644
index caf1b2c..0000000
--- a/app/views/events/_event_push.html.haml
+++ /dev/null
@@ -1,27 +0,0 @@
-%div
-  .event_icon= image_tag "event_push.png"
-  = image_tag gravatar_icon(event.author_email), class: "avatar"
-
-  .event-title
-    %span.author_name= link_to_author event
-    %span.event_label.pushed #{event.push_action_name} #{event.ref_type}
-    = link_to project_commits_path(event.project, event.ref_name) do
-      %strong= event.ref_name
-    at
-    %strong= link_to event.project.name, event.project
-
-  - if event.push_with_commits?
-    - project = event.project
-    .event-body
-      %ul.unstyled.event_commits
-        - few_commits = event.commits[0...2]
-        - few_commits.each do |commit|
-          = render "events/commit", commit: commit, project: project
-
-        - if event.commits_count > 1
-          %li.commits-stat
-            - if event.commits_count > 2
-              %span ... and #{event.commits_count - 2} more commits.
-            = link_to project_compare_path(event.project, from: event.parent_commit.id, to: event.last_commit.id) do
-              %strong Compare &rarr; #{event.parent_commit.id[0..7]}...#{event.last_commit.id[0..7]}
-  .clearfix
diff --git a/app/views/events/event/_common.html.haml b/app/views/events/event/_common.html.haml
new file mode 100644
index 0000000..ffcaa81
--- /dev/null
+++ b/app/views/events/event/_common.html.haml
@@ -0,0 +1,7 @@
+.event-title
+  %span.author_name= link_to_author event
+  %span.event_label{class: event.action_name}= event_action_name(event)
+  = link_to [event.project, event.target] do
+    %strong= truncate event.target_title
+  at
+  = link_to_project event.project
diff --git a/app/views/events/event/_push.html.haml b/app/views/events/event/_push.html.haml
new file mode 100644
index 0000000..e0c0b61
--- /dev/null
+++ b/app/views/events/event/_push.html.haml
@@ -0,0 +1,22 @@
+.event-title
+  %span.author_name= link_to_author event
+  %span.event_label.pushed #{event.push_action_name} #{event.ref_type}
+  = link_to project_commits_path(event.project, event.ref_name) do
+    %strong= event.ref_name
+  at
+  %strong= link_to event.project.name, event.project
+
+- if event.push_with_commits?
+  - project = event.project
+  .event-body
+    %ul.unstyled.event_commits
+      - few_commits = event.commits[0...2]
+      - few_commits.each do |commit|
+        = render "events/commit", commit: commit, project: project
+
+      - if event.commits_count > 1
+        %li.commits-stat
+          - if event.commits_count > 2
+            %span ... and #{event.commits_count - 2} more commits.
+          = link_to project_compare_path(event.project, from: event.parent_commit.id, to: event.last_commit.id) do
+            %strong Compare &rarr; #{event.parent_commit.id[0..7]}...#{event.last_commit.id[0..7]}
--
libgit2 0.21.2