なるようになるブログ

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

rails commit log流し読み(2016/12/29)

2016/12/29分のコミットです。

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

actionview/CHANGELOG.md

railties/CHANGELOG.md

**

Optimize Journey::Route#score

actionpack/lib/action_dispatch/journey/formatter.rbactionpack/lib/action_dispatch/journey/route.rbの修正です。

ActionDispatch::Journey::Route#scoreメソッドの改善として、ループの削除、不要な型変換の削除等を行っています。


Fix Rubocop violations and fix documentation visibility

Rubocopの設定に合わせてコードを修正、及び、Privatize unneededly protected methods in Active Recordでprivate APIが誤ってpublic APIになってしまっていたのを、再度private APIに戻しています。


Merge pull request #27463 from y-yagi/fix_26931

actionview/lib/action_view/helpers/form_helper.rbの修正です。

fields_forメソッドの後にhelper methodを使用した場合に、タグに使用されるobject nameの値がおかしな値になってしまうバグがあったのを修正しています。


Merge pull request #27402 from yui-knk/execute

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

ActiveRecord::ConnectionAdapters::AbstractMysqlAdapter#configure_connectionメソッドでqueryを実行するのに@connection.queryではなくexecuteメソッドを実行するよう修正しています。

@connection.queryだとinstrumentation APIを経由せず、SQLのdebugに困る為、instrumentation APIを経由するexecuteメソッドを使用するようにしたとの事です。


Upgrade to nio4r 2.0.0

Gemfile.lockactioncable/actioncable.gemspecの修正です。

nio4rのバージョンを2.0.0に更新しています。


Revert "Revert "Merge pull request #17943 from jeremywadsack/doc_cache_importability""

Add Guides note to change cache namespaceをrevertしたコミットをrevertしています。ようは追加された状態に戻っています。

元のコミットはrails guideのA Guide for Upgrading Ruby on Railsの修正で、Upgrading from Rails 3.2 to Rails 4.0Cacheの項を追加し、アップグレード時にcacheのネームスペースを変更する必要がある旨、説明を追加したコミットでした。

が、これが必要な状態というのがバグで、4.0.xの最新バージョンで直そうとしていた為、revertされました。が、対応出来る内容では無く、ユーザが対応する必要があるケースの為、再度情報を載せるようにしたとの事です。


Merge pull request #26443 from y-yagi/clear_all_environments_log_by_default

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

log:clearタスクがデフォルトで全てのenv(config/environments配下にファイルがある環境)のログファイルをトランケートするよう修正しています。


Should not update children when the parent creation with no reason

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

has_many through association を使用している場合に、親レコードの作成時に子レコードの更新処理も発生してしまうバグがあったのを修正しています。


Merge pull request #26376 from kamipo/remove_polymorphic_base_class_for_array

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

ActiveRecord::PredicateBuilder::AssociationQueryValue#polymorphic_base_class_from_valueメソッドから不要でvalueがArrayの場合の処理を削除しています。

valueがArrayの場合AssociationQueryValueクラスで処理が行われ、AssociationQueryValueクラスで処理が行われない為。


Sync the visibility of sql_for_insert to private

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

sql_for_insertメソッドの可視性をprotected -> privateに修正しています。

Privatize unneededly protected methods in Active Recordprotectedメソッドの可視性をまとめてprivateに変更する対応が行われており、合わせる為sql_for_insertメソッドについても修正しています。


Update request.rb --ci skip

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

ActionDispatch::Request#key?メソッドにdocを追加しています。


Use proper output format [ci skip] (#27498)

actionpack/lib/action_dispatch/http/request.rbのdoc、及び、guides/source/active_record_postgresql.mdの修正です。

doc内のコードの実行結果を表示する箇所で、スペースが不足している箇所があったのを修正しています。

-    #    request.key? :ip_spoofing_check #=> true
+    #    request.key? :ip_spoofing_check # => true