なるようになるブログ

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

rails commit log流し読み(2015/08/31)

2015/08/31分のコミットです。

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


[ci skip] Update what methods Migration#change can reverse

activerecord/lib/active_record/migration/command_recorder.rbguides/source/active_record_migrations.mdのdocの修正です。

Migration#changeメソッドに指定可能なメソッドの一覧(reveribleなメソッド)が最新化されてなかったので、漏れていたメソッドを追加し、ドキュメントを最新化しています。


Silence callback deprecation warning if testing AJ

activejob/test/helper.rbの修正です。

ActiveSupport.halt_callback_chains_on_return_falseにfalseを設定するよう修正しています。

Introduce explicit way of halting callback chains by throwing :abort. Deprecate current implicit behavior of halting callback chains by returning false in apps ported to Rails 5.0. Completely remove that behavior in brand new Rails 5.0 apps. by claudiob · Pull Request #17227 · rails/railsの対応から、callback chainを停止させる条件がfalseをreturnする事から、throw(:abort)する事に変更になったのですが、ActiveJobの幾つかのadapterのテストのsetup/teardown処理でfalseを返す処理がある為、callback処理のwarning("DEPRECATION WARNING: Returning false in a callback will not implicitly halt a callback chain in the next release of Rails. To explicitly halt a callback chain, please use throw :abort instead.")が表示されてしまっていました。

falseが返されても処理としては問題無いのですが、とりあえず無用なwarninigが表示されなくなるようにするために、call back処理の停止条件を変更したとの事です。