fixes #143: Janitor trials issues

app/controllers/janitor_trials_controller.rb

@@ -19,18 +19,22 @@ class JanitorTrialsController < ApplicationController
   
   def create
     @janitor_trial = JanitorTrial.create(params[:janitor_trial])
-    respond_with(@janitor_trial)
+    respond_with(@janitor_trial, :location => janitor_trials_path)
   end
   
   def promote
     @janitor_trial = JanitorTrial.find(params[:id])
     @janitor_trial.promote!
-    respond_with(@janitor_trial)
+    respond_with(@janitor_trial) do |format|
+      format.js
+    end
   end
   
   def demote
     @janitor_trial = JanitorTrial.find(params[:id])
     @janitor_trial.demote!
-    respond_with(@janitor_trial)
+    respond_with(@janitor_trial) do |format|
+      format.js
+    end
   end
 end

app/models/janitor_trial.rb

@@ -15,6 +15,10 @@ class JanitorTrial < ActiveRecord::Base
     self.original_level = user.level
   end
   
+  def user_name
+    user.try(:name)
+  end
+  
   def user_name=(name)
     self.user_id = User.name_to_id(name)
   end

app/views/janitor_trials/_secondary_links.html.erb

@@ -0,0 +1,6 @@
+<% content_for(:secondary_links) do %>
+  <menu>
+    <li><%= link_to "Listing", janitor_trials_path %></li>
+    <li><%= link_to "New", new_janitor_trial_path %></li>
+  </menu>
+<% end %>

app/views/janitor_trials/demote.js.erb

@@ -0,0 +1 @@
+location.reload();

app/views/janitor_trials/index.html.erb

@@ -2,7 +2,7 @@
   <div id="a-index">
     <h1>Janitor Trials</h1>
 
-    <table>
+    <table class="striped">
       <thead>
         <tr>
           <th>User</th>
@@ -14,7 +14,7 @@
         <% @janitor_trials.each do |janitor_trial| %>
           <tr>
             <td><%= janitor_trial.user.name %></td>
-            <td><%= janitor_trial.created_at %></td>
+            <td><%= time_ago_in_words janitor_trial.created_at %></td>
             <td>
               <%= link_to "Promote", promote_janitor_trial_path(janitor_trial), :remote => true, :method => :put %>
               | <%= link_to "Demote", demote_janitor_trial_path(janitor_trial), :remote => true, :method => :put %>
@@ -25,3 +25,5 @@
     </table>
   </div>
 </div>
+
+<%= render "secondary_links" %>

app/views/janitor_trials/new.html.erb

@@ -3,8 +3,10 @@
     <h1>New Janitor Trial</h1>
 
     <%= simple_form_for(@janitor_trial) do |f| %>
-      <%= f.input :user_id %>
+      <%= f.input :user_name %>
-      <%= f.button :submit %>
+      <%= f.button :submit, "Submit" %>
     <% end %>
   </div>
 </div>
+
+<%= render "secondary_links" %>

app/views/janitor_trials/promote.js.erb

@@ -0,0 +1 @@
+location.reload();