なるようになるブログ

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

rails commit log流し読み(2017/09/02)

2017/09/02分のコミットです。

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


sqlite3 adapter returns integer value which used to be string

activerecord/test/cases/calculations_test.rbactiverecord/test/cases/query_cache_test.rbの修正です。

SQLite3 adapterで、SQLでcalcucateした結果をto_iでIntegerに明示的に変換していたのを、to_iを削除しています。元々は結果がStringで返ってきた為to_iで変換していたのですが、今はgem(sqlite3)側でIntegerに変換するようになり、Rails側での変換処理は不要になった為削除したとの事です。


Bump PostgreSQL version to the latest version 9.6 at Travis CI

.travis.ymlの修正です。

Travisで使用するPostgreSQLのバージョンを9.6に更新しています。


Don’t call scope.eager_loading? when reflection_scope.where_clause is empty

activerecord/lib/active_record/associations/preloader/through_association.rbの修正です。

Preloader::ThroughAssociation#through_scopeメソッドのリファクタリングとして、reflection_scope.where_clauseが空の場合、scope.eager_loading?を呼び出さないよう修正しています。

reflection_scope.where_clauseが空の場合、scopeは変更されておらず、scope.eager_loading?は必ずfalseになるため。


values[:includes] in reflection_scope is not compatible with through_scope

activerecord/lib/active_record/associations/preloader/through_association.rbの修正です。

through associationでsourceを指定している、かつ、そのassocaitionをperloadした場合、エラーになってしまうバグがあったのを修正しています。


Add a test case for preloading through association with implicit source

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

先のコミットで対応したthrough associationをpreloadした際の挙動について、テストを追加しています。


Replace unnecessary link with typewriter text [ci skip]

actionpack/lib/abstract_controller/callbacks.rbのdocの修正です。

AbstractController::Callbacks#process_actionメソッドのdoc内のメソッド名をttタグで囲むよう修正しています。


Accept GCS client options

activestorage/lib/active_storage/service/gcs_service.rbの修正です。

ActiveStorage::Service::GCSService#newGoogle::Cloud::Storageに渡す為のオプションを指定出来るよう修正しています。


Fix can't modify frozen String error in AC::Rendering

ActionController::Rendering#render_to_stringメソッドでcan't modify frozen Stringエラーが発生してしまうバグがあったのを修正しています。


Fix preloading through association with custom scope

activerecord/lib/active_record/associations/preloader/through_association.rbの修正です。

custom scope blockを指定しているthrough associationをpreloadした場合にエラーになってしまうバグがあったのを修正しています。


Retry if rubygems misbehaves: it’s probably just the network

.travis.ymlの修正です。

Travis上でgem updateでエラーになった場合、リトライするよう修正しています。エラーになる原因は恐らくネットワークに関する問題の為、リトライすれば解決する事があるため。


Fix outdated comment [ci skip]

actionview/lib/action_view/template.rbの修正です。

ActionView::Template#locals_codeメソッド内の、keyの設定処理のコメントにダブルアサインを使用している旨記載されいたのを削除しています。

double assign is no longer an effective workaround for unused variable warning の対応でダブルアサインは使用されなくなった為。


Don’t expose find_all_ordered utility method in tests

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

EagerAssociationTestでだけで使用するヘルパーメソッド(find_all_ordered)の可視性をprivateに変更しています。