jro_apps/redmine Commit - r682:72b5cd388977

Trac importer now migrates status changes....

Jean-Philippe Lang -

r682:72b5cd388977

parent child

Context file:

r682:72b5cd388977

Collapse all files

lib/tasks/migrate_from_trac.rake +24 -17

                     set_table_name :ticket
                     set_inheritance_column :none
-                    has_many :comments, :class_name => "TracTicketChange", :foreign_key => :ticket, :conditions => "field = 'comment'"
+                    # ticket changes: only migrate status changes and comments
+                    has_many :changes, :class_name => "TracTicketChange", :foreign_key => :ticket, :conditions => "field = 'comment' OR field='status'"
                     has_many :attachments, :class_name => "TracAttachment", :foreign_key => :id, :conditions => "type = 'ticket'"
                     has_many :customs, :class_name => "TracTicketCustom", :foreign_key => :ticket
                     migrated_components = 0
                     migrated_milestones = 0
                     migrated_tickets = 0
-                    migrated_ticket_comments = 0
                     migrated_custom_values = 0
-                    trac_ticket_comments = 0
                     migrated_ticket_attachments = 0
                     migrated_wiki_edits = 0
                     # Custom fields
                     # TODO: read trac.ini instead
-                    print "Custom fields"
+                    print "Migrating custom fields"
                     custom_field_map = {}
                     TracTicketCustom.find_by_sql("SELECT DISTINCT name FROM #{TracTicketCustom.table_name}").each do |field|
                       print '.'
-                      f = IssueCustomField.new :name => encode(field.name[0, limit_for(IssueCustomField, 'name')]),
+                      f = IssueCustomField.new :name => encode(field.name[0, limit_for(IssueCustomField, 'name')]).humanize,
-                                               :field_format => 'string',
+                                               :field_format => 'string'
-                                               :is_for_all => true
                       next unless f.save
                       f.trackers = Tracker.find(:all)
+                      f.projects << @target_project
                       custom_field_map[field.name] = f
                     end
                     puts
                           i.save
                         end
-                    	# Comments
+                    	# Comments and status changes
-                    	# TODO: migrate status changes history
+                    	ticket.changes.group_by(&:time).each do |time, changeset|
-                    	ticket.comments.each do |comment|
+                          status_change = changeset.select {|change| change.field == 'status'}.first
-                    	  next if comment.newvalue.blank?
+                          comment_change = changeset.select {|change| change.field == 'comment'}.first
-                    	  trac_ticket_comments += 1
-                          n = Journal.new :notes => convert_wiki_text(encode(comment.newvalue)),
+                          n = Journal.new :notes => (comment_change ? convert_wiki_text(encode(comment_change.newvalue)) : ''),
-                                          :created_on => comment.time
+                                          :created_on => time
-                          n.user = find_or_create_user(comment.author)
+                          n.user = find_or_create_user(changeset.first.author)
                           n.journalized = i
-                          migrated_ticket_comments += 1 if n.save
+                          if status_change &&
+                               STATUS_MAPPING[status_change.oldvalue] &&
+                               STATUS_MAPPING[status_change.newvalue] &&
+                               (STATUS_MAPPING[status_change.oldvalue] != STATUS_MAPPING[status_change.newvalue])
+                            n.details << JournalDetail.new(:property => 'attr',
+                                                           :prop_key => 'status_id',
+                                                           :old_value => STATUS_MAPPING[status_change.oldvalue].id,
+                                                           :value => STATUS_MAPPING[status_change.newvalue].id)
+                          end
+                          n.save
                     	end
                     	# Attachments
                     puts "Components:      #{migrated_components}/#{TracComponent.count}"
                     puts "Milestones:      #{migrated_milestones}/#{TracMilestone.count}"
                     puts "Tickets:         #{migrated_tickets}/#{TracTicket.count}"
-                    puts "Ticket comments: #{migrated_ticket_comments}/#{trac_ticket_comments}"
                     puts "Ticket files:    #{migrated_ticket_attachments}/" + TracAttachment.count("type = 'ticket'").to_s
                     puts "Custom values:   #{migrated_custom_values}/#{TracTicketCustom.count}"
                     puts "Wiki edits:      #{migrated_wiki_edits}/#{TracWikiPage.count}"

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