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

-          value = value.nil? ? "<NULL binary data>" : "<#{value.bytesize} bytes of binary data>"
+          value = value ? "<#{value.bytesize} bytes of binary data>" : "<NULL binary data>"

valueがnilでないケースの方が多いから、という理由だそうです。なるほど。

Merge pull request #14949 from bogdan/empty-hash-array-parameterization

Hash#to_paramメソッドの修正です。空のHash/Arrayがあった場合のserializationの結果を修正しています。

# before
{a: []}.to_query # => "a%5B%5D="

# after
{a: []}.to_query # => ""

rails 4.1で動作が変わってしまったのを、また戻した形です。

No need the else clause

同じくHash#to_paramメソッドの修正です。不要なelseを削除しています。

Improve CHANGELOG [ci skip]

CHANGELOGの修正です。上記to_paramの更新についての記載箇所について、ピリオドを追加しています。

Merge pull request #11166 from xavier/callable_constraint_verification

ActionDispatch::Routing::Constraintsの修正です。

ルーティングの:constraintsオプションには、成約を定義したオブジェクトを渡せるのですが、渡すオブジェクトは:callまたは:matches?メソッドが定義されている必要がありますが。

が、現状どちらのメソッドが定義されていなくても、エラーが起きず握りつぶしてしまっていたのですが、メソッドが定義されていない場合にArgumentErrorが起こすように修正しています。

:constraintsにオブジェクトを渡せるの知りませんでした。以下使い方についてのメモ(rails guideからの抜粋)。

class BlacklistConstraint
  def initialize
    @ips = Blacklist.retrieve_ips
  end

  def matches?(request)
    @ips.include?(request.remote_ip)
  end
end

Rails.application.routes.draw do
  get '*path', to: 'blacklist#index',
    constraints: BlacklistConstraint.new
end

Do not use short-circuit return

直ぐ上のconstraints対応のコミットについて、return if ~している箇所を修正しています。

-  return if callable_constraint.respond_to?(:call) || callable_constraint.respond_to?(:matches?)
-  raise ArgumentError, "Invalid constraint: #{callable_constraint.inspect} must respond to :call or :matches?"
+  unless callable_constraint.respond_to?(:call) || callable_constraint.respond_to?(:matches?)
+    raise ArgumentError, "Invalid constraint: #{callable_constraint.inspect} must respond to :call or :matches?"
+  end