以下の内容はhttps://y-yagi.hatenablog.com/entry/2024/08/01/050647より取得しました。


rails commit log流し読み(2024/07/31)

2024/07/31分のコミットです。

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

actionpack/CHANGELOG.md


lib/assets is too rare of a use case to warrant a default directory (#52447)

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

rails newで新規にアプリケーションを作成した際に、lib/assetsディレクトリを作成するようになっていたのを、作成しないよう修正しています。lib/assetsを使う事はレアと思われ、デフォルトで作成する必要は無い、とのことです。


Edits to the scripts entry in CHANGELOG

railties/CHANGELOG.mdの修正です。

scripts generatorに関するエントリーについて、フォーマットや実行するためのexampleコードを適切な内容に更新しています。


Revise formatting in railties/CHANGELOG.md

railties/CHANGELOG.mdの修正です。

CHANGELOG全体について、フォーマットの修正を行っています。


Still more CHANGELOG formatting

railties/CHANGELOG.mdの修正です。

scripts generatorに関するエントリーについて、引き続きフォーマットの修正を行っています。


Merge pull request #52448 from jhawthorn/symbol_as_json

activesupport/lib/active_support/core_ext/object/json.rbの修正でs.

Symbol#as_jsonのメソッドの戻り値をto_sからnameに変更しています。mainでサポートしているRubyでは、Symbol#nameが必ず使用出来るため。


Get rid of unused AbstractAdapter#log method

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

使用していないAbstractAdapter#logメソッドを削除しています。


Postgresql: clear results after accessing cmd_tuples

activerecord/lib/active_record/connection_adapters/postgresql/database_statements.rbの修正です。

PostgreSQL adapteeでcmd_tuplesメソッドを呼び出した後にresultがclearするよう修正しています。次に実行する際に正しい結果を取得出来るようにするため。


Deprecate hash key path mapping

Action Packの修正です。

hash key pathを使用してのrouting(e.g. get "/users" => "users#index")をdeprecatedにしています。

# Before
get "/users" => "users#index"
post "/logout" => :sessions
mount MyApp => "/my_app"

# After
get "/users", to: "users#index"
post "/logout", to: "sessions#logout"
mount MyApp, at: "/my_app"

hash key pathを使用してのroutingがある事により処理が複雑になってしまっており、この処理を削除する事でroutingの高速化が望めるため、との事です。


Added declaration to generate errors when running a Docker build with warnings.

railties/lib/rails/generators/rails/app/templates/Dockerfile.ttの修正です。

check=error=true declarationを追加して、warningがある場合、Docker buildがエラーになるよう修正しています。


Revert "Get rid of unused AbstractAdapter#log method"

使用していないAbstractAdapter#logメソッドを削除した、Get rid of unused AbstractAdapter#log methodをRevertしています。この時点では使用してなかったのですが、他のPRでこのメソッドを使っていた為。


AbstractAdapter#cast_result should return a result with columns even if empty

Active Recordの修正です。

columnが空の場合も、AbstractAdapter#cast_resultが値を返すよう修正しています。rowsが空でもcolumnは取得出来るケースはあり、その場合columnは返すべきだろう、とのことです。


Merge pull request #52446 from jhawthorn/activerecord_log

activerecord/lib/active_record/connection_adapters/abstract/database_statements.rbの修正です。

Refactor Active Record adapters to have a similar internal interfaceで削除された、DatabaseStatements#raw_execute内のlogメソッドの呼び出しを戻しています。このlogメソッドをオーバライドして処理を行っている事があるらしく、その処理を維持出来るようにする為。


Refactor SQLite3Adapter to simplify transaction management

Active Recordの修正です。

SQLite3Adapterのtransactionの管理処理がシンプルになるようリファクタリングしています。


Get rid of unused reset_read_uncommitted method

activerecord/lib/active_record/connection_adapters/sqlite3/database_statements.rbの修正です。

使用していないreset_read_uncommittedメソッドを削除しています。


sql_for_insert doesn't need to concern itself about binds

activerecord/lib/active_record/connection_adapters/abstract/database_statements.rbactiverecord/lib/active_record/connection_adapters/trilogy/database_statements.rbの修正です。

sql_for_insertメソッドから使用していないbinds引数を削除しています。


Get rid of unused Sqlite3Adapter#use_insert_returning?

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

使用していないSqlite3Adapter#use_insert_returning?メソッドを削除しています。


Revert "Merge pull request #52464 from Shopify/sql-for-insert-no-binds"

sql_for_insertメソッドから使用していないbinds引数を削除した、sql_for_insert doesn't need to concern itself about binds をRevertしています。3rd partyのadapter(oracle-enhanced adapter)が使用している為。




以上の内容はhttps://y-yagi.hatenablog.com/entry/2024/08/01/050647より取得しました。
このページはhttp://font.textar.tv/のウェブフォントを使用してます

不具合報告/要望等はこちらへお願いします。
モバイルやる夫Viewer Ver0.14