なるようになるブログ

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

rails commit log流し読み(2014/10/20)

2014/10/20分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

activerecord/CHANGELOG.md


Make _status_code methods nodoc

actionpack/lib/action_controller/metal.rbactionpack/lib/action_dispatch/http/response.rbの修正です。

_status_codeメソッド:nodoc:を追加しています。


Fixed test throwing unused variable warning

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

テスト内で使用してない変数を削除しています。


Enable emitting of warnings from ActiveJob tests.

activejob/Rakefileの修正です。

テスト実行時にwarningが出力されるよう、Rake::TestTask#warningにtrueを設定しています。


- Removed unused variable warnings from ActiveJob Adapters ActiveJobのテストの修正です。

使用してない変数の削除を行っています。


remove duplicate method (_status_code) in action_dispatch

actionpack/lib/action_dispatch/http/response.rbactionpack/lib/action_controller/metal/rack_delegation.rbの修正。

Response#_status_codeメソッドを削除しています。

ActionController_status_codeメソッドをResponseにdelegateするようにしていたのですが、そもそもActionController::Metalに_status_codeメソッドがあるので、delegateの必要が無いので、削除したようです。


[ci skip] merge docs

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

Relation::HashMerger#mergeメソッドのdocを追加しています。が、直後のコミットで削除される事に。


[ci skip] Make merge method nodoc

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

Relation::HashMerger#mergeメソッドのdocを削除しています。こちらはprivate APIのようで、代わりに:nodoc:を追加しています。


Mention perform_now in AJ base docs to specify how to immediately invoke a job. [ci skip]

activejob/lib/active_job/base.rbのdocの修正です。

ActiveJobの説明を行っている箇所に、perform_nowについての説明を追加しています。


AR::UnknownAttributeError should include the class name of a record

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

ActiveRecord::UnknownAttributeErrorにエラーが起きたレコードのクラス名を含むよう対応しています。

User.new(name: "Yuki Nishijima", project_attributes: {name: "kaminari"})
# => ActiveRecord::UnknownAttributeError: unknown attribute on User: name

docs, It's "Active Job" not "Active job". [ci skip]

activejob/lib/active_job/base.rbのdocの修正です。

"Active job"を"Active Job"に修正しています。


No need to specify log level, run at log level as all other Rails components.

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

helperの中でlog levelを指定していたのを削除しています。他のコンポーネントと同じログレベルで実行されるようにする為、ですかねえ。


- Remove duplication in AJ logging test for setting logger.

activejob/test/cases/logging_test.rbactivejob/test/cases/rescue_test.rbの修正です。

loggerの設定処理を、set_loggerメソッドで行うよう修正しています。 あと合わせて、不要なrequireの削除を行っています。