なるようになるブログ

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

rails commit log流し読み(2019/02/11)

2019/02/11分のコミットです。

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


Revert "Merge pull request #35211 from y-yagi/fix_broken_association_test" (#35217)

ajvのバージョンを6.9.0に固定した、Avoid ajv@6.9.0 for nowをRevertしています。ajv 6.9.1がリリースされてbuildの問題が解消した為。


More exercise table name prefix and suffix tests

Active Recordの修正です。

sqlite3 adapterでforeign keyを変更する際に、table_name_prefix / table_name_suffixが指定された場合の考慮が不足していたのを修正、及び、table_name_prefix / table_name_suffixを指定している場合にforeign keyを変更するテストの追加等を行っています。


SQLite3: Implement add_foreign_key and remove_foreign_key

Active Recordの修正です。

sqlite3 adapterにadd_foreign_key / remove_foreign_keyメソッドを追加しています。

create_tableでforeign keyの指定はSQLite: Foreign Key Supportで出来るようになっていたのですが、上記メソッドについては実装されていませんでした。

ただ上記メソッドが無いとschema.rbにforeign keyに関する処理が定義されず、結果db:schema:loadを実行するとdb:migrateした状態と違う状態になってしまう(foreign keysが無い状態)になってしまう、という問題が合った為、実装が追加されています。


Revert "Merge pull request #33563 from lzap/foreign-key-note-doc"

SQLiteではadd_foreign_keyメソッドをサポートしていない旨説明を追加した、Clarify note that SQLite3 adapter doesn't support add_foreign_key [ci skip]をRevertしています。先のコミットでサポートされた為。


Fix typo a -> an, an -> a [ci skip]

各docの修正です。

グラマーの修正(a -> an, an -> a)を行っています。


Merge pull request #35184 from y-yagi/avoid_extra_package_install

railtiesのテストの修正です。

assetsに関するテストで、テスト内でyarn installしてパッケージのインストールを行っていたのを、テストの前に必要なパッケージのインストールを行い、それを使用するよう修正しています。


Merge pull request #35216 from CHTJonas/patch-1

rails guideのWorking with JavaScript in Railsの修正です。

Server-Side Concernsの項にあるjs.erbのexampleコードを修正しています。

が、修正内容に色々と間違いがあり、後ほどRevertしています。


Add remove_foreign_key for change_table

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

change_tableメソッドのblock内でremove_foreign_keyメソッドを使用出来るようにする為に、Table#remove_foreign_keyメソッドを追加しています。


Revert "Merge pull request #35216 from CHTJonas/patch-1"

先ほどコミットされた、rails guideのWorking with JavaScript in Railsのexampleコードの修正をRevertしています。