なるようになるブログ

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

rails commit log流し読み(2024/04/19)

2024/04/19分のコミットです。

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

actionpack/CHANGELOG.md

activerecord/CHANGELOG.md


Add print styles to Rails guides (#51458)

rails guideの修正です。

印刷用のCSS が全体のCSSよりインラインになるように修正しています。


Merge pull request #51588 from duduribeiro/add-devcontainer-forward-ports

railties/lib/rails/generators/devcontainer.rbrailties/lib/rails/generators/rails/app/templates/.devcontainer/devcontainer.json.ttの修正です。

rails newで生成するdevcontainer.jsonで、forward portsを使用するよう修正しています。forward portsを自動で行ってくれない環境でもDevcontainerにアクセス出来るようにするため。


Merge pull request #51479 from nhorton/patch-5

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

driven_byでremote browserを指定するよう設定している場合に、browserのpreload処理が出来ずロード時にエラーになってしまうバグがあったのを修正しています。


Merge pull request #50826 from hexdevs/ta/document-in-order-of-with-enum

activerecord/lib/active_record/relation/query_methods.rbのdocの修正です。

ActiveRecord::QueryMethods#in_order_ofenumnilを指定した場合のexampleを追加しています。


test: update karma npm from 3.1.4 to 6.4.2

actioncable/package.jsonの修正です。

karma npmのバージョンを最新バージョンに更新しています。


Merge pull request #49774 from stevepolitodesign/sp-disable-inheritance

docの修正です。

rails guide及びinheritance_columnのdocにinheritance columnのオーバーライド、及び、無効化についての方法の説明を追加しています。


Add save_and_open_page helper to IntegrationTest

Action Packの修正です。

IntegrationTestでsave_and_open_pageメソッドを使用出来るよう対応しています。IntegrationTestの場合、レスポンスをファイルに保存し、それをブラウザで開く(オープン処理はcopiousfreetime/launchy gemを使用)よう対応されています。


Merge pull request #51036 from zzak/railties/asto-integration

railties/test/application/active_storage/uploads_integration_test.rbの修正です。

Active Storageのfile upload処理のintegrationテストを追加しています。


Don't silently execute statements on migrations when they can't be reversed

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

migrationファイル内でupdateメソッドを使用している場合に、そのmigrationをrevertした際に、updateメソッドが暗黙的に実行されてしまうバグがあったのを修正しています。

が、修正内容に問題があったため、後ほどRevertされています。


Remove unused variable assignment in test

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

使用していない変数を削除しています。


Action View Tests: Use #with_routing helper

actionview/test/abstract_unit.rbactionview/test/actionpack/controller/view_paths_test.rbの修正です。

Action Viewのテストで、Make with_routing test helper work for integration testsで追加されたwith_routing helperを使うよう修正しています。


Revert "Don't silently execute statements on migrations when they can't be reversed"

そのmigrationをrevertした際に、migration内のupdateメソッドが暗黙的に実行されてしまうバグがあったのを修正した、Don't silently execute statements on migrations when they can't be re…をRevertしています。修正というより既存の挙動を変更してしまう対応だったため。


Strict loading using :n_plus_one_only does not eagerly load child associations.

activerecord/lib/active_record/associations/collection_association.rbactiverecord/lib/active_record/core.rbの修正です。

strict_loading:n_plus_one_onlyオプションを指定していた場合に、指定したmodelの子associationをfirstlastでloadした場合にActiveRecord::StrictLoadingViolationErrorがraiseされてしまうバグがあったのを修正しています。


Fix some more ignored block warnings

blockを使用していないメソッドにblockを渡していた箇所があったのを、blockを渡さないよう修正しています。


Arel: only wrap SELECT statements in UNION if they involve ORDER BY, LIMIT or OFFSET

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

ArelでUNIQO用のSQLを生成する際にSELECTを括弧で括るようFixes/union select parenthesesで修正したのですが、、括らないよう戻しています。SQLiteでエラーになるSQLが生成されるようになってしまった為。


Fix indentation on save_and_open_page tests

actionpack/test/controller/integration_test.rbの修正です

save_and_open_pageメソッドのテストのインデントを修正しています。


Fix typo in page_dump_helper.rb

actionpack/lib/action_dispatch/testing/test_helpers/page_dump_helper.rbの修正です。

defaultdefaulにタイポしていたのを修正しています。