Merge pull request #1200 from Br3nda/bw/planting_slug

Simplified logic in planting.planting_slug
This commit is contained in:
pozorvlak
2017-02-12 15:46:22 +00:00
committed by GitHub

View File

@@ -64,15 +64,11 @@ class Planting < ActiveRecord::Base
end
def planting_slug
if garden.present? && crop.present?
"#{owner.login_name}-#{garden.name}-#{crop.name}"
elsif garden.present?
"#{owner.login_name}-#{garden.name}-null"
elsif crop.present?
"#{owner.login_name}-null-#{crop.name}"
else
"#{owner.login_name}-null-null"
end.downcase.gsub(' ', '-')
[
owner.login_name,
garden.present? ? garden.name : 'null',
crop.present? ? crop.name : 'null'
].join('-').tr(' ', '-').downcase
end
# location = garden owner + garden name, i.e. "Skud's backyard"