なるようになるブログ

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

rails commit log流し読み(2016/08/06)

2016/08/06分のコミットです。

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


Add note about limitations of Windows Subsystem for Linux

rails guideのGetting Started with Railsの修正です。

Creating the Blog Applicationの項で、Windows Subsystem for Linuxではfile system notificationsが動作しない為、rails newする際に、spring及びlistenを使わないよう、--skip-spring --skip-listenオプションを指定するよう注記を追加しています。

Windows Subsystem for Linuxではlibinotifyが動作しないとの事です。参考:Filesystem watchers like libinotify do not work · Issue #216 · Microsoft/BashOnWindows


Fix GET JSON integration test request to use method override

actionpack/lib/action_dispatch/testing/integration.rbactionpack/test/abstract_unit.rbの修正です。

integration testのgetメソッドのasオプションにjsonを指定した場合に、パラメータがJSONとして扱われないバグがあったのを修正しています。


Merge pull request #26051 from kamipo/make_name_and_binds_to_optional_args

Active Recordの修正です。

#exec_insert#exec_delete#exec_updateメソッドのnamebinds引数のデフォルトにnilを指定しています。

#select_all#select_one#select_value等のメソッドの引数と合わせる為、との事です。


Merge pull request #25665 from kamipo/remove_unused_table_arg

activerecord/lib/active_record/validations/uniqueness.rbの修正です。

ActiveRecord::Validations::UniquenessValidator#scope_relationメソッドから、使用していないtable引数を削除しています。


[ci skip] Fix broken URL in guides

rails guideのRuby on Rails 2.3 Release NotesRuby on Rails 3.0 Release Notesの修正です。

Rails Documentation Projectsへのリンクが壊れていたのを修正しています。


Don't passing a nil value to case_sensitive_comparison

Active Recordの修正です。

各adapter内のcase_sensitive_comparisonメソッドで、引数のvaluenilでないかどうかチェックをしていたのを、呼び出し元(ActiveRecord::Validations::UniquenessValidator#build_relation)でvalueがnilかどうかチェックし、nilの場合early return するよう修正しています。


Revert passing arel node with splat binds for where

activerecord/lib/active_record/relation/where_clause_factory.rbactiverecord/lib/active_record/validations/uniqueness.rbの修正です。

Refactor `case_{sensitive|insensitive}_comparison`の対応により、whereメソッドの引数にarel node を渡せるようになってしまったのですが、whereに直接arel nodeを渡せるのは良くない(arel nodeはあくまで内部的に使用するクラスの為)、という事で、該当の処理をrevertしています。


Merge pull request #26023 from tekin/correct-acceptance-guidelines

rails guideのActive Record Validationsの修正です。

acceptance validatorのexampleコードに誤りがあったのを修正しています。


Merge pull request #25998 from kamipo/enum_with_alias_attribute

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

alias_attributeで定義したattributeにenumを使用した場合に、正常に動作しないバグがあったのを修正しています。