なるようになるブログ

読書感想文かrailsについてかrubyについてか

rails commit log流し読み(2017/05/24)

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メソッドの第二引数のchoicesEnumerableを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する際のメソッド名が誤っていたのを修正しています。