remove dumb menu helpers

Author: Austin Schneider

app/controllers/locations_controller.rb

@@ -1,5 +1,4 @@
 class LocationsController < ApplicationController
-
   respond_to :html
 
   helper_method :location
@@ -20,8 +19,6 @@ def destroy
     respond_with location
   end
 
-##################################################
-
   # TODO: test
   def location
     @location ||= begin
@@ -32,5 +29,4 @@ def location
       end
     end
   end
-
 end

app/helpers/application_helper.rb

@@ -29,18 +29,6 @@ def render_paginated_locations
     end
   end
 
-  def main_menu
-    [
-      ["Add URL", new_location_path],
-      ["Email callbacks", email_callbacks_path],
-      ["Alerts", alerts_path]
-    ]
-  end
-
-  def email_callbacks_menu
-    [['Add email callback', new_email_callback_path]]
-  end
-
   def alerts
     @alerts ||= Alert.joins { location }.order { locations.title.asc }
   end

app/views/email_callbacks/index.html.haml

@@ -1,6 +1,6 @@
 #email_callbacks
   %h2 Email callbacks
 
-  %p= render 'shared/piped_links', link_args: email_callbacks_menu
+  %p= link_to 'Add email callback', new_email_callback_path
 
   = render_email_callbacks

app/views/locations/index.html.haml

@@ -1,7 +1,12 @@
 #locations
   %h2 URLs
 
-  %p= render 'shared/piped_links', link_args: main_menu
+  %p
+    = link_to "Add URL", new_location_path
+    |
+    = link_to "Email callbacks", email_callbacks_path
+    |
+    = link_to "Alerts", alerts_path
 
   = render_paginated_locations
   = paginate paginated_locations

app/views/shared/_piped_links.html.haml

@@ -95,16 +95,6 @@
     end
   end
 
-  describe "main_menu" do
-    its :main_menu do
-      should == [
-        ["Add URL", new_location_path],
-        ["Email callbacks", email_callbacks_path],
-        ["Alerts", alerts_path]
-      ]
-    end
-  end
-
   describe "alerts" do
     context "when there are alerts" do
       before do
@@ -122,12 +112,6 @@
     end
   end
 
-  describe "email_callbacks_menu" do
-    its :email_callbacks_menu do
-      should == [["Add email callback", new_email_callback_path]]
-    end
-  end
-
   describe "email_callbacks" do
     context "when there are email callbacks" do
       before do