rails commit log流し読み(2015/11/24) - なるようになるブログ

2015/11/24分のコミットです。

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

Add schema dumping support for PostgreSQL geometric data types

Test if each_object(singleton_class) works, since JRuby added it.

activesupport/lib/active_support/core_ext/class/subclasses.rbの修正です。

Class#descendantsメソッドを定義する際に、ObjectSpace.each_objectメソッドの引数にオブジェクトを指定していたのを、singleton classを指定するよう修正しています。

-    ObjectSpace.each_object(Class.new) {}
+    # Test if this Ruby supports each_object against singleton_class
+    ObjectSpace.each_object(Numeric.singleton_class) {}

元々JRubyではObjectSpace.each_objectの引数にsingleton classは指定出来なかったのですが、JRuby 9.0.5.0から指定出来るようになった、かつ、singleton classで見れない場合、descendantsメソッドではObjectSpace.each_objectにClassを指定してチェックするようにしているのですが、Class経由だと数が多く、性能が悪いため修正したとの事です。

Remove blanket array delegation from Relation

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

ActiveRecord::Delegationから、RelationからArrayにdelegation出来ないメソッドの一覧を定義していたBLACKLISTED_ARRAY_METHODS定数を削除しています。

black listで全てのメソッドを対応出来てはいなかった、かつ、現在RelationはEnumerableをincludeしており、Arrayにdelegateしなくてはならないメソッドは少なくなった為、black listの方は削除し、Arrayにdelegateするメソッドの定義の方を更新し、対応しています。

Merge pull request #22372 from greysteil/clearer-ip-spoofing-comment

actionpack/lib/action_dispatch/middleware/remote_ip.rbの修正です。

check_ip_spoofing -> ip_spoofing_checkに変数名を変更、及びIpSpoofAttackErrorが発生する場合の振る舞いについての詳細な説明を追加しています。

Move migration/postgresql_geometric_types_test.rb in adapters/postgresql/geometric_test.rb

activerecord/test/cases/adapters/postgresql/geometric_test.rb、 activerecord/test/cases/migration/postgresql_geometric_types_test.rbの修正です。

PostgreSQLのgeometric型に関するテストを、activerecord/test/cases/adapters/postgresql/geometric_test.rbからactiverecord/test/cases/migration/postgresql_geometric_types_test.rbに移動しています。

Add schema dumping support for PostgreSQL geometric data types

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

PostgreSQLのGeometric Types(line、lseg、box、path、polygon、circle)がschema dump時に正しく出力されるよう修正しています。

Merge pull request #18446 from cloud8421/actionmailer-unregister-interceptor

actionmailer/lib/action_mailer/base.rb、actionmailer/lib/action_mailer/preview.rbの修正です。

動的にinterceptor、preview_interceptorの解除をする為のBase.unregister_interceptor, Base.unregister_interceptors, Base.unregister_preview_interceptor 及び Base.unregister_preview_interceptorsメソッドを追加しています。

Merge pull request #17928 from sergey-alekseev/remove-unused-form-data-method

actionpack/lib/action_dispatch/http/request.rbの修正です。

ActionDispatch::Request#form_data?メソッドで、media typeのチェックで、content_mime_typeを直接参照していたのを、ラッパーメソッドである#media_typeメソッドを使用するよう修正しています。

Merge pull request #17013 from gsamokovarov/fix-null-resolver

actionview/lib/action_view/testing/resolvers.rbの修正です。

ActionView::NullResolver#queryメソッドでActionView::Templateのインスタンスを作成する際、参照する値を謝っていたのを修正しています。

   class NullResolver < PathResolver
     def query(path, exts, formats)
       handler, format, variant = extract_handler_and_format_and_variant(path, formats)
-      [ActionView::Template.new("Template generated by Null Resolver", path, handler, :virtual_path => path, :format => format, :variant => variant)]
+      [ActionView::Template.new("Template generated by Null Resolver", path.virtual, handler, :virtual_path => path.virtual, :format => format, :variant => variant)]
     end
   end

Merge pull request #21000 from twalpole/find_or_parameter_issues

Active Record / Active Modelの修正です。

ActionController::ParametersがHashWithIndifferentAccessの子クラスで無くなった影響で、forbidden_attributes_protectionのテスト用クラスをHashWithIndifferentAccessから継承させないよう修正、及びWhereChain#notメソッドで引数のsanitizeチェックを行うよう修正しています。

Revert "Merge pull request #18446 from cloud8421/actionmailer-unregister-interceptor"

テストでmochaを使うようになっていたのですが、このPRを投げられた後に、テストでmochaを使わないようになっており、その影響でテストがコケてしまった為、一旦revertしたとの事です。

Fix test failures caused by #21000

activemodel/test/cases/attribute_assignment_test.rbの修正です。

active_support/core_ext/hash/indifferent_accessのrequireが不足していたのを追加しています。

set_field_encoding is only needed for MysqlAdapter

activerecord/lib/active_record/connection_adapters/abstract_mysql_adapter.rb、 activerecord/lib/active_record/connection_adapters/mysql2_adapter.rb、 activerecord/lib/active_record/connection_adapters/mysql_adapter.rbの修正です。

引数の値をそのまま返すだけだったActiveRecord::ConnectionAdapters::Mysql2Adapter#set_field_encodingメソッドを削除、及びActiveRecord::ConnectionAdapters::AbstractMysqlAdapte#columnメソッドで直接set_field_encodingメソッドを呼び出さないよう修正しています。

Fix more test failures caused by #21000

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

各テスト用modelのrequireが不足していたので追加しています。

Merge pull request #22272 from y-yagi/remove_test_files_in_plugin_gemspec

railties/lib/rails/generators/rails/plugin/templates/%name%.gemspecの修正です。

rails plugin用のgemspecのテンプレートでtest_filesオプションを使用していたのを、削除しています。

test_filesオプションがdeprecateになっている(Documentation about 'test_files' is confusing · Issue #90 · rubygems/guides)と思われる為、削除されています。

Remove valgrind note about linux only support

rails guideのDebugging Rails Applicationsの修正です。

Valgrindについて説明している箇所に、Linuxだけのアプリである旨記載されてあったのですが、Valgrind 3.11.0からMac OSXもサポートするようになった(Valgrind)為、該当部分の記載を削除しています。

Merge pull request #22287 from dharamgollapudi/patch-2

railties/lib/rails/tasks/dev.rakeの修正です。

dev:cache taskについてのdescを、task -Tでタスク名が表示されるよう、taskの定義の上に移動しています。

Merge pull request #22120 from hnatt/refactor-actionmailer