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
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
# - 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
@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
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
end
def author
- target.author
+ @author ||= User.find(author_id)
end
def commits
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
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
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
Event.create(
:project => self,
:action => Event::Pushed,
- :data => data
+ :data => data,
+ :author_id => Key.find_by_identifier(author_key_id).user.id
)
end
.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
--- /dev/null
+= 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.
--- /dev/null
+= 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
+→
+%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
= 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
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
--- /dev/null
+class AddAuthorIdToEvent < ActiveRecord::Migration
+ def change
+ add_column :events, :author_id, :integer, :null => true
+ end
+end
#
# 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"
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|