##// END OF EJS Templates
Code cleanup....
Jean-Philippe Lang -
r8225:20ead6d8f73d
parent child
Show More
@@ -936,12 +936,13 class Issue < ActiveRecord::Base
936 }
936 }
937 # custom fields changes
937 # custom fields changes
938 custom_values.each {|c|
938 custom_values.each {|c|
939 next if (@custom_values_before_change[c.custom_field_id]==c.value ||
939 before = @custom_values_before_change[c.custom_field_id]
940 (@custom_values_before_change[c.custom_field_id].blank? && c.value.blank?))
940 after = c.value
941 next if before == after || (before.blank? && after.blank?)
941 @current_journal.details << JournalDetail.new(:property => 'cf',
942 @current_journal.details << JournalDetail.new(:property => 'cf',
942 :prop_key => c.custom_field_id,
943 :prop_key => c.custom_field_id,
943 :old_value => @custom_values_before_change[c.custom_field_id],
944 :old_value => before,
944 :value => c.value)
945 :value => after)
945 }
946 }
946 @current_journal.save
947 @current_journal.save
947 # reset current journal
948 # reset current journal
General Comments 0
You need to be logged in to leave comments. Login now