refactored revsions controller to act as a nested resource of nodes. boy that cleaned up some stuff quite a bit. also having tests for that is just feeling great

6 files changed, 97 insertions, 48 deletions

diff --git a/app/views/nodes/edit.html.erb b/app/views/nodes/edit.html.erb
index 0635dac..0063dd9 100644
--- a/app/views/nodes/edit.html.erb
+++ b/app/views/nodes/edit.html.erb
@@ -3,7 +3,7 @@
   <%= link_to 'Show', @node %>
   <%= link_to 'Preview', preview_page_path(@draft) %>
   <%= link_to 'Publish', publish_node_path, :method => :put, :confirm => "Publish this draft?" %>
-  <%= link_to 'Revisions', revision_path(params[:id]) %>  
+  <%= link_to 'Revisions', node_revisions_path(@node) %>  
 <% end %>
 
 <div id="page_editor">
diff --git a/app/views/nodes/index.html.erb b/app/views/nodes/index.html.erb
index 0603c08..d78a997 100644
--- a/app/views/nodes/index.html.erb
+++ b/app/views/nodes/index.html.erb
@@ -21,7 +21,7 @@
       </td>
       <td class="actions">
           <%= link_to 'show',      node_path(node) %>
-          <%= link_to 'Revisions', :controller => :revisions, :action => :show, :id => node.id %>
+          <%= link_to 'Revisions', node_revisions_path(node) %>
           <%# 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>
diff --git a/app/views/nodes/show.html.erb b/app/views/nodes/show.html.erb
index 8d8a985..8f56c7c 100644
--- a/app/views/nodes/show.html.erb
+++ b/app/views/nodes/show.html.erb
@@ -1,7 +1,7 @@
 <% content_for :subnavigation do %>
   <%= link_to 'Edit', edit_node_path(@node), :class => "unselected" %>
   <%= link_to 'Preview', preview_page_path(@page) %>
-  <%= link_to 'Revisions', revision_path(params[:id]) %>
+  <%= link_to 'Revisions', node_revisions_path(@node) %>
   <%= link_to 'Unlock', unlock_node_path(@node), :method => :put, :confirm => "Are you sure you want to unlock?" %>
 <% end %>
 
diff --git a/app/views/revisions/diff.html.erb b/app/views/revisions/diff.html.erb
index 9428b7a..f7ad79c 100644
--- a/app/views/revisions/diff.html.erb
+++ b/app/views/revisions/diff.html.erb
@@ -1,11 +1,11 @@
 <% content_for :subnavigation do %>
-  <%= link_to 'Edit',     edit_node_path(@node) %>
-  <%= link_to 'Revisions', revision_path(params[:id]) %>
+  <%= link_to 'Edit',       edit_node_path(@node) %>
+  <%= link_to 'Revisions',  node_revisions_path(@node) %>
 <% end %>
 
 <h1>Revisions#diff</h1>
 
-<% form_tag diff_revision_path do %>
+<% form_tag diff_node_revisions_path do %>
   <%= select_tag :start_revision, options_for_select(@node.pages.map{|x| x.revision}, params[:start_revision].to_i) %>
   <%= select_tag :end_revision, options_for_select(@node.pages.map{|x| x.revision}, params[:end_revision].to_i) %>
   <%= submit_tag 'Diff' %>
