Merge branch 'master' of ssh://git@svn.medienhaus.udk-berlin.de/usr/local/git/cccms

11 files changed, 78 insertions, 56 deletions

diff --git a/app/controllers/content_controller.rb b/app/controllers/content_controller.rb
index 706cfcd..d548a34 100644
--- a/app/controllers/content_controller.rb
+++ b/app/controllers/content_controller.rb
@@ -10,7 +10,7 @@ class ContentController < ApplicationController
     
     @page = Node.find_page(path)
     
-    if @page
+    if @page and @page.public?
      template = @page.valid_template
 
       render(
@@ -25,4 +25,5 @@ class ContentController < ApplicationController
     end
     
   end
+
 end
diff --git a/app/controllers/nodes_controller.rb b/app/controllers/nodes_controller.rb
index 81ead62..0c961b7 100644
--- a/app/controllers/nodes_controller.rb
+++ b/app/controllers/nodes_controller.rb
@@ -32,7 +32,7 @@ class NodesController < ApplicationController
     
     if parent and @node.save
       @node.move_to_child_of parent
-      redirect_to(@node)
+      redirect_to(edit_node_path(@node))
     else
       @node.errors.add("Parent node")
       render :action => :new
@@ -40,7 +40,15 @@ class NodesController < ApplicationController
   end
 
   def show
-    @nodes = Node.find(params[:id]).children
+    @page = Node.find(params[:id]).draft
+    
+    if @page
+     template = @page.valid_template
+     render(
+       :file => template,
+       :layout => "application"
+     )
+    end
   end
 
   def edit
diff --git a/app/helpers/link_helper.rb b/app/helpers/link_helper.rb
index 68586c1..33d8a06 100644
--- a/app/helpers/link_helper.rb
+++ b/app/helpers/link_helper.rb
@@ -1,11 +1,14 @@
 module LinkHelper
   
-  def link_to_path path
-    url_for(
+  def link_to_path title, path
+    params[:locale] ||= I18n.locale
+    
+    link_to( 
+      title,
       :controller => :content,
-      :action     => :render_page,
-      :language   => I18n.locale,
-      :page_path  => path
+      :action => :render_page,
+      :locale => params[:locale],
+      :page_path => path
     )
   end
   
diff --git a/app/models/node.rb b/app/models/node.rb
index 7f2bca6..d2db4ba 100644
--- a/app/models/node.rb
+++ b/app/models/node.rb
@@ -59,7 +59,6 @@ class Node < ActiveRecord::Base
   def create_new_draft user
     empty_page = self.pages.new
     empty_page.user = user
-    
     empty_page.clone_attributes_from self.head
     
     self.draft = empty_page
@@ -70,11 +69,9 @@ class Node < ActiveRecord::Base
   def publish_draft!
     if self.draft
       self.head = self.draft
+      self.head.save!
       self.draft = nil
       self.save!
-      
-      self.head.published_at = Time.now
-      self.head.save!
     else
       nil
     end
diff --git a/app/models/page.rb b/app/models/page.rb
index a4817a3..af820e3 100644
--- a/app/models/page.rb
+++ b/app/models/page.rb
@@ -1,3 +1,5 @@
+require 'xml'
+
 class Page < ActiveRecord::Base
   
   PUBLIC_TEMPLATE_PATH = File.join(%w(custom page_templates public))
@@ -17,7 +19,7 @@ class Page < ActiveRecord::Base
   before_save :rewrite_links_in_body
   
   # Security
-  attr_accessible :title, :abstract, :body, :template_name
+  attr_accessible :title, :abstract, :body, :template_name, :published_at
   
   # Class Methods
   
@@ -57,24 +59,6 @@ class Page < ActiveRecord::Base
   end
   
   # Instance Methods
-  
-  def clone_attributes_from page
-    return nil unless page
-  
-    self.tag_list = page.tag_list.join(", ")
-    
-    locale_before = I18n.locale
-    
-    I18n.available_locales.each do |l|
-      next if l == :root
-      I18n.locale = l
-      self.title    = page.title
-      self.abstract = page.abstract
-      self.body     = page.body
-    end
-  
-    I18n.locale = locale_before
-  end
 
   def public_template_path
     File.join(PUBLIC_TEMPLATE_PATH, template_name)
@@ -104,8 +88,13 @@ class Page < ActiveRecord::Base
   def clone_attributes_from page
     return nil unless page
   
+    # Clone untranslated attributes
+  
     self.tag_list = page.tag_list.join(", ")
     self.template_name = page.template_name
+    self.published_at = page.published_at
+    
+    # Clone translated attributes
     
     locale_before = I18n.locale
     
@@ -120,27 +109,35 @@ class Page < ActiveRecord::Base
     I18n.locale = locale_before
   end
   
+  def public?
+    published_at.nil? ? true : published_at < Time.now 
+  end
+  
   private
     
     def rewrite_links_in_body
-      if self.body
-        tmp_body = "<div>#{self.body}</div>"
-        xml_string = XML::Parser.string( tmp_body )
-        xml_doc = xml_string.parse
-        links = xml_doc.find("a[not(starts-with(@href, 'http://'))]")
-        
-        locales = I18n.available_locales.reject {|l| l == :root}
-        
-        links.each do |link|
-          unless locales.include? link[:href].slice(1,2).to_sym
-            link[:href] = link[:href].sub(/^\//, "/#{I18n.locale}/")
+      begin
+        if self.body
+          tmp_body = "<div>#{self.body}</div>"
+          xml_string = XML::Parser.string( tmp_body )
+          xml_doc = xml_string.parse
+          links = xml_doc.find("//a[not(starts-with(@href, 'http://'))]")
+          
+          locales = I18n.available_locales.reject {|l| l == :root}
+          
+          links.each do |link|
+            unless locales.include? link[:href].slice(1,2).to_sym
+              link[:href] = link[:href].sub(/^\//, "/#{I18n.locale}/")
+            end
           end
+          
+          tmp_body = xml_doc.to_s.gsub(/(\n\<div\>|\<\/div\>\n)/, "")
+          tmp_body.gsub!("<?xml version=\"1.0\" encoding=\"UTF-8\"?>", "")
+          
+          self.body = tmp_body
         end
-        
-        tmp_body = xml_doc.to_s.gsub(/(\n\<div\>|\<\/div\>\n)/, "")
-        tmp_body.gsub!("<?xml version=\"1.0\" encoding=\"UTF-8\"?>", "")
-        
-        self.body = tmp_body
+      rescue
+        nil
       end
     end
   
diff --git a/app/views/layouts/admin.html.erb b/app/views/layouts/admin.html.erb
index 840c37c..bede781 100644
--- a/app/views/layouts/admin.html.erb
+++ b/app/views/layouts/admin.html.erb
@@ -20,7 +20,8 @@
       theme_advanced_buttons2 : "",
       theme_advanced_buttons3 : "",
       extended_valid_elements : "aggregate[tags|limit|order_by|order_direction|partial]",
-      relative_urls : false
+      relative_urls : false,
+      entity_encoding : "raw"
     });
     </script>
   </head>
diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
index be7e044..e6317fd 100644
--- a/app/views/layouts/application.html.erb
+++ b/app/views/layouts/application.html.erb
@@ -12,10 +12,11 @@
         <![endif]-->
     </head>
     <body>
+        <%= render :partial => 'nodes/preview' if params[:action] == "show" %>
         <div id="logo">
             <a href="index.html" title="Startseite"><img src="/images/chaosknot.gif" alt="Chaosknoten" /></a>
         </div>
-        <!-- HEAD -->
+        <!-- HEAD -->        
         <div id="header">
             <div id="yellow">
             </div>
@@ -25,19 +26,19 @@
             <div id="topnav">
                 <ul id="navlist">
                     <li>
-                        <a href="<%= params[:locale] %>/club">Kennenlernen</a>
+                      <%= link_to_path "Kennenlernen", "club" %>
                     </li>
                     <li>
                         &bull;
                     </li>
                     <li>
-                        <a href="<%= params[:locale] %>/support">Unterstützen</a>
+                      <%= link_to_path "Unterstützen", "support" %>
                     </li>
                     <li>
                         &bull;
                     </li>
                     <li>
-                        <a href="<%= params[:locale] %>/contact">Erreichen</a>
+                      <%= link_to_path "Erreichen", "contact" %>
                     </li>
                     <% if current_user %>
                     <li>
diff --git a/app/views/nodes/_preview.html.erb b/app/views/nodes/_preview.html.erb
new file mode 100644
index 0000000..e8d7b9f
--- /dev/null
+++ b/app/views/nodes/_preview.html.erb
@@ -0,0 +1,5 @@
+<div id="preview_disclaimer">
+  <strong>Preview draft.</strong>
+  Clicking on links will redirected you to the public site.
+  <%= link_to 'Back to admin interface', nodes_path %>
+</div
\ No newline at end of file
diff --git a/app/views/nodes/edit.html.erb b/app/views/nodes/edit.html.erb
index 929bbf6..2dade0f 100644
--- a/app/views/nodes/edit.html.erb
+++ b/app/views/nodes/edit.html.erb
@@ -13,9 +13,14 @@
     <%= f.error_messages %>
     
     <% fields_for @draft do |d| %>
+    
+    <p>
+      <%= d.label :published_at %><br />
+      <%= d.datetime_select :published_at %>
+    </p>
     <p>
       <%= d.label :template_name %>
-      <%= d.select :template_name, custom_page_templates %>
+      <%= d.select :template_name, custom_page_templates, {:prompt => 'Select Template'} %>
     </p>
     <p>
       <%= d.label :title %><br />
diff --git a/app/views/nodes/index.html.erb b/app/views/nodes/index.html.erb
index 833ffed..a2a42b1 100644
--- a/app/views/nodes/index.html.erb
+++ b/app/views/nodes/index.html.erb
@@ -11,6 +11,7 @@
       <th class="path">Path</th>
       <th class="actions">Actions</th>
       <th class="editor">Locked by</th>
+      <th class="revision">Rev.</th>
     </tr>
     <% @nodes.each do |node| %>
     <tr class="<%= cycle("even", "odd") %>">
@@ -29,6 +30,9 @@
       <td>
         <%= "#{node.draft.user.login}" if node.draft && node.draft.user %>
       </td>
+      <td>
+        <%= node.head.revision if node.head %>
+      </td>
     </tr>
     <% end %>
   </table>
diff --git a/app/views/nodes/show.html.erb b/app/views/nodes/show.html.erb
index dd37c28..67dd7fc 100644
--- a/app/views/nodes/show.html.erb
+++ b/app/views/nodes/show.html.erb
@@ -1,5 +1,5 @@
 <h1>Node</h1>
 
-<%= @node.unique_name %>
-
-<%= link_to 'Edit', edit_node_path %>
\ No newline at end of file
+There is no draft to preview. Click <%= link_to 'edit', edit_node_path %> to 
+create one or view the currently 
+<%= link_to_path 'published version', @node.unique_path %>. 
\ No newline at end of file