From ce573c1e55f4713b209e87f77e90dfb1a4afce9f Mon Sep 17 00:00:00 2001
From: Brenda Wallace <brenda@wallace.net.nz>
Date: Fri, 10 Feb 2017 22:41:28 +1300
Subject: [PATCH] Simplified logic in planting.planting_slug

---
 app/models/planting.rb | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/app/models/planting.rb b/app/models/planting.rb
index 112f53ba3..c11f1335f 100644
--- a/app/models/planting.rb
+++ b/app/models/planting.rb
@@ -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'
+    ].downcase.tr(' ', '-')
   end
 
   # location = garden owner + garden name, i.e. "Skud's backyard"