summaryrefslogtreecommitdiff
path: root/engines/support/app/helpers
diff options
context:
space:
mode:
authorAzul <azul@leap.se>2014-07-05 13:10:50 +0200
committerAzul <azul@leap.se>2014-07-09 13:26:36 +0200
commit9d83092405745f6d1b07a57fd8d2aa8e9bffd9fa (patch)
treee8ad9a5044131ce4bdeb52bd52e990d199c6d066 /engines/support/app/helpers
parentc8fc45c21d72837d5a6bd41ffca18b3ac52a305f (diff)
minor: cleanup some links
only submit the params that differ from the defaults
Diffstat (limited to 'engines/support/app/helpers')
-rw-r--r--engines/support/app/helpers/auto_tickets_path_helper.rb6
-rw-r--r--engines/support/app/helpers/tickets_helper.rb8
2 files changed, 7 insertions, 7 deletions
diff --git a/engines/support/app/helpers/auto_tickets_path_helper.rb b/engines/support/app/helpers/auto_tickets_path_helper.rb
index 5638222..c26aac7 100644
--- a/engines/support/app/helpers/auto_tickets_path_helper.rb
+++ b/engines/support/app/helpers/auto_tickets_path_helper.rb
@@ -45,10 +45,8 @@ module AutoTicketsPathHelper
private
def ticket_view_options
- hsh = {}
- hsh[:open_status] = params[:open_status] if params[:open_status] && !params[:open_status].empty?
- hsh[:sort_order] = params[:sort_order] if params[:sort_order] && !params[:sort_order].empty?
- hsh
+ hash = params.slice(:open_status, :sort_order)
+ hash.reject {|k,v| v.blank?}
end
end
diff --git a/engines/support/app/helpers/tickets_helper.rb b/engines/support/app/helpers/tickets_helper.rb
index 7db31dc..58b67ea 100644
--- a/engines/support/app/helpers/tickets_helper.rb
+++ b/engines/support/app/helpers/tickets_helper.rb
@@ -36,7 +36,8 @@ module TicketsHelper
def link_to_status(new_status)
label = ".#{new_status}"
- link_to_navigation label, auto_tickets_path(open_status: new_status, sort_order: search_order)
+ link_to_navigation label, auto_tickets_path(open_status: new_status),
+ active: search_status == new_status
end
def link_to_order(order_field)
@@ -45,11 +46,12 @@ module TicketsHelper
# for not-currently-filtered field link to descending direction
direction ||= 'desc'
label = ".#{order_field}"
- link_to_navigation label, auto_tickets_path(sort_order: order_field + '_at_' + direction, open_status: search_status),
+ link_to_navigation label,
+ auto_tickets_path(sort_order: order_field + '_at_' + direction),
+ active: search_order.start_with?(order_field),
icon: icon
end
-
def new_direction_for_order(order_field)
# return if we're not filtering by this field
return unless search_order.start_with?(order_field)