なるようになるブログ

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

rails commit log流し読み(2015/03/15)

2015/03/15分のコミットです。

CHANGELOGへの追加はありませんでした。


Fix before_commit when updating a record on the callback

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

before_commitコールバックの中でレコードの更新処理を行った場合に、正常に動作してなかったバグがあったのを修正しています。


[ci skip] Subject-verb agreement

rails guideのContributing to Ruby on Railsの修正です。

Updating the Gemfile.lockの項を"Some changes requires"を"Some changes require"にグラマーの修正を行っています。


Deleting unused fixtures

使用していないテスト用ファイルの actionview/test/fixtures/multipart/bracketed_utf8_paramactionview/test/fixtures/multipart/single_utf8_paramを削除しています。


Cleaning unused methods from abstract_unit

actionview/test/abstract_unit.rbの修正です。

使用していないテスト用のクラス、メソッドを削除しています。


Zlib isn't required for AssetTagHelperTest anymore

actionview/test/template/asset_tag_helper_test.rbの修正です。

不要なzlibのrequireを削除しています。


Cleaning unused views from actionpack tests inside actionview

使用していないテスト用ファイルの actionview/test/actionpack/abstract/views/abstract_controller/testing/me5/index.erbを削除しています。


Cleaning JavaScriptHelperTest

actionview/test/template/javascript_helper_test.rbの修正です。

使用していないメソッドインスタンス変数を削除しています。


Fix typo in Action Pack changelog. [ci skip]

actionpack/CHANGELOG.mdの修正です。

implemet -> implementにタイポを修正しています。


add mailer suffix to comments for email previews

railties/lib/rails/generators/test_unit/mailer/templates/preview.rbの修正です。

email previews用のクラスのコメントに記載されているpreview確認用に、mailer suffixを追加しています。Add _mailer suffix to generated file names. の漏れ対応です。


Doc fix about association hierarchy

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

Associationクラスの下位クラスについて説明している箇所で、ForeignAssociationクラスが漏れていたので、追加しています。


Fix leaky chain on polymorphic association

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

Reflection#chainメソッドでthrough reflectionの値を取得する際、dupした値を取得するよう修正しています。

-          b = through_reflection.chain
+          b = through_reflection.chain.map(&:dup)

polymorphic has many through associationをscope付きで呼び出した後に、scope無しのhas many through associationを呼び出した場合に、 最初に呼び出した際のscopeがキャッシュされてしまい、2回目のscope無しの呼び出しの方に影響を与えてしまうバグがあったのを修正しています。