Merge pull request #4636 from nonamethanks/fix_replacements
Uploads preprocessing: get rid of same-source existence check
This commit is contained in:
@@ -66,20 +66,6 @@ class UploadService
|
|||||||
def start!
|
def start!
|
||||||
raise NotImplementedError, "No login credentials configured for #{strategy.site_name}." unless strategy.class.enabled?
|
raise NotImplementedError, "No login credentials configured for #{strategy.site_name}." unless strategy.class.enabled?
|
||||||
|
|
||||||
if Utils.is_downloadable?(source)
|
|
||||||
if Post.system_tag_match("source:#{canonical_source}").where.not(id: original_post_id).exists?
|
|
||||||
raise ActiveRecord::RecordNotUnique, "A post with source #{canonical_source} already exists"
|
|
||||||
end
|
|
||||||
|
|
||||||
if Upload.where(source: source, status: "completed").exists?
|
|
||||||
raise ActiveRecord::RecordNotUnique, "A completed upload with source #{source} already exists"
|
|
||||||
end
|
|
||||||
|
|
||||||
if Upload.where(source: source).where("status like ?", "error%").exists?
|
|
||||||
raise ActiveRecord::RecordNotUnique, "An errored upload with source #{source} already exists"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
params[:rating] ||= "q"
|
params[:rating] ||= "q"
|
||||||
params[:tag_string] ||= "tagme"
|
params[:tag_string] ||= "tagme"
|
||||||
upload = Upload.create!(params)
|
upload = Upload.create!(params)
|
||||||
|
|||||||
Reference in New Issue
Block a user