なるようになるブログ

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

rails commit log流し読み(2018/08/29)

2018/08/29分のコミットです。

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


Focus search input after page load on /rails/info/routes (#33683)

actionpack/lib/action_dispatch/middleware/templates/routes/_table.html.erbの修正です。

routes用のページ(/rails/info/routes)にアクセスした際に検索ボックスにフォーカスがあたるよう修正しています。


Add test to make sure the custom object key can't be serialized

activejob/test/cases/argument_serialization_test.rbの修正です。

customしたobject serializer keyはserialize出来ない事を確認するテストを追加しています。


Generate the same value as a label of view in system test template

railties/lib/rails/generators/test_unit/scaffold/templates/system_test.rb.ttの修正です。

scaffoldで使用するsystem test用のテンプレートファイル内のattributeに対して値を入力する処理で、attributeに対してtitleizeメソッドを使用していたのを使用しないよう修正しています。

Capybaraのfill_inメソッドを使用して値を入力するコードなのですが、scaffoldで使用するviewのテンプレートファイルではtitleizeを使用しておらず、system testの方だけtitleizeを使用すると、viewで生成されるlabelとattribute名が一致せず、結果テストを実行するとエラーになってしまう、という問題があった為。


Merge pull request #33718 from kddeisz/permit-list

各コードでblacklist / whitelistという言葉を使用していたのを、permitted list / restricted listを使用するよう修正しています。

Merge pull request #33681 from minaslater/replace-white-and-blacklist ではdocの修正がメインでしたが、こちらはコードの修正メイン。


Add "Ruby on Rails 6.0 Release Notes" guide [ci skip]

rails guideのRuby on Rails 6.0 Release Notesを追加しています。

書かれているのはParallel Testingについての説明だけで、とりあえず雛形を追加しただけの状態です。


Prevent leaking of user's DB credentials on rails db:create failure

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

rails db:createでエラーになった際に、エラーメッセージ内でconfigの内容をそのまま表示していたのを、テーブル名だけ表示するよう修正しています。

configの内容にはDB接続用のパスワードも含まれており、セキュリティ的に良くないだろう、という理由の為。