なるようになるブログ

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

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

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

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


Remove duplicate 'select' database statement

ActiveRecordの修正です。

selectメソッドは各Adapterクラスで定義されていたのですが、処理的に殆どのDBで変わらないという事で、DatabaseStatementsに切り出しています。mysql_adapterだけ他と異なる処理が必要なので、selectメソッドが残ったままになっています。


- Fix mentioned shortcut, to what the shortcut actually is, and that it accepts blocks for assert_no_enqueued_jobs and assert_no_performed_jobs test helpers.

ActiveJobのdocの修正です。

args->argumentsに修正、assert_no_enqueued_jobsassert_no_performed_jobsのdocに、足りてなかった引数を追加しています。


- Changed description for AJ enqueuing methods to provide a sane description.

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

グラマーの修正を行っています。


Merge pull request #17331 from vipulnsward/activejob-cleanups


[ci skip] Provide more reasoning to not accept cosmetic changes

rails guideのContributing to Ruby on Railsの修正です。

Railsでは、スペースの削除、改行の削除等の体裁だけの修正のPRは受け付けてないらしのですが、その点についての詳細説明が記載されているissueへのリンクを追加しています。


Oops, missed a closing bracket [ci skip]

rails guideのContributing to Ruby on Railsの修正です。

上記修正で")"が足りてなかったのを修正しています。


fix url generation error message

actionpack/lib/action_dispatch/journey/formatter.rbの修正です。

URL generation時のエラーメッセージの出力の判定処理に誤りがあったのを修正しています。

-        message << " missing required keys: #{missing_keys.sort.inspect}" if name
+        message << " missing required keys: #{missing_keys.sort.inspect}" unless missing_keys.empty?

チェックする値が間違えてたんですねえ。