jro_apps/redmine Commit - r3646:f3cc84b3437a

Show subproject versions on the Roadmap....

Eric Davis -

r3646:f3cc84b3437a

parent child

Context file:

r3646:f3cc84b3437a

Collapse all files

app/controllers/projects_controller.rb +3 -1

              # Redmine - project management software
              # Copyright (C) 2006-2009  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 ProjectsController < ApplicationController
                menu_item :overview
                menu_item :activity, :only => :activity
                menu_item :roadmap, :only => :roadmap
                menu_item :files, :only => [:list_files, :add_file]
                menu_item :settings, :only => :settings
                before_filter :find_project, :except => [ :index, :list, :add, :copy, :activity ]
                before_filter :find_optional_project, :only => :activity
                before_filter :authorize, :except => [ :index, :list, :add, :copy, :archive, :unarchive, :destroy, :activity ]
                before_filter :authorize_global, :only => :add
                before_filter :require_admin, :only => [ :copy, :archive, :unarchive, :destroy ]
                accept_key_auth :activity
                after_filter :only => [:add, :edit, :archive, :unarchive, :destroy] do |controller|
                  if controller.request.post?
                    controller.send :expire_action, :controller => 'welcome', :action => 'robots.txt'
                  end
                end
                helper :sort
                include SortHelper
                helper :custom_fields
                include CustomFieldsHelper
                helper :issues
                helper :queries
                include QueriesHelper
                helper :repositories
                include RepositoriesHelper
                include ProjectsHelper
                # Lists visible projects
                def index
                  respond_to do |format|
                    format.html {
                      @projects = Project.visible.find(:all, :order => 'lft')
                    }
                    format.xml  {
                      @projects = Project.visible.find(:all, :order => 'lft')
                    }
                    format.atom {
                      projects = Project.visible.find(:all, :order => 'created_on DESC',
                                                            :limit => Setting.feeds_limit.to_i)
                      render_feed(projects, :title => "#{Setting.app_title}: #{l(:label_project_latest)}")
                    }
                  end
                end
                # Add a new project
                def add
                  @issue_custom_fields = IssueCustomField.find(:all, :order => "#{CustomField.table_name}.position")
                  @trackers = Tracker.all
                  @project = Project.new(params[:project])
                  if request.get?
                    @project.identifier = Project.next_identifier if Setting.sequential_project_identifiers?
                    @project.trackers = Tracker.all
                    @project.is_public = Setting.default_projects_public?
                    @project.enabled_module_names = Setting.default_projects_modules
                  else
                    @project.enabled_module_names = params[:enabled_modules]
                    if validate_parent_id && @project.save
                      @project.set_allowed_parent!(params[:project]['parent_id']) if params[:project].has_key?('parent_id')
                      # Add current user as a project member if he is not admin
                      unless User.current.admin?
                        r = Role.givable.find_by_id(Setting.new_project_user_role_id.to_i) || Role.givable.first
                        m = Member.new(:user => User.current, :roles => [r])
                        @project.members << m
                      end
                      respond_to do |format|
                        format.html {
                          flash[:notice] = l(:notice_successful_create)
                          redirect_to :controller => 'projects', :action => 'settings', :id => @project
                        }
                        format.xml  { head :created, :location => url_for(:controller => 'projects', :action => 'show', :id => @project.id) }
                      end
                    else
                      respond_to do |format|
                        format.html
                        format.xml  { render :xml => @project.errors, :status => :unprocessable_entity }
                      end
                    end
                  end
                end
                def copy
                  @issue_custom_fields = IssueCustomField.find(:all, :order => "#{CustomField.table_name}.position")
                  @trackers = Tracker.all
                  @root_projects = Project.find(:all,
                                                :conditions => "parent_id IS NULL AND status = #{Project::STATUS_ACTIVE}",
                                                :order => 'name')
                  @source_project = Project.find(params[:id])
                  if request.get?
                    @project = Project.copy_from(@source_project)
                    if @project
                      @project.identifier = Project.next_identifier if Setting.sequential_project_identifiers?
                    else
                      redirect_to :controller => 'admin', :action => 'projects'
                    end
                  else
                    Mailer.with_deliveries(params[:notifications] == '1') do
                      @project = Project.new(params[:project])
                      @project.enabled_module_names = params[:enabled_modules]
                      if validate_parent_id && @project.copy(@source_project, :only => params[:only])
                        @project.set_allowed_parent!(params[:project]['parent_id']) if params[:project].has_key?('parent_id')
                        flash[:notice] = l(:notice_successful_create)
                        redirect_to :controller => 'admin', :action => 'projects'
                      elsif !@project.new_record?
                        # Project was created
                        # But some objects were not copied due to validation failures
                        # (eg. issues from disabled trackers)
                        # TODO: inform about that
                        redirect_to :controller => 'admin', :action => 'projects'
                      end
                    end
                  end
                rescue ActiveRecord::RecordNotFound
                  redirect_to :controller => 'admin', :action => 'projects'
                end
                # Show @project
                def show
                  if params[:jump]
                    # try to redirect to the requested menu item
                    redirect_to_project_menu_item(@project, params[:jump]) && return
                  end
                  @users_by_role = @project.users_by_role
                  @subprojects = @project.children.visible
                  @news = @project.news.find(:all, :limit => 5, :include => [ :author, :project ], :order => "#{News.table_name}.created_on DESC")
                  @trackers = @project.rolled_up_trackers
                  cond = @project.project_condition(Setting.display_subprojects_issues?)
                  @open_issues_by_tracker = Issue.visible.count(:group => :tracker,
                                                          :include => [:project, :status, :tracker],
                                                          :conditions => ["(#{cond}) AND #{IssueStatus.table_name}.is_closed=?", false])
                  @total_issues_by_tracker = Issue.visible.count(:group => :tracker,
                                                          :include => [:project, :status, :tracker],
                                                          :conditions => cond)
                  TimeEntry.visible_by(User.current) do
                    @total_hours = TimeEntry.sum(:hours,
                                                 :include => :project,
                                                 :conditions => cond).to_f
                  end
                  @key = User.current.rss_key
                  respond_to do |format|
                    format.html
                    format.xml
                  end
                end
                def settings
                  @issue_custom_fields = IssueCustomField.find(:all, :order => "#{CustomField.table_name}.position")
                  @issue_category ||= IssueCategory.new
                  @member ||= @project.members.new
                  @trackers = Tracker.all
                  @repository ||= @project.repository
                  @wiki ||= @project.wiki
                end
                # Edit @project
                def edit
                  if request.get?
                  else
                    @project.attributes = params[:project]
                    if validate_parent_id && @project.save
                      @project.set_allowed_parent!(params[:project]['parent_id']) if params[:project].has_key?('parent_id')
                      respond_to do |format|
                        format.html {
                          flash[:notice] = l(:notice_successful_update)
                          redirect_to :action => 'settings', :id => @project
                        }
                        format.xml  { head :ok }
                      end
                    else
                      respond_to do |format|
                        format.html {
                          settings
                          render :action => 'settings'
                        }
                        format.xml  { render :xml => @project.errors, :status => :unprocessable_entity }
                      end
                    end
                  end
                end
                def modules
                  @project.enabled_module_names = params[:enabled_modules]
                  flash[:notice] = l(:notice_successful_update)
                  redirect_to :action => 'settings', :id => @project, :tab => 'modules'
                end
                def archive
                  if request.post?
                    unless @project.archive
                      flash[:error] = l(:error_can_not_archive_project)
                    end
                  end
                  redirect_to(url_for(:controller => 'admin', :action => 'projects', :status => params[:status]))
                end
                def unarchive
                  @project.unarchive if request.post? && !@project.active?
                  redirect_to(url_for(:controller => 'admin', :action => 'projects', :status => params[:status]))
                end
                # Delete @project
                def destroy
                  @project_to_destroy = @project
                  if request.get?
                    # display confirmation view
                  else
                    if params[:format] == 'xml' || params[:confirm]
                      @project_to_destroy.destroy
                      respond_to do |format|
                        format.html { redirect_to :controller => 'admin', :action => 'projects' }
                        format.xml  { head :ok }
                      end
                    end
                  end
                  # hide project in layout
                  @project = nil
                end
                def add_file
                  if request.post?
                    container = (params[:version_id].blank? ? @project : @project.versions.find_by_id(params[:version_id]))
                    attachments = Attachment.attach_files(container, params[:attachments])
                    render_attachment_warning_if_needed(container)
                    if !attachments.empty? && Setting.notified_events.include?('file_added')
                      Mailer.deliver_attachments_added(attachments[:files])
                    end
                    redirect_to :controller => 'projects', :action => 'list_files', :id => @project
                    return
                  end
                  @versions = @project.versions.sort
                end
                def save_activities
                  if request.post? && params[:enumerations]
                    Project.transaction do
                      params[:enumerations].each do |id, activity|
                        @project.update_or_create_time_entry_activity(id, activity)
                      end
                    end
                    flash[:notice] = l(:notice_successful_update)
                  end
                  redirect_to :controller => 'projects', :action => 'settings', :tab => 'activities', :id => @project
                end
                def reset_activities
                  @project.time_entry_activities.each do |time_entry_activity|
                    time_entry_activity.destroy(time_entry_activity.parent)
                  end
                  flash[:notice] = l(:notice_successful_update)
                  redirect_to :controller => 'projects', :action => 'settings', :tab => 'activities', :id => @project
                end
                def list_files
                  sort_init 'filename', 'asc'
                  sort_update 'filename' => "#{Attachment.table_name}.filename",
                              'created_on' => "#{Attachment.table_name}.created_on",
                              'size' => "#{Attachment.table_name}.filesize",
                              'downloads' => "#{Attachment.table_name}.downloads"
                  @containers = [ Project.find(@project.id, :include => :attachments, :order => sort_clause)]
                  @containers += @project.versions.find(:all, :include => :attachments, :order => sort_clause).sort.reverse
                  render :layout => !request.xhr?
                end
                def roadmap
                  @trackers = @project.trackers.find(:all, :order => 'position')
                  retrieve_selected_tracker_ids(@trackers, @trackers.select {|t| t.is_in_roadmap?})
                  @with_subprojects = params[:with_subprojects].nil? ? Setting.display_subprojects_issues? : (params[:with_subprojects] == '1')
                  project_ids = @with_subprojects ? @project.self_and_descendants.collect(&:id) : [@project.id]
