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

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

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

Raise ArgumentError for bad strptime arguments

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

先日行われたgroup + calculationメソッドを使用した場合にadapterによって結果のHashのkeyがtype castされてない状態になってしまっているというバグを修正した対応(Always prefer class types to query types when casting group)で、テストの内容に誤りがあったのを修正しています。

AS::Duration should serialize empty values correctly. (#25656)

activesupport/lib/active_support/duration/iso8601_serializer.rbの修正です。

ActiveSupport::Duration::ISO8601Serializerに0を指定した場合に、予期せぬエラーが発生してしまうバグがあったのを修正しています。

# before
ActiveSupport::Duration.parse(0.minutes.iso8601)
# => ActiveSupport::Duration::ISO8601Parser::ParsingError: Invalid ISO 8601 duration: "-P" is empty duration

# after
ActiveSupport::Duration.parse(0.minutes.iso8601)
# => 0 seconds

値が0のときに、ActiveSupport::Duration::ISO8601Serializer#serializeが返す値に誤りがあった為エラーになっていました(PT0Sを返すべきが、Pを返してしまっていた)。

Merge pull request #25735 from timrogers/actioncontroller-parameters-dup

actionpack/lib/action_controller/metal/strong_parameters.rbの修正です。

ActionController::Parameters#dupで@parametersがdupされないなかったのを、@parametersもdupするよう修正しています。

params = ActionController::Parameters.new(foo: "bar")
duplicated_params = params.dup
duplicated_params.delete(:foo)

params == duplicated_params  #=> trueになっていた

#initialize_copyを追加し、そこで@parametersをdupするよう修正しています。

Remove duplicate test and fix a typo in the test

actionpack/test/controller/parameters/dup_test.rb、 actionpack/test/controller/parameters/parameters_permit_test.rbの修正です。

テスト名でtoをtpにタイポしている箇所があったのを修正、及び重複していたテストを削除しています。

Raise ArgumentError for bad strptime arguments

activesupport/lib/active_support/values/time_zone.rbの修正です。

ActiveSupport::TimeZone#strptimeに不正な値を渡した場合に、NoMethodError(undefined method empty?' for nil:NilClass)になってしまっていたのを、ArgumentError`(invalid date)を返すよう修正しています。

# before
ActiveSupport::TimeZone['Eastern Time (US & Canada)'].strptime('1999-12-31', '%Y/%m/%d')
#=> NoMethodError: undefined method `empty?' for nil:NilClass

# after
ActiveSupport::TimeZone['Eastern Time (US & Canada)'].strptime('1999-12-31', '%Y/%m/%d')
# => ArgumentError: invalid date