なるようになるブログ

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

rails commit log流し読み(2018/11/14)

2018/11/14分のコミットです。

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

actionpack/CHANGELOG.md

railties/CHANGELOG.md


add a nice alias for the --webpack option

railties/lib/rails/generators/rails/app/app_generator.rbの修正です。

rails new--webpackオプションのaliasとして--webpackerを指定出来るよう修正しています。オプションが--webpack--webpackerなのか混乱する事がある為、両方指定出来るようにしたとのことです。


Merge pull request #34429 from bogdanvlviv/ensure-that-connected_to-establish_connection

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

ActiveRecord::Base#connected_to:databaseオプションを指定した場合に、その指定した際に接続が行われる事を確認するテストを追加しています。


Adjust bind length of SQLite to default (999)

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

SQLite3 adapterで、bind params lengthを999にしています。

SQLiteはデフォルトでは999までしかparameterを指定出来ない為。参考:Implementation Limits For SQLite


Emit single pair of parens for UNION and UNION ALL

activerecord/lib/arel/visitors/mysql.rbactiverecord/lib/arel/visitors/to_sql.rbの修正です。

複数のUNION / UNION ALLを使用した場合に、不要な括弧が生成されてしまうバグがあったのを修正しています。

元々MySQL + UNIONの場合は問題無かった(Visitors::MySQLに独自の処理があった)のですが、UNION ALLの場合、及び、他のadapterでは問題があり、全ての処理で正しいSQLが生成されるように修正しています。


Merge pull request #34411 from N0xFF/master

actionpack/lib/action_dispatch/system_testing/test_helpers/setup_and_teardown.rbの修正です。

system testのテスト失敗時に行うscreenshot取得処理でエラーになった場合にもCapybaraのsessionをリセットするよう修正しています。


Merge pull request #34400 from gmcgibbon/rm_autoload_app_javascripts

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

デフォルトのeager_load_pathsautoload_pathsからapp/assets及びapp/javascriptを除外するよう修正しています。

app/assetsapp/javascript配下にはRuby用のファイルは無いはず、かつ、上記ディレクトリがeager_load_pathsautoload_pathsの対象に入ってしまっていると、不要なmoduleが定義されてしまう(例えば、app/javascript/foo/barというディレクトリがあった場合に、Foo::Barというmoduleが定義されてしまう)為。


Allow rescue from parameter parse errors

Action Packの修正です。

parameter parse error(ActionDispatch::Http::Parameters::ParseError)をcontrollerのrescue_fromでキャッチ出来るよう修正しています。


Describe how has_many's :dependent option affects #delete

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

Options for has_manyの項、:dependentオプションについて説明している箇所に、:destroyオプションと:delete_allオプションはcollection.deletecollection=の挙動にも影響を与える旨説明を追加しています。


fix typo on association_basics.md [ci skip]

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

semanticssamanticsにタイポしている箇所があったのを修正しています。