-                 @versions = @project.shared_versions.sort
+                 @versions = @project.shared_versions || []
+                 @versions += @project.rolled_up_versions.visible if @with_subprojects
+                 @versions = @versions.uniq.sort
                  @versions.reject! {|version| version.closed? || version.completed? } unless params[:completed]
                  @issues_by_version = {}
                  unless @selected_tracker_ids.empty?
                    @versions.each do |version|
                      issues = version.fixed_issues.visible.find(:all,
                                                                 :include => [:project, :status, :tracker, :priority],
                                                                 :conditions => {:tracker_id => @selected_tracker_ids, :project_id => project_ids},
                                                                 :order => "#{Project.table_name}.lft, #{Tracker.table_name}.position, #{Issue.table_name}.id")
                      @issues_by_version[version] = issues
                    end
                  end
                  @versions.reject! {|version| !project_ids.include?(version.project_id) && @issues_by_version[version].blank?}
                end
                def activity
                  @days = Setting.activity_days_default.to_i
                  if params[:from]
                    begin; @date_to = params[:from].to_date + 1; rescue; end
                  end
                  @date_to ||= Date.today + 1
                  @date_from = @date_to - @days
                  @with_subprojects = params[:with_subprojects].nil? ? Setting.display_subprojects_issues? : (params[:with_subprojects] == '1')
                  @author = (params[:user_id].blank? ? nil : User.active.find(params[:user_id]))
                  @activity = Redmine::Activity::Fetcher.new(User.current, :project => @project,
                                                                           :with_subprojects => @with_subprojects,
                                                                           :author => @author)
                  @activity.scope_select {|t| !params["show_#{t}"].nil?}
                  @activity.scope = (@author.nil? ? :default : :all) if @activity.scope.empty?
                  events = @activity.events(@date_from, @date_to)
                  if events.empty? || stale?(:etag => [events.first, User.current])
                    respond_to do |format|
                      format.html {
                        @events_by_day = events.group_by(&:event_date)
                        render :layout => false if request.xhr?
                      }
                      format.atom {
                        title = l(:label_activity)
                        if @author
                          title = @author.name
                        elsif @activity.scope.size == 1
                          title = l("label_#{@activity.scope.first.singularize}_plural")
                        end
                        render_feed(events, :title => "#{@project || Setting.app_title}: #{title}")
                      }
                    end
                  end
                rescue ActiveRecord::RecordNotFound
                  render_404
                end
              private
                def find_optional_project
                  return true unless params[:id]
                  @project = Project.find(params[:id])
                  authorize
                rescue ActiveRecord::RecordNotFound
                  render_404
                end
                def retrieve_selected_tracker_ids(selectable_trackers, default_trackers=nil)
                  if ids = params[:tracker_ids]
                    @selected_tracker_ids = (ids.is_a? Array) ? ids.collect { |id| id.to_i.to_s } : ids.split('/').collect { |id| id.to_i.to_s }
                  else
                    @selected_tracker_ids = (default_trackers || selectable_trackers).collect {|t| t.id.to_s }
                  end
                end
                # Validates parent_id param according to user's permissions
                # TODO: move it to Project model in a validation that depends on User.current
                def validate_parent_id
                  return true if User.current.admin?
                  parent_id = params[:project] && params[:project][:parent_id]
                  if parent_id || @project.new_record?
                    parent = parent_id.blank? ? nil : Project.find_by_id(parent_id.to_i)
                    unless @project.allowed_parents.include?(parent)
                      @project.errors.add :parent_id, :invalid
                      return false
                    end
                  end
                  true
                end
              end

