diff options
author | OZAWA Sakuro <sakuro@2238club.org> | 2013-03-09 15:37:26 +0900 |
---|---|---|
committer | OZAWA Sakuro <sakuro@2238club.org> | 2013-03-09 15:38:02 +0900 |
commit | c3a26c592ca99e92056ae46328ca69336e400882 (patch) | |
tree | 466a123035f20af6d90c730cde81fce4918a2af3 /activerecord/lib | |
parent | 11dd15a5c2bd08f399793f45c04075c2fbb20861 (diff) | |
download | rails-c3a26c592ca99e92056ae46328ca69336e400882.tar.gz rails-c3a26c592ca99e92056ae46328ca69336e400882.tar.bz2 rails-c3a26c592ca99e92056ae46328ca69336e400882.zip |
Preserve magic comments and content encoding of copied migrations.
During insertion of "# This migration comes from ... " comment at the beginning of
a migration, presence of magic comment was not considered.
Diffstat (limited to 'activerecord/lib')
-rw-r--r-- | activerecord/lib/active_record/migration.rb | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/activerecord/lib/active_record/migration.rb b/activerecord/lib/active_record/migration.rb index 3d71b784e7..5d7762ec3a 100644 --- a/activerecord/lib/active_record/migration.rb +++ b/activerecord/lib/active_record/migration.rb @@ -634,8 +634,17 @@ module ActiveRecord source_migrations = ActiveRecord::Migrator.migrations(path) source_migrations.each do |migration| - source = File.read(migration.filename) - source = "# This migration comes from #{scope} (originally #{migration.version})\n#{source}" + source = File.binread(migration.filename) + inserted_comment = "# This migration comes from #{scope} (originally #{migration.version})\n" + if /\A#.*\b(?:en)?coding:\s*\S+/ =~ source + # If we have a magic comment in the original migration, + # insert our comment after the first newline(end of the magic comment line) + # so the magic keep working. + # Note that magic comments must be at the first line(except sh-bang). + source[/\n/] = "\n#{inserted_comment}" + else + source = "#{inserted_comment}#{source}" + end if duplicate = destination_migrations.detect { |m| m.name == migration.name } if options[:on_skip] && duplicate.scope != scope.to_s @@ -649,7 +658,7 @@ module ActiveRecord old_path, migration.filename = migration.filename, new_path last = migration - File.open(migration.filename, "w") { |f| f.write source } + File.binwrite(migration.filename, source) copied << migration options[:on_copy].call(scope, migration, old_path) if options[:on_copy] destination_migrations << migration |