summaryrefslogtreecommitdiff
path: root/help/app
diff options
context:
space:
mode:
authorjessib <jessib@riseup.net>2012-10-31 17:02:02 -0700
committerjessib <jessib@riseup.net>2012-10-31 17:02:02 -0700
commit8b5f7fb6ef438ef034541d6fb500b5ab7e12f3f2 (patch)
tree568d0048c0625fec84a17e6cef7339a31831c2f3 /help/app
parent73a4e9f06840a85225d93734c1f4fb2ebb3b5a93 (diff)
Very basic filtering of tickets based on status. Will want this ajax-y.
Diffstat (limited to 'help/app')
-rw-r--r--help/app/controllers/tickets_controller.rb13
-rw-r--r--help/app/models/ticket.rb1
-rw-r--r--help/app/views/tickets/index.html.haml4
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