app/models/project.rb +7 0

              # redMine - project management software
              # Copyright (C) 2006  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
                # Project statuses
                STATUS_ACTIVE     = 1
                STATUS_ARCHIVED   = 9
                # 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 => :delete_all, :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'
                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, :enabled_module_names
                validates_presence_of :name, :identifier
                validates_uniqueness_of :name, :identifier
                validates_associated :repository, :wiki
                validates_length_of :name, :maximum => 30
                validates_length_of :homepage, :maximum => 255
                validates_length_of :identifier, :in => 1..20
                # 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, :destroy_children
                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 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={})
                  statements = []
                  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?
                    # no restriction
                  else
                    statements << "1=0"
                    if user.logged?
                      if Role.non_member.allowed_to?(permission) && !options[:member]
                        statements << "#{Project.table_name}.is_public = #{connection.quoted_true}"
                      end
                      allowed_project_ids = user.memberships.select {|m| m.roles.detect {|role| role.allowed_to?(permission)}}.collect {|m| m.project_id}
                      statements << "#{Project.table_name}.id IN (#{allowed_project_ids.join(',')})" if allowed_project_ids.any?
                    else
                      if Role.anonymous.allowed_to?(permission) && !options[:member]
                        # anonymous user allowed on public project
                        statements << "#{Project.table_name}.is_public = #{connection.quoted_true}"
                      end
                    end
                  end
                  statements.empty? ? base_statement : "((#{base_statement}) AND (#{statements.join(' OR ')}))"
                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
                # 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 ||=
                    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 >= #{root.lft} AND #{Project.table_name}.rgt <= #{root.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
                # 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
                  members.select {|m| m.mail_notification? || m.user.mail_notification?}.collect {|m| m.user.mail}
                end
                # Returns the users that should be notified on project events
                def notified_users
                  members.select {|m| m.mail_notification? || m.user.mail_notification?}.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
                # 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)
                    # 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)}
                  else
                    enabled_modules.clear
                  end
                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
                private
                # Destroys children before destroying self
                def destroy_children
                  children.each do |child|
                    child.destroy
                  end
                end
                # 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
                    issues_map[issue.id] = new_issue
                  end
                  # Relations after in case issues related each other
                  project.issues.each do |issue|
                    new_issue = issues_map[issue.id]
                    # 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)
                  project.memberships.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 +4 -2

              # 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.dirname(__FILE__) + '/../test_helper'
              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 "#add" do
                  context "by admin user" do
                    setup do
                      @request.session[:user_id] = 1
                    end
                    should "accept get" do
                      get :add
                      assert_response :success
                      assert_template 'add'
                    end
                    should "accept post" do
                      post :add, :project => { :name => "blog",
                                               :description => "weblog",
                                               :identifier => "blog",
                                               :is_public => 1,
                                               :custom_field_values => { '3' => 'Beta' }
                                              }
                      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_nil project.parent
                    end
                    should "accept post with parent" do
                      post :add, :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 get" do
                      get :add
                      assert_response :success
                      assert_template 'add'
                      assert_no_tag :select, :attributes => {:name => 'project[parent_id]'}
                    end
                    should "accept post" do
                      post :add, :project => { :name => "blog",
                                               :description => "weblog",
                                               :identifier => "blog",
                                               :is_public => 1,
                                               :custom_field_values => { '3' => 'Beta' }
                                              }
                      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?
                      # 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 :add, :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 "accept get" do
                      get :add, :parent_id => 'ecookbook'
                      assert_response :success
                      assert_template 'add'
                      # 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
                    should "accept post with parent_id" do
                      post :add, :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 :add, :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 :add, :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_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)
                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 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_edit
                  @request.session[:user_id] = 2 # manager
                  post :edit, :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_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_add_file
                  set_tmp_attachments_directory
                  @request.session[:user_id] = 2
                  Setting.notified_events = ['file_added']
                  ActionMailer::Base.deliveries.clear
                  assert_difference 'Attachment.count' do
                    post :add_file, :id => 1, :version_id => '',
                         :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
                  end
                  assert_redirected_to 'projects/ecookbook/files'
                  a = Attachment.find(:first, :order => 'created_on DESC')
                  assert_equal 'testfile.txt', a.filename
                  assert_equal Project.find(1), a.container
                  mail = ActionMailer::Base.deliveries.last
                  assert_kind_of TMail::Mail, mail
                  assert_equal "[eCookbook] New file", mail.subject
                  assert mail.body.include?('testfile.txt')
                end
                def test_add_version_file
                  set_tmp_attachments_directory
                  @request.session[:user_id] = 2
                  Setting.notified_events = ['file_added']
                  assert_difference 'Attachment.count' do
                    post :add_file, :id => 1, :version_id => '2',
                         :attachments => {'1' => {'file' => uploaded_test_file('testfile.txt', 'text/plain')}}
                  end
                  assert_redirected_to 'projects/ecookbook/files'
                  a = Attachment.find(:first, :order => 'created_on DESC')
                  assert_equal 'testfile.txt', a.filename
                  assert_equal Version.find(2), a.container
                end
                def test_list_files
                  get :list_files, :id => 1
                  assert_response :success
                  assert_template 'list_files'
                  assert_not_nil assigns(:containers)
                  # file attached to the project
                  assert_tag :a, :content => 'project_file.zip',
                                 :attributes => { :href => '/attachments/download/8/project_file.zip' }
                  # file attached to a project's version
                  assert_tag :a, :content => 'version_file.zip',
                                 :attributes => { :href => '/attachments/download/9/version_file.zip' }
                end
                def test_roadmap
                  get :roadmap, :id => 1
                  assert_response :success
                  assert_template 'roadmap'
                  assert_not_nil assigns(:versions)
                  # Version with no date set appears
                  assert assigns(:versions).include?(Version.find(3))
                  # Completed version doesn't appear
                  assert !assigns(:versions).include?(Version.find(1))
                end
                def test_roadmap_with_completed_versions
                  get :roadmap, :id => 1, :completed => 1
                  assert_response :success
                  assert_template 'roadmap'
                  assert_not_nil assigns(:versions)
                  # Version with no date set appears
                  assert assigns(:versions).include?(Version.find(3))
                  # Completed version appears
                  assert assigns(:versions).include?(Version.find(1))
                end
                def test_roadmap_showing_subprojects_versions
