REVERT: DEV: should ignore missing post uploads when a user export destroyed

REVERT: DEV: should ignore missing post uploads when a user export destroyed

Reverts 793915fe6aa1024b6a08cb8b042b7fb1e0bbece8. We no longer need this since we’re destroying each posts in commit 028121b95b982500e0c63b11d216a6162d47a7bb.

diff --git a/app/models/user_export.rb b/app/models/user_export.rb
index e6687b1..a7b5b67 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 4dfca38..507e485 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

GitHub sha: 2ba4de2d