jro_apps/redmine Commit - r1161:421d4203113a

Trac importer fix:...

Jean-Philippe Lang -

r1161:421d4203113a

parent child

Context file:

r1161:421d4203113a

Collapse all files

lib/tasks/migrate_from_trac.rake +45 -5

                                         'low' => priorities[0],
                                         'normal' => priorities[1],
                                         'high' => priorities[2],
-                                        'highest' => priorities[3]
+                                        'highest' => priorities[3],
+                                        # ---
+                                        'trivial' => priorities[0],
+                                        'minor' => priorities[1],
+                                        'major' => priorities[2],
+                                        'critical' => priorities[3],
+                                        'blocker' => priorities[4]
                                         }
                     TRACKER_BUG = Tracker.find_by_position(1)
                     def time; Time.at(read_attribute(:time)) end
                   end
+                  TRAC_WIKI_PAGES = %w(TracAccessibility TracAdmin TracBackup TracBrowser TracCgi TracChangeset \
+                                       TracEnvironment TracFastCgi TracGuide TracImport TracIni TracInstall TracInterfaceCustomization \
+                                       TracLinks TracLogging TracModPython TracNotification TracPermissions TracPlugins TracQuery \
+                                       TracReports TracRoadmap TracRss TracSearch TracStandalone TracSupport TracSyntaxColoring TracTickets \
+                                       TracTicketsCustomFields TracTimeline TracUnicode TracUpgrade TracWiki WikiDeletePage WikiFormatting \
+                                       WikiHtml WikiMacros WikiNewPage WikiPageNames WikiProcessors WikiRestructuredText WikiRestructuredTextLinks \
+                                       CamelCase TitleIndex)
                   class TracWikiPage < ActiveRecord::Base
                     set_table_name :wiki
+                    set_primary_key :name
+                    has_many :attachments, :class_name => "TracAttachment", :foreign_key => :id, :conditions => "#{TracMigrate::TracAttachment.table_name}.type = 'wiki'"
                     def self.columns
                       # Hides readonly Trac field to prevent clash with AR readonly? method (Rails 2.0)
                     text = text.gsub(/\[(\d+)\]/, 'r\1')
                     # Ticket number re-writing
                     text = text.gsub(/#(\d+)/) do |s|
+                      if $1.length < 10
                         TICKET_MAP[$1.to_i] ||= $1
                         "\##{TICKET_MAP[$1.to_i] || $1}"
+                      else
+                        s
+                      end
                     end
                     # Preformatted blocks
                     text = text.gsub(/\{\{\{/, '<pre>')
                     migrated_custom_values = 0
                     migrated_ticket_attachments = 0
                     migrated_wiki_edits = 0
+                    migrated_wiki_attachments = 0
                     # Components
                     print "Migrating components"
                     @target_project.wiki.destroy if @target_project.wiki
                     @target_project.reload
                     wiki = Wiki.new(:project => @target_project, :start_page => 'WikiStart')
+                    wiki_edit_count = 0
                     if wiki.save
                       TracWikiPage.find(:all, :order => 'name, version').each do |page|
+                        # Do not migrate Trac manual wiki pages
+                        next if TRAC_WIKI_PAGES.include?(page.name)
+                        wiki_edit_count += 1
                         print '.'
                         STDOUT.flush
                         p = wiki.find_or_new_page(page.name)
                         p.content.author = find_or_create_user(page.author) unless page.author.blank? || page.author == 'trac'
                         p.content.comments = page.comment
                         p.new_record? ? p.save : p.content.save
-                        migrated_wiki_edits += 1 unless p.content.new_record?
+                        next if p.content.new_record?
+                        migrated_wiki_edits += 1
+                        # Attachments
+                        page.attachments.each do |attachment|
+                          next unless attachment.exist?
+                          next if p.attachments.find_by_filename(attachment.filename.gsub(/^.*(\\|\/)/, '').gsub(/[^\w\.\-]/,'_')) #add only once per page
+                          a = Attachment.new :created_on => attachment.time
+                          a.file = attachment
+                          a.author = find_or_create_user(attachment.author)
+                          a.container = p
+                          migrated_wiki_attachments += 1 if a.save
+                        end
                       end
                       wiki.reload
                     puts "Components:      #{migrated_components}/#{TracComponent.count}"
                     puts "Milestones:      #{migrated_milestones}/#{TracMilestone.count}"
                     puts "Tickets:         #{migrated_tickets}/#{TracTicket.count}"
-                    puts "Ticket files:    #{migrated_ticket_attachments}/" + TracAttachment.count("type = 'ticket'").to_s
+                    puts "Ticket files:    #{migrated_ticket_attachments}/" + TracAttachment.count(:conditions => {:type => 'ticket'}).to_s
                     puts "Custom values:   #{migrated_custom_values}/#{TracTicketCustom.count}"
-                    puts "Wiki edits:      #{migrated_wiki_edits}/#{TracWikiPage.count}"
+                    puts "Wiki edits:      #{migrated_wiki_edits}/#{wiki_edit_count}"
+                    puts "Wiki files:      #{migrated_wiki_attachments}/" + TracAttachment.count(:conditions => {:type => 'wiki'}).to_s
                   end
                   def self.limit_for(klass, attribute)
                 DEFAULT_PORTS = {'mysql' => 3306, 'postgresl' => 5432}
-                prompt('Trac directory') {|directory| TracMigrate.set_trac_directory directory}
+                prompt('Trac directory') {|directory| TracMigrate.set_trac_directory directory.strip}
                 prompt('Trac database adapter (sqlite, sqlite3, mysql, postgresql)', :default => 'sqlite') {|adapter| TracMigrate.set_trac_adapter adapter}
                 unless %w(sqlite sqlite3).include?(TracMigrate.trac_adapter)
                   prompt('Trac database host', :default => 'localhost') {|host| TracMigrate.set_trac_db_host host}

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