なるようになるブログ

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

rails commit log流し読み(2016/01/15)

2016/01/15分のコミットです。

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


[ci skip] fix typo in docs

rails guideのActive Record Query Interfaceの修正です。

Retrieving Objects from the Databaseの項、使用出来るメソッドの一覧を記載している箇所でfindbindにタイポしていたのを修正しています。


Merge pull request #23017 from schneems/schneems/migration-sets-environment-value

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

実行するmigrationが無い場合でも、active_record_internal_metadatasテーブルに現在のenvironmentを保存するよう修正しています。

active_record_internal_metadatasテーブルのenvironmentの値が無いと、DB破壊系のtask(db:dropdb:purge)等を指定された環境で動作させないようにする処理が動作しない為、実行するmigrationが無い場合でも値を保存するよう対応したようです。

が、テストがコケるようになってしまった為、後ほどrevertされています。


Ensure enum scopes work properly with STI and inheritance

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

STIを使用していた場合に、enumのscopeメソッドが正しく動作しないバグがあったのを修正しています。


Properly include the from clause when merging relations

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

relationの#mergeメソッドfrom句を指定した場合に、正しく動作しないバグがあったのを修正しています。


Revert "Merge pull request #20835 from glittershark/if-and-unless-in-secure-token"

Active Recordの修正です。

has_secure_tokenメソッド:if及び:unlessオプションを指定出来るよう対応した、Support :if and :unless in has_secure_tokenをrevertしています。

has_secure_tokenは該当のmodelがtokenを保持している事を宣言する為のメソッドであり、そこに条件を定義するのはおかしいだろう、という事でrevertされたようです。多分。


Revert "Set environment even when no migration runs"

実行するmigrationが無い場合でもactive_record_internal_metadatasテーブルに現在のenvironmentを保存するよう修正をrevertしています。テストがコケるようになってしまった為。


Fix intermittent test failures

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

internal metadataのテストで、ActiveRecord::Migrator.upの呼び出しが不足していた為テストがコケていた為、メソッドの呼び出しを追加し対応しています。


Merge pull request #23052 from kamipo/update_deprecation_horizon

activesupport/lib/active_support/deprecation/method_wrappers.rbのdocの修正です。

deprecate_methodsメソッドのdoc内のexampleが、実際の結果と異なっていたのを、実際の内容と合うよう修正しています。


Merge pull request #23014 from jonatack/improve-generated-config-files

rails/railties/lib/rails/generators/rails/app/templates/config/initializers/active_record_belongs_to_required_by_default.rbrails/railties/lib/rails/generators/rails/app/templates/config/initializers/callback_terminator.rbのdocの修正です。

initializers内のコメントについて、グラマーの修正、及び1行80文字に収まるよう適切な位置に改行を追加しています。


Clean up internal metadata definition

activerecord/lib/active_record/internal_metadata.rbactiverecord/lib/active_record/schema_migration.rbの修正です。

internal metadataテーブル作成処理で、indexの作成にadd_indexを使用していたのをt.indexを使用するよう修正、及び、create_tableprimary_keyオプションを指定していたのを削除しています。

primary_keyオプションを削除したのは、create_tableのオプションにid: falseを指定しており、id: falseが指定されている場合primary_keyオプションは意味が無い為。


last_insert_id_value and last_insert_id are unused anymore

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

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