なるようになるブログ

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

rails commit log流し読み(2017/12/07)

2017/12/07分のコミットです。

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


Merge pull request #31344 from kamipo/use_string_instead_of_text

activerecord/test/cases/json_attribute_test.rbactiverecord/test/cases/json_shared_test_cases.rbの修正です。

Execute JsonAttributeTest only if supports_json? returns true を revert、及び、Oracle enhanced adapterでもJSON attributesのテストが通るよう修正しています。

テストが通らなかったのはテストに使用していた型(CLOBが使われていたのが不味かった?)、及び、テスト用データの作成処理でprimary key valueが不足していた為だったようなので、左記に関する修正を行っています。


Correct routing test spelling mistake.

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

routingのテストでprecedenceprecidenceにタイポしていたのを修正しています。


[ci skip] Make Todo classes inherit ApplicationRecord

activesupport/lib/active_support/core_ext/module/concerning.rbのdocの修正です。

Module::Concerning moduleのdocのexampleコードで使用しているclassの親クラスにApplicationRecordを指定しています。class内でhas_manybefore_createメソッドを使用しており、親クラスにActiveRecordがいる前提になっていた為。


Merge pull request #31327 from aellispierce/custom-id-change-table-sqlite

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

DBにsqliteを使用している場合に、primary keyにid以外のtypeを指定しているtableに対してchange_tableを実行するとエラーになるバグがあったのを修正しています。