なるようになるブログ

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

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

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

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


Enable multiple submit buttons for Active Storage forms

activestorage/app/assets/javascripts/activestorage.jsactivestorage/app/javascript/activestorage/ujs.jsの修正です。

Active StorageのJSが、formに複数のボタンがあるケースを考慮してなかった(clickされたbuttonではなく、findして最初に見つかったbuttonに対して処理を行っていた)のを修正しています。


Enable Style/ParenthesesAroundCondition cop

.rubocop.ymlStyle/ParenthesesAroundConditioncopを追加、及び、各ファイルにそれらのcopを適応しています。


Ensure foreign_keys assertions after alter table

activerecord/test/cases/migration/foreign_key_test.rbの修正です。

test_change_column_of_parent_tableでalter tableする前にforeign keysを取得していたのを、alter tableした後に取得するよう修正しています。alter tableでforeign keysの情報がロストしてしまった際に、検知出来るようにする為。


Address intermittent CI failure due to non-determined sort order

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

test_values_cast_from_nil_are_persisted_as_nilでassertionで値を比較する際に、値をidでsortするよう修正しています。

sortを指定していと結果が不定になり、seedによってはテストが失敗してしまう為。


Remove superfluous test

activestorage/test/jobs/purge_job_test.rbの修正です。

ActiveStorage::PurgeJobにattach済みのblobを指定した際に、エラーにならない事を確認するテストを削除しています。

今は、ActiveStorage::PurgeJobではなくActiveStorage::Blob#purgeで無視する用の処理が入っている、かつ、ActiveStorage::Blobのテストで同じテストがある為。


Retry ActiveStorage::AnalyzeJobs on download integrity check failures

activestorage/app/jobs/active_storage/analyze_job.rbの修正です。

ActiveStorage::AnalyzeJobで、ActiveStorage::IntegrityErrorエラーが発生した際にretryするよう修正しています。


Update google-cloud-storage gem version [ci skip]

rails guideのActive Storage Overviewの修正です。

Google Cloud Storage Serviceの項に記載されているgoogle-cloud-storage gemのバージョンを1.8以上から1.11以上に修正しています。

1.11以上じゃないとstreaming downloadsが動作しない為。


Merge pull request #33653 from LemonAndroid/patch-1

rails gudieのActive Record Query Interfaceの修正です。

Scopesの項にあるscopeとclass methodの違いについて説明している箇所に、scopeは常にActiveRecord::Relationを返す旨説明を追加、及び、exampleコードを追加しています。


Fix syntax error in the doc [ci skip]

rails gudieのActive Record Query Interfaceの修正です。

先のscopeとclass methodの違いについての説明の修正で追加されたexampleコードに、シンタックスエラーになるコードがあったのを修正しています。