Patched hole in the opportunities controller for the tags passing through with whitespace and causing rocket tag errors

just3ws · just3ws · commit 07804ba504e7 · 2014-05-23T11:40:29.000-05:00
diff --git a/app/controllers/opportunities_controller.rb b/app/controllers/opportunities_controller.rb
@@ -124,8 +124,8 @@ def header_ok
 
   def cleanup_params_to_prevent_rocket_tag_error
     if params[:opportunity] && params[:opportunity][:tags]
-      params[:opportunity][:tags] = params[:opportunity][:tags].split(",").compact.join(",")
-      params[:opportunity][:tags] = nil if params[:opportunity][:tags].blank?
+      params[:opportunity][:tags] = "#{params[:opportunity][:tags]}".split(',').map(&:strip).reject(&:empty?).join(",")
+      params[:opportunity][:tags] = nil if params[:opportunity][:tags].strip.blank?
     end
   end
 
