DEV: Remove bypass_bump from CookedPostProcessor (#9468)

It was only passing it along to `PullHotlinkedImages` and that class have not used that arg since April 2014 (c52ee665b4)
This commit is contained in:
Jarek Radosz
2020-04-21 03:48:19 +02:00
committed by GitHub
parent 9a6e4b1fa1
commit 5a81e3999c
3 changed files with 7 additions and 7 deletions

View File

@@ -32,7 +32,7 @@ class CookedPostProcessor
@omit_nofollow = post.omit_nofollow?
end
def post_process(bypass_bump: false, new_post: false)
def post_process(new_post: false)
DistributedMutex.synchronize("post_process_#{@post.id}", validity: 10.minutes) do
DiscourseEvent.trigger(:before_post_process_cooked, @doc, @post)
remove_full_quote_on_direct_reply if new_post
@@ -43,7 +43,7 @@ class CookedPostProcessor
remove_user_ids
update_post_image
enforce_nofollow
pull_hotlinked_images(bypass_bump)
pull_hotlinked_images
grant_badges
@post.link_post_uploads(fragments: @doc)
DiscourseEvent.trigger(:post_process_cooked, @doc, @post)
@@ -655,7 +655,7 @@ class CookedPostProcessor
end
end
def pull_hotlinked_images(bypass_bump = false)
def pull_hotlinked_images
# have we enough disk space?
disable_if_low_on_disk_space # But still enqueue the job
# don't download remote images for posts that are more than n days old
@@ -666,7 +666,7 @@ class CookedPostProcessor
Jobs.cancel_scheduled_job(:pull_hotlinked_images, post_id: @post.id)
# schedule the job
delay = SiteSetting.editing_grace_period + 1
Jobs.enqueue_in(delay.seconds.to_i, :pull_hotlinked_images, post_id: @post.id, bypass_bump: bypass_bump)
Jobs.enqueue_in(delay.seconds.to_i, :pull_hotlinked_images, post_id: @post.id)
end
def disable_if_low_on_disk_space