jro_apps/redmine Commit - r3298:541d830d2a96

Fixed: Project copy loses wiki pages hierarchy (#4797)....

Jean-Philippe Lang -

r3298:541d830d2a96

parent child

Context file:

r3298:541d830d2a96

app/models/project.rb +12 0

                  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

test/fixtures/wiki_contents.yml +24 0

test/fixtures/wiki_pages.yml +21 0

test/unit/project_test.rb +10 -2

                    assert_equal "Start page", @project.wiki.start_page
                  end
-                 should "copy wiki pages and content" do
+                 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 1, @project.wiki.pages.length
+                   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

General Comments 0

You need to be logged in to leave comments. Login now

No TODOs yet

	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