upload.rb: separate out post creation from upload processing.

This commit is contained in:
evazion
2017-03-31 13:38:48 -05:00
parent 3b414db844
commit b4a38c68b4

View File

@@ -105,7 +105,7 @@ class Upload < ActiveRecord::Base
end end
module ConversionMethods module ConversionMethods
def process_once def process_upload
CurrentUser.scoped(uploader, uploader_ip_addr) do CurrentUser.scoped(uploader, uploader_ip_addr) do
update_attribute(:status, "processing") update_attribute(:status, "processing")
self.source = strip_source self.source = strip_source
@@ -129,16 +129,19 @@ class Upload < ActiveRecord::Base
move_file move_file
validate_md5_confirmation_after_move validate_md5_confirmation_after_move
save save
post = convert_to_post end
post.distribute_files end
if post.save
User.where(id: CurrentUser.id).update_all("post_upload_count = post_upload_count + 1") def create_post_from_upload
create_artist_commentary(post) if include_artist_commentary? post = convert_to_post
ugoira_service.save_frame_data(post) if is_ugoira? post.distribute_files
update_attributes(:status => "completed", :post_id => post.id) if post.save
else User.where(id: CurrentUser.id).update_all("post_upload_count = post_upload_count + 1")
update_attribute(:status, "error: " + post.errors.full_messages.join(", ")) create_artist_commentary(post) if include_artist_commentary?
end ugoira_service.save_frame_data(post) if is_ugoira?
update_attributes(:status => "completed", :post_id => post.id)
else
update_attribute(:status, "error: " + post.errors.full_messages.join(", "))
end end
end end
@@ -146,7 +149,8 @@ class Upload < ActiveRecord::Base
@tries ||= 0 @tries ||= 0
return if !force && status =~ /processing|completed|error/ return if !force && status =~ /processing|completed|error/
process_once process_upload
create_post_from_upload
rescue Timeout::Error, Net::HTTP::Persistent::Error => x rescue Timeout::Error, Net::HTTP::Persistent::Error => x
if @tries > 3 if @tries > 3