jro_apps/redmine Commit - r879:fa95501fe5e8

Added issues status changes on the activity view (initial patch by Cyril Mougel)....

Jean-Philippe Lang -

r879:fa95501fe5e8

parent child

Context file:

r879:fa95501fe5e8

Collapse all files

test/unit/journal_test.rb created 644 +39 0

@@ -0,0 +1,39
	1	# redMine - project management software
	2	# Copyright (C) 2006-2007 Jean-Philippe Lang
	3	#
	4	# This program is free software; you can redistribute it and/or
	5	# modify it under the terms of the GNU General Public License
	6	# as published by the Free Software Foundation; either version 2
	7	# of the License, or (at your option) any later version.
	8	#
	9	# This program is distributed in the hope that it will be useful,
	10	# but WITHOUT ANY WARRANTY; without even the implied warranty of
	11	# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
	12	# GNU General Public License for more details.
	13	#
	14	# You should have received a copy of the GNU General Public License
	15	# along with this program; if not, write to the Free Software
	16	# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
	17
	18	require File.dirname(__FILE__) + '/../test_helper'
	19
	20	class JournalTest < Test::Unit::TestCase
	21	fixtures :issues, :issue_statuses, :journals, :journal_details
	22
	23	def setup
	24	@journal = Journal.find 1
	25	end
	26
	27	def test_journalized_is_an_issue
	28	issue = @journal.issue
	29	assert_kind_of Issue, issue
	30	assert_equal 1, issue.id
	31	end
	32
	33	def test_new_status
	34	status = @journal.new_status
	35	assert_not_nil status
	36	assert_kind_of IssueStatus, status
	37	assert_equal 2, status.id
	38	end
	39	end

app/controllers/projects_controller.rb +1 0

                 if @scope.include?('issues')
                   @events += @project.issues.find(:all, :include => [:author, :tracker], :conditions => ["#{Issue.table_name}.created_on>=? and #{Issue.table_name}.created_on<=?", @date_from, @date_to] )
+                  @events += @project.issues_status_changes(@date_from, @date_to)
                 end
                 if @scope.include?('news')

app/models/journal.rb +8 -1

                                  :project_key => "#{Issue.table_name}.project_id",
                                  :date_column => "#{Issue.table_name}.created_on"
-              acts_as_event :title => Proc.new {|o| "#{o.issue.tracker.name} ##{o.issue.id}: #{o.issue.subject}"},
+              acts_as_event :title => Proc.new {|o| "#{o.issue.tracker.name} ##{o.issue.id}: #{o.issue.subject}" + ((s = o.new_status) ? " (#{s})" : '') },
                             :description => :notes,
+                            :author => :user,
                             :url => Proc.new {|o| {:controller => 'issues', :action => 'show', :id => o.issue.id}}
               def save
                 # Do not save an empty journal
                 (details.empty? && notes.blank?) ? false : super
               end
+              # Returns the new status if the journal contains a status change, otherwise nil
+              def new_status
+                c = details.detect {|detail| detail.prop_key == 'status_id'}
+                (c && c.value) ? IssueStatus.find_by_id(c.value.to_i) : nil
+              end
             end

app/models/project.rb +10 0

@@ -76,6 +76,16 class Project < ActiveRecord::Base
76	end	76	end
77	end	77	end
78		78
		79	# Return all issues status changes for the project between the 2 given dates
		80	def issues_status_changes(from, to)
		81	Journal.find(:all, :include => [:issue, :details, :user],
		82	:conditions => ["#{Journal.table_name}.journalized_type = 'Issue'" +
		83	" AND #{Issue.table_name}.project_id = ?" +
		84	" AND #{JournalDetail.table_name}.prop_key = 'status_id'" +
		85	" AND #{Journal.table_name}.created_on BETWEEN ? AND ?",
		86	id, from, to+1])
		87	end
		88
79	# returns latest created projects	89	# returns latest created projects
80	# non public projects will be returned only if user is a member of those	90	# non public projects will be returned only if user is a member of those
81	def self.latest(user=nil, count=5)	91	def self.latest(user=nil, count=5)

