ActiveJobからメールが送れない

Question

実現したいこと

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

この事象について

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

参考文献

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 = '[e-mail]'

    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

Answer 1

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 で機能分割することをおすすめします

Comments

[sabinuki]

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