作成したzipファイルをcarrierwaveでレコードに保存する方法について

Question

carrierwaveとrubyzipのgemを利用しています。
zipファイルを作成後、そのzipファイルの保存するところができません。
zipファイルをファイルとして認識していないことが原因かと思っています。
作成したzipファイルをcarrierwaveにファイルとして認識されるにはどうしたらよいでしょうか。

class SendMaskInsuranceFile < ApplicationRecord
  mount_uploader :csv_url,  DocumentFileUploader
  mount_uploader :zip_url, DocumentFileUploader

def self.generate_pdf_file_with_save
    ActiveRecord::Base.transaction do
      SendMaskInsuranceFile.where(created_at: Time.current.yesterday..Time.current).each do |send_mask_insurance_file|
        open(send_mask_insurance_file.csv_url.path) do |f|
          csv = CSV.new(f, headers: :first_row)
          csv.each do |row|
            staff = Staff.find_by(id: row[3])
            if staff
              mask_insurance_tempfile = Tempfile.new(["#{Time.current.to_i}", ".pdf"], Rails.root.join('tmp'))
              zip = send_mask_insurance_file.generate_zip_file(mask_insurance_tempfile)
              send_mask_insurance_file.update!(zip_url: zip.zipfile)
              **↑エラーは起きませんがzipファイルの保存ができていません。**
              mask_insurance_tempfile.close
            end
          end
        end
      end
    end
  end

  def generate_zip_file(mask_insurance_tempfile)
    zip_tempfile = Tempfile.new(["#{Time.current.to_i}", ".zip"], Rails.root.join('tmp'))
    Zip::File.open(zip_tempfile.path, Zip::File::CREATE) do |zipfile|
      zipfile.add(File.basename(mask_insurance_tempfile.path), mask_insurance_tempfile.path)
    end
  end
end

追記

class DocumentFileUploader < CarrierWave::Uploader::Base
  include CarrierWave::MiniMagick

  process :fix_exif_rotation, if: :image?
  process :strip, if: :image?

  version :thumb, if: :image? do
    process resize_and_pad: [300, 300]
  end

  def store_dir
    "uploads/#{model.class.to_s.underscore}/#{mounted_as}/#{model.id}"
  end

  def size_range
    1..2.megabytes
  end

  def extension_allowlist
    %w(jpg jpeg gif png pdf docx xlsx xlsm pptx doc xls ppt txt zip csv msg)
  end

  private

  def image?(file)
    file.content_type.include? 'image'
  end
end
```追記

Comments

[winterboum]

DocumentFileUploader を載せてください

[ssk]

コメントありがとうございます。
情報を追記いたしました。

Answer 1

extension_allowlist に zip が入ってないのでは、と予想してたのですが、入ってますね ｗ

すると、拡張子が zip になっていないのかも、と思ったのですが、 Tempfile.new の書き方も問題なさそう。

すると私の未知の世界ですが、
generate_zip_file は Zip::File.open の結果を返しますが、Zip::File に zipfile という method見つからなかったのですが、有りますか？
zip_tempfile とか zip_tempfile.path を返すようにしてはどうでしょうか

Comments

[ssk]

下記のzipfileはブロックの中で利用しているものになります。

Zip::File.open(zip_tempfile.path, Zip::File::CREATE) do |zipfile|
zipfile.add(File.basename(mask_insurance_tempfile.path), mask_insurance_tempfile.path)
end

[ssk]

明示的にzip_tempfileを返したところ、無事保存することができました。
ご回答いただき、ありがとうございました。