2018/02/22分のコミットです。
CHANGELOGへの追加はありませんでした。
activerecord/lib/active_record/tasks/database_tasks.rb
、
railties/lib/rails/application/configuration.rb
の修正です。
database configurationをロードした際に、default
セクションを削除するよう修正しています。
元々はそのまま保持していたのですが、DatabaseTasks
でconfigurationを参照する際にdefault
セクションの場合は処理をスキップする、という処理が入っていた為、ロード時に削除し、チェックは行わないすむよう修正しています。
activerecord/lib/active_record/railties/databases.rake
、
activesupport/lib/active_support/xml_mini.rb
の修正です。
クラスメソッドを呼び出すのに::
を使用していたのを、.
を使用するよう修正しています。
- if data = YAML::load(ERB.new(IO.read(file)).result) + if data = YAML.load(ERB.new(IO.read(file)).result)
Do not add routes when actions are not specified
railties/lib/rails/generators/rails/controller/controller_generator.rb
の修正です。
controller generatorでactionが指定されていない場合は、routes
にnamespaceを追加しないよう修正しています。Rails 5.1までと挙動を合わせる為。
We should call methods with .method_name
not ::method_name
.
他にもクラスメソッドを呼び出すのに::
を使用している箇所があったのを、.
を使用するよう修正しています。
Add cop for preferring 'Foo.method' over 'Foo::method'
.rubocop.yml
にStyle/ColonMethodCall
copを追加、及び、各ファイルにcopを適応しています。
railties/lib/rails/application/configuration.rb
の修正です。
database configurationをロードした際に、database
の設定が無いセクションを削除するよう修正しています。
Delete default configurationでdefault
セクションを削除するよう修正したのですが、それだとdefault
セクションにdatabase
の設定があった場合に問題になる(YAMLのアンカーを使っている場合)為、database
の設定が無い箇所を削除する修正しています。
Properly escape column name embedded into regexp
activerecord/test/cases/base_test.rb
の修正です。
test_default_values_are_deeply_dupped
でqueryにquoted_id
が含まれているか確認する際に、quoted_id
をRegexp.escape
で囲むよう修正しています。
- assert_match(/SELECT #{quoted_id}.* FROM developers/, query) + assert_match(/SELECT #{Regexp.escape(quoted_id)}.* FROM developers/, query)
SQLServerAdapterではカラム名に中括弧を使うようになっており(e.g.[id]
)、括弧をエスケープしないと正しくマッチしない為。