From 3704f4ba2e937e32677d461d54fdaa3172a1df8b Mon Sep 17 00:00:00 2001 From: David Heinemeier Hansson Date: Fri, 28 Mar 2008 22:14:04 +0000 Subject: [PATCH] Fix new migration versions git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@9126 5ecf4fe2-1ee6-0310-87b1-e25e094e27de --- .../lib/action_view/helpers/active_record_helper.rb | 10 ++++++---- activerecord/lib/active_record/migration.rb | 2 +- railties/lib/tasks/databases.rake | 3 +++ 3 files changed, 10 insertions(+), 5 deletions(-) diff --git a/actionpack/lib/action_view/helpers/active_record_helper.rb b/actionpack/lib/action_view/helpers/active_record_helper.rb index 7569cc381d..6b9543c3be 100644 --- a/actionpack/lib/action_view/helpers/active_record_helper.rb +++ b/actionpack/lib/action_view/helpers/active_record_helper.rb @@ -203,8 +203,9 @@ def to_tag(options = {}) alias_method :tag_without_error_wrapping, :tag def tag(name, options) - if object.respond_to?("errors") && object.errors.respond_to?("on") - error_wrapping(tag_without_error_wrapping(name, options), object.errors.on(@method_name)) + if object.respond_to?("errors") && object.errors.respond_to?("on") && object.errors.on(@method_name) + # error_wrapping(tag_without_error_wrapping(name, options), object.errors.on(@method_name)) + tag_without_error_wrapping(name, options.merge({ "class" => options["class"] ? "#{options["class"]} errors" : 'errors' })) else tag_without_error_wrapping(name, options) end @@ -212,8 +213,9 @@ def tag(name, options) alias_method :content_tag_without_error_wrapping, :content_tag def content_tag(name, value, options) - if object.respond_to?("errors") && object.errors.respond_to?("on") - error_wrapping(content_tag_without_error_wrapping(name, value, options), object.errors.on(@method_name)) + if object.respond_to?("errors") && object.errors.respond_to?("on") && object.errors.on(@method_name) + # error_wrapping(content_tag_without_error_wrapping(name, value, options), object.errors.on(@method_name)) + content_tag_without_error_wrapping(name, value, options.merge({ "class" => options["class"] ? "#{options["class"]} errors" : 'errors' })) else content_tag_without_error_wrapping(name, value, options) end diff --git a/activerecord/lib/active_record/migration.rb b/activerecord/lib/active_record/migration.rb index 58374a01ad..4c10cd806c 100644 --- a/activerecord/lib/active_record/migration.rb +++ b/activerecord/lib/active_record/migration.rb @@ -343,7 +343,7 @@ def down(migrations_path, target_version = nil) end def run(direction, migrations_path, target_version) - self.new(direction, migrations_path, target_version) + self.new(direction, migrations_path, target_version).run end def schema_info_table_name diff --git a/railties/lib/tasks/databases.rake b/railties/lib/tasks/databases.rake index bd96b23351..40b60c1959 100644 --- a/railties/lib/tasks/databases.rake +++ b/railties/lib/tasks/databases.rake @@ -104,6 +104,7 @@ namespace :db do version = ENV["VERSION"] ? ENV["VERSION"].to_i : nil raise "VERSION is required" unless version ActiveRecord::Migrator.run(:up, "db/migrate/", version) + Rake::Task["db:schema:dump"].invoke if ActiveRecord::Base.schema_format == :ruby end desc 'Runs the "down" for a given migration VERSION.' @@ -111,6 +112,7 @@ namespace :db do version = ENV["VERSION"] ? ENV["VERSION"].to_i : nil raise "VERSION is required" unless version ActiveRecord::Migrator.run(:down, "db/migrate/", version) + Rake::Task["db:schema:dump"].invoke if ActiveRecord::Base.schema_format == :ruby end end @@ -118,6 +120,7 @@ namespace :db do task :rollback => :environment do step = ENV['STEP'] ? ENV['STEP'].to_i : 1 ActiveRecord::Migrator.rollback('db/migrate/', step) + Rake::Task["db:schema:dump"].invoke if ActiveRecord::Base.schema_format == :ruby end desc 'Drops and recreates the database from db/schema.rb for the current environment.' -- GitLab