From 262f4e0c2abfefbd6965358fdfd9a8c332a38d57 Mon Sep 17 00:00:00 2001
From: hukl <contact@smyck.org>
Date: Wed, 7 Oct 2009 23:36:03 +0200
Subject: 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

---
 app/controllers/revisions_controller.rb | 21 +++-----
 app/views/nodes/edit.html.erb           |  2 +-
 app/views/nodes/index.html.erb          |  2 +-
 app/views/nodes/show.html.erb           |  2 +-
 app/views/revisions/diff.html.erb       |  6 +--
 app/views/revisions/index.html.erb      | 45 ++++++++++++++++-
 app/views/revisions/show.html.erb       | 88 ++++++++++++++++++---------------
 7 files changed, 104 insertions(+), 62 deletions(-)

(limited to 'app')

diff --git a/app/controllers/revisions_controller.rb b/app/controllers/revisions_controller.rb
index 32110ae..05e8acc 100644
--- a/app/controllers/revisions_controller.rb
+++ b/app/controllers/revisions_controller.rb
@@ -7,12 +7,12 @@ class RevisionsController < ApplicationController
   layout 'admin'
   
   def index
+    @node = Node.find(params[:node_id])
   end
 
   def diff
-    @node = Node.find(params[:id])
+    @node = Node.find(params[:node_id])
     
-    puts @node.pages.length
     if @node.pages.length > 1
       params[:start_revision]  ||= @node.pages.all[-2].revision
       params[:end_revision]    ||= @node.pages.all[-1].revision
@@ -20,26 +20,19 @@ class RevisionsController < ApplicationController
       params[:start], params[:end] = 1, 1
     end
     
-    @start = Page.find( :first, :conditions => {
-      :node_id => params[:id],
-      :revision => params[:start_revision]
-    })
-    
-    @end = Page.find( :first, :conditions => {
-      :node_id => params[:id],
-      :revision => params[:end_revision]
-    })
-    
+    @start  = @node.pages.find_by_revision( params[:start_revision] )
+    @end    = @node.pages.find_by_revision( params[:end_revision] )
   end
 
   def show
-    @node = Node.find(params[:id])
+    @node     = Node.find(params[:node_id])
+    @page     = @node.pages.find(params[:id])
   end
 
   def restore
     page = Page.find(params[:id])
     page.node.restore_revision! page.revision
     flash[:notice] = "Revision #{page.revision} restored"
-    redirect_to :back
+    redirect_to node_path(page.node)
   end
 end
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
-- 
cgit v1.3