ActiveJobからメールが送れない

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 61

sabinuki

score 8

実現したいこと

sidekiqを用いて、ActionMailerから、PDFファイルを添付してメールを送信をしたいと思っております。
Pdfクラスを用いてメール送信した場合はメール送信に成功します。
Pdfクラスを継承したMultiPdfインスタンスを用いてメールを送信した場合にActiveJob::Arguments#deserializeにてエラーが発生してしまいます。

この事象について

調べてみるとActiveJobに対してActiveRecordインスタンスを渡すとシリアライズされずに、Global IDというURIに変換されるようです。
デバッグしてみると、以下のようなGlobalIDが引数で渡されていました。
{"_aj_globalid"=>"gid://myapp/MultiPdf/hoge"}, ["fuga@example.com", "", "", "", ""]

参考文献

PdfクラスのインスタンスはActiveRecordインスタンスで、DBにpdfsテーブルが存在します。
MultiPdfクラスはPdfクラスを継承しているので、ActiveRecordを親に持ちますが、DBにテーブルは存在しません。

上記の場合についてMultiPdfインスタンスでメールを送信する場合の方法や対応策についてご教授願います。

エラー内容

  1. sidekiqのエラー内容
    ActiveJob::DeserializationError: Error while trying to deserialize arguments: Couldn't find MultiPdf with 'id'=hoge

該当のソースコード

DBにpfdテーブルがあり、以下のpdfはActiveRecordインスタンスモデルです。

  class Pdf < ActiveRecord::Base
# Table name: pdfs
#
#  id          :integer          not null, primary key
#  status      :integer          default(0)
#  created_at  :datetime         not null
#  updated_at  :datetime         not null
#  deleted_at  :datetime
...

...以降メソッドなど色々
end


上記のPdfクラスを継承したMultiPdfクラスがあります。

class MultiPdf < Pdf
  def id
    self.class.id
  end

  def title
    'マルチPDF'
  end
  ...
end
class ProposalsController < ApplicationController
  ...
  def mail
    email = 'fuga@example.com'

    if @form.valid?
      EmailMailer.pdf(@pdf, email).deliver_later!

      redirect_to home_path
    else
      flash[:alert] = @form.errors.full_messages
    end
  end
end
class EmailMailer < ApplicationMailer
  def pdf(@pdf, email)
    @pdf = pdf

    attachments[@pdf.decorate.title] = WickedPdf.new.pdf_from_string(
      render_to_string(
        template: 'pdf/show.pdf',
        format:   'pdf',
        pdf:      @pdf.title,
        encoding: 'utf-8',
        layout:   'hoge_pdf.html.haml'
      )
    )

    mail(
      to: email,
      subject: "hogehoge"
    )
  end
  ...
end
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

ActiveJob から見るシリアライズとデシリアライズ

ActiveJob はキューに一旦保存する関係で、オブジェクトを渡す場合シリアライズしています。
ActiveRecordのインスタンスであれば、テーブルを持つ前提なので、IDをJobに渡し、findするという動作を裏でしています。
今回動かすだけであれば、オブジェクトをまるごと渡すのではなく、 Hashや title などの String のような必要な情報のみ渡せば良いとおもいます。

EmailMailer.pdf(@pdf.title, @pdf.decorate.title, email).deliver_later!

質問とはそれますが、継承は is a の関係になる (親の機能を子は全て持っている) のを想定しているため、 MultiPdf のクラスがテーブルを持たないのであれば、継承は利用すべきではなさそうです。
(親がテーブルを持っているのに子は持っていない == is a ではない

継承ではなく共通な機能を module や別のクラスに切り出し、 Pdf と MultiPdf にそれぞれもたせる等 Composition で機能分割することをおすすめします

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/09/06 11:02

    回答ありがとうございます!エラー回避することができました。
    継承についてのアドバイスもありがとうございます!勉強になります。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.22%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる