rails commit log流し読み(2014/12/13) - なるようになるブログ

2014/12/13分のコミットです。

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

rails guideのActive Job Basicsの修正です。

デフォルトのキュー名のデリミタについての説明の箇所で、エスケープが足りておらずアンダースコアが空として表示されてしまっていたので、エスケープを追加しています。

-The default queue name prefix delimiter is '_'.  This can be changed by setting
+The default queue name prefix delimiter is '\_'.  This can be changed by setting

Align the GemfileEntry API with stable branches

railties/lib/rails/generators/app_base.rbの修正です。

Gemfileを生成する際、githubを指定する為のメソッドとして、GemfileEntry.githubというメソッドがあるのですが、このメソッドにブランチ名を指定出来るよう修正しています。

Rails master requires arel master

railties/lib/rails/generators/app_base.rbの修正です。

dev / edge を使用している場合、arelのmasterを使用するよう修正しています。

New applications should use sass-rails 5.0

railties/lib/rails/generators/app_base.rbの修正です。

sass-rails 5.0を使用するよう修正しています。

Pass through the prepend option to AS::Callback

activemodel/lib/active_model/validations.rbの修正です。

ActiveModelのvalidateメソッドのオプションに、:prependを指定出来るよう修正しています。

このコミットでオプションのチェックを行うようになり、 railsの中で使用していないオプションはエラーになるようになったのですが、どうも、:prependというオプションを使用しているユーザがいた為、下位互換の為に、許容出来るようにしたようです。

何故:prependが必要なのかがissueには説明無いんですよねえ。

Mention that Action Mailer methods are not deferred [ci skip]

rails guideのA Guide for Upgrading Ruby on Railsの修正です。

Action Mailerの項を追加しています。

Active Jobの導入に伴い、Action Mailerの挙動も変わってしまっているので、その旨説明を追加しています。

以下例。

class Notifier < ActionMailer::Base
  def notify(user, ...)
    puts "Called"
    mail(to: user.email, ...)
  end
end

mail = Notifier.notify(user, ...) # Notifier#notify はまだ呼ばれない
mail = mail.deliver_now           # ここで "Called"が表示される

以前の挙動では、mailerのメソッドが呼ばれた時点で処理が行われたのですが、Rails 4.2からはdeliver_now又はdeliver_laterが呼ばれるまで処理が行われなくなりました。

通常は問題無いのですが、mailerでないメソッドから処理を呼ぼうとすると、問題が起きます。

class TestMailer < ActionMailer::Base
  def msg
    mail to:"foo@bar.com",
         subject:"asd",
         from: "a@a.com",
         body: "test"
  end
  def msg_and_deliver
    msg.deliver
  end
end

TestMailer.msg.deliver     # => OK
TestMailer.msg_and_deliver # => NG