Added the ability to copy a project in the Project Administration panel.

* Added Copy project button.
* Added Project#copy_from to duplicate a project to be modified and saved by the user
* Added a ProjectsController#copy based off the add method
** Used Project#copy_from to create a duplicate project in memory
* Implemented Project#copy to copy data for a project from another and save it.
** Members
** Project level queries
** Project custom fields
* Added a plugin hook for Project#copy.

  #1125  #1556  #886  #309

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2704 e93f8b46-1217-0410-a6f0-8f06a7374b81

4 files changed, 134 insertions, 6 deletions

diff --git a/test/fixtures/issues.yml b/test/fixtures/issues.yml
index 921ba40c4..856f80289 100644
--- a/test/fixtures/issues.yml
+++ b/test/fixtures/issues.yml
@@ -58,7 +58,7 @@ issues_004:
   category_id: 

   description: Issue on project 2

   tracker_id: 1

-  assigned_to_id: 

+  assigned_to_id: 2

   author_id: 2

   status_id: 1

 issues_005: 

@@ -125,4 +125,4 @@ issues_008:
   start_date: 

   due_date: 

   lock_version: 0

-  
\ No newline at end of file
+
diff --git a/test/fixtures/queries.yml b/test/fixtures/queries.yml
index a1bb08eff..563bf583a 100644
--- a/test/fixtures/queries.yml
+++ b/test/fixtures/queries.yml
@@ -106,4 +106,32 @@ queries_006:
     --- 

     - - priority

       - desc

-  
\ No newline at end of file
+queries_007: 

+  id: 7

+  project_id: 2

+  is_public: true

+  name: Public query for project 2

+  filters: |

+    --- 

+    tracker_id: 

+      :values: 

+      - "3"

+      :operator: "="

+

+  user_id: 2

+  column_names: 

+queries_008: 

+  id: 8

+  project_id: 2

+  is_public: false

+  name: Private query for project 2

+  filters: |

+    --- 

+    tracker_id: 

+      :values: 

+      - "3"

+      :operator: "="

+

+  user_id: 2

+  column_names: 

+
diff --git a/test/functional/projects_controller_test.rb b/test/functional/projects_controller_test.rb
index 1aded6429..4393ac075 100644
--- a/test/functional/projects_controller_test.rb
+++ b/test/functional/projects_controller_test.rb
@@ -453,7 +453,6 @@ class ProjectsControllerTest < Test::Unit::TestCase
     6.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,
@@ -462,7 +461,24 @@ class ProjectsControllerTest < Test::Unit::TestCase
       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'
diff --git a/test/unit/project_test.rb b/test/unit/project_test.rb
index f579e14ff..f9a17e2ec 100644
--- a/test/unit/project_test.rb
+++ b/test/unit/project_test.rb
@@ -20,7 +20,8 @@ require File.dirname(__FILE__) + '/../test_helper'
 class ProjectTest < Test::Unit::TestCase

   fixtures :projects, :enabled_modules, 

            :issues, :issue_statuses, :journals, :journal_details,

-           :users, :members, :roles, :projects_trackers, :trackers, :boards

+           :users, :members, :roles, :projects_trackers, :trackers, :boards,

+           :queries
 

   def setup

     @ecookbook = Project.find(1)

@@ -221,6 +222,7 @@ class ProjectTest < Test::Unit::TestCase
     assert_nil Project.next_identifier

   end

   

+
   def test_enabled_module_names_should_not_recreate_enabled_modules

     project = Project.find(1)

     # Remove one module

@@ -233,4 +235,86 @@ class ProjectTest < Test::Unit::TestCase
     # 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

+  

+  # Context: Project#copy

+  def test_copy_should_copy_issues

+    # Setup

+    ProjectCustomField.destroy_all # Custom values are a mess to isolate in tests

+    source_project = Project.find(2)

+    Project.destroy_all :identifier => "copy-test"

+    project = Project.new(:name => 'Copy Test', :identifier => 'copy-test')

+    project.trackers = source_project.trackers

+    assert project.valid?

+    

+    assert project.issues.empty?

+    assert project.copy(source_project)

+

+    # Tests

+    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

+  end

+  

+  def test_copy_should_copy_members

+    # Setup

+    ProjectCustomField.destroy_all # Custom values are a mess to isolate in tests

+    source_project = Project.find(2)

+    project = Project.new(:name => 'Copy Test', :identifier => 'copy-test')

+    project.trackers = source_project.trackers

+    project.enabled_modules = source_project.enabled_modules

+    assert project.valid?

+

+    assert project.members.empty?

+    assert project.copy(source_project)

+

+    # Tests

+    assert_equal source_project.members.size, project.members.size

+    project.members.each do |member|

+      assert member

+      assert_equal project, member.project

+    end

+  end

+

+  def test_copy_should_copy_project_level_queries

+    # Setup

+    ProjectCustomField.destroy_all # Custom values are a mess to isolate in tests

+    source_project = Project.find(2)

+    project = Project.new(:name => 'Copy Test', :identifier => 'copy-test')

+    project.trackers = source_project.trackers

+    project.enabled_modules = source_project.enabled_modules

+    assert project.valid?

+

+    assert project.queries.empty?

+    assert project.copy(source_project)

+

+    # Tests

+    assert_equal source_project.queries.size, project.queries.size

+    project.queries.each do |query|

+      assert query

+      assert_equal project, query.project

+    end

+  end

+

 end