Discource-C
/
discourse
mirror of https://github.com/discourse/discourse.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

FIX: Don't raise an exception if a TopicLink cannot be created 

This can happen under concurrency and it's fine. If a `TopicLink`
already exists we can continue on without raising an error.
This commit is contained in:
Robin Ward 2020-05-12 11:14:57 -04:00
parent 0344ad14c1
commit 088c11a12c
1 changed files with 12 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
app/models/topic_link.rb
View File

@ -213,7 +213,8 @@ class TopicLink < ActiveRecord::Base
unless TopicLink.exists?(topic_id: post.topic_id, post_id: post.id, url: url) unless TopicLink.exists?(topic_id: post.topic_id, post_id: post.id, url: url)
file_extension = File.extname(parsed.path)[1..10].downcase unless parsed.path.nil? || File.extname(parsed.path).empty? file_extension = File.extname(parsed.path)[1..10].downcase unless parsed.path.nil? || File.extname(parsed.path).empty?
begin begin
TopicLink.create!(post_id: post.id, TopicLink.create(
post_id: post.id,
user_id: post.user_id, user_id: post.user_id,
topic_id: post.topic_id, topic_id: post.topic_id,
url: url, url: url,
@ -222,7 +223,8 @@ class TopicLink < ActiveRecord::Base
link_topic_id: topic&.id, link_topic_id: topic&.id,
link_post_id: reflected_post.try(:id), link_post_id: reflected_post.try(:id),
quote: link.is_quote, quote: link.is_quote,
extension: file_extension) extension: file_extension
)
rescue ActiveRecord::RecordNotUnique rescue ActiveRecord::RecordNotUnique
# it's fine # it's fine
end end