diff --git a/app/models/user_export.rb b/app/models/user_export.rb index e6687b1ac0..a7b5b6784c 100644 --- a/app/models/user_export.rb +++ b/app/models/user_export.rb @@ -5,16 +5,8 @@ class UserExport < ActiveRecord::Base belongs_to :upload, dependent: :destroy belongs_to :topic, dependent: :destroy - around_destroy :ignore_missing_post_uploads - DESTROY_CREATED_BEFORE = 2.days.ago - def ignore_missing_post_uploads - post_ids = upload.post_uploads.pluck(:post_id) - yield - post_ids.each { |post_id| PostCustomField.create!(post_id: post_id, name: Post::MISSING_UPLOADS_IGNORED, value: "t") } - end - def self.remove_old_exports UserExport.where('created_at < ?', DESTROY_CREATED_BEFORE).find_each do |user_export| UserExport.transaction do diff --git a/spec/models/user_export_spec.rb b/spec/models/user_export_spec.rb index 4dfca38250..507e485bb1 100644 --- a/spec/models/user_export_spec.rb +++ b/spec/models/user_export_spec.rb @@ -41,22 +41,4 @@ RSpec.describe UserExport do expect(Topic.exists?(id: topic_2.id)).to eq(true) end end - - describe '#destroy!' do - it 'should create post custom field for ignored missing uploads' do - upload = Fabricate(:upload, created_at: 3.days.ago) - export = UserExport.create!( - file_name: "test", - user: user, - upload_id: upload.id, - created_at: 3.days.ago - ) - post = Fabricate(:post, raw: "![#{upload.original_filename}](#{upload.short_url})") - post.link_post_uploads - - export.destroy! - - expect(PostCustomField.exists?(post_id: post.id, name: Post::MISSING_UPLOADS_IGNORED)).to eq(true) - end - end end