Skip to content
Snippets Groups Projects
Commit 704feaae authored by Oleg Valter's avatar Oleg Valter
Browse files

user_id is implicit when accessing user.user_websites

parent 23c1a021
No related branches found
No related tags found
No related merge requests found
...@@ -7,14 +7,15 @@ module Maintenance ...@@ -7,14 +7,15 @@ module Maintenance
end end
def process(user) def process(user)
unless user.user_websites.where(user_id: user.id, position: 1).size.positive? unless user.user_websites.where(position: 1).size.positive?
if user.website.present? if user.website.present?
UserWebsite.create!(user_id: user.id, position: 1, label: 'website', url: user.website) UserWebsite.create!(user_id: user.id, position: 1, label: 'website', url: user.website)
else else
UserWebsite.create!(user_id: user.id, position: 1) UserWebsite.create!(user_id: user.id, position: 1)
end end
end end
unless user.user_websites.where(user_id: user.id, position: 2).size.positive?
unless user.user_websites.where(position: 2).size.positive?
if user.twitter.present? if user.twitter.present?
UserWebsite.create!(user_id: user.id, position: 2, label: 'Twitter', UserWebsite.create!(user_id: user.id, position: 2, label: 'Twitter',
url: "https://twitter.com/#{user.twitter}") url: "https://twitter.com/#{user.twitter}")
...@@ -22,8 +23,9 @@ module Maintenance ...@@ -22,8 +23,9 @@ module Maintenance
UserWebsite.create!(user_id: user.id, position: 2) UserWebsite.create!(user_id: user.id, position: 2)
end end
end end
# This check *should* be superfluous, but just in case... # This check *should* be superfluous, but just in case...
unless user.user_websites.where(user_id: user.id, position: 3).size.positive? unless user.user_websites.where(position: 3).size.positive?
UserWebsite.create!(user_id: user.id, position: 3) UserWebsite.create!(user_id: user.id, position: 3)
end end
end end
... ...
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment