kolan
/
Redmine
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

remove trailing white-spaces from app/models/issue_status.rb. 

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@6504 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
Toshi MARUYAMA 2011-08-21 01:55:08 +00:00
parent 67169fe9dd
commit b335f1dc87
1 changed files with 16 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
app/models/issue_status.rb
View File

@ -1,43 +1,43 @@
# redMine - project management software # Redmine - project management software
# Copyright (C) 2006 Jean-Philippe Lang # Copyright (C) 2006-2011 Jean-Philippe Lang
# #
# This program is free software; you can redistribute it and/or # This program is free software; you can redistribute it and/or
# modify it under the terms of the GNU General Public License # modify it under the terms of the GNU General Public License
# as published by the Free Software Foundation; either version 2 # as published by the Free Software Foundation; either version 2
# of the License, or (at your option) any later version. # of the License, or (at your option) any later version.
# #
# This program is distributed in the hope that it will be useful, # This program is distributed in the hope that it will be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of # but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details. # GNU General Public License for more details.
# #
# You should have received a copy of the GNU General Public License # You should have received a copy of the GNU General Public License
# along with this program; if not, write to the Free Software # along with this program; if not, write to the Free Software
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
class IssueStatus < ActiveRecord::Base class IssueStatus < ActiveRecord::Base
before_destroy :check_integrity before_destroy :check_integrity
has_many :workflows, :foreign_key => "old_status_id" has_many :workflows, :foreign_key => "old_status_id"
acts_as_list acts_as_list
before_destroy :delete_workflows before_destroy :delete_workflows
validates_presence_of :name validates_presence_of :name
validates_uniqueness_of :name validates_uniqueness_of :name
validates_length_of :name, :maximum => 30 validates_length_of :name, :maximum => 30
validates_inclusion_of :default_done_ratio, :in => 0..100, :allow_nil => true validates_inclusion_of :default_done_ratio, :in => 0..100, :allow_nil => true
named_scope :named, lambda {|arg| { :conditions => ["LOWER(#{table_name}.name) = LOWER(?)", arg.to_s.strip]}} named_scope :named, lambda {|arg| { :conditions => ["LOWER(#{table_name}.name) = LOWER(?)", arg.to_s.strip]}}
def after_save def after_save
IssueStatus.update_all("is_default=#{connection.quoted_false}", ['id <> ?', id]) if self.is_default? IssueStatus.update_all("is_default=#{connection.quoted_false}", ['id <> ?', id]) if self.is_default?
end end
# Returns the default status for new issues # Returns the default status for new issues
def self.default def self.default
find(:first, :conditions =>["is_default=?", true]) find(:first, :conditions =>["is_default=?", true])
end end
# Update all the +Issues+ setting their done_ratio to the value of their +IssueStatus+ # Update all the +Issues+ setting their done_ratio to the value of their +IssueStatus+
def self.update_issue_done_ratios def self.update_issue_done_ratios
if Issue.use_status_for_done_ratio? if Issue.use_status_for_done_ratio?
@ -57,7 +57,7 @@ class IssueStatus < ActiveRecord::Base
role_ids = roles.collect(&:id) role_ids = roles.collect(&:id)
transitions = workflows.select do |w| transitions = workflows.select do |w|
role_ids.include?(w.role_id) && role_ids.include?(w.role_id) &&
w.tracker_id == tracker.id && w.tracker_id == tracker.id &&
((!w.author && !w.assignee) || (author && w.author) || (assignee && w.assignee)) ((!w.author && !w.assignee) || (author && w.author) || (assignee && w.assignee))
end end
transitions.collect{|w| w.new_status}.compact.sort transitions.collect{|w| w.new_status}.compact.sort
@ -65,7 +65,7 @@ class IssueStatus < ActiveRecord::Base
[] []
end end
end end
# Same thing as above but uses a database query # Same thing as above but uses a database query
# More efficient than the previous method if called just once # More efficient than the previous method if called just once
def find_new_statuses_allowed_to(roles, tracker, author=false, assignee=false) def find_new_statuses_allowed_to(roles, tracker, author=false, assignee=false)
@ -73,10 +73,10 @@ class IssueStatus < ActiveRecord::Base
conditions = "(author = :false AND assignee = :false)" conditions = "(author = :false AND assignee = :false)"
conditions << " OR author = :true" if author conditions << " OR author = :true" if author
conditions << " OR assignee = :true" if assignee conditions << " OR assignee = :true" if assignee
workflows.find(:all, workflows.find(:all,
:include => :new_status, :include => :new_status,
:conditions => ["role_id IN (:role_ids) AND tracker_id = :tracker_id AND (#{conditions})", :conditions => ["role_id IN (:role_ids) AND tracker_id = :tracker_id AND (#{conditions})",
{:role_ids => roles.collect(&:id), :tracker_id => tracker.id, :true => true, :false => false} {:role_ids => roles.collect(&:id), :tracker_id => tracker.id, :true => true, :false => false}
] ]
).collect{|w| w.new_status}.compact.sort ).collect{|w| w.new_status}.compact.sort
@ -88,14 +88,14 @@ class IssueStatus < ActiveRecord::Base
def <=>(status) def <=>(status)
position <=> status.position position <=> status.position
end end
def to_s; name end def to_s; name end
private private
def check_integrity def check_integrity
raise "Can't delete status" if Issue.find(:first, :conditions => ["status_id=?", self.id]) raise "Can't delete status" if Issue.find(:first, :conditions => ["status_id=?", self.id])
end end
# Deletes associated workflows # Deletes associated workflows
def delete_workflows def delete_workflows
Workflow.delete_all(["old_status_id = :id OR new_status_id = :id", {:id => id}]) Workflow.delete_all(["old_status_id = :id OR new_status_id = :id", {:id => id}])