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

-        transaction =
-          if @stack.empty?
-            RealTransaction.new(@connection, current_transaction, options)
-          else
-            SavepointTransaction.new(@connection, current_transaction, options)
-          end
+        transaction_class = @stack.empty? ? RealTransaction : SavepointTransaction
+        transaction = transaction_class.new(@connection, current_transaction, options)

ちょいリファクタリング。

remove some caching

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

optimize_routes_generation?メソッドでインスンタンス変数に値をキャッシュしていたのを止めています。

-        return @_optimized_routes if defined?(@_optimized_routes)
-        @_optimized_routes = _routes.optimize_routes_generation? && default_url_options.empty?
+        _routes.optimize_routes_generation? && default_url_options.empty?

このキャッシュは性能の改善にはならないのに、複雑になってしまっているので、キャッシュを削除したとの事です。

remove the mounted? method

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

不要なmounted?メソッドを削除しています。

remove blank lines in the start of the ActiveRecord files

ActiveRecordの各ファイルの先頭にあった不要な空行を削除しています。

Move object test files under object

Objectクラスのテストをactivesupport/test/core_ext/object/配下に移動しています。

Give password_confirmation div the "field" class in erb form template

railties/lib/rails/generators/erb/scaffold/templates/_form.html.erbの修正です。

scaffoldで生成されるtemplateからclass="field"が漏れていたのを追加しています。

Added failing test case for #16131

activesupport/test/core_ext/object/json_cherry_pick_test.rbの修正です。

ojとmulti_jsonと一緒にactive_support/core_ext/objectを使用した際に、エラーになる問題があったらしく、その確認の為のテストを追加しています。

Fixed a compatibility issue with the Oj gem

activesupport/lib/active_support/core_ext/object/json.rbの修正です。

先にテストを上げた問題について対応しています。

active_support/jsonをloadせずにactive_support/core_ext/object/jsonを使用した場合に問題があったようです。

-    if ActiveSupport.use_standard_json_time_format
+    if ActiveSupport::JSON::Encoding.use_standard_json_time_format

active_support/jsonがロードされるよう、autoloadのトリガーを修正しています。

Raise an exception when attachments are added after mail was called.

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

mailメソッドを呼び出した後にattachmentsを設定した場合にRuntimeErrorがraiseされるよう対応しています。

class LateAttachmentMailer < ActionMailer::Base
  def welcome
    mail body: "yay", from: "welcome@example.com", to: "to@example.com"
    attachments['invoice.pdf'] = 'This is test File content'
  end
end

上記のようなケースがエラーになるようになっています。そもそも、正常に動作してなかったので、問題がを起きている事を明確にする為に、Errorを起こすようにしたようです。

class LateAttachmentAccessorMailer < ActionMailer::Base
  def welcome
    attachments['invoice.pdf'] = 'This is test File content'
    mail body: "yay", from: "welcome@example.com", to: "to@example.com"
  end
end