なるようになるブログ

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

rails commit log流し読み(2017/06/22)

2017/06/22分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

activerecord/CHANGELOG.md


Merge pull request #27063 from MaxLap/merge_keep_inner_join

activerecord/lib/active_record/associations/join_dependency.rbの修正です。

ネストされた2つのrelationをマージする際、マージされたrelationのJOINがLEFT OUTER JOINになってしまっていたのを、INNER JOINのままになるよう修正しています。

# Before
Author.joins(:posts).merge(Post.joins(:comments))
#=> SELECT ... FROM authors INNER JOIN posts ON ... LEFT OUTER JOIN comments ON...

# After
Author.joins(:posts).merge(Post.joins(:comments))
#=> SELECT ... FROM authors INNER JOIN posts ON ... INNER JOIN comments ON...

Remove unused variable

activerecord/test/cases/relation_test.rbの修正です。

test_relation_merging_with_merged_symbol_joins_keeps_inner_joinsから使用していない変数を削除しています。


Fix hash conditions documentation [ci skip]

rails guideのActive Record Query Interfaceの修正です。

Equality Conditionsの項、conditionの値にSymbolが使えない旨説明がされていたのですが、実際はSymbolを使用出来るので不要な説明を削除しています。


Merge pull request #29524 from y-yagi/do_not_use_private_api_in_bug_report_templates

guides/bug_report_templates/active_record_migrations_gem.rbguides/bug_report_templates/active_record_migrations_master.rbの修正です。

bug report templatesでprivate API(ActiveRecord::Migrator)を使用していたのを、public API(ActiveRecord::Migration#migrate)を使用するよう修正しています。