なるようになるブログ

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

rails commit log流し読み(2018/04/10)

2018/04/10分のコミットです。

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


Allow schema/structure load for multiple databases

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

複数DBを使用していた場合に、schema/structureのload処理で正しいファイルを読み込まないバグがあったのを修正しています。


Add multidb application test

railties/test/application/rake/multi_dbs_test.rbn修正です。

複数DBを使用している場合の各種db tasksについてのテストを追加しています。


Don't create namespaced tasks if single db application

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

database.ymlが2階層構成だった場合にもspec nameを含むrake task(e.g. db:create:primary)が作成されていたのを、作成しないよう修正しています。spec nameを含むtaskは複数DB用(database.ymlが3階層構成)の為。


Merge pull request #32488 from swrobel/patch-4

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

headles chromeを起動する際に、必ず--disable-gpuオプションを指定していたのを、Windows環境の場合でのみ指定するよう修正しています。

Windows以外(LinuxMac)では--disable-gpuオプションの指定はもう不要な為。

参考:737678 - Headless: make --disable-gpu flag unnecessary - chromium - Monorail


Correct Maintenance Policy for Rails 5.2

rails guideのMaintenance Policy for Ruby on Railsの修正です。

Rails 5.2がリリースされた為、それに合わせてサポートするバージョンに5.2.Zを追加、及び、4.2.Zをサポートしているバージョンから削除しています。

が、これは誤りでSevere Security Issuesでは4.2.Zはまだサポートしている為、後ほど修正されています。


Update bug report templates

bug report templatesの修正です。

各ファイルのRails関連のgemのバージョンを5.2.0.rc1 -> 5.2.0に修正しています。


Add TestCaseTest#test_request_format_kwarg_doesnt_mutate_params to master

actionpack/test/controller/test_case_test.rbの修正です。

GET request エミュレート用のメソッドでparamsが変更されない事を確認するテストを追加しています。


Merge pull request #32498 from eugeneius/mutation_tracker_merge_changes

activemodel/lib/active_model/attribute_mutation_tracker.rbの修正です。

attributesにHashのArrayを指定した場合に、changes_to_saveメソッドでその値を変更(keyがSymbolだった場合に、Stringになってしまう)されてしまっていたのを、元の値を維持するよう修正しています。


4.2 is still maintained

rails guideのMaintenance Policy for Ruby on Railsの修正です。

Severe Security Issuesの対象に4.2.Zを追加、及び、5.0.Zを削除しています。

サポートするバージョンは、最新のリリース(5.2)、一つ前のリリース(5.1)、そして、最後のシリーズの最後のリリース(4.2)の為。という訳で5.0.Zは表向きはSecurity Issuesのサポート外になっています。表向きは。


Always exclude JSON root from direct_uploads#create response

activestorage/app/controllers/active_storage/direct_uploads_controller.rbの修正です。

ActiveStorage::DirectUploadsControlle#direct_upload_jsonメソッドでdirect upload用のURLやヘッダーをJSONで返す際に、JSONにroot要素を含まないよう修正しています。

元々はActiveRecord::Base.include_root_in_jsonにtrueが指定されていた場合、rootが含まれるようになっていたのですが、JS側がrootが無いことを前提として実装となっていた為、include_root_in_jsonの設定に関わらずrootは含まないようにしています。