なるようになるブログ

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

rails commit log流し読み(2014/12/01)

2014/12/01分のコミットです。

CHANGELOGへの追加はありませんでした。


FIX: absolute_path may be nil for code added via instance_eval

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

inheritedメソッドでcall_stack取得する際、absolute_pathメソッドを使用するよう修正が以前入ったのですが、absolute_pathメソッドnilを返すケースがあるらしく、 absolute_pathnilの場合はpathを取得するよう修正しています。

absolute_pathnilになるのはおかしくないか、という話もありましたが、とりあえずは上記対応が入ったようです。


Pretty-print request params on exception page

actionpack/lib/action_dispatch/middleware/exception_wrapper.rbactionpack/lib/action_dispatch/middleware/templates/rescues/_request_and_response.html.erbの修正です。

エラーページで表示されるquery paramsを、pretty printで表示されるよう修正しています。

PRより。

before

before

after

after

これは見やすくなって良いですねえ。


Ensure numericality validations work with mutation

activemodel/lib/active_model/validations/numericality.rbactiverecord/lib/active_record/attribute_methods/dirty.rbの修正です。

before_validationでattributeの値を修正した場合に、NumericalityValidatorで変更後の値でチェックが行われないregressionがあったのを修正しています。


Fix value extracted from negative integers for PostgreSQL.

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

integer型のカラムに"-1"を設定した場合に、numericalityのvalidationがエラーになってしまうregressionがあったのを修正しています。 正規表現に誤りがあったようです。

             # Numeric types
-            when /\A\(?(-?\d+(\.\d*)?\)?(::bigint)?)\z/
+            when /\A\(?(-?\d+(\.\d*)?)\)?(::bigint)?\z/

Merge pull request #17863 from tgxworld/fix_extracated_default_value_for_negative_integers