horribly rough toggling of open/closed status

practicingruby · Mar 17, 2012 · d66215c · d66215c
1 parent 9bb7ebe
commit d66215c
Show file tree

Hide file tree

Showing 2 changed files with 41 additions and 11 deletions.
diff --git a/app/controllers/discussions_controller.rb b/app/controllers/discussions_controller.rb
@@ -2,15 +2,28 @@ class DiscussionsController < ApplicationController
   before_filter :find_course
 
   def index
-    @discussions = case params[:category]
-    when "conversation"
-      @course.discussions.active.conversations
-    when "review"
-      @course.discussions.active.reviews
-    when "evaluation"
-      @course.discussions.active.evaluations
+    if params[:state] == "closed"
+      @discussions = case params[:category]
+      when "conversation"
+        @course.discussions.inactive.conversations
+      when "review"
+        @course.discussions.inactive.reviews
+      when "evaluation"
+        @course.discussions.inactive.evaluations
+      else
+        @course.discussions.inactive.conversations
+      end
     else
-      @course.discussions.active.conversations
+      @discussions = case params[:category]
+      when "conversation"
+        @course.discussions.active.conversations
+      when "review"
+        @course.discussions.active.reviews
+      when "evaluation"
+        @course.discussions.active.evaluations
+      else
+        @course.discussions.active.conversations
+      end
     end
   end
 

diff --git a/app/views/discussions/index.html.haml b/app/views/discussions/index.html.haml
@@ -1,8 +1,25 @@
-= link_to "Conversations", course_discussions_path(:category => "conversation")
+= link_to "Conversations", 
+  course_discussions_path(params.merge(:category => "conversation"))
 |
-= link_to "Reviews", course_discussions_path(:category => "review")
+= link_to "Reviews", 
+  course_discussions_path(params.merge(:category => "review"))
 |
-= link_to "Evaluations", course_discussions_path(:category => "evaluation")
+= link_to "Evaluations", 
+  course_discussions_path(params.merge(:category => "evaluation"))
+
+%br
+%br
+
+  - if params[:state] == "closed"
+    State: 
+    = link_to "Open", course_discussions_path(params.merge(:state => "open"))
+    |
+    Closed
+  -else
+    State: 
+    Open 
+    |
+    = link_to  "Closed", course_discussions_path(params.merge(:state => "closed"))
 
 %br
 %br