Merge branch 'master' of dev.gitlabhq.com:gitlabhq

Dmitriy Zaporozhets
2 parents 05044658 c9b64974
Showing 15 changed files with 102 additions and 23 deletions Show diff stats
app/assets/stylesheets/common.scss
app/controllers/issues_controller.rb
app/controllers/merge_requests_controller.rb
app/models/activity_observer.rb
app/models/event.rb
app/models/issue.rb
app/models/merge_request.rb
app/models/project.rb
app/views/dashboard/index.html.haml
app/views/events/_event.html.haml
app/views/events/_event_changed_issue.html.haml
app/views/events/_event_changed_merge_request.html.haml
app/views/events/_event_push.html.haml
db/migrate/20120307095918_add_author_id_to_event.rb
db/schema.rb
@@ -599,7 +599,7 @@ p.time {
     }
   }
   .dashboard_block { 
-    width:700px;
+    width:840px;
     margin:auto;
     .wll { 
@@ -76,7 +76,7 @@ class IssuesController &lt; ApplicationController
   end
   def update
-    @issue.update_attributes(params[:issue])
+    @issue.update_attributes(params[:issue].merge(:author_id_of_changes => current_user.id))
     respond_to do |format|
       format.js
@@ -87,7 +87,7 @@ class MergeRequestsController &lt; ApplicationController
   def update
     respond_to do |format|
-      if @merge_request.update_attributes(params[:merge_request])
+      if @merge_request.update_attributes(params[:merge_request].merge(:author_id_of_changes => current_user.id))
         format.html { redirect_to [@project, @merge_request], notice: 'Merge request was successfully updated.' }
         format.json { head :ok }
       else
 class ActivityObserver < ActiveRecord::Observer
-  observe :issue, :merge_request, :note
+  observe :issue, :merge_request
   def after_create(record)
     Event.create(
       :project => record.project,
       :target_id => record.id,
       :target_type => record.class.name,
-      :action => Event.determine_action(record)
+      :action => Event.determine_action(record),
+      :author_id => record.author_id
     )
   end
+
+  def after_save(record)
+    if record.changed.include?("closed") 
+      Event.create(
+        :project => record.project,
+        :target_id => record.id,
+        :target_type => record.class.name,
+        :action => (record.closed ? Event::Closed : Event::Reopened),
+        :author_id => record.author_id_of_changes
+      )
+    end
+  end
 end
 class Event < ActiveRecord::Base
+  default_scope where("author_id IS NOT NULL")
+
   Created   = 1
   Updated   = 2
   Closed    = 3
@@ -26,13 +28,22 @@ class Event &lt; ActiveRecord::Base
   #  - new issue
   #  - merge request
   def allowed?
-    push? || new_issue? || new_merge_request?
+    push? || new_issue? || new_merge_request? || 
+      changed_merge_request? || changed_issue?
   end
   def push?
     action == self.class::Pushed
   end
+  def closed?
+    action == self.class::Closed
+  end
+
+  def reopened?
+    action == self.class::Reopened
+  end
+
   def new_tag? 
     data[:ref]["refs/tags"]
   end
@@ -57,10 +68,6 @@ class Event &lt; ActiveRecord::Base
     @tag_name ||= data[:ref].gsub("refs/tags/", "")
   end
-  def pusher
-    User.find_by_id(data[:user_id])
-  end
-
   def new_issue? 
     target_type == "Issue" && 
       action == Created
@@ -71,6 +78,16 @@ class Event &lt; ActiveRecord::Base
       action == Created
   end
+  def changed_merge_request? 
+    target_type == "MergeRequest" && 
+      [Closed, Reopened].include?(action)
+  end
+
+  def changed_issue? 
+    target_type == "Issue" && 
+      [Closed, Reopened].include?(action)
+  end
+
   def issue 
     target if target_type == "Issue"
   end
@@ -80,7 +97,7 @@ class Event &lt; ActiveRecord::Base
   end
   def author 
-    target.author
+    @author ||= User.find(author_id)
   end
   def commits
@@ -89,7 +106,6 @@ class Event &lt; ActiveRecord::Base
     end
   end
-  delegate :id, :name, :email, :to => :pusher, :prefix => true, :allow_nil => true
   delegate :name, :email, :to => :author, :prefix => true, :allow_nil => true
   delegate :title, :to => :issue, :prefix => true, :allow_nil => true
   delegate :title, :to => :merge_request, :prefix => true, :allow_nil => true
@@ -5,6 +5,7 @@ class Issue &lt; ActiveRecord::Base
   has_many :notes, :as => :noteable, :dependent => :destroy
   attr_protected :author, :author_id, :project, :project_id
+  attr_accessor :author_id_of_changes
   validates_presence_of :project_id
   validates_presence_of :assignee_id
@@ -5,6 +5,7 @@ class MergeRequest &lt; ActiveRecord::Base
   has_many :notes, :as => :noteable, :dependent => :destroy
   attr_protected :author, :author_id, :project, :project_id
+  attr_accessor :author_id_of_changes
   validates_presence_of :project_id
   validates_presence_of :assignee_id
@@ -68,7 +68,8 @@ class Project &lt; ActiveRecord::Base
     Event.create(
       :project => self,
       :action => Event::Pushed,
-      :data => data
+      :data => data,
+      :author_id => Key.find_by_identifier(author_key_id).user.id
     )
   end
@@ -259,7 +260,7 @@ class Project &lt; ActiveRecord::Base
   def commit(commit_id = nil)
     commit = if commit_id
-               repo.commits(commit_id).first
+               repo.commit(commit_id)
              else
                repo.commits.first
              end
@@ -20,8 +20,8 @@
   .row
     .dashboard_block
       .row
-        .span9= render "dashboard/projects_feed"
-        .span3.right
+        .span10= render "dashboard/projects_feed"
+        .span4.right
           - if current_user.can_create_project?
             .alert-message.block-message.warning
               You can create up to
@@ -2,7 +2,11 @@
   .event_feed
     - if event.new_issue? 
       = render "events/event_new_issue", :event => event
-    - if event.new_merge_request? 
+    - elsif event.new_merge_request? 
       = render "events/event_new_merge_request", :event => event
+    - elsif event.changed_merge_request? 
+      = render "events/event_changed_merge_request", :event => event
+    - elsif event.changed_issue? 
+      = render "events/event_changed_issue", :event => event
     - elsif event.push?
       = render "events/event_push", :event => event
@@ -0,0 +1,14 @@
+= image_tag gravatar_icon(event.author_email), :class => "avatar"
+%strong #{event.author_name} 
+- if event.closed?
+  closed
+- else 
+  reopened
+issue 
+= link_to project_issue_path(event.project, event.issue) do
+  %strong= truncate event.issue_title
+at 
+%strong= link_to event.project.name, event.project
+%span.cgray
+  = time_ago_in_words(event.created_at)
+  ago.
@@ -0,0 +1,19 @@
+= image_tag gravatar_icon(event.author_email), :class => "avatar"
+%strong #{event.author_name} 
+- if event.closed?
+  closed
+- else 
+  reopened
+merge request 
+= link_to project_merge_request_path(event.project, event.merge_request) do
+  %strong= truncate event.merge_request_title
+at 
+%strong= link_to event.project.name, event.project
+%span.cgray
+  = time_ago_in_words(event.created_at)
+  ago.
+%br
+%span.label= event.merge_request.source_branch
+&rarr;
+%span.label= event.merge_request.target_branch
+
 - if event.new_branch? || event.new_tag?
-  = image_tag gravatar_icon(event.pusher_email), :class => "avatar"
-  %strong #{event.pusher_name} 
+  = image_tag gravatar_icon(event.author_email), :class => "avatar"
+  %strong #{event.author_name} 
   pushed new 
   - if event.new_tag?
     tag
@@ -16,8 +16,8 @@
     = time_ago_in_words(event.created_at)
     ago.
 - else 
-  = image_tag gravatar_icon(event.pusher_email), :class => "avatar"
-  %strong #{event.pusher_name}
+  = image_tag gravatar_icon(event.author_email), :class => "avatar"
+  %strong #{event.author_name}
   pushed to 
   = link_to project_commits_path(event.project, :ref => event.branch_name) do
     %strong= event.branch_name
@@ -31,6 +31,10 @@
       Compare #{event.commits.first.commit.id[0..8]}...#{event.commits.last.id[0..8]}
   - @project = event.project  
   %ul.unstyled
-    = render event.commits
+    - if event.commits.size > 4
+      = render event.commits[0..2]
+      %li ... and #{event.commits.size - 3} more commits
+    - else 
+      = render event.commits
@@ -0,0 +1,5 @@
+class AddAuthorIdToEvent < ActiveRecord::Migration
+  def change
+    add_column :events, :author_id, :integer, :null => true
+  end
+end
@@ -11,7 +11,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
-ActiveRecord::Schema.define(:version => 20120301185805) do
+ActiveRecord::Schema.define(:version => 20120307095918) do
   create_table "events", :force => true do |t|
     t.string   "target_type"
@@ -22,6 +22,7 @@ ActiveRecord::Schema.define(:version =&gt; 20120301185805) do
     t.datetime "created_at",  :null => false
     t.datetime "updated_at",  :null => false
     t.integer  "action"
+    t.integer  "author_id"
   end
   create_table "issues", :force => true do |t|
	@@ -599,7 +599,7 @@ p.time {		@@ -599,7 +599,7 @@ p.time {
599	}	599	}
600	}	600	}
601	.dashboard_block {	601	.dashboard_block {
602	- width:700px;	602	+ width:840px;
603	margin:auto;	603	margin:auto;
604		604
605	.wll {	605	.wll {
	@@ -76,7 +76,7 @@ class IssuesController < ApplicationController		@@ -76,7 +76,7 @@ class IssuesController < ApplicationController
76	end	76	end
77		77
78	def update	78	def update
79	- @issue.update_attributes(params[:issue])	79	+ @issue.update_attributes(params[:issue].merge(:author_id_of_changes => current_user.id))
80		80
81	respond_to do \|format\|	81	respond_to do \|format\|
82	format.js	82	format.js
	@@ -87,7 +87,7 @@ class MergeRequestsController < ApplicationController		@@ -87,7 +87,7 @@ class MergeRequestsController < ApplicationController
87		87
88	def update	88	def update
89	respond_to do \|format\|	89	respond_to do \|format\|
90	- if @merge_request.update_attributes(params[:merge_request])	90	+ if @merge_request.update_attributes(params[:merge_request].merge(:author_id_of_changes => current_user.id))
91	format.html { redirect_to [@project, @merge_request], notice: 'Merge request was successfully updated.' }	91	format.html { redirect_to [@project, @merge_request], notice: 'Merge request was successfully updated.' }
92	format.json { head :ok }	92	format.json { head :ok }
93	else	93	else
1	class ActivityObserver < ActiveRecord::Observer	1	class ActivityObserver < ActiveRecord::Observer
2	- observe :issue, :merge_request, :note	2	+ observe :issue, :merge_request
3		3
4	def after_create(record)	4	def after_create(record)
5	Event.create(	5	Event.create(
6	:project => record.project,	6	:project => record.project,
7	:target_id => record.id,	7	:target_id => record.id,
8	:target_type => record.class.name,	8	:target_type => record.class.name,
9	- :action => Event.determine_action(record)	9	+ :action => Event.determine_action(record),
		10	+ :author_id => record.author_id
10	)	11	)
11	end	12	end
		13	+
		14	+ def after_save(record)
		15	+ if record.changed.include?("closed")
		16	+ Event.create(
		17	+ :project => record.project,
		18	+ :target_id => record.id,
		19	+ :target_type => record.class.name,
		20	+ :action => (record.closed ? Event::Closed : Event::Reopened),
		21	+ :author_id => record.author_id_of_changes
		22	+ )
		23	+ end
		24	+ end
12	end	25	end
	@@ -5,6 +5,7 @@ class Issue < ActiveRecord::Base		@@ -5,6 +5,7 @@ class Issue < ActiveRecord::Base
5	has_many :notes, :as => :noteable, :dependent => :destroy	5	has_many :notes, :as => :noteable, :dependent => :destroy
6		6
7	attr_protected :author, :author_id, :project, :project_id	7	attr_protected :author, :author_id, :project, :project_id
		8	+ attr_accessor :author_id_of_changes
8		9
9	validates_presence_of :project_id	10	validates_presence_of :project_id
10	validates_presence_of :assignee_id	11	validates_presence_of :assignee_id
	@@ -5,6 +5,7 @@ class MergeRequest < ActiveRecord::Base		@@ -5,6 +5,7 @@ class MergeRequest < ActiveRecord::Base
5	has_many :notes, :as => :noteable, :dependent => :destroy	5	has_many :notes, :as => :noteable, :dependent => :destroy
6		6
7	attr_protected :author, :author_id, :project, :project_id	7	attr_protected :author, :author_id, :project, :project_id
		8	+ attr_accessor :author_id_of_changes
8		9
9	validates_presence_of :project_id	10	validates_presence_of :project_id
10	validates_presence_of :assignee_id	11	validates_presence_of :assignee_id
@@ -0,0 +1,14 @@		@@ -0,0 +1,14 @@
	1	+= image_tag gravatar_icon(event.author_email), :class => "avatar"
	2	+%strong #{event.author_name}
	3	+- if event.closed?
	4	+ closed
	5	+- else
	6	+ reopened
	7	+issue
	8	+= link_to project_issue_path(event.project, event.issue) do
	9	+ %strong= truncate event.issue_title
	10	+at
	11	+%strong= link_to event.project.name, event.project
	12	+%span.cgray
	13	+ = time_ago_in_words(event.created_at)
	14	+ ago.
@@ -0,0 +1,19 @@		@@ -0,0 +1,19 @@
	1	+= image_tag gravatar_icon(event.author_email), :class => "avatar"
	2	+%strong #{event.author_name}
	3	+- if event.closed?
	4	+ closed
	5	+- else
	6	+ reopened
	7	+merge request
	8	+= link_to project_merge_request_path(event.project, event.merge_request) do
	9	+ %strong= truncate event.merge_request_title
	10	+at
	11	+%strong= link_to event.project.name, event.project
	12	+%span.cgray
	13	+ = time_ago_in_words(event.created_at)
	14	+ ago.
	15	+%br
	16	+%span.label= event.merge_request.source_branch
	17	+→
	18	+%span.label= event.merge_request.target_branch
	19	+
@@ -0,0 +1,5 @@		@@ -0,0 +1,5 @@
	1	+class AddAuthorIdToEvent < ActiveRecord::Migration
	2	+ def change
	3	+ add_column :events, :author_id, :integer, :null => true
	4	+ end
	5	+end