jro_apps/redmine Commit - r2852:5b787785b476

Project copy: let the user choose what to copy from the source project (everything by default)....

Jean-Philippe Lang -

r2852:5b787785b476

parent child

Context file:

r2852:5b787785b476

Collapse all files

app/controllers/projects_controller.rb +1 -1

                  else
                    @project = Project.new(params[:project])
                    @project.enabled_module_names = params[:enabled_modules]
-                   if @project.copy(params[:id])
+                   if @project.copy(params[:id], :only => params[:only])
                      flash[:notice] = l(:notice_successful_create)
                      redirect_to :controller => 'admin', :action => 'projects'
                    end

app/models/project.rb +65 -37

                end
                # Copies and saves the Project instance based on the +project+.
-               # Will duplicate the source project's:
+               # Duplicates the source project's:
+               # * Wiki
+               # * Versions
+               # * Categories
                # * Issues
                # * Members
                # * Queries
-               def copy(project)
+               #
+               # 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)
+                 to_be_copied = to_be_copied & options[:only].to_a unless options[:only].nil?
                  Project.transaction do
-                   # Wikis
+                   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)
+                   self.save
+                 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('name', 'identifier', 'id', 'status')
+                     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
+               # Copies wiki from +project+
+               def copy_wiki(project)
-                   self.wiki = Wiki.new(project.wiki.attributes.dup.except("project_id"))
-                   project.wiki.pages.each do |page|
-                     new_wiki_content = WikiContent.new(page.content.attributes.dup.except("page_id"))
-                     new_wiki_page = WikiPage.new(page.attributes.dup.except("wiki_id"))
-                     new_wiki_page.content = new_wiki_content
-                     self.wiki.pages << new_wiki_page
-                   end
+               end
-                   # Versions
+               # Copies versions from +project+
+               def copy_versions(project)
-                   project.versions.each do |version|
-                     new_version = Version.new
-                     new_version.attributes = version.attributes.dup.except("project_id")
-                     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("project_id")
-                     self.issue_categories << new_issue_category
-                   end
+               end
-                   # Issues
+               # Copies issues from +project+
+               def copy_issues(project)
-                   project.issues.each do |issue|
-                     new_issue = Issue.new
-                     new_issue.copy_from(issue)
-                     if issue.category
-                       new_issue.category = self.issue_categories.select {|c| c.name == issue.category.name}.first
-                     end
-                     self.issues << new_issue
-                   end
+               end
-                   # Members
+               # Copies members from +project+
+               def copy_members(project)
-                   project.members.each do |member|
-                     new_member = Member.new
-                     new_member.attributes = member.attributes.dup.except("project_id")
-                     new_member.project = self
-                     self.members << new_member
-                   end
+               end
-                   # Queries
+               # Copies queries from +project+
+               def copy_queries(project)
-                   project.queries.each do |query|
-                     new_query = Query.new
-                     new_query.attributes = query.attributes.dup.except("project_id", "sort_criteria")
-                     new_query.project = self
-                     self.queries << new_query
-                   end
-                   Redmine::Hook.call_hook(:model_project_copy_before_save, :source_project => project, :destination_project => self)
-                   self.save
-                 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('name', 'identifier', 'id', 'status')
-                     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
                def allowed_permissions
                  @allowed_permissions ||= begin
                    module_names = enabled_modules.collect {|m| m.name}

app/views/projects/copy.rhtml +10 0

              <% end %>
              </fieldset>
+             <fieldset class="box"><legend><%= l(:button_copy) %></legend>
+               <label class="floating"><%= check_box_tag 'only[]', 'members', true %> <%= l(:label_member_plural) %></label>
+               <label class="floating"><%= check_box_tag 'only[]', 'versions', true %> <%= l(:label_version_plural) %></label>
+               <label class="floating"><%= check_box_tag 'only[]', 'issue_categories', true %> <%= l(:label_issue_category_plural) %></label>
+               <label class="floating"><%= check_box_tag 'only[]', 'issues', true %> <%= l(:label_issue_plural) %></label>
+               <label class="floating"><%= check_box_tag 'only[]', 'queries', true %> <%= l(:label_query_plural) %></label>
+               <label class="floating"><%= check_box_tag 'only[]', 'wiki', true %> <%= l(:label_wiki) %></label>
+               <%= hidden_field_tag 'only[]', '' %>
+             </fieldset>
              <%= submit_tag l(:button_copy) %>
              <% end %>

test/unit/project_test.rb +12 0

                    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
                  should "copy issue relations"
                  should "link issue relations if cross project issue relations are valid"

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