diff --git a/app/views/revisions/index.html.erb b/app/views/revisions/index.html.erb
index e3134d7..dbb612b 100644
--- a/app/views/revisions/index.html.erb
+++ b/app/views/revisions/index.html.erb
@@ -1 +1,44 @@
-<h1>Revisions#index</h1>
+<% content_for :subnavigation do %>
+  <%= link_to 'Edit', edit_node_path(@node) %>
+<% end %>
+
+<h2>Revisions for Node: <%= @node.unique_name %></h2>
+
+<% form_tag diff_node_revisions_path(@node) do %>
+<table id="revisions">
+  <tr class="header">
+    <th>First</th>
+    <th>Last</th>
+    <th>Rev.</th>
+    <th>Title</th>
+    <th>User</th>
+    <th>Date</th>
+    <th></th>
+  </tr>
+<% @node.pages.reverse.each do |page| %>
+  <tr>
+    <td><%= radio_button_tag :start_revision, page.revision %></td>
+    <td><%= radio_button_tag :end_revision, page.revision %></td>
+    <td class="revision"><%= page.revision %></td>
+    <td class="title"><%= page.title %></td>
+    <td class="user"><%= page.user.try(:login)  %></td>
+    <td class="date"><%= page.updated_at %></td>
+    <td>
+      <%= link_to 'show', node_revision_path(@node, page) %>
+    </td>
+    <td>
+      <%= link_to(
+            'restore', 
+            restore_node_revision_path(@node, page), 
+            :method => :put, 
+            :confirm => "Restore this revision?"
+          ) %>
+    </td>
+  </tr>
+<% end %>
+  <tr class="no_hover">
+    <td colspan="8" class="right"><%= submit_tag 'Diff revisions' %></td>
+  </tr>
+</table>
+
+<% end %>
\ No newline at end of file
diff --git a/app/views/revisions/show.html.erb b/app/views/revisions/show.html.erb
index ebda18d..154d31b 100644
--- a/app/views/revisions/show.html.erb
+++ b/app/views/revisions/show.html.erb
@@ -1,44 +1,50 @@
 <% content_for :subnavigation do %>
-  <%= link_to 'Edit',     edit_node_path(@node) %>
+  <%= link_to 'show all revisions', node_revisions_path(@node) %>
 <% end %>
 
-<h2>Revisions for Node: <%= @node.unique_name %></h2>
-
-<% form_tag diff_revision_path do %>
-<table id="revisions">
-  <tr class="header">
-    <th>First</th>
-    <th>Last</th>
-    <th>Rev.</th>
-    <th>Title</th>
-    <th>User</th>
-    <th>Date</th>
-    <th></th>
-  </tr>
-<% @node.pages.reverse.each do |page| %>
-  <tr>
-    <td><%= radio_button_tag :start_revision, page.revision %></td>
-    <td><%= radio_button_tag :end_revision, page.revision %></td>
-    <td class="revision"><%= page.revision %></td>
-    <td class="title"><%= page.title %></td>
-    <td class="user"><%= page.user.try(:login)  %></td>
-    <td class="date"><%= page.updated_at %></td>
-    <td>
-      <%= link_to 'show', node_path(page.node) %>
-    </td>
-    <td>
-      <%= link_to(
-            'restore', 
-            restore_revision_path(page), 
-            :method => :put, 
-            :confirm => "Restore this revision?"
-          ) %>
-    </td>
-  </tr>
-<% end %>
-  <tr class="no_hover">
-    <td colspan="8" class="right"><%= submit_tag 'Diff revisions' %></td>
-  </tr>
-</table>
-
-<% end %>
\ No newline at end of file
+<div id="page_editor" class="show_node">
+  <table id="content">
+    <tr>
+      <td class="description">Path</td>
+      <td><%= @page.public_link %></td>
+    </tr>
+    <tr>
+      <td class="description">Author</td>
+      <td><%= @page.user.try(:login) %></td>
+    </tr>
+    <% if @page.node.locked? %>
+    <tr>
+      <td class="description">Locked by</td>
+      <td><%= @page.node.lock_owner.login %></td>
+    </tr>
+    <% end %>
+    <tr>
+      <td class="description">Last updated</td>
+      <td><%= @page.updated_at %></td>
+    </tr>
+    <tr>
+      <td class="description">Published at</td>
+      <td><%= @page.published_at %></td>
+    </tr>
+    <tr>
+      <td class="description">Revision</td>
+      <td><%= @page.revision %></td>
+    </tr>
+    <tr>
+      <td class="description"><strong>Title</strong></td>
+      <td><%= @page.title %></td>
+    </tr>
+    <tr>
+      <td class="description"><strong>Abstract</strong></td>
+      <td><%= @page.abstract %></td>
+    </tr>
+    <tr>
+      <td class="description"><strong>Body</strong></td>
+      <td><%= @page.body %></td>
+    </tr>
+    <tr>
+      <td></td>
+      <td class="right"></td>
+    </tr>
+  </table>
+</div>
\ No newline at end of file