2017/05/24分のコミットです。
CHANGELOGへの追加はありませんでした。
Merge pull request #29184 from kirs/nested-attributes-friendly-error
activerecord/lib/active_record/nested_attributes.rb
の修正です。
nested attributes accessorsに不正な値が指定された場合のエラーメッセージに、association名を含むよう修正しています。
Merge pull request #29183 from kamipo/refactor_making_join_constraints
activerecord/lib/active_record/associations/join_dependency.rb
の修正です。
重複した内容が定義されていたAssociations::JoinDependency#make_left_outer_joins
、#make_inner_joins
メソッドをリファクタして#make_join_constraints
メソッドにまとめています。
Merge pull request #29182 from kamipo/remove_unused_left_joins_values_generation
activerecord/lib/active_record/relation.rb
の修正です。
ActiveRecord::Relation
クラスから使用していないleft_joins_values
メソッドの生成処理を削除しています。
Merge pull request #29179 from y-yagi/remove_rubygems_version_lock
.travis.yml
の修正です。
CIを実行する際にrubygems
のバージョンを2.6.11に固定していたのを、バージョンロックを削除しています。
rubygems
側の問題でRailsのテストがコケてしまう、という問題があった為ロックしていたのですが、bundler 1.15でその問題が対処されており、ロックは不要になった為削除しています。
Merge pull request #29174 from kamipo/remove_unused_join_part_name
activerecord/lib/active_record/associations/join_dependency/join_part.rb
の修正です。
使用されていないJoinPart#name
メソッドを削除しています。
Merge pull request #29167 from koic/remove_duplicate_inverse_associations_test
activerecord/test/cases/associations/inverse_associations_test.rb
の修正です。
inverse associationに関する重複したテストがあった為、片方削除しています。
Merge pull request #29027 from mohitnatoo/foreign-key-name
activerecord/lib/active_record/connection_adapters/abstract/schema_statements.rb
の修正です。
SchemaStatements#foreign_key_name
メソッドでname
が既に定義されている場合に、不要なidentifierの生成処理を行わないよう修正しています。
Merge pull request #28644 from mtsmfm/set-capybara-app-host
actionpack/lib/action_dispatch/system_testing/test_helpers/setup_and_teardown.rb
の修正です。
system testでcapybaraが使用するホスト(Capybara.app_host
)をhost!
メソッド経由で設定するよう修正しています。
アプリ側でCapybara.app_host
を上書き出来るようにする為。
Merge pull request #29145 from kamipo/reference_type_should_be_not_null
activerecord/lib/active_record/connection_adapters/abstract/schema_definitions.rb
の修正です。
migrationでpolymorphic: true
及びnull: false
を両方定義した際に、xxx_id
カラムのみNOT NULL制約が設定されて、xxx_type
カラムにNOT NULL制約が設定されないバグがあったのを修正しています。
Merge pull request #29141 from kamipo/make_helper_methods_to_private
activerecord/test/cases/associations/has_many_through_associations_test.rb
の修正です。
テスト用のヘルパーメソッドの可視性をprivateに変更しています。
Merge pull request #29119 from spohlenz/fix-select-with-enumerable
actionview/lib/action_view/helpers/tags/select.rb
の修正です。
ActionView::Helpers::Tags::Select#rouped_choices?
メソッドで、@choices
が空かどうかのチェックにempty?
を使用していたのを、blank?
を使用するよう修正しています。
select
メソッドの第二引数のchoices
にEnumerable
をincludeしたオブジェクトを指定した場合に、empty?
はEnumerable
に定義されておらずエラーになってしまう為、エラーになるのを避けるためblank?
を使用するようにしたとの事です。
Bump rubocop and dependent gem versions
Gemfile.lock
の修正です。
rubocopのバージョンを0.49.0
に更新しています。
Fix a RuboCop offences using rubocop -a
各ファイルにrubocopのルールを適応しています。
Merge pull request #29201 from willnet/fix-docs-has-one
rails guideのActive Record Associations
の修正です。
has_one Association Reference
の項でhas_one
associationをbuildする際のメソッド名が誤っていたのを修正しています。