rails commit log流し読み(2014/09/26) - なるようになるブログ

2014/09/26分のコミットです。

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

actionview/CHANGELOG.md

Changed the meaning of render "foo/bar".

Use released rails-dom-testing

各モジュールのgemspecの更新です。

rails-dom-testingのバージョンを1.0.3に更新しています。

Use release web-console

railties/lib/rails/generators/rails/app/templates/Gemfileの修正です。

web-consoleのバージョンを2.0.0.beta4に修正してます。

Remove internal options from query string of paths

actionpack/lib/action_dispatch/routing/polymorphic_routes.rbの修正です。

rails 4.2でpolymorphic routesを使用した際に、不要なパラメータが付与されてしまっていたバグを修正しています。

edit_polymorphic_path(User.first)
# => => "/users/1/edit?routing_type=path"

"routing_type=path"というパラメータが付与されてしまっていたようです。

Improve welcome page css and table cell aignments.

railties/lib/rails/info.rbの修正です。

welcomeページの表示の改善を行っています。 railsのバージョンを最初に表示、モジュールのバージョンの表示を削除、cssの改善等を行っています。

Use released rails-html-sanitizer and rails-deprecated_sanitizer

各モジュールのgemspecの更新です。

rails-html-sanitizerのバージョンを1.0.1に更新しています。

Revert "Revert "Merge pull request #16888 from jejacks0n/render_template""

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

render "foo/bar"した際の挙動が、render file: "foo/bar"からtemplate: "foo/bar"`に変更になっています。

fileだと任意のファイルを指定して処理が行われてしまうので、セキュリティに良くないのでは、という事で変更になったようです。

詳細についてはPR参照。

Fix actionpack test cases broken by #16888

actionpack/test/controller/new_base/render_file_test.rb、 actionpack/test/controller/new_base/render_template_test.rbの修正です。

上記コミットで壊れてしまったテストを修正しています。

Fix string/gid collision in job arguments

activejob/lib/active_job/arguments.rbの修正です。

シリアライズ処理を行った際に、_aj_globalidというkeyで値を管理するようにしており、デシリアライズ処理を行う際にそのkeyがあるかどうのチェック行っています。

a = ActiveJob::Arguments.serialize({ 1 => Todo.first })
# => [[1, {"_aj_globalid"=>"gid://master/Todo/1"}]]

ActiveJob::Arguments.deserialize(a)
=> [["_aj_globalid",
  #<Todo:0x007fcdacd217f8
   id: 1,
   name: "xx1",
   limit_date: Thu, 21 Aug 2014,
   deleted: false,
   user_id: nil,
   created_at: Tue, 19 Aug 2014 06:22:58 UTC +00:00,
   updated_at: Thu, 21 Aug 2014 15:32:28 UTC +00:00,
   format: nil>]]

:nail_care: the CSS

railties/lib/rails/templates/rails/welcome/index.html.erbの修正です。

改行の削除/追加等を行っています。

Remove unused code

railties/lib/rails/info.rbの修正です。

使用していないコードの削除を行っています。

rails-html-sanitizer is now the default

Gemfile、guides/source/upgrading_ruby_on_rails.mdの修正です。

Gemfileからrails-html-sanitizerの削除及び、rails guideに古い挙動をサポートしたい場合、rails-deprecated_sanitizerを使用するよう説明を追加しています。

No need to install the deprecated_sanitizer gem

Gemfileの修正です。

rails-deprecated_sanitizerを削除しています。

Keep the original implementation to not having to allocate new objects

actionpack/lib/action_dispatch/routing/polymorphic_routes.rbの修正です。

先に行われたpolymorphic routesの修正で元々のコードを消してしまっていたのですが、そのコードを戻しています。

No need CHANGLOG entry for a bug in a non released version

actionpack/CHANGELOG.mdの修正です。

先に行われたpolymorphic routesの修正でCHANGELOGにエントリーを追加していたのですが、non released versionのみでのバグという事で、CHANGELOGからは削除しています。

Ensure named path params are symbols (Fixes #16958)

actionpack/lib/action_dispatch/routing/route_set.rbの修正です。

rails 4.2で、named pathのkeyにStringを渡した際に動作していなかったバグがあったのを修正しています。

チケットより。

render text: root_path(controller: "test", action: "index")         # 正常に動作する
render text: root_path("controller" => "test", "action" => "index") # 動作しない