app/views/projects/activity.rhtml +1 -1

               <% @events_by_day[day].sort {|x,y| y.event_datetime <=> x.event_datetime }.each do |e| %>
                 <li><p><%= e.event_datetime.strftime("%H:%M") %> <%= link_to truncate(e.event_title, 100), e.event_url %><br />
                 <% unless e.event_description.blank? %><em><%= truncate(e.event_description, 500) %></em><br /><% end %>
-                <span class="author"><%= e.event_author if e.respond_to?(:author) %></span></p></li>
+                <span class="author"><%= e.event_author if e.respond_to?(:event_author) %></span></p></li>
               <% end %>
               </ul>
             <% end %>

test/fixtures/issues.yml +2 -2

             ---
             issues_001:
-              created_on: 2006-07-19 21:02:17 +02:00
+              created_on: <%= 3.days.ago.to_date.to_s(:db) %>
               project_id: 1
-              updated_on: 2006-07-19 21:04:30 +02:00
+              updated_on: <%= 1.day.ago.to_date.to_s(:db) %>
               priority_id: 4
               subject: Can't print recipes
               id: 1

test/fixtures/journals.yml +1 -1

             ---
             journals_001:
-              created_on: 2007-01-26 19:58:40 +01:00
+              created_on: <%= 2.days.ago.to_date.to_s(:db) %>
               notes: "Journal notes"
               id: 1
               journalized_type: Issue

test/functional/projects_controller_test.rb +19 -1

@@ -22,7 +22,7 require 'projects_controller'
22	class ProjectsController; def rescue_action(e) raise e end; end	22	class ProjectsController; def rescue_action(e) raise e end; end
23		23
24	class ProjectsControllerTest < Test::Unit::TestCase	24	class ProjectsControllerTest < Test::Unit::TestCase
25	fixtures :projects, :users, :roles, :members, :issues, :enabled_modules, :enumerations	25	fixtures :projects, :users, :roles, :members, :issues, :journals, :journal_details, :trackers, :issue_statuses, :enabled_modules, :enumerations
26		26
27	def setup	27	def setup
28	@controller = ProjectsController.new	28	@controller = ProjectsController.new
@@ -93,6 +93,24 class ProjectsControllerTest < Test::Unit::TestCase
93	assert_response :success	93	assert_response :success
94	assert_template 'activity'	94	assert_template 'activity'
95	assert_not_nil assigns(:events_by_day)	95	assert_not_nil assigns(:events_by_day)
		96
		97	assert_tag :tag => "h3",
		98	:content => /#{2.days.ago.to_date.day}/,
		99	:sibling => { :tag => "ul",
		100	:child => { :tag => "li",
		101	:child => { :tag => "p",
		102	:content => /(#{IssueStatus.find(2).name})/,
		103	}
		104	}
		105	}
		106	assert_tag :tag => "h3",
		107	:content => /#{3.day.ago.to_date.day}/,
		108	:sibling => { :tag => "ul", :child => { :tag => "li",
		109	:child => { :tag => "p",
		110	:content => /#{Issue.find(1).subject}/,
		111	}
		112	}
		113	}
96	end	114	end
97		115
98	def test_archive	116	def test_archive

test/unit/project_test.rb +10 -1

             require File.dirname(__FILE__) + '/../test_helper'
             class ProjectTest < Test::Unit::TestCase
-              fixtures :projects
+              fixtures :projects, :issues, :issue_statuses, :journals, :journal_details
               def setup
                 @ecookbook = Project.find(1)
                 sub.parent = Project.find(2)
                 assert !sub.save
               end
+              def test_issues_status_changes
+                journals = @ecookbook.issues_status_changes 3.days.ago.to_date, Date.today
+                assert_equal 1, journals.size
+                assert_kind_of Journal, journals.first
+                journals = @ecookbook.issues_status_changes 30.days.ago.to_date, 10.days.ago.to_date
+                assert_equal 0, journals.size
+              end
             end

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages