From 8b5f7fb6ef438ef034541d6fb500b5ab7e12f3f2 Mon Sep 17 00:00:00 2001 From: jessib Date: Wed, 31 Oct 2012 17:02:02 -0700 Subject: Very basic filtering of tickets based on status. Will want this ajax-y. --- help/app/controllers/tickets_controller.rb | 13 +++++++++++-- help/app/models/ticket.rb | 1 + help/app/views/tickets/index.html.haml | 4 +++- 3 files changed, 15 insertions(+), 3 deletions(-) diff --git a/help/app/controllers/tickets_controller.rb b/help/app/controllers/tickets_controller.rb index d2c9643..4684a40 100644 --- a/help/app/controllers/tickets_controller.rb +++ b/help/app/controllers/tickets_controller.rb @@ -41,7 +41,7 @@ class TicketsController < ApplicationController @ticket = Ticket.find(params[:id]) @ticket.attributes = params[:ticket] - # what if there is an update and no new comment? Confirm that there is a new comment to update posted_by + # what if there is an update and no new comment? Confirm that there is a new comment to update posted_by. will @tickets.comments_changed? work? @ticket.comments.last.posted_by = (current_user ? current_user.id : nil) #protecting posted_by isn't working, so this should protect it. if @ticket.save @@ -57,7 +57,16 @@ class TicketsController < ApplicationController def index # @tickets = Ticket.by_title #not actually what we will want - respond_with(@tickets = Ticket.all) #we'll want only tickets that this user can access + #we'll want only tickets that this user can access + # @tickets = Ticket.by_is_open.key(params[:status]) + if params[:status] == 'open' + @tickets = Ticket.by_is_open.key(true) + elsif params[:status] == 'closed' + @tickets = Ticket.by_is_open.key(false) + else + @tickets = Ticket.all + end + respond_with(@tickets) end private diff --git a/help/app/models/ticket.rb b/help/app/models/ticket.rb index 15d4d3d..0407012 100644 --- a/help/app/models/ticket.rb +++ b/help/app/models/ticket.rb @@ -37,6 +37,7 @@ class Ticket < CouchRest::Model::Base design do view :by_title + view :by_is_open end validates :title, :presence => true diff --git a/help/app/views/tickets/index.html.haml b/help/app/views/tickets/index.html.haml index 6db2140..1f46433 100644 --- a/help/app/views/tickets/index.html.haml +++ b/help/app/views/tickets/index.html.haml @@ -3,7 +3,9 @@ Create a = link_to "new ticket", new_ticket_path = # below shouldn't be unless logged in %h2 Tickets -= # want to have selection option to see tickets, that are open, closed or all += form_tag (tickets_path, :method => :get) do # want to redo as ajax, and make sure it displays the selected option + = select_tag :status, options_for_select(["open", "closed", "all"]) + = submit_tag "filter" - @tickets.each do |ticket| %p = link_to ticket.title, ticket -- cgit v1.2.3