jro_apps/redmine Commit - r5145:8b39f62a4ce7

Modules selection lost on project form after validation failure (#8012)....

Jean-Philippe Lang -

r5145:8b39f62a4ce7

parent child

Context file:

r5145:8b39f62a4ce7

Collapse all files

app/models/project.rb +1 -4

              # Redmine - project management software
              # Copyright (C) 2006-2011  Jean-Philippe Lang
              #
              # This program is free software; you can redistribute it and/or
              # modify it under the terms of the GNU General Public License
              # as published by the Free Software Foundation; either version 2
              # of the License, or (at your option) any later version.
              #
              # This program is distributed in the hope that it will be useful,
              # but WITHOUT ANY WARRANTY; without even the implied warranty of
              # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
              # GNU General Public License for more details.
              #
              # You should have received a copy of the GNU General Public License
              # along with this program; if not, write to the Free Software
              # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
              class Project < ActiveRecord::Base
                include Redmine::SafeAttributes
                # Project statuses
                STATUS_ACTIVE     = 1
                STATUS_ARCHIVED   = 9
                # Maximum length for project identifiers
                IDENTIFIER_MAX_LENGTH = 100
                # Specific overidden Activities
                has_many :time_entry_activities
                has_many :members, :include => [:user, :roles], :conditions => "#{User.table_name}.type='User' AND #{User.table_name}.status=#{User::STATUS_ACTIVE}"
                has_many :memberships, :class_name => 'Member'
                has_many :member_principals, :class_name => 'Member',
                                             :include => :principal,
                                             :conditions => "#{Principal.table_name}.type='Group' OR (#{Principal.table_name}.type='User' AND #{Principal.table_name}.status=#{User::STATUS_ACTIVE})"
                has_many :users, :through => :members
                has_many :principals, :through => :member_principals, :source => :principal
                has_many :enabled_modules, :dependent => :delete_all
                has_and_belongs_to_many :trackers, :order => "#{Tracker.table_name}.position"
                has_many :issues, :dependent => :destroy, :order => "#{Issue.table_name}.created_on DESC", :include => [:status, :tracker]
                has_many :issue_changes, :through => :issues, :source => :journals
                has_many :versions, :dependent => :destroy, :order => "#{Version.table_name}.effective_date DESC, #{Version.table_name}.name DESC"
                has_many :time_entries, :dependent => :delete_all
                has_many :queries, :dependent => :delete_all
                has_many :documents, :dependent => :destroy
                has_many :news, :dependent => :destroy, :include => :author
                has_many :issue_categories, :dependent => :delete_all, :order => "#{IssueCategory.table_name}.name"
                has_many :boards, :dependent => :destroy, :order => "position ASC"
                has_one :repository, :dependent => :destroy
                has_many :changesets, :through => :repository
                has_one :wiki, :dependent => :destroy
                # Custom field for the project issues
                has_and_belongs_to_many :issue_custom_fields,
                                        :class_name => 'IssueCustomField',
                                        :order => "#{CustomField.table_name}.position",
                                        :join_table => "#{table_name_prefix}custom_fields_projects#{table_name_suffix}",
                                        :association_foreign_key => 'custom_field_id'
                acts_as_nested_set :order => 'name', :dependent => :destroy
                acts_as_attachable :view_permission => :view_files,
                                   :delete_permission => :manage_files
                acts_as_customizable
                acts_as_searchable :columns => ['name', 'identifier', 'description'], :project_key => 'id', :permission => nil
                acts_as_event :title => Proc.new {|o| "#{l(:label_project)}: #{o.name}"},
                              :url => Proc.new {|o| {:controller => 'projects', :action => 'show', :id => o}},
                              :author => nil
                attr_protected :status
                validates_presence_of :name, :identifier
                validates_uniqueness_of :identifier
                validates_associated :repository, :wiki
                validates_length_of :name, :maximum => 255
                validates_length_of :homepage, :maximum => 255
                validates_length_of :identifier, :in => 1..IDENTIFIER_MAX_LENGTH
                # donwcase letters, digits, dashes but not digits only
                validates_format_of :identifier, :with => /^(?!\d+$)[a-z0-9\-]*$/, :if => Proc.new { |p| p.identifier_changed? }
                # reserved words
                validates_exclusion_of :identifier, :in => %w( new )
                before_destroy :delete_all_members
                named_scope :has_module, lambda { |mod| { :conditions => ["#{Project.table_name}.id IN (SELECT em.project_id FROM #{EnabledModule.table_name} em WHERE em.name=?)", mod.to_s] } }
                named_scope :active, { :conditions => "#{Project.table_name}.status = #{STATUS_ACTIVE}"}
                named_scope :all_public, { :conditions => { :is_public => true } }
                named_scope :visible, lambda { { :conditions => Project.visible_by(User.current) } }
                def initialize(attributes = nil)
                  super
                  initialized = (attributes || {}).stringify_keys
                  if !initialized.key?('identifier') && Setting.sequential_project_identifiers?
                    self.identifier = Project.next_identifier
                  end
                  if !initialized.key?('is_public')
                    self.is_public = Setting.default_projects_public?
                  end
                  if !initialized.key?('enabled_module_names')
                    self.enabled_module_names = Setting.default_projects_modules
                  end
                  if !initialized.key?('trackers') && !initialized.key?('tracker_ids')
                    self.trackers = Tracker.all
                  end
                end
                def identifier=(identifier)
                  super unless identifier_frozen?
                end
                def identifier_frozen?
                  errors[:identifier].nil? && !(new_record? || identifier.blank?)
                end
                # returns latest created projects
                # non public projects will be returned only if user is a member of those
                def self.latest(user=nil, count=5)
                  find(:all, :limit => count, :conditions => visible_by(user), :order => "created_on DESC")
                end
                # Returns a SQL :conditions string used to find all active projects for the specified user.
                #
                # Examples:
                #     Projects.visible_by(admin)        => "projects.status = 1"
                #     Projects.visible_by(normal_user)  => "projects.status = 1 AND projects.is_public = 1"
                def self.visible_by(user=nil)
                  user ||= User.current
                  if user && user.admin?
                    return "#{Project.table_name}.status=#{Project::STATUS_ACTIVE}"
                  elsif user && user.memberships.any?
                    return "#{Project.table_name}.status=#{Project::STATUS_ACTIVE} AND (#{Project.table_name}.is_public = #{connection.quoted_true} or #{Project.table_name}.id IN (#{user.memberships.collect{|m| m.project_id}.join(',')}))"
                  else
                    return "#{Project.table_name}.status=#{Project::STATUS_ACTIVE} AND #{Project.table_name}.is_public = #{connection.quoted_true}"
                  end
                end
                def self.allowed_to_condition(user, permission, options={})
                  base_statement = "#{Project.table_name}.status=#{Project::STATUS_ACTIVE}"
                  if perm = Redmine::AccessControl.permission(permission)
                    unless perm.project_module.nil?
                      # If the permission belongs to a project module, make sure the module is enabled
                      base_statement << " AND #{Project.table_name}.id IN (SELECT em.project_id FROM #{EnabledModule.table_name} em WHERE em.name='#{perm.project_module}')"
                    end
                  end
                  if options[:project]
                    project_statement = "#{Project.table_name}.id = #{options[:project].id}"
                    project_statement << " OR (#{Project.table_name}.lft > #{options[:project].lft} AND #{Project.table_name}.rgt < #{options[:project].rgt})" if options[:with_subprojects]
                    base_statement = "(#{project_statement}) AND (#{base_statement})"
                  end
                  if user.admin?
                    base_statement
                  else
                    statement_by_role = {}
                    if user.logged?
                      if Role.non_member.allowed_to?(permission) && !options[:member]
                        statement_by_role[Role.non_member] = "#{Project.table_name}.is_public = #{connection.quoted_true}"
                      end
                      user.projects_by_role.each do |role, projects|
                        if role.allowed_to?(permission)
                          statement_by_role[role] = "#{Project.table_name}.id IN (#{projects.collect(&:id).join(',')})"
                        end
                      end
                    else
                      if Role.anonymous.allowed_to?(permission) && !options[:member]
                        statement_by_role[Role.anonymous] = "#{Project.table_name}.is_public = #{connection.quoted_true}"
                      end
                    end
                    if statement_by_role.empty?
                      "1=0"
                    else
                      "((#{base_statement}) AND (#{statement_by_role.values.join(' OR ')}))"
                    end
                  end
                end
                # Returns the Systemwide and project specific activities
                def activities(include_inactive=false)
                  if include_inactive
                    return all_activities
                  else
                    return active_activities
                  end
                end
                # Will create a new Project specific Activity or update an existing one
                #
                # This will raise a ActiveRecord::Rollback if the TimeEntryActivity
                # does not successfully save.
                def update_or_create_time_entry_activity(id, activity_hash)
                  if activity_hash.respond_to?(:has_key?) && activity_hash.has_key?('parent_id')
                    self.create_time_entry_activity_if_needed(activity_hash)
                  else
                    activity = project.time_entry_activities.find_by_id(id.to_i)
                    activity.update_attributes(activity_hash) if activity
                  end
                end
                # Create a new TimeEntryActivity if it overrides a system TimeEntryActivity
                #
                # This will raise a ActiveRecord::Rollback if the TimeEntryActivity
                # does not successfully save.
                def create_time_entry_activity_if_needed(activity)
                  if activity['parent_id']
                    parent_activity = TimeEntryActivity.find(activity['parent_id'])
                    activity['name'] = parent_activity.name
                    activity['position'] = parent_activity.position
                    if Enumeration.overridding_change?(activity, parent_activity)
                      project_activity = self.time_entry_activities.create(activity)
                      if project_activity.new_record?
                        raise ActiveRecord::Rollback, "Overridding TimeEntryActivity was not successfully saved"
                      else
                        self.time_entries.update_all("activity_id = #{project_activity.id}", ["activity_id = ?", parent_activity.id])
                      end
                    end
                  end
                end
                # Returns a :conditions SQL string that can be used to find the issues associated with this project.
                #
                # Examples:
                #   project.project_condition(true)  => "(projects.id = 1 OR (projects.lft > 1 AND projects.rgt < 10))"
                #   project.project_condition(false) => "projects.id = 1"
                def project_condition(with_subprojects)
                  cond = "#{Project.table_name}.id = #{id}"
                  cond = "(#{cond} OR (#{Project.table_name}.lft > #{lft} AND #{Project.table_name}.rgt < #{rgt}))" if with_subprojects
                  cond
                end
                def self.find(*args)
                  if args.first && args.first.is_a?(String) && !args.first.match(/^\d*$/)
                    project = find_by_identifier(*args)
                    raise ActiveRecord::RecordNotFound, "Couldn't find Project with identifier=#{args.first}" if project.nil?
                    project
                  else
                    super
                  end
                end
                def to_param
                  # id is used for projects with a numeric identifier (compatibility)
                  @to_param ||= (identifier.to_s =~ %r{^\d*$} ? id : identifier)
                end
                def active?
                  self.status == STATUS_ACTIVE
                end
                def archived?
                  self.status == STATUS_ARCHIVED
                end
                # Archives the project and its descendants
                def archive
                  # Check that there is no issue of a non descendant project that is assigned
                  # to one of the project or descendant versions
                  v_ids = self_and_descendants.collect {|p| p.version_ids}.flatten
                  if v_ids.any? && Issue.find(:first, :include => :project,
                                                      :conditions => ["(#{Project.table_name}.lft < ? OR #{Project.table_name}.rgt > ?)" +
                                                                      " AND #{Issue.table_name}.fixed_version_id IN (?)", lft, rgt, v_ids])
                    return false
                  end
                  Project.transaction do
                    archive!
                  end
                  true
                end
                # Unarchives the project
                # All its ancestors must be active
                def unarchive
                  return false if ancestors.detect {|a| !a.active?}
                  update_attribute :status, STATUS_ACTIVE
                end
                # Returns an array of projects the project can be moved to
                # by the current user
                def allowed_parents
                  return @allowed_parents if @allowed_parents
                  @allowed_parents = Project.find(:all, :conditions => Project.allowed_to_condition(User.current, :add_subprojects))
                  @allowed_parents = @allowed_parents - self_and_descendants
                  if User.current.allowed_to?(:add_project, nil, :global => true) || (!new_record? && parent.nil?)
                    @allowed_parents << nil
                  end
                  unless parent.nil? || @allowed_parents.empty? || @allowed_parents.include?(parent)
                    @allowed_parents << parent
                  end
                  @allowed_parents
                end
                # Sets the parent of the project with authorization check
                def set_allowed_parent!(p)
                  unless p.nil? || p.is_a?(Project)
                    if p.to_s.blank?
                      p = nil
                    else
                      p = Project.find_by_id(p)
                      return false unless p
                    end
                  end
                  if p.nil?
                    if !new_record? && allowed_parents.empty?
                      return false
                    end
                  elsif !allowed_parents.include?(p)
                    return false
                  end
                  set_parent!(p)
                end
                # Sets the parent of the project
                # Argument can be either a Project, a String, a Fixnum or nil
                def set_parent!(p)
                  unless p.nil? || p.is_a?(Project)
                    if p.to_s.blank?
                      p = nil
                    else
                      p = Project.find_by_id(p)
                      return false unless p
                    end
                  end
                  if p == parent && !p.nil?
                    # Nothing to do
                    true
                  elsif p.nil? || (p.active? && move_possible?(p))
                    # Insert the project so that target's children or root projects stay alphabetically sorted
                    sibs = (p.nil? ? self.class.roots : p.children)
                    to_be_inserted_before = sibs.detect {|c| c.name.to_s.downcase > name.to_s.downcase }
                    if to_be_inserted_before
                      move_to_left_of(to_be_inserted_before)
                    elsif p.nil?
                      if sibs.empty?
                        # move_to_root adds the project in first (ie. left) position
                        move_to_root
                      else
                        move_to_right_of(sibs.last) unless self == sibs.last
                      end
                    else
                      # move_to_child_of adds the project in last (ie.right) position
                      move_to_child_of(p)
                    end
                    Issue.update_versions_from_hierarchy_change(self)
                    true
                  else
                    # Can not move to the given target
                    false
                  end
                end
                # Returns an array of the trackers used by the project and its active sub projects
                def rolled_up_trackers
                  @rolled_up_trackers ||=
                    Tracker.find(:all, :include => :projects,
                                       :select => "DISTINCT #{Tracker.table_name}.*",
                                       :conditions => ["#{Project.table_name}.lft >= ? AND #{Project.table_name}.rgt <= ? AND #{Project.table_name}.status = #{STATUS_ACTIVE}", lft, rgt],
                                       :order => "#{Tracker.table_name}.position")
                end
                # Closes open and locked project versions that are completed
                def close_completed_versions
                  Version.transaction do
                    versions.find(:all, :conditions => {:status => %w(open locked)}).each do |version|
                      if version.completed?
                        version.update_attribute(:status, 'closed')
                      end
                    end
                  end
                end
                # Returns a scope of the Versions on subprojects
                def rolled_up_versions
                  @rolled_up_versions ||=
                    Version.scoped(:include => :project,
                                   :conditions => ["#{Project.table_name}.lft >= ? AND #{Project.table_name}.rgt <= ? AND #{Project.table_name}.status = #{STATUS_ACTIVE}", lft, rgt])
                end
                # Returns a scope of the Versions used by the project
                def shared_versions
                  @shared_versions ||= begin
                    r = root? ? self : root
                    Version.scoped(:include => :project,
                                   :conditions => "#{Project.table_name}.id = #{id}" +
                                                  " OR (#{Project.table_name}.status = #{Project::STATUS_ACTIVE} AND (" +
                                                        " #{Version.table_name}.sharing = 'system'" +
                                                        " OR (#{Project.table_name}.lft >= #{r.lft} AND #{Project.table_name}.rgt <= #{r.rgt} AND #{Version.table_name}.sharing = 'tree')" +
                                                        " OR (#{Project.table_name}.lft < #{lft} AND #{Project.table_name}.rgt > #{rgt} AND #{Version.table_name}.sharing IN ('hierarchy', 'descendants'))" +
                                                        " OR (#{Project.table_name}.lft > #{lft} AND #{Project.table_name}.rgt < #{rgt} AND #{Version.table_name}.sharing = 'hierarchy')" +
                                                        "))")
                  end
                end
                # Returns a hash of project users grouped by role
                def users_by_role
                  members.find(:all, :include => [:user, :roles]).inject({}) do |h, m|
                    m.roles.each do |r|
                      h[r] ||= []
                      h[r] << m.user
                    end
                    h
                  end
                end
                # Deletes all project's members
                def delete_all_members
                  me, mr = Member.table_name, MemberRole.table_name
                  connection.delete("DELETE FROM #{mr} WHERE #{mr}.member_id IN (SELECT #{me}.id FROM #{me} WHERE #{me}.project_id = #{id})")
                  Member.delete_all(['project_id = ?', id])
                end
                # Users issues can be assigned to
                def assignable_users
                  members.select {|m| m.roles.detect {|role| role.assignable?}}.collect {|m| m.user}.sort
                end
                # Returns the mail adresses of users that should be always notified on project events
                def recipients
                  notified_users.collect {|user| user.mail}
                end
                # Returns the users that should be notified on project events
                def notified_users
                  # TODO: User part should be extracted to User#notify_about?
                  members.select {|m| m.mail_notification? || m.user.mail_notification == 'all'}.collect {|m| m.user}
                end
                # Returns an array of all custom fields enabled for project issues
                # (explictly associated custom fields and custom fields enabled for all projects)
                def all_issue_custom_fields
                  @all_issue_custom_fields ||= (IssueCustomField.for_all + issue_custom_fields).uniq.sort
                end
                def project
                  self
                end
                def <=>(project)
                  name.downcase <=> project.name.downcase
                end
                def to_s
                  name
                end
                # Returns a short description of the projects (first lines)
                def short_description(length = 255)
                  description.gsub(/^(.{#{length}}[^\n\r]*).*$/m, '\1...').strip if description
                end
                def css_classes
                  s = 'project'
                  s << ' root' if root?
                  s << ' child' if child?
                  s << (leaf? ? ' leaf' : ' parent')
                  s
                end
                # The earliest start date of a project, based on it's issues and versions
                def start_date
                  [
                   issues.minimum('start_date'),
                   shared_versions.collect(&:effective_date),
                   shared_versions.collect(&:start_date)
                  ].flatten.compact.min
                end
                # The latest due date of an issue or version
                def due_date
                  [
                   issues.maximum('due_date'),
                   shared_versions.collect(&:effective_date),
                   shared_versions.collect {|v| v.fixed_issues.maximum('due_date')}
                  ].flatten.compact.max
                end
                def overdue?
                  active? && !due_date.nil? && (due_date < Date.today)
                end
                # Returns the percent completed for this project, based on the
                # progress on it's versions.
                def completed_percent(options={:include_subprojects => false})
                  if options.delete(:include_subprojects)
                    total = self_and_descendants.collect(&:completed_percent).sum
                    total / self_and_descendants.count
                  else
                    if versions.count > 0
                      total = versions.collect(&:completed_pourcent).sum
                      total / versions.count
                    else
 
                    end
                  end
                end
                # Return true if this project is allowed to do the specified action.
                # action can be:
                # * a parameter-like Hash (eg. :controller => 'projects', :action => 'edit')
                # * a permission Symbol (eg. :edit_project)
                def allows_to?(action)
                  if action.is_a? Hash
                    allowed_actions.include? "#{action[:controller]}/#{action[:action]}"
                  else
                    allowed_permissions.include? action
                  end
                end
                def module_enabled?(module_name)
                  module_name = module_name.to_s
                  enabled_modules.detect {|m| m.name == module_name}
                end
                def enabled_module_names=(module_names)
                  if module_names && module_names.is_a?(Array)
                    module_names = module_names.collect(&:to_s).reject(&:blank?)
-                   # remove disabled modules
-                   enabled_modules.each {|mod| mod.destroy unless module_names.include?(mod.name)}
-                   # add new modules
-                   module_names.reject {|name| module_enabled?(name)}.each {|name| enabled_modules << EnabledModule.new(:name => name)}
+                   self.enabled_modules = module_names.collect {|name| enabled_modules.detect {|mod| mod.name == name} || EnabledModule.new(:name => name)}
                  else
                    enabled_modules.clear
                  end
                end
                # Returns an array of the enabled modules names
                def enabled_module_names
                  enabled_modules.collect(&:name)
                end
                safe_attributes 'name',
                  'description',
                  'homepage',
                  'is_public',
                  'identifier',
                  'custom_field_values',
                  'custom_fields',
                  'tracker_ids',
                  'issue_custom_field_ids'
                safe_attributes 'enabled_module_names',
                  :if => lambda {|project, user| project.new_record? || user.allowed_to?(:select_project_modules, project) }
                # Returns an array of projects that are in this project's hierarchy
                #
                # Example: parents, children, siblings
                def hierarchy
                  parents = project.self_and_ancestors || []
                  descendants = project.descendants || []
                  project_hierarchy = parents | descendants # Set union
                end
                # Returns an auto-generated project identifier based on the last identifier used
                def self.next_identifier
                  p = Project.find(:first, :order => 'created_on DESC')
                  p.nil? ? nil : p.identifier.to_s.succ
                end
                # Copies and saves the Project instance based on the +project+.
                # Duplicates the source project's:
                # * Wiki
                # * Versions
                # * Categories
                # * Issues
                # * Members
                # * Queries
                #
                # Accepts an +options+ argument to specify what to copy
                #
                # Examples:
                #   project.copy(1)                                    # => copies everything
                #   project.copy(1, :only => 'members')                # => copies members only
                #   project.copy(1, :only => ['members', 'versions'])  # => copies members and versions
                def copy(project, options={})
                  project = project.is_a?(Project) ? project : Project.find(project)
                  to_be_copied = %w(wiki versions issue_categories issues members queries boards)
                  to_be_copied = to_be_copied & options[:only].to_a unless options[:only].nil?
                  Project.transaction do
                    if save
                      reload
                      to_be_copied.each do |name|
                        send "copy_#{name}", project
                      end
                      Redmine::Hook.call_hook(:model_project_copy_before_save, :source_project => project, :destination_project => self)
                      save
                    end
                  end
                end
                # Copies +project+ and returns the new instance.  This will not save
                # the copy
                def self.copy_from(project)
                  begin
                    project = project.is_a?(Project) ? project : Project.find(project)
                    if project
                      # clear unique attributes
                      attributes = project.attributes.dup.except('id', 'name', 'identifier', 'status', 'parent_id', 'lft', 'rgt')
                      copy = Project.new(attributes)
                      copy.enabled_modules = project.enabled_modules
                      copy.trackers = project.trackers
                      copy.custom_values = project.custom_values.collect {|v| v.clone}
                      copy.issue_custom_fields = project.issue_custom_fields
                      return copy
                    else
                      return nil
                    end
                  rescue ActiveRecord::RecordNotFound
                    return nil
                  end
                end
                # Yields the given block for each project with its level in the tree
                def self.project_tree(projects, &block)
                  ancestors = []
                  projects.sort_by(&:lft).each do |project|
                    while (ancestors.any? && !project.is_descendant_of?(ancestors.last))
                      ancestors.pop
                    end
                    yield project, ancestors.size
                    ancestors << project
                  end
                end
                private
                # Copies wiki from +project+
                def copy_wiki(project)
                  # Check that the source project has a wiki first
                  unless project.wiki.nil?
                    self.wiki ||= Wiki.new
                    wiki.attributes = project.wiki.attributes.dup.except("id", "project_id")
                    wiki_pages_map = {}
                    project.wiki.pages.each do |page|
                      # Skip pages without content
                      next if page.content.nil?
                      new_wiki_content = WikiContent.new(page.content.attributes.dup.except("id", "page_id", "updated_on"))
                      new_wiki_page = WikiPage.new(page.attributes.dup.except("id", "wiki_id", "created_on", "parent_id"))
                      new_wiki_page.content = new_wiki_content
                      wiki.pages << new_wiki_page
                      wiki_pages_map[page.id] = new_wiki_page
                    end
                    wiki.save
                    # Reproduce page hierarchy
                    project.wiki.pages.each do |page|
                      if page.parent_id && wiki_pages_map[page.id]
                        wiki_pages_map[page.id].parent = wiki_pages_map[page.parent_id]
                        wiki_pages_map[page.id].save
                      end
                    end
                  end
                end
                # Copies versions from +project+
                def copy_versions(project)
                  project.versions.each do |version|
                    new_version = Version.new
                    new_version.attributes = version.attributes.dup.except("id", "project_id", "created_on", "updated_on")
                    self.versions << new_version
                  end
                end
                # Copies issue categories from +project+
                def copy_issue_categories(project)
                  project.issue_categories.each do |issue_category|
                    new_issue_category = IssueCategory.new
                    new_issue_category.attributes = issue_category.attributes.dup.except("id", "project_id")
                    self.issue_categories << new_issue_category
                  end
                end
                # Copies issues from +project+
                def copy_issues(project)
                  # Stores the source issue id as a key and the copied issues as the
                  # value.  Used to map the two togeather for issue relations.
                  issues_map = {}
                  # Get issues sorted by root_id, lft so that parent issues
                  # get copied before their children
                  project.issues.find(:all, :order => 'root_id, lft').each do |issue|
                    new_issue = Issue.new
                    new_issue.copy_from(issue)
                    new_issue.project = self
                    # Reassign fixed_versions by name, since names are unique per
                    # project and the versions for self are not yet saved
                    if issue.fixed_version
                      new_issue.fixed_version = self.versions.select {|v| v.name == issue.fixed_version.name}.first
                    end
                    # Reassign the category by name, since names are unique per
                    # project and the categories for self are not yet saved
                    if issue.category
                      new_issue.category = self.issue_categories.select {|c| c.name == issue.category.name}.first
                    end
                    # Parent issue
                    if issue.parent_id
                      if copied_parent = issues_map[issue.parent_id]
                        new_issue.parent_issue_id = copied_parent.id
                      end
                    end
                    self.issues << new_issue
                    if new_issue.new_record?
                      logger.info "Project#copy_issues: issue ##{issue.id} could not be copied: #{new_issue.errors.full_messages}" if logger && logger.info
                    else
                      issues_map[issue.id] = new_issue unless new_issue.new_record?
                    end
                  end
                  # Relations after in case issues related each other
                  project.issues.each do |issue|
                    new_issue = issues_map[issue.id]
                    unless new_issue
                      # Issue was not copied
                      next
                    end
                    # Relations
                    issue.relations_from.each do |source_relation|
                      new_issue_relation = IssueRelation.new
                      new_issue_relation.attributes = source_relation.attributes.dup.except("id", "issue_from_id", "issue_to_id")
                      new_issue_relation.issue_to = issues_map[source_relation.issue_to_id]
                      if new_issue_relation.issue_to.nil? && Setting.cross_project_issue_relations?
                        new_issue_relation.issue_to = source_relation.issue_to
                      end
                      new_issue.relations_from << new_issue_relation
                    end
                    issue.relations_to.each do |source_relation|
                      new_issue_relation = IssueRelation.new
                      new_issue_relation.attributes = source_relation.attributes.dup.except("id", "issue_from_id", "issue_to_id")
                      new_issue_relation.issue_from = issues_map[source_relation.issue_from_id]
                      if new_issue_relation.issue_from.nil? && Setting.cross_project_issue_relations?
                        new_issue_relation.issue_from = source_relation.issue_from
                      end
                      new_issue.relations_to << new_issue_relation
                    end
                  end
                end
                # Copies members from +project+
                def copy_members(project)
                  # Copy users first, then groups to handle members with inherited and given roles
                  members_to_copy = []
                  members_to_copy += project.memberships.select {|m| m.principal.is_a?(User)}
                  members_to_copy += project.memberships.select {|m| !m.principal.is_a?(User)}
                  members_to_copy.each do |member|
                    new_member = Member.new
                    new_member.attributes = member.attributes.dup.except("id", "project_id", "created_on")
                    # only copy non inherited roles
                    # inherited roles will be added when copying the group membership
                    role_ids = member.member_roles.reject(&:inherited?).collect(&:role_id)
                    next if role_ids.empty?
                    new_member.role_ids = role_ids
                    new_member.project = self
                    self.members << new_member
                  end
                end
                # Copies queries from +project+
                def copy_queries(project)
                  project.queries.each do |query|
                    new_query = Query.new
                    new_query.attributes = query.attributes.dup.except("id", "project_id", "sort_criteria")
                    new_query.sort_criteria = query.sort_criteria if query.sort_criteria
                    new_query.project = self
                    self.queries << new_query
                  end
                end
                # Copies boards from +project+
                def copy_boards(project)
                  project.boards.each do |board|
                    new_board = Board.new
                    new_board.attributes = board.attributes.dup.except("id", "project_id", "topics_count", "messages_count", "last_message_id")
                    new_board.project = self
                    self.boards << new_board
                  end
                end
                def allowed_permissions
                  @allowed_permissions ||= begin
                    module_names = enabled_modules.all(:select => :name).collect {|m| m.name}
                    Redmine::AccessControl.modules_permissions(module_names).collect {|p| p.name}
                  end
                end
                def allowed_actions
                  @actions_allowed ||= allowed_permissions.inject([]) { |actions, permission| actions += Redmine::AccessControl.allowed_actions(permission) }.flatten
                end
                # Returns all the active Systemwide and project specific activities
                def active_activities
                  overridden_activity_ids = self.time_entry_activities.collect(&:parent_id)
                  if overridden_activity_ids.empty?
                    return TimeEntryActivity.shared.active
                  else
                    return system_activities_and_project_overrides
                  end
                end
                # Returns all the Systemwide and project specific activities
                # (inactive and active)
                def all_activities
                  overridden_activity_ids = self.time_entry_activities.collect(&:parent_id)
                  if overridden_activity_ids.empty?
                    return TimeEntryActivity.shared
                  else
                    return system_activities_and_project_overrides(true)
                  end
                end
                # Returns the systemwide active activities merged with the project specific overrides
                def system_activities_and_project_overrides(include_inactive=false)
                  if include_inactive
                    return TimeEntryActivity.shared.
                      find(:all,
                           :conditions => ["id NOT IN (?)", self.time_entry_activities.collect(&:parent_id)]) +
                      self.time_entry_activities
                  else
                    return TimeEntryActivity.shared.active.
                      find(:all,
                           :conditions => ["id NOT IN (?)", self.time_entry_activities.collect(&:parent_id)]) +
                      self.time_entry_activities.active
                  end
                end
                # Archives subprojects recursively
                def archive!
                  children.each do |subproject|
                    subproject.send :archive!
                  end
                  update_attribute :status, STATUS_ARCHIVED
                end
              end

test/functional/projects_controller_test.rb +16 0

              # Redmine - project management software
              # Copyright (C) 2006-2008  Jean-Philippe Lang
              #
              # This program is free software; you can redistribute it and/or
              # modify it under the terms of the GNU General Public License
              # as published by the Free Software Foundation; either version 2
              # of the License, or (at your option) any later version.
              #
              # This program is distributed in the hope that it will be useful,
              # but WITHOUT ANY WARRANTY; without even the implied warranty of
              # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
              # GNU General Public License for more details.
              #
              # You should have received a copy of the GNU General Public License
              # along with this program; if not, write to the Free Software
              # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
              require File.expand_path('../../test_helper', __FILE__)
              require 'projects_controller'
              # Re-raise errors caught by the controller.
              class ProjectsController; def rescue_action(e) raise e end; end
              class ProjectsControllerTest < ActionController::TestCase
                fixtures :projects, :versions, :users, :roles, :members, :member_roles, :issues, :journals, :journal_details,
                         :trackers, :projects_trackers, :issue_statuses, :enabled_modules, :enumerations, :boards, :messages,
                         :attachments, :custom_fields, :custom_values, :time_entries
                def setup
                  @controller = ProjectsController.new
                  @request    = ActionController::TestRequest.new
                  @response   = ActionController::TestResponse.new
                  @request.session[:user_id] = nil
                  Setting.default_language = 'en'
                end
                def test_index
                  get :index
                  assert_response :success
                  assert_template 'index'
                  assert_not_nil assigns(:projects)
                  assert_tag :ul, :child => {:tag => 'li',
                                             :descendant => {:tag => 'a', :content => 'eCookbook'},
                                             :child => { :tag => 'ul',
                                                         :descendant => { :tag => 'a',
                                                                          :content => 'Child of private child'
                                                                         }
                                                        }
                                             }
                  assert_no_tag :a, :content => /Private child of eCookbook/
                end
                def test_index_atom
                  get :index, :format => 'atom'
                  assert_response :success
                  assert_template 'common/feed.atom.rxml'
                  assert_select 'feed>title', :text => 'Redmine: Latest projects'
                  assert_select 'feed>entry', :count => Project.count(:conditions => Project.visible_by(User.current))
                end
                context "#index" do
                  context "by non-admin user with view_time_entries permission" do
                    setup do
                      @request.session[:user_id] = 3
                    end
                    should "show overall spent time link" do
                      get :index
                      assert_template 'index'
                      assert_tag :a, :attributes => {:href => '/time_entries'}
                    end
                  end
                  context "by non-admin user without view_time_entries permission" do
                    setup do
                      Role.find(2).remove_permission! :view_time_entries
                      Role.non_member.remove_permission! :view_time_entries
                      Role.anonymous.remove_permission! :view_time_entries
                      @request.session[:user_id] = 3
                    end
                    should "not show overall spent time link" do
                      get :index
                      assert_template 'index'
                      assert_no_tag :a, :attributes => {:href => '/time_entries'}
                    end
                  end
                end
                context "#new" do
                  context "by admin user" do
                    setup do
                      @request.session[:user_id] = 1
                    end
                    should "accept get" do
                      get :new
                      assert_response :success
                      assert_template 'new'
                    end
                  end
                  context "by non-admin user with add_project permission" do
                    setup do
                      Role.non_member.add_permission! :add_project
                      @request.session[:user_id] = 9
                    end
                    should "accept get" do
                      get :new
                      assert_response :success
                      assert_template 'new'
                      assert_no_tag :select, :attributes => {:name => 'project[parent_id]'}
                    end
                  end
                  context "by non-admin user with add_subprojects permission" do
                    setup do
                      Role.find(1).remove_permission! :add_project
                      Role.find(1).add_permission! :add_subprojects
                      @request.session[:user_id] = 2
                    end
                    should "accept get" do
                      get :new, :parent_id => 'ecookbook'
                      assert_response :success
                      assert_template 'new'
                      # parent project selected
                      assert_tag :select, :attributes => {:name => 'project[parent_id]'},
                                          :child => {:tag => 'option', :attributes => {:value => '1', :selected => 'selected'}}
                      # no empty value
                      assert_no_tag :select, :attributes => {:name => 'project[parent_id]'},
                                             :child => {:tag => 'option', :attributes => {:value => ''}}
                    end
                  end
                end
                context "POST :create" do
                  context "by admin user" do
                    setup do
                      @request.session[:user_id] = 1
                    end
                    should "create a new project" do
                      post :create,
                        :project => {
                          :name => "blog",
                          :description => "weblog",
                          :homepage => 'http://weblog',
                          :identifier => "blog",
                          :is_public => 1,
                          :custom_field_values => { '3' => 'Beta' },
                          :tracker_ids => ['1', '3'],
                          # an issue custom field that is not for all project
                          :issue_custom_field_ids => ['9'],
                          :enabled_module_names => ['issue_tracking', 'news', 'repository']
                        }
                      assert_redirected_to '/projects/blog/settings'
                      project = Project.find_by_name('blog')
                      assert_kind_of Project, project
                      assert project.active?
                      assert_equal 'weblog', project.description
                      assert_equal 'http://weblog', project.homepage
                      assert_equal true, project.is_public?
                      assert_nil project.parent
                      assert_equal 'Beta', project.custom_value_for(3).value
                      assert_equal [1, 3], project.trackers.map(&:id).sort
                      assert_equal ['issue_tracking', 'news', 'repository'], project.enabled_module_names.sort
                      assert project.issue_custom_fields.include?(IssueCustomField.find(9))
                    end
                    should "create a new subproject" do
                      post :create, :project => { :name => "blog",
                                               :description => "weblog",
                                               :identifier => "blog",
                                               :is_public => 1,
                                               :custom_field_values => { '3' => 'Beta' },
                                               :parent_id => 1
                                              }
                      assert_redirected_to '/projects/blog/settings'
                      project = Project.find_by_name('blog')
                      assert_kind_of Project, project
                      assert_equal Project.find(1), project.parent
                    end
                  end
                  context "by non-admin user with add_project permission" do
                    setup do
                      Role.non_member.add_permission! :add_project
                      @request.session[:user_id] = 9
                    end
                    should "accept create a Project" do
                      post :create, :project => { :name => "blog",
                                               :description => "weblog",
                                               :identifier => "blog",
                                               :is_public => 1,
                                               :custom_field_values => { '3' => 'Beta' },
                                               :tracker_ids => ['1', '3'],
                                               :enabled_module_names => ['issue_tracking', 'news', 'repository']
                                              }
                      assert_redirected_to '/projects/blog/settings'
                      project = Project.find_by_name('blog')
                      assert_kind_of Project, project
                      assert_equal 'weblog', project.description
                      assert_equal true, project.is_public?
                      assert_equal [1, 3], project.trackers.map(&:id).sort
                      assert_equal ['issue_tracking', 'news', 'repository'], project.enabled_module_names.sort
                      # User should be added as a project member
                      assert User.find(9).member_of?(project)
                      assert_equal 1, project.members.size
                    end
                    should "fail with parent_id" do
                      assert_no_difference 'Project.count' do
                        post :create, :project => { :name => "blog",
                                                 :description => "weblog",
                                                 :identifier => "blog",
                                                 :is_public => 1,
                                                 :custom_field_values => { '3' => 'Beta' },
                                                 :parent_id => 1
                                                }
                      end
                      assert_response :success
                      project = assigns(:project)
                      assert_kind_of Project, project
                      assert_not_nil project.errors.on(:parent_id)
                    end
                  end
                  context "by non-admin user with add_subprojects permission" do
                    setup do
                      Role.find(1).remove_permission! :add_project
                      Role.find(1).add_permission! :add_subprojects
                      @request.session[:user_id] = 2
                    end
                    should "create a project with a parent_id" do
                      post :create, :project => { :name => "blog",
                                               :description => "weblog",
                                               :identifier => "blog",
                                               :is_public => 1,
                                               :custom_field_values => { '3' => 'Beta' },
                                               :parent_id => 1
                                              }
                      assert_redirected_to '/projects/blog/settings'
                      project = Project.find_by_name('blog')
                    end
                    should "fail without parent_id" do
                      assert_no_difference 'Project.count' do
                        post :create, :project => { :name => "blog",
                                                 :description => "weblog",
                                                 :identifier => "blog",
                                                 :is_public => 1,
                                                 :custom_field_values => { '3' => 'Beta' }
                                                }
                      end
                      assert_response :success
                      project = assigns(:project)
                      assert_kind_of Project, project
                      assert_not_nil project.errors.on(:parent_id)
                    end
                    should "fail with unauthorized parent_id" do
                      assert !User.find(2).member_of?(Project.find(6))
                      assert_no_difference 'Project.count' do
                        post :create, :project => { :name => "blog",
                                                 :description => "weblog",
                                                 :identifier => "blog",
                                                 :is_public => 1,
                                                 :custom_field_values => { '3' => 'Beta' },
                                                 :parent_id => 6
                                                }
                      end
                      assert_response :success
                      project = assigns(:project)
                      assert_kind_of Project, project
                      assert_not_nil project.errors.on(:parent_id)
                    end
                  end
                end
+               def test_create_should_preserve_modules_on_validation_failure
+                 with_settings :default_projects_modules => ['issue_tracking', 'repository'] do
+                   @request.session[:user_id] = 1
+                   assert_no_difference 'Project.count' do
+                     post :create, :project => {
+                       :name => "blog",
+                       :identifier => "",
+                       :enabled_module_names => %w(issue_tracking news)
+                     }
+                   end
+                   assert_response :success
+                   project = assigns(:project)
+                   assert_equal %w(issue_tracking news), project.enabled_module_names.sort
+                 end
+               end
                def test_create_should_not_accept_get
                  @request.session[:user_id] = 1
                  get :create
                  assert_response :method_not_allowed
                end
                def test_show_by_id
                  get :show, :id => 1
                  assert_response :success
                  assert_template 'show'
                  assert_not_nil assigns(:project)
                end
                def test_show_by_identifier
                  get :show, :id => 'ecookbook'
                  assert_response :success
                  assert_template 'show'
                  assert_not_nil assigns(:project)
                  assert_equal Project.find_by_identifier('ecookbook'), assigns(:project)
                  assert_tag 'li', :content => /Development status/
                end
                def test_show_should_not_display_hidden_custom_fields
                  ProjectCustomField.find_by_name('Development status').update_attribute :visible, false
                  get :show, :id => 'ecookbook'
                  assert_response :success
                  assert_template 'show'
                  assert_not_nil assigns(:project)
                  assert_no_tag 'li', :content => /Development status/
                end
                def test_show_should_not_fail_when_custom_values_are_nil
                  project = Project.find_by_identifier('ecookbook')
                  project.custom_values.first.update_attribute(:value, nil)
                  get :show, :id => 'ecookbook'
                  assert_response :success
                  assert_template 'show'
                  assert_not_nil assigns(:project)
                  assert_equal Project.find_by_identifier('ecookbook'), assigns(:project)
                end
                def show_archived_project_should_be_denied
                  project = Project.find_by_identifier('ecookbook')
                  project.archive!
                  get :show, :id => 'ecookbook'
                  assert_response 403
                  assert_nil assigns(:project)
                  assert_tag :tag => 'p', :content => /archived/
                end
                def test_private_subprojects_hidden
                  get :show, :id => 'ecookbook'
                  assert_response :success
                  assert_template 'show'
                  assert_no_tag :tag => 'a', :content => /Private child/
                end
                def test_private_subprojects_visible
                  @request.session[:user_id] = 2 # manager who is a member of the private subproject
                  get :show, :id => 'ecookbook'
                  assert_response :success
                  assert_template 'show'
                  assert_tag :tag => 'a', :content => /Private child/
                end
                def test_settings
                  @request.session[:user_id] = 2 # manager
                  get :settings, :id => 1
                  assert_response :success
                  assert_template 'settings'
                end
                def test_update
                  @request.session[:user_id] = 2 # manager
                  post :update, :id => 1, :project => {:name => 'Test changed name',
                                                     :issue_custom_field_ids => ['']}
                  assert_redirected_to '/projects/ecookbook/settings'
                  project = Project.find(1)
                  assert_equal 'Test changed name', project.name
                end
                def test_modules
                  @request.session[:user_id] = 2
                  Project.find(1).enabled_module_names = ['issue_tracking', 'news']
                  post :modules, :id => 1, :enabled_module_names => ['issue_tracking', 'repository', 'documents']
                  assert_redirected_to '/projects/ecookbook/settings/modules'
                  assert_equal ['documents', 'issue_tracking', 'repository'], Project.find(1).enabled_module_names.sort
                end
                def test_modules_should_not_allow_get
                  @request.session[:user_id] = 1
                  get :modules, :id => 1
                  assert_response :method_not_allowed
                end
                def test_get_destroy
                  @request.session[:user_id] = 1 # admin
                  get :destroy, :id => 1
                  assert_response :success
                  assert_template 'destroy'
                  assert_not_nil Project.find_by_id(1)
                end
                def test_post_destroy
                  @request.session[:user_id] = 1 # admin
                  post :destroy, :id => 1, :confirm => 1
                  assert_redirected_to '/admin/projects'
                  assert_nil Project.find_by_id(1)
                end
                def test_archive
                  @request.session[:user_id] = 1 # admin
                  post :archive, :id => 1
                  assert_redirected_to '/admin/projects'
                  assert !Project.find(1).active?
                end
                def test_unarchive
                  @request.session[:user_id] = 1 # admin
                  Project.find(1).archive
                  post :unarchive, :id => 1
                  assert_redirected_to '/admin/projects'
                  assert Project.find(1).active?
                end
                def test_project_breadcrumbs_should_be_limited_to_3_ancestors
                  CustomField.delete_all
                  parent = nil
 .times do |i|
                    p = Project.create!(:name => "Breadcrumbs #{i}", :identifier => "breadcrumbs-#{i}")
                    p.set_parent!(parent)
                    get :show, :id => p
                    assert_tag :h1, :parent => { :attributes => {:id => 'header'}},
                                    :children => { :count => [i, 3].min,
                                                   :only => { :tag => 'a' } }
                    parent = p
                  end
                end
                def test_copy_with_project
                  @request.session[:user_id] = 1 # admin
                  get :copy, :id => 1
                  assert_response :success
                  assert_template 'copy'
                  assert assigns(:project)
                  assert_equal Project.find(1).description, assigns(:project).description
                  assert_nil assigns(:project).id
                end
                def test_copy_without_project
                  @request.session[:user_id] = 1 # admin
                  get :copy
                  assert_response :redirect
                  assert_redirected_to :controller => 'admin', :action => 'projects'
                end
                context "POST :copy" do
                  should "TODO: test the rest of the method"
                  should "redirect to the project settings when successful" do
                    @request.session[:user_id] = 1 # admin
                    post :copy, :id => 1, :project => {:name => 'Copy', :identifier => 'unique-copy'}
                    assert_response :redirect
                    assert_redirected_to :controller => 'projects', :action => 'settings', :id => 'unique-copy'
                  end
                end
                def test_jump_should_redirect_to_active_tab
                  get :show, :id => 1, :jump => 'issues'
                  assert_redirected_to '/projects/ecookbook/issues'
                end
                def test_jump_should_not_redirect_to_inactive_tab
                  get :show, :id => 3, :jump => 'documents'
                  assert_response :success
                  assert_template 'show'
                end
                def test_jump_should_not_redirect_to_unknown_tab
                  get :show, :id => 3, :jump => 'foobar'
                  assert_response :success
                  assert_template 'show'
                end
                # A hook that is manually registered later
                class ProjectBasedTemplate < Redmine::Hook::ViewListener
                  def view_layouts_base_html_head(context)
                    # Adds a project stylesheet
                    stylesheet_link_tag(context[:project].identifier) if context[:project]
                  end
                end
                # Don't use this hook now
                Redmine::Hook.clear_listeners
                def test_hook_response
                  Redmine::Hook.add_listener(ProjectBasedTemplate)
                  get :show, :id => 1
                  assert_tag :tag => 'link', :attributes => {:href => '/stylesheets/ecookbook.css'},
                                             :parent => {:tag => 'head'}
                  Redmine::Hook.clear_listeners
                end
              end

test/unit/project_test.rb +8 0

              # Redmine - project management software
              # Copyright (C) 2006-2011  Jean-Philippe Lang
              #
              # This program is free software; you can redistribute it and/or
              # modify it under the terms of the GNU General Public License
              # as published by the Free Software Foundation; either version 2
              # of the License, or (at your option) any later version.
              #
              # This program is distributed in the hope that it will be useful,
              # but WITHOUT ANY WARRANTY; without even the implied warranty of
              # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
              # GNU General Public License for more details.
              #
              # You should have received a copy of the GNU General Public License
              # along with this program; if not, write to the Free Software
              # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
              require File.expand_path('../../test_helper', __FILE__)
              class ProjectTest < ActiveSupport::TestCase
                fixtures :all
                def setup
                  @ecookbook = Project.find(1)
                  @ecookbook_sub1 = Project.find(3)
                  User.current = nil
                end
                should_validate_presence_of :name
                should_validate_presence_of :identifier
                should_validate_uniqueness_of :identifier
                context "associations" do
                  should_have_many :members
                  should_have_many :users, :through => :members
                  should_have_many :member_principals
                  should_have_many :principals, :through => :member_principals
                  should_have_many :enabled_modules
                  should_have_many :issues
                  should_have_many :issue_changes, :through => :issues
                  should_have_many :versions
                  should_have_many :time_entries
                  should_have_many :queries
                  should_have_many :documents
                  should_have_many :news
                  should_have_many :issue_categories
                  should_have_many :boards
                  should_have_many :changesets, :through => :repository
                  should_have_one :repository
                  should_have_one :wiki
                  should_have_and_belong_to_many :trackers
                  should_have_and_belong_to_many :issue_custom_fields
                end
                def test_truth
                  assert_kind_of Project, @ecookbook
                  assert_equal "eCookbook", @ecookbook.name
                end
                def test_default_attributes
                  with_settings :default_projects_public => '1' do
                    assert_equal true, Project.new.is_public
                    assert_equal false, Project.new(:is_public => false).is_public
                  end
                  with_settings :default_projects_public => '0' do
                    assert_equal false, Project.new.is_public
                    assert_equal true, Project.new(:is_public => true).is_public
                  end
                  with_settings :sequential_project_identifiers => '1' do
                    assert !Project.new.identifier.blank?
                    assert Project.new(:identifier => '').identifier.blank?
                  end
                  with_settings :sequential_project_identifiers => '0' do
                    assert Project.new.identifier.blank?
                    assert !Project.new(:identifier => 'test').blank?
                  end
                  with_settings :default_projects_modules => ['issue_tracking', 'repository'] do
                    assert_equal ['issue_tracking', 'repository'], Project.new.enabled_module_names
                  end
                  assert_equal Tracker.all, Project.new.trackers
                  assert_equal Tracker.find(1, 3), Project.new(:tracker_ids => [1, 3]).trackers
                end
                def test_update
                  assert_equal "eCookbook", @ecookbook.name
                  @ecookbook.name = "eCook"
                  assert @ecookbook.save, @ecookbook.errors.full_messages.join("; ")
                  @ecookbook.reload
                  assert_equal "eCook", @ecookbook.name
                end
                def test_validate_identifier
                  to_test = {"abc" => true,
                             "ab12" => true,
                             "ab-12" => true,
                             "12" => false,
                             "new" => false}
                  to_test.each do |identifier, valid|
                    p = Project.new
                    p.identifier = identifier
                    p.valid?
                    assert_equal valid, p.errors.on('identifier').nil?
                  end
                end
                def test_members_should_be_active_users
                  Project.all.each do |project|
                    assert_nil project.members.detect {|m| !(m.user.is_a?(User) && m.user.active?) }
                  end
                end
                def test_users_should_be_active_users
                  Project.all.each do |project|
                    assert_nil project.users.detect {|u| !(u.is_a?(User) && u.active?) }
                  end
                end
                def test_archive
                  user = @ecookbook.members.first.user
                  @ecookbook.archive
                  @ecookbook.reload
                  assert !@ecookbook.active?
                  assert @ecookbook.archived?
                  assert !user.projects.include?(@ecookbook)
                  # Subproject are also archived
                  assert !@ecookbook.children.empty?
                  assert @ecookbook.descendants.active.empty?
                end
                def test_archive_should_fail_if_versions_are_used_by_non_descendant_projects
                  # Assign an issue of a project to a version of a child project
                  Issue.find(4).update_attribute :fixed_version_id, 4
                  assert_no_difference "Project.count(:all, :conditions => 'status = #{Project::STATUS_ARCHIVED}')" do
                    assert_equal false, @ecookbook.archive
                  end
                  @ecookbook.reload
                  assert @ecookbook.active?
                end
                def test_unarchive
                  user = @ecookbook.members.first.user
                  @ecookbook.archive
                  # A subproject of an archived project can not be unarchived
                  assert !@ecookbook_sub1.unarchive
                  # Unarchive project
                  assert @ecookbook.unarchive
                  @ecookbook.reload
                  assert @ecookbook.active?
                  assert !@ecookbook.archived?
                  assert user.projects.include?(@ecookbook)
                  # Subproject can now be unarchived
                  @ecookbook_sub1.reload
                  assert @ecookbook_sub1.unarchive
                end
                def test_destroy
                  # 2 active members
                  assert_equal 2, @ecookbook.members.size
                  # and 1 is locked
                  assert_equal 3, Member.find(:all, :conditions => ['project_id = ?', @ecookbook.id]).size
                  # some boards
                  assert @ecookbook.boards.any?
                  @ecookbook.destroy
                  # make sure that the project non longer exists
                  assert_raise(ActiveRecord::RecordNotFound) { Project.find(@ecookbook.id) }
                  # make sure related data was removed
                  assert_nil Member.first(:conditions => {:project_id => @ecookbook.id})
                  assert_nil Board.first(:conditions => {:project_id => @ecookbook.id})
                  assert_nil Issue.first(:conditions => {:project_id => @ecookbook.id})
                end
                def test_destroying_root_projects_should_clear_data
                  Project.roots.each do |root|
                    root.destroy
                  end
                  assert_equal 0, Project.count, "Projects were not deleted: #{Project.all.inspect}"
                  assert_equal 0, Member.count, "Members were not deleted: #{Member.all.inspect}"
                  assert_equal 0, MemberRole.count
                  assert_equal 0, Issue.count
                  assert_equal 0, Journal.count
                  assert_equal 0, JournalDetail.count
                  assert_equal 0, Attachment.count
                  assert_equal 0, EnabledModule.count
                  assert_equal 0, IssueCategory.count
                  assert_equal 0, IssueRelation.count
                  assert_equal 0, Board.count
                  assert_equal 0, Message.count
                  assert_equal 0, News.count
                  assert_equal 0, Query.count(:conditions => "project_id IS NOT NULL")
                  assert_equal 0, Repository.count
                  assert_equal 0, Changeset.count
                  assert_equal 0, Change.count
                  assert_equal 0, Comment.count
                  assert_equal 0, TimeEntry.count
                  assert_equal 0, Version.count
                  assert_equal 0, Watcher.count
                  assert_equal 0, Wiki.count
                  assert_equal 0, WikiPage.count
                  assert_equal 0, WikiContent.count
                  assert_equal 0, WikiContent::Version.count
                  assert_equal 0, Project.connection.select_all("SELECT * FROM projects_trackers").size
                  assert_equal 0, Project.connection.select_all("SELECT * FROM custom_fields_projects").size
                  assert_equal 0, CustomValue.count(:conditions => {:customized_type => ['Project', 'Issue', 'TimeEntry', 'Version']})
                end
                def test_move_an_orphan_project_to_a_root_project
                  sub = Project.find(2)
                  sub.set_parent! @ecookbook
                  assert_equal @ecookbook.id, sub.parent.id
                  @ecookbook.reload
                  assert_equal 4, @ecookbook.children.size
                end
                def test_move_an_orphan_project_to_a_subproject
                  sub = Project.find(2)
                  assert sub.set_parent!(@ecookbook_sub1)
                end
                def test_move_a_root_project_to_a_project
                  sub = @ecookbook
                  assert sub.set_parent!(Project.find(2))
                end
                def test_should_not_move_a_project_to_its_children
                  sub = @ecookbook
                  assert !(sub.set_parent!(Project.find(3)))
                end
                def test_set_parent_should_add_roots_in_alphabetical_order
                  ProjectCustomField.delete_all
                  Project.delete_all
                  Project.create!(:name => 'Project C', :identifier => 'project-c').set_parent!(nil)
                  Project.create!(:name => 'Project B', :identifier => 'project-b').set_parent!(nil)
                  Project.create!(:name => 'Project D', :identifier => 'project-d').set_parent!(nil)
                  Project.create!(:name => 'Project A', :identifier => 'project-a').set_parent!(nil)
                  assert_equal 4, Project.count
                  assert_equal Project.all.sort_by(&:name), Project.all.sort_by(&:lft)
                end
                def test_set_parent_should_add_children_in_alphabetical_order
                  ProjectCustomField.delete_all
                  parent = Project.create!(:name => 'Parent', :identifier => 'parent')
                  Project.create!(:name => 'Project C', :identifier => 'project-c').set_parent!(parent)
                  Project.create!(:name => 'Project B', :identifier => 'project-b').set_parent!(parent)
                  Project.create!(:name => 'Project D', :identifier => 'project-d').set_parent!(parent)
                  Project.create!(:name => 'Project A', :identifier => 'project-a').set_parent!(parent)
                  parent.reload
                  assert_equal 4, parent.children.size
                  assert_equal parent.children.sort_by(&:name), parent.children
                end
                def test_rebuild_should_sort_children_alphabetically
                  ProjectCustomField.delete_all
                  parent = Project.create!(:name => 'Parent', :identifier => 'parent')
                  Project.create!(:name => 'Project C', :identifier => 'project-c').move_to_child_of(parent)
                  Project.create!(:name => 'Project B', :identifier => 'project-b').move_to_child_of(parent)
                  Project.create!(:name => 'Project D', :identifier => 'project-d').move_to_child_of(parent)
                  Project.create!(:name => 'Project A', :identifier => 'project-a').move_to_child_of(parent)
                  Project.update_all("lft = NULL, rgt = NULL")
                  Project.rebuild!
                  parent.reload
                  assert_equal 4, parent.children.size
                  assert_equal parent.children.sort_by(&:name), parent.children
                end
                def test_set_parent_should_update_issue_fixed_version_associations_when_a_fixed_version_is_moved_out_of_the_hierarchy
                  # Parent issue with a hierarchy project's fixed version
                  parent_issue = Issue.find(1)
                  parent_issue.update_attribute(:fixed_version_id, 4)
                  parent_issue.reload
                  assert_equal 4, parent_issue.fixed_version_id
                  # Should keep fixed versions for the issues
                  issue_with_local_fixed_version = Issue.find(5)
                  issue_with_local_fixed_version.update_attribute(:fixed_version_id, 4)
                  issue_with_local_fixed_version.reload
                  assert_equal 4, issue_with_local_fixed_version.fixed_version_id
                  # Local issue with hierarchy fixed_version
                  issue_with_hierarchy_fixed_version = Issue.find(13)
                  issue_with_hierarchy_fixed_version.update_attribute(:fixed_version_id, 6)
                  issue_with_hierarchy_fixed_version.reload
                  assert_equal 6, issue_with_hierarchy_fixed_version.fixed_version_id
                  # Move project out of the issue's hierarchy
                  moved_project = Project.find(3)
                  moved_project.set_parent!(Project.find(2))
                  parent_issue.reload
                  issue_with_local_fixed_version.reload
                  issue_with_hierarchy_fixed_version.reload
                  assert_equal 4, issue_with_local_fixed_version.fixed_version_id, "Fixed version was not keep on an issue local to the moved project"
                  assert_equal nil, issue_with_hierarchy_fixed_version.fixed_version_id, "Fixed version is still set after moving the Project out of the hierarchy where the version is defined in"
                  assert_equal nil, parent_issue.fixed_version_id, "Fixed version is still set after moving the Version out of the hierarchy for the issue."
                end
                def test_parent
                  p = Project.find(6).parent
                  assert p.is_a?(Project)
                  assert_equal 5, p.id
                end
                def test_ancestors
                  a = Project.find(6).ancestors
                  assert a.first.is_a?(Project)
                  assert_equal [1, 5], a.collect(&:id)
                end
                def test_root
                  r = Project.find(6).root
                  assert r.is_a?(Project)
                  assert_equal 1, r.id
                end
                def test_children
                  c = Project.find(1).children
                  assert c.first.is_a?(Project)
                  assert_equal [5, 3, 4], c.collect(&:id)
                end
                def test_descendants
                  d = Project.find(1).descendants
                  assert d.first.is_a?(Project)
                  assert_equal [5, 6, 3, 4], d.collect(&:id)
                end
                def test_allowed_parents_should_be_empty_for_non_member_user
                  Role.non_member.add_permission!(:add_project)
                  user = User.find(9)
                  assert user.memberships.empty?
                  User.current = user
                  assert Project.new.allowed_parents.compact.empty?
                end
                def test_allowed_parents_with_add_subprojects_permission
                  Role.find(1).remove_permission!(:add_project)
                  Role.find(1).add_permission!(:add_subprojects)
                  User.current = User.find(2)
                  # new project
                  assert !Project.new.allowed_parents.include?(nil)
                  assert Project.new.allowed_parents.include?(Project.find(1))
                  # existing root project
                  assert Project.find(1).allowed_parents.include?(nil)
                  # existing child
                  assert Project.find(3).allowed_parents.include?(Project.find(1))
                  assert !Project.find(3).allowed_parents.include?(nil)
                end
                def test_allowed_parents_with_add_project_permission
                  Role.find(1).add_permission!(:add_project)
                  Role.find(1).remove_permission!(:add_subprojects)
                  User.current = User.find(2)
                  # new project
                  assert Project.new.allowed_parents.include?(nil)
                  assert !Project.new.allowed_parents.include?(Project.find(1))
                  # existing root project
                  assert Project.find(1).allowed_parents.include?(nil)
                  # existing child
                  assert Project.find(3).allowed_parents.include?(Project.find(1))
                  assert Project.find(3).allowed_parents.include?(nil)
                end
                def test_allowed_parents_with_add_project_and_subprojects_permission
                  Role.find(1).add_permission!(:add_project)
                  Role.find(1).add_permission!(:add_subprojects)
                  User.current = User.find(2)
                  # new project
                  assert Project.new.allowed_parents.include?(nil)
                  assert Project.new.allowed_parents.include?(Project.find(1))
                  # existing root project
                  assert Project.find(1).allowed_parents.include?(nil)
                  # existing child
                  assert Project.find(3).allowed_parents.include?(Project.find(1))
                  assert Project.find(3).allowed_parents.include?(nil)
                end
                def test_users_by_role
                  users_by_role = Project.find(1).users_by_role
                  assert_kind_of Hash, users_by_role
                  role = Role.find(1)
                  assert_kind_of Array, users_by_role[role]
                  assert users_by_role[role].include?(User.find(2))
                end
                def test_rolled_up_trackers
                  parent = Project.find(1)
                  parent.trackers = Tracker.find([1,2])
                  child = parent.children.find(3)
                  assert_equal [1, 2], parent.tracker_ids
                  assert_equal [2, 3], child.trackers.collect(&:id)
                  assert_kind_of Tracker, parent.rolled_up_trackers.first
                  assert_equal Tracker.find(1), parent.rolled_up_trackers.first
                  assert_equal [1, 2, 3], parent.rolled_up_trackers.collect(&:id)
                  assert_equal [2, 3], child.rolled_up_trackers.collect(&:id)
                end
                def test_rolled_up_trackers_should_ignore_archived_subprojects
                  parent = Project.find(1)
                  parent.trackers = Tracker.find([1,2])
                  child = parent.children.find(3)
                  child.trackers = Tracker.find([1,3])
                  parent.children.each(&:archive)
                  assert_equal [1,2], parent.rolled_up_trackers.collect(&:id)
                end
                context "#rolled_up_versions" do
                  setup do
                    @project = Project.generate!
                    @parent_version_1 = Version.generate!(:project => @project)
                    @parent_version_2 = Version.generate!(:project => @project)
                  end
                  should "include the versions for the current project" do
                    assert_same_elements [@parent_version_1, @parent_version_2], @project.rolled_up_versions
                  end
                  should "include versions for a subproject" do
                    @subproject = Project.generate!
                    @subproject.set_parent!(@project)
                    @subproject_version = Version.generate!(:project => @subproject)
                    assert_same_elements [
                                          @parent_version_1,
                                          @parent_version_2,
                                          @subproject_version
                                         ], @project.rolled_up_versions
                  end
                  should "include versions for a sub-subproject" do
                    @subproject = Project.generate!
                    @subproject.set_parent!(@project)
                    @sub_subproject = Project.generate!
                    @sub_subproject.set_parent!(@subproject)
                    @sub_subproject_version = Version.generate!(:project => @sub_subproject)
                    @project.reload
                    assert_same_elements [
                                          @parent_version_1,
                                          @parent_version_2,
                                          @sub_subproject_version
                                         ], @project.rolled_up_versions
                  end
                  should "only check active projects" do
                    @subproject = Project.generate!
                    @subproject.set_parent!(@project)
                    @subproject_version = Version.generate!(:project => @subproject)
                    assert @subproject.archive
                    @project.reload
                    assert !@subproject.active?
                    assert_same_elements [@parent_version_1, @parent_version_2], @project.rolled_up_versions
                  end
                end
                def test_shared_versions_none_sharing
                  p = Project.find(5)
                  v = Version.create!(:name => 'none_sharing', :project => p, :sharing => 'none')
                  assert p.shared_versions.include?(v)
                  assert !p.children.first.shared_versions.include?(v)
                  assert !p.root.shared_versions.include?(v)
                  assert !p.siblings.first.shared_versions.include?(v)
                  assert !p.root.siblings.first.shared_versions.include?(v)
                end
                def test_shared_versions_descendants_sharing
                  p = Project.find(5)
                  v = Version.create!(:name => 'descendants_sharing', :project => p, :sharing => 'descendants')
                  assert p.shared_versions.include?(v)
                  assert p.children.first.shared_versions.include?(v)
                  assert !p.root.shared_versions.include?(v)
                  assert !p.siblings.first.shared_versions.include?(v)
                  assert !p.root.siblings.first.shared_versions.include?(v)
                end
                def test_shared_versions_hierarchy_sharing
                  p = Project.find(5)
                  v = Version.create!(:name => 'hierarchy_sharing', :project => p, :sharing => 'hierarchy')
                  assert p.shared_versions.include?(v)
                  assert p.children.first.shared_versions.include?(v)
                  assert p.root.shared_versions.include?(v)
                  assert !p.siblings.first.shared_versions.include?(v)
                  assert !p.root.siblings.first.shared_versions.include?(v)
                end
                def test_shared_versions_tree_sharing
                  p = Project.find(5)
                  v = Version.create!(:name => 'tree_sharing', :project => p, :sharing => 'tree')
                  assert p.shared_versions.include?(v)
                  assert p.children.first.shared_versions.include?(v)
                  assert p.root.shared_versions.include?(v)
                  assert p.siblings.first.shared_versions.include?(v)
                  assert !p.root.siblings.first.shared_versions.include?(v)
                end
                def test_shared_versions_system_sharing
                  p = Project.find(5)
                  v = Version.create!(:name => 'system_sharing', :project => p, :sharing => 'system')
                  assert p.shared_versions.include?(v)
                  assert p.children.first.shared_versions.include?(v)
                  assert p.root.shared_versions.include?(v)
                  assert p.siblings.first.shared_versions.include?(v)
                  assert p.root.siblings.first.shared_versions.include?(v)
                end
                def test_shared_versions
                  parent = Project.find(1)
                  child = parent.children.find(3)
                  private_child = parent.children.find(5)
                  assert_equal [1,2,3], parent.version_ids.sort
                  assert_equal [4], child.version_ids
                  assert_equal [6], private_child.version_ids
                  assert_equal [7], Version.find_all_by_sharing('system').collect(&:id)
                  assert_equal 6, parent.shared_versions.size
                  parent.shared_versions.each do |version|
                    assert_kind_of Version, version
                  end
                  assert_equal [1,2,3,4,6,7], parent.shared_versions.collect(&:id).sort
                end
                def test_shared_versions_should_ignore_archived_subprojects
                  parent = Project.find(1)
                  child = parent.children.find(3)
                  child.archive
                  parent.reload
                  assert_equal [1,2,3], parent.version_ids.sort
                  assert_equal [4], child.version_ids
                  assert !parent.shared_versions.collect(&:id).include?(4)
                end
                def test_shared_versions_visible_to_user
                  user = User.find(3)
                  parent = Project.find(1)
                  child = parent.children.find(5)
                  assert_equal [1,2,3], parent.version_ids.sort
                  assert_equal [6], child.version_ids
                  versions = parent.shared_versions.visible(user)
                  assert_equal 4, versions.size
                  versions.each do |version|
                    assert_kind_of Version, version
                  end
                  assert !versions.collect(&:id).include?(6)
                end
                def test_next_identifier
                  ProjectCustomField.delete_all
                  Project.create!(:name => 'last', :identifier => 'p2008040')
                  assert_equal 'p2008041', Project.next_identifier
                end
                def test_next_identifier_first_project
                  Project.delete_all
                  assert_nil Project.next_identifier
                end
+               def test_enabled_module_names
+                 with_settings :default_projects_modules => ['issue_tracking', 'repository'] do
+                   project = Project.new
+                   project.enabled_module_names = %w(issue_tracking news)
+                   assert_equal %w(issue_tracking news), project.enabled_module_names.sort
+                 end
+               end
                def test_enabled_module_names_should_not_recreate_enabled_modules
                  project = Project.find(1)
                  # Remove one module
                  modules = project.enabled_modules.slice(0..-2)
                  assert modules.any?
                  assert_difference 'EnabledModule.count', -1 do
                    project.enabled_module_names = modules.collect(&:name)
                  end
                  project.reload
                  # Ids should be preserved
                  assert_equal project.enabled_module_ids.sort, modules.collect(&:id).sort
                end
                def test_copy_from_existing_project
                  source_project = Project.find(1)
                  copied_project = Project.copy_from(1)
                  assert copied_project
                  # Cleared attributes
                  assert copied_project.id.blank?
                  assert copied_project.name.blank?
                  assert copied_project.identifier.blank?
                  # Duplicated attributes
                  assert_equal source_project.description, copied_project.description
                  assert_equal source_project.enabled_modules, copied_project.enabled_modules
                  assert_equal source_project.trackers, copied_project.trackers
                  # Default attributes
                  assert_equal 1, copied_project.status
                end
                def test_activities_should_use_the_system_activities
                  project = Project.find(1)
                  assert_equal project.activities, TimeEntryActivity.find(:all, :conditions => {:active => true} )
                end
                def test_activities_should_use_the_project_specific_activities
                  project = Project.find(1)
                  overridden_activity = TimeEntryActivity.new({:name => "Project", :project => project})
                  assert overridden_activity.save!
                  assert project.activities.include?(overridden_activity), "Project specific Activity not found"
                end
                def test_activities_should_not_include_the_inactive_project_specific_activities
                  project = Project.find(1)
                  overridden_activity = TimeEntryActivity.new({:name => "Project", :project => project, :parent => TimeEntryActivity.find(:first), :active => false})
                  assert overridden_activity.save!
                  assert !project.activities.include?(overridden_activity), "Inactive Project specific Activity found"
                end
                def test_activities_should_not_include_project_specific_activities_from_other_projects
                  project = Project.find(1)
                  overridden_activity = TimeEntryActivity.new({:name => "Project", :project => Project.find(2)})
                  assert overridden_activity.save!
                  assert !project.activities.include?(overridden_activity), "Project specific Activity found on a different project"
                end
                def test_activities_should_handle_nils
                  overridden_activity = TimeEntryActivity.new({:name => "Project", :project => Project.find(1), :parent => TimeEntryActivity.find(:first)})
                  TimeEntryActivity.delete_all
                  # No activities
                  project = Project.find(1)
                  assert project.activities.empty?
                  # No system, one overridden
                  assert overridden_activity.save!
                  project.reload
                  assert_equal [overridden_activity], project.activities
                end
                def test_activities_should_override_system_activities_with_project_activities
                  project = Project.find(1)
                  parent_activity = TimeEntryActivity.find(:first)
                  overridden_activity = TimeEntryActivity.new({:name => "Project", :project => project, :parent => parent_activity})
                  assert overridden_activity.save!
                  assert project.activities.include?(overridden_activity), "Project specific Activity not found"
                  assert !project.activities.include?(parent_activity), "System Activity found when it should have been overridden"
                end
                def test_activities_should_include_inactive_activities_if_specified
                  project = Project.find(1)
                  overridden_activity = TimeEntryActivity.new({:name => "Project", :project => project, :parent => TimeEntryActivity.find(:first), :active => false})
                  assert overridden_activity.save!
                  assert project.activities(true).include?(overridden_activity), "Inactive Project specific Activity not found"
                end
                test 'activities should not include active System activities if the project has an override that is inactive' do
                  project = Project.find(1)
                  system_activity = TimeEntryActivity.find_by_name('Design')
                  assert system_activity.active?
                  overridden_activity = TimeEntryActivity.generate!(:project => project, :parent => system_activity, :active => false)
                  assert overridden_activity.save!
                  assert !project.activities.include?(overridden_activity), "Inactive Project specific Activity not found"
                  assert !project.activities.include?(system_activity), "System activity found when the project has an inactive override"
                end
                def test_close_completed_versions
                  Version.update_all("status = 'open'")
                  project = Project.find(1)
                  assert_not_nil project.versions.detect {|v| v.completed? && v.status == 'open'}
                  assert_not_nil project.versions.detect {|v| !v.completed? && v.status == 'open'}
                  project.close_completed_versions
                  project.reload
                  assert_nil project.versions.detect {|v| v.completed? && v.status != 'closed'}
                  assert_not_nil project.versions.detect {|v| !v.completed? && v.status == 'open'}
                end
                context "Project#copy" do
                  setup do
                    ProjectCustomField.destroy_all # Custom values are a mess to isolate in tests
                    Project.destroy_all :identifier => "copy-test"
                    @source_project = Project.find(2)
                    @project = Project.new(:name => 'Copy Test', :identifier => 'copy-test')
                    @project.trackers = @source_project.trackers
                    @project.enabled_module_names = @source_project.enabled_modules.collect(&:name)
                  end
                  should "copy issues" do
                    @source_project.issues << Issue.generate!(:status => IssueStatus.find_by_name('Closed'),
                                                              :subject => "copy issue status",
                                                              :tracker_id => 1,
                                                              :assigned_to_id => 2,
                                                              :project_id => @source_project.id)
                    assert @project.valid?
                    assert @project.issues.empty?
                    assert @project.copy(@source_project)
                    assert_equal @source_project.issues.size, @project.issues.size
                    @project.issues.each do |issue|
                      assert issue.valid?
                      assert ! issue.assigned_to.blank?
                      assert_equal @project, issue.project
                    end
                    copied_issue = @project.issues.first(:conditions => {:subject => "copy issue status"})
                    assert copied_issue
                    assert copied_issue.status
                    assert_equal "Closed", copied_issue.status.name
                  end
                  should "change the new issues to use the copied version" do
                    User.current = User.find(1)
                    assigned_version = Version.generate!(:name => "Assigned Issues", :status => 'open')
                    @source_project.versions << assigned_version
                    assert_equal 3, @source_project.versions.size
                    Issue.generate_for_project!(@source_project,
                                                :fixed_version_id => assigned_version.id,
                                                :subject => "change the new issues to use the copied version",
                                                :tracker_id => 1,
                                                :project_id => @source_project.id)
                    assert @project.copy(@source_project)
                    @project.reload
                    copied_issue = @project.issues.first(:conditions => {:subject => "change the new issues to use the copied version"})
                    assert copied_issue
                    assert copied_issue.fixed_version
                    assert_equal "Assigned Issues", copied_issue.fixed_version.name # Same name
                    assert_not_equal assigned_version.id, copied_issue.fixed_version.id # Different record
                  end
                  should "copy issue relations" do
                    Setting.cross_project_issue_relations = '1'
                    second_issue = Issue.generate!(:status_id => 5,
                                                   :subject => "copy issue relation",
                                                   :tracker_id => 1,
                                                   :assigned_to_id => 2,
                                                   :project_id => @source_project.id)
                    source_relation = IssueRelation.generate!(:issue_from => Issue.find(4),
                                                              :issue_to => second_issue,
                                                              :relation_type => "relates")
                    source_relation_cross_project = IssueRelation.generate!(:issue_from => Issue.find(1),
                                                                            :issue_to => second_issue,
                                                                            :relation_type => "duplicates")
                    assert @project.copy(@source_project)
                    assert_equal @source_project.issues.count, @project.issues.count
                    copied_issue = @project.issues.find_by_subject("Issue on project 2") # Was #4
                    copied_second_issue = @project.issues.find_by_subject("copy issue relation")
                    # First issue with a relation on project
                    assert_equal 1, copied_issue.relations.size, "Relation not copied"
                    copied_relation = copied_issue.relations.first
                    assert_equal "relates", copied_relation.relation_type
                    assert_equal copied_second_issue.id, copied_relation.issue_to_id
                    assert_not_equal source_relation.id, copied_relation.id
                    # Second issue with a cross project relation
                    assert_equal 2, copied_second_issue.relations.size, "Relation not copied"
                    copied_relation = copied_second_issue.relations.select {|r| r.relation_type == 'duplicates'}.first
                    assert_equal "duplicates", copied_relation.relation_type
                    assert_equal 1, copied_relation.issue_from_id, "Cross project relation not kept"
                    assert_not_equal source_relation_cross_project.id, copied_relation.id
                  end
                  should "copy memberships" do
                    assert @project.valid?
                    assert @project.members.empty?
                    assert @project.copy(@source_project)
                    assert_equal @source_project.memberships.size, @project.memberships.size
                    @project.memberships.each do |membership|
                      assert membership
                      assert_equal @project, membership.project
                    end
                  end
                  should "copy memberships with groups and additional roles" do
                    group = Group.create!(:lastname => "Copy group")
                    user = User.find(7)
                    group.users << user
                    # group role
                    Member.create!(:project_id => @source_project.id, :principal => group, :role_ids => [2])
                    member = Member.find_by_user_id_and_project_id(user.id, @source_project.id)
                    # additional role
                    member.role_ids = [1]
                    assert @project.copy(@source_project)
                    member = Member.find_by_user_id_and_project_id(user.id, @project.id)
                    assert_not_nil member
                    assert_equal [1, 2], member.role_ids.sort
                  end
                  should "copy project specific queries" do
                    assert @project.valid?
                    assert @project.queries.empty?
                    assert @project.copy(@source_project)
                    assert_equal @source_project.queries.size, @project.queries.size
                    @project.queries.each do |query|
                      assert query
                      assert_equal @project, query.project
                    end
                  end
                  should "copy versions" do
                    @source_project.versions << Version.generate!
                    @source_project.versions << Version.generate!
                    assert @project.versions.empty?
                    assert @project.copy(@source_project)
                    assert_equal @source_project.versions.size, @project.versions.size
                    @project.versions.each do |version|
                      assert version
                      assert_equal @project, version.project
                    end
                  end
                  should "copy wiki" do
                    assert_difference 'Wiki.count' do
                      assert @project.copy(@source_project)
                    end
                    assert @project.wiki
                    assert_not_equal @source_project.wiki, @project.wiki
                    assert_equal "Start page", @project.wiki.start_page
                  end
                  should "copy wiki pages and content with hierarchy" do
                    assert_difference 'WikiPage.count', @source_project.wiki.pages.size do
                      assert @project.copy(@source_project)
                    end
                    assert @project.wiki
                    assert_equal @source_project.wiki.pages.size, @project.wiki.pages.size
                    @project.wiki.pages.each do |wiki_page|
                      assert wiki_page.content
                      assert !@source_project.wiki.pages.include?(wiki_page)
                    end
                    parent = @project.wiki.find_page('Parent_page')
                    child1 = @project.wiki.find_page('Child_page_1')
                    child2 = @project.wiki.find_page('Child_page_2')
                    assert_equal parent, child1.parent
                    assert_equal parent, child2.parent
                  end
                  should "copy issue categories" do
                    assert @project.copy(@source_project)
                    assert_equal 2, @project.issue_categories.size
                    @project.issue_categories.each do |issue_category|
                      assert !@source_project.issue_categories.include?(issue_category)
                    end
                  end
                  should "copy boards" do
                    assert @project.copy(@source_project)
                    assert_equal 1, @project.boards.size
                    @project.boards.each do |board|
                      assert !@source_project.boards.include?(board)
                    end
                  end
                  should "change the new issues to use the copied issue categories" do
                    issue = Issue.find(4)
                    issue.update_attribute(:category_id, 3)
                    assert @project.copy(@source_project)
                    @project.issues.each do |issue|
                      assert issue.category
                      assert_equal "Stock management", issue.category.name # Same name
                      assert_not_equal IssueCategory.find(3), issue.category # Different record
                    end
                  end
                  should "limit copy with :only option" do
                    assert @project.members.empty?
                    assert @project.issue_categories.empty?
                    assert @source_project.issues.any?
                    assert @project.copy(@source_project, :only => ['members', 'issue_categories'])
                    assert @project.members.any?
                    assert @project.issue_categories.any?
                    assert @project.issues.empty?
                  end
                end
                context "#start_date" do
                  setup do
                    ProjectCustomField.destroy_all # Custom values are a mess to isolate in tests
                    @project = Project.generate!(:identifier => 'test0')
                    @project.trackers << Tracker.generate!
                  end
                  should "be nil if there are no issues on the project" do
                    assert_nil @project.start_date
                  end
                  should "be tested when issues have no start date"
                  should "be the earliest start date of it's issues" do
                    early = 7.days.ago.to_date
                    Issue.generate_for_project!(@project, :start_date => Date.today)
                    Issue.generate_for_project!(@project, :start_date => early)
                    assert_equal early, @project.start_date
                  end
                end
                context "#due_date" do
                  setup do
                    ProjectCustomField.destroy_all # Custom values are a mess to isolate in tests
                    @project = Project.generate!(:identifier => 'test0')
                    @project.trackers << Tracker.generate!
                  end
                  should "be nil if there are no issues on the project" do
                    assert_nil @project.due_date
                  end
                  should "be tested when issues have no due date"
                  should "be the latest due date of it's issues" do
                    future = 7.days.from_now.to_date
                    Issue.generate_for_project!(@project, :due_date => future)
                    Issue.generate_for_project!(@project, :due_date => Date.today)
                    assert_equal future, @project.due_date
                  end
                  should "be the latest due date of it's versions" do
                    future = 7.days.from_now.to_date
                    @project.versions << Version.generate!(:effective_date => future)
                    @project.versions << Version.generate!(:effective_date => Date.today)
                    assert_equal future, @project.due_date
                  end
                  should "pick the latest date from it's issues and versions" do
                    future = 7.days.from_now.to_date
                    far_future = 14.days.from_now.to_date
                    Issue.generate_for_project!(@project, :due_date => far_future)
                    @project.versions << Version.generate!(:effective_date => future)
                    assert_equal far_future, @project.due_date
                  end
                end
                context "Project#completed_percent" do
                  setup do
                    ProjectCustomField.destroy_all # Custom values are a mess to isolate in tests
                    @project = Project.generate!(:identifier => 'test0')
                    @project.trackers << Tracker.generate!
                  end
                  context "no versions" do
                    should "be 100" do
                      assert_equal 100, @project.completed_percent
                    end
                  end
                  context "with versions" do
                    should "return 0 if the versions have no issues" do
                      Version.generate!(:project => @project)
                      Version.generate!(:project => @project)
                      assert_equal 0, @project.completed_percent
                    end
                    should "return 100 if the version has only closed issues" do
                      v1 = Version.generate!(:project => @project)
                      Issue.generate_for_project!(@project, :status => IssueStatus.find_by_name('Closed'), :fixed_version => v1)
                      v2 = Version.generate!(:project => @project)
                      Issue.generate_for_project!(@project, :status => IssueStatus.find_by_name('Closed'), :fixed_version => v2)
                      assert_equal 100, @project.completed_percent
                    end
                    should "return the averaged completed percent of the versions (not weighted)" do
                      v1 = Version.generate!(:project => @project)
                      Issue.generate_for_project!(@project, :status => IssueStatus.find_by_name('New'), :estimated_hours => 10, :done_ratio => 50, :fixed_version => v1)
                      v2 = Version.generate!(:project => @project)
                      Issue.generate_for_project!(@project, :status => IssueStatus.find_by_name('New'), :estimated_hours => 10, :done_ratio => 50, :fixed_version => v2)
                      assert_equal 50, @project.completed_percent
                    end
                  end
                end
                context "#notified_users" do
                  setup do
                    @project = Project.generate!
                    @role = Role.generate!
                    @user_with_membership_notification = User.generate!(:mail_notification => 'selected')
                    Member.generate!(:project => @project, :roles => [@role], :principal => @user_with_membership_notification, :mail_notification => true)
                    @all_events_user = User.generate!(:mail_notification => 'all')
                    Member.generate!(:project => @project, :roles => [@role], :principal => @all_events_user)
                    @no_events_user = User.generate!(:mail_notification => 'none')
                    Member.generate!(:project => @project, :roles => [@role], :principal => @no_events_user)
                    @only_my_events_user = User.generate!(:mail_notification => 'only_my_events')
                    Member.generate!(:project => @project, :roles => [@role], :principal => @only_my_events_user)
                    @only_assigned_user = User.generate!(:mail_notification => 'only_assigned')
                    Member.generate!(:project => @project, :roles => [@role], :principal => @only_assigned_user)
                    @only_owned_user = User.generate!(:mail_notification => 'only_owner')
                    Member.generate!(:project => @project, :roles => [@role], :principal => @only_owned_user)
                  end
                  should "include members with a mail notification" do
                    assert @project.notified_users.include?(@user_with_membership_notification)
                  end
                  should "include users with the 'all' notification option" do
                    assert @project.notified_users.include?(@all_events_user)
                  end
                  should "not include users with the 'none' notification option" do
                    assert !@project.notified_users.include?(@no_events_user)
                  end
                  should "not include users with the 'only_my_events' notification option" do
                    assert !@project.notified_users.include?(@only_my_events_user)
                  end
                  should "not include users with the 'only_assigned' notification option" do
                    assert !@project.notified_users.include?(@only_assigned_user)
                  end
                  should "not include users with the 'only_owner' notification option" do
                    assert !@project.notified_users.include?(@only_owned_user)
                  end
                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