jro_apps/redmine Commit - r5461:fab9aca747d8

Fixed: selected modules are not activated when copying a project (#8244)....

Jean-Philippe Lang -

r5461:fab9aca747d8

parent child

Context file:

r5461:fab9aca747d8

Collapse all files

app/controllers/projects_controller.rb 0 -1

                   Mailer.with_deliveries(params[:notifications] == '1') do
                     @project = Project.new
                     @project.safe_attributes = 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)

test/functional/projects_controller_test.rb +8 -3

                 end
               end
-              def test_copy_with_project
+              def test_get_copy
                 @request.session[:user_id] = 1 # admin
                 get :copy, :id => 1
                 assert_response :success
                 assert assigns(:project)
                 assert_equal Project.find(1).description, assigns(:project).description
                 assert_nil assigns(:project).id
+                assert_tag :tag => 'input',
+                  :attributes => {:name => 'project[enabled_module_names][]', :value => 'issue_tracking'}
               end
-              def test_copy_without_project
+              def test_get_copy_without_project
                 @request.session[:user_id] = 1 # admin
                 get :copy
                 assert_response :redirect
                       :name => 'Copy',
                       :identifier => 'unique-copy',
                       :tracker_ids => ['1', '2', '3', ''],
-                      :enabled_modules => %w(issue_tracking time_tracking)
+                      :enabled_module_names => %w(issue_tracking time_tracking)
                     },
                     :only => %w(issues versions)
                 end
                 project = Project.find('unique-copy')
                 source = Project.find(1)
+                assert_equal %w(issue_tracking time_tracking), project.enabled_module_names.sort
                 assert_equal source.versions.count, project.versions.count, "All versions were not copied"
                 # issues assigned to a closed version won't be copied
                 assert_equal source.issues.select {|i| i.fixed_version.nil? || i.fixed_version.open?}.size,

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