+                 @subproject_version = Version.generate!(:project => Project.find(3))
                  get :roadmap, :id => 1, :with_subprojects => 1
                  assert_response :success
                  assert_template 'roadmap'
                  assert_not_nil assigns(:versions)
-                 # Version on subproject appears
-                 assert assigns(:versions).include?(Version.find(4))
+                 assert assigns(:versions).include?(Version.find(4)), "Shared version not found"
+                 assert assigns(:versions).include?(@subproject_version), "Subproject version not found"
                end
                def test_project_activity
                  get :activity, :id => 1, :with_subprojects => 0
                  assert_response :success
                  assert_template 'activity'
                  assert_not_nil assigns(:events_by_day)
                  assert_tag :tag => "h3",
                             :content => /#{2.days.ago.to_date.day}/,
                             :sibling => { :tag => "dl",
                               :child => { :tag => "dt",
                                 :attributes => { :class => /issue-edit/ },
                                 :child => { :tag => "a",
                                   :content => /(#{IssueStatus.find(2).name})/,
                                 }
                               }
                             }
                end
                def test_previous_project_activity
                  get :activity, :id => 1, :from => 3.days.ago.to_date
                  assert_response :success
                  assert_template 'activity'
                  assert_not_nil assigns(:events_by_day)
                  assert_tag :tag => "h3",
                             :content => /#{3.day.ago.to_date.day}/,
                             :sibling => { :tag => "dl",
                               :child => { :tag => "dt",
                                 :attributes => { :class => /issue/ },
                                 :child => { :tag => "a",
                                   :content => /#{Issue.find(1).subject}/,
                                 }
                               }
                             }
                end
                def test_global_activity
                  get :activity
                  assert_response :success
                  assert_template 'activity'
                  assert_not_nil assigns(:events_by_day)
                  assert_tag :tag => "h3",
                             :content => /#{5.day.ago.to_date.day}/,
                             :sibling => { :tag => "dl",
                               :child => { :tag => "dt",
                                 :attributes => { :class => /issue/ },
                                 :child => { :tag => "a",
                                   :content => /#{Issue.find(5).subject}/,
                                 }
                               }
                             }
                end
                def test_user_activity
                  get :activity, :user_id => 2
                  assert_response :success
                  assert_template 'activity'
                  assert_not_nil assigns(:events_by_day)
                  assert_tag :tag => "h3",
                             :content => /#{3.day.ago.to_date.day}/,
                             :sibling => { :tag => "dl",
                               :child => { :tag => "dt",
                                 :attributes => { :class => /issue/ },
                                 :child => { :tag => "a",
                                   :content => /#{Issue.find(1).subject}/,
                                 }
                               }
                             }
                end
                def test_activity_atom_feed
                  get :activity, :format => 'atom'
                  assert_response :success
                  assert_template 'common/feed.atom.rxml'
                  assert_tag :tag => 'entry', :child => {
                    :tag => 'link',
                    :attributes => {:href => 'http://test.host/issues/11'}}
                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
                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
                def test_reset_activities
                  @request.session[:user_id] = 2 # manager
                  project_activity = TimeEntryActivity.new({
                                                             :name => 'Project Specific',
                                                             :parent => TimeEntryActivity.find(:first),
                                                             :project => Project.find(1),
                                                             :active => true
                                                           })
                  assert project_activity.save
                  project_activity_two = TimeEntryActivity.new({
                                                                 :name => 'Project Specific Two',
                                                                 :parent => TimeEntryActivity.find(:last),
                                                                 :project => Project.find(1),
                                                                 :active => true
                                                               })
                  assert project_activity_two.save
                  delete :reset_activities, :id => 1
                  assert_response :redirect
                  assert_redirected_to 'projects/ecookbook/settings/activities'
                  assert_nil TimeEntryActivity.find_by_id(project_activity.id)
                  assert_nil TimeEntryActivity.find_by_id(project_activity_two.id)
                end
                def test_reset_activities_should_reassign_time_entries_back_to_the_system_activity
                  @request.session[:user_id] = 2 # manager
                  project_activity = TimeEntryActivity.new({
                                                             :name => 'Project Specific Design',
                                                             :parent => TimeEntryActivity.find(9),
                                                             :project => Project.find(1),
                                                             :active => true
                                                           })
                  assert project_activity.save
                  assert TimeEntry.update_all("activity_id = '#{project_activity.id}'", ["project_id = ? AND activity_id = ?", 1, 9])
                  assert 3, TimeEntry.find_all_by_activity_id_and_project_id(project_activity.id, 1).size
                  delete :reset_activities, :id => 1
                  assert_response :redirect
                  assert_redirected_to 'projects/ecookbook/settings/activities'
                  assert_nil TimeEntryActivity.find_by_id(project_activity.id)
                  assert_equal 0, TimeEntry.find_all_by_activity_id_and_project_id(project_activity.id, 1).size, "TimeEntries still assigned to project specific activity"
                  assert_equal 3, TimeEntry.find_all_by_activity_id_and_project_id(9, 1).size, "TimeEntries still assigned to project specific activity"
                end
                def test_save_activities_to_override_system_activities
                  @request.session[:user_id] = 2 # manager
                  billable_field = TimeEntryActivityCustomField.find_by_name("Billable")
                  post :save_activities, :id => 1, :enumerations => {
                    "9"=> {"parent_id"=>"9", "custom_field_values"=>{"7" => "1"}, "active"=>"0"}, # Design, De-activate
                    "10"=> {"parent_id"=>"10", "custom_field_values"=>{"7"=>"0"}, "active"=>"1"}, # Development, Change custom value
                    "14"=>{"parent_id"=>"14", "custom_field_values"=>{"7"=>"1"}, "active"=>"1"}, # Inactive Activity, Activate with custom value
                    "11"=>{"parent_id"=>"11", "custom_field_values"=>{"7"=>"1"}, "active"=>"1"} # QA, no changes
                  }
                  assert_response :redirect
                  assert_redirected_to 'projects/ecookbook/settings/activities'
                  # Created project specific activities...
                  project = Project.find('ecookbook')
                  # ... Design
                  design = project.time_entry_activities.find_by_name("Design")
                  assert design, "Project activity not found"
                  assert_equal 9, design.parent_id # Relate to the system activity
                  assert_not_equal design.parent.id, design.id # Different records
                  assert_equal design.parent.name, design.name # Same name
                  assert !design.active?
                  # ... Development
                  development = project.time_entry_activities.find_by_name("Development")
                  assert development, "Project activity not found"
                  assert_equal 10, development.parent_id # Relate to the system activity
                  assert_not_equal development.parent.id, development.id # Different records
                  assert_equal development.parent.name, development.name # Same name
                  assert development.active?
                  assert_equal "0", development.custom_value_for(billable_field).value
                  # ... Inactive Activity
                  previously_inactive = project.time_entry_activities.find_by_name("Inactive Activity")
                  assert previously_inactive, "Project activity not found"
                  assert_equal 14, previously_inactive.parent_id # Relate to the system activity
                  assert_not_equal previously_inactive.parent.id, previously_inactive.id # Different records
                  assert_equal previously_inactive.parent.name, previously_inactive.name # Same name
                  assert previously_inactive.active?
                  assert_equal "1", previously_inactive.custom_value_for(billable_field).value
                  # ... QA
                  assert_equal nil, project.time_entry_activities.find_by_name("QA"), "Custom QA activity created when it wasn't modified"
                end
                def test_save_activities_will_update_project_specific_activities
                  @request.session[:user_id] = 2 # manager
                  project_activity = TimeEntryActivity.new({
                                                             :name => 'Project Specific',
                                                             :parent => TimeEntryActivity.find(:first),
                                                             :project => Project.find(1),
                                                             :active => true
                                                           })
                  assert project_activity.save
                  project_activity_two = TimeEntryActivity.new({
                                                                 :name => 'Project Specific Two',
                                                                 :parent => TimeEntryActivity.find(:last),
                                                                 :project => Project.find(1),
                                                                 :active => true
                                                               })
                  assert project_activity_two.save
                  post :save_activities, :id => 1, :enumerations => {
                    project_activity.id => {"custom_field_values"=>{"7" => "1"}, "active"=>"0"}, # De-activate
                    project_activity_two.id => {"custom_field_values"=>{"7" => "1"}, "active"=>"0"} # De-activate
                  }
                  assert_response :redirect
                  assert_redirected_to 'projects/ecookbook/settings/activities'
                  # Created project specific activities...
                  project = Project.find('ecookbook')
                  assert_equal 2, project.time_entry_activities.count
                  activity_one = project.time_entry_activities.find_by_name(project_activity.name)
                  assert activity_one, "Project activity not found"
                  assert_equal project_activity.id, activity_one.id
                  assert !activity_one.active?
                  activity_two = project.time_entry_activities.find_by_name(project_activity_two.name)
                  assert activity_two, "Project activity not found"
                  assert_equal project_activity_two.id, activity_two.id
                  assert !activity_two.active?
                end
                def test_save_activities_when_creating_new_activities_will_convert_existing_data
                  assert_equal 3, TimeEntry.find_all_by_activity_id_and_project_id(9, 1).size
                  @request.session[:user_id] = 2 # manager
                  post :save_activities, :id => 1, :enumerations => {
                    "9"=> {"parent_id"=>"9", "custom_field_values"=>{"7" => "1"}, "active"=>"0"} # Design, De-activate
                  }
                  assert_response :redirect
                  # No more TimeEntries using the system activity
                  assert_equal 0, TimeEntry.find_all_by_activity_id_and_project_id(9, 1).size, "Time Entries still assigned to system activities"
                  # All TimeEntries using project activity
                  project_specific_activity = TimeEntryActivity.find_by_parent_id_and_project_id(9, 1)
                  assert_equal 3, TimeEntry.find_all_by_activity_id_and_project_id(project_specific_activity.id, 1).size, "No Time Entries assigned to the project activity"
                end
                def test_save_activities_when_creating_new_activities_will_not_convert_existing_data_if_an_exception_is_raised
                  # TODO: Need to cause an exception on create but these tests
                  # aren't setup for mocking.  Just create a record now so the
                  # second one is a dupicate
                  parent = TimeEntryActivity.find(9)
                  TimeEntryActivity.create!({:name => parent.name, :project_id => 1, :position => parent.position, :active => true})
                  TimeEntry.create!({:project_id => 1, :hours => 1.0, :user => User.find(1), :issue_id => 3, :activity_id => 10, :spent_on => '2009-01-01'})
                  assert_equal 3, TimeEntry.find_all_by_activity_id_and_project_id(9, 1).size
                  assert_equal 1, TimeEntry.find_all_by_activity_id_and_project_id(10, 1).size
                  @request.session[:user_id] = 2 # manager
                  post :save_activities, :id => 1, :enumerations => {
                    "9"=> {"parent_id"=>"9", "custom_field_values"=>{"7" => "1"}, "active"=>"0"}, # Design
                    "10"=> {"parent_id"=>"10", "custom_field_values"=>{"7"=>"0"}, "active"=>"1"} # Development, Change custom value
                  }
                  assert_response :redirect
                  # TimeEntries shouldn't have been reassigned on the failed record
                  assert_equal 3, TimeEntry.find_all_by_activity_id_and_project_id(9, 1).size, "Time Entries are not assigned to system activities"
                  # TimeEntries shouldn't have been reassigned on the saved record either
                  assert_equal 1, TimeEntry.find_all_by_activity_id_and_project_id(10, 1).size, "Time Entries are not assigned to system activities"
                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 +53 0

              # redMine - project management software
              # Copyright (C) 2006-2007  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.dirname(__FILE__) + '/../test_helper'
              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 :name
                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_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 !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 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_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_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 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
              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