adding revision links to the interface. disabling destroy for now

3 files changed, 26 insertions, 5 deletions

diff --git a/app/controllers/revisions_controller.rb b/app/controllers/revisions_controller.rb
index 20991ef..565a25f 100644
--- a/app/controllers/revisions_controller.rb
+++ b/app/controllers/revisions_controller.rb
@@ -8,8 +8,12 @@ class RevisionsController < ApplicationController
   def diff
     @node = Node.find(params[:id])
     
-    params[:start]  ||= @node.pages.all[-1].revision
-    params[:end]    ||= @node.pages.all[-2].revision
+    if @node.pages.length > 1
+      params[:start]  ||= @node.pages.all[-1].revision
+      params[:end]    ||= @node.pages.all[-2].revision
+    else
+      params[:start], params[:end] = 1, 1
+    end
     
     @start = Page.find( :first, :conditions => {
       :node_id => params[:id],
@@ -24,6 +28,7 @@ class RevisionsController < ApplicationController
   end
 
   def show
+    @node = Node.find(params[:id])
   end
 
 end
diff --git a/app/views/nodes/index.html.erb b/app/views/nodes/index.html.erb
index a2a42b1..31cd53c 100644
--- a/app/views/nodes/index.html.erb
+++ b/app/views/nodes/index.html.erb
@@ -24,7 +24,8 @@
       <td>
         <%= link_to 'Show',     node_path(node) %>
         <%= link_to 'Edit',     edit_node_path(node) %>
-        <%= link_to 'Destroy',  node, :method => :delete, :confirm => "Are you sure you want to delete this node?" %>
+        <%= link_to 'Revision', :controller => :revisions, :action => :show, :id => node.id %>
+        <%# link_to 'Destroy',  node, :method => :delete, :confirm => "Are you sure you want to delete this node?" %>
         <%= link_to 'Unlock', unlock_node_path(node), :method => :put, :confirm => "Are you sure you want to unlock?" %>
       </td>
       <td>
diff --git a/app/views/revisions/show.html.erb b/app/views/revisions/show.html.erb
index 20fc39c..c63e7e8 100644
--- a/app/views/revisions/show.html.erb
+++ b/app/views/revisions/show.html.erb
@@ -1,2 +1,17 @@
-<h1>Revisions#show</h1>
-<p>Find me in app/views/revisions/show.html.erb</p>
+<div id="subnavigation">
+  <%= link_to 'Diff revisions', :action => :diff, :id => params[:id] %>
+</div>
+
+<h2>Revisions for Node: <%= @node.unique_name %></h2>
+
+<h3>Current title: <%= @node.head.title %></h3>
+<table>
+<% @node.pages.reverse.each do |page| %>
+  <tr>
+    <td><%= page.revision %></td>
+    <td><%= page.title %></td>
+    <td><%= page.user.try(:login)  %></td>
+    <td><%= page.updated_at %></td>
+  </tr>
+<% end %>
+</table>
\ No newline at end of file