なるようになるブログ

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

rails commit log流し読み(2017/11/30)

2017/11/30分のコミットです。

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


Merge pull request #31214 from chopraanmol1/bug_fix_has_one_inverse_owner_reload_from_validation

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

inverse associationをvalidation内で使用した際に、associationのインスタンスがreloadされてしまい、結果値が正しく取得出来ない、というバグがあったのを修正しています。


Make screenshots default to "simple" format

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

System Testで取得するscreenshotsのデフォルトをsimpleからinlineに変更しています。

simpleはiTerm image protocolを使用して表示する為、Macだとターミナルに画像が正しく表示されます。ただ、それ以外の環境(Ubuntuとか)だと画像が表示されず、ただバイナリが表示されてしまいます。

で、これは流石にデフォルトとしては良くないだろう、という事でinline(screenshotのpathだけ表示)に変更されました。issueが複数来た為。


Build the root folder before specific files

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

rails newで新規にアプリを生成する際に、rootディレクトリ(tmpvendor)を最初に作成するようにしています。

元々は、上記rootディレクトリよりActive Storage用のファイルが先に作成されていた、かつ、Active Storageはtmpディレクトリを作成する為、tmpディレクトリを作成する際に既に存在している旨メッセージが表示されてしまっていたました。 そのメッセージが表示されてしまうのを避ける為、rootディレクトリを先に作るようにしたようです。


Add :nodoc: to StatementPool which is internal used [ci skip]

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

内部用のクラスであるStatementPool:nodoc:を指定しています。


Do not overwrite by default if credentials already exists

railties/lib/rails/generators/rails/credentials/credentials_generator.rbの修正です。

credentials を編集済みの状態でcredentials:editコマンドを実行した際に、編集済みの値を無視してデフォルトで上書きしてしまうバグがあったのを修正しています。