なるようになるブログ

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

rails commit log流し読み(2015/02/05)

2015/02/05分のコミットです。

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

activerecord/CHANGELOG.md


Respect custom primary keys for associations in Relation#where

activerecord/lib/active_record/relation/predicate_builder/association_query_handler.rbactiverecord/lib/active_record/table_metadata.rbの修正です。

任意のprimary keyを指定してassociationsを使用した場合に、Relation#whereが指定されたprimary keyを使用しないバグがあったのを修正しています。


Merge pull request #18809 from meltheadorable/bugfix/postgres_money

activerecord/lib/active_record/connection_adapters/postgresql/schema_definitions.rbの修正です。

PostgreSQLmoney型を使用しようした場合に、db:migrateStandardErrorが発生してしまうバグがあったのを修正しています。

ColumnMethods#moneyメソッドの第二引数のoptionsが必須になってしまっていた為、optionsが無い場合にエラーになってしまっていました。


load "rack/test" before "action_controller/railtie" in bug report templates [ci skip]

guides/bug_report_templates/action_controller_gem.rbの修正です。

action_controller/railtieより先にrack/testをrequireするよう修正しています。

action_controller/railtiestrong_parameterが読み込まれるのですが、strong_parameterRack::Test::UploadedFileを使用しており、先にrack/testをロードしておかないとLoadErrorが起きてしまうので、先に読み込むよう修正しています。

strong_parameterrack/testをrequireすべきな気がするのですが、それはどうも反対されているようです。詳細はPR参照


fix transaction rollback in case of aborting thread

activerecord/lib/active_record/connection_adapters/abstract/transaction.rbの修正です。

within_new_transactionメソッドで、rollback_transactionをする際、transactionの値のチェックを追加しています。

           if Thread.current.status == 'aborting'
-            rollback_transaction
+            rollback_transaction if transaction

スレッドを使用している場合に、transaction変数がセットされる前に、within_new_transactionが呼ばれる場合があるので、対応したとの事です。


Move ruby-head to allowed failures [ci skip]

.travis.ymlの取得です。

allow_failuresruby-headを追加しています。リリースまで1年近くあるので、今はテストがコケててもしょうがない、との事です。