FIX: Delete PostUploads on Post deletion (#10090)

Related uploads will then be removed by Jobs::CleanUpUploads
This commit is contained in:
Jarek Radosz
2020-06-19 17:45:08 +02:00
committed by GitHub
parent 4a2871f7f6
commit 58a5293d9b
2 changed files with 11 additions and 1 deletions

View File

@ -38,7 +38,7 @@ class Post < ActiveRecord::Base
has_many :topic_links
has_many :group_mentions, dependent: :destroy
has_many :post_uploads
has_many :post_uploads, dependent: :delete_all
has_many :uploads, through: :post_uploads
has_one :post_stat

View File

@ -1375,6 +1375,16 @@ describe Post do
let(:post) { Fabricate(:post, raw: raw_multiple) }
it "removes post uploads on destroy" do
post.link_post_uploads
post.trash!
expect(PostUpload.count).to eq(6)
post.destroy!
expect(PostUpload.count).to eq(0)
end
context "#link_post_uploads" do
it "finds all the uploads in the post" do
post.custom_fields[Post::DOWNLOADED_IMAGES] = {