Modify edit page. Still need to handle "form" with drop down lists

app/controllers/admin/pages_controller.rb

@@ -26,6 +26,11 @@ class Admin::PagesController < ApplicationController
   end
 
   def edit
+    @item = Page.find(params[:id])
+    @i18n_variable = @item.i18n_variable
+    @designs = Design.all.entries
+    @design = @item.design
+    @default_design = @design.themes.detect {|t| t.id == @item.theme_id }
   end
 
   def create
@@ -48,6 +53,18 @@ class Admin::PagesController < ApplicationController
   end
 
   def update
+    @item = Page.find(params[:id])
+    if @item.update_attributes(params[:item])
+      flash[:notice] = t('admin.update_success_page')
+      respond_to do |format|
+        format.html {
+          redirect_to admin_page_url(@item) 
+        }
+        format.js {}
+      end
+    else
+      render :action => "edit"
+    end
   end
 
   def destroy

app/helpers/admin/item_helper.rb

@@ -12,6 +12,7 @@ module Admin::ItemHelper
       ret << "<ul>"
       ret << "<li>"
       ret << (link_to node.name, dest)
+      ret << ' | ' << (link_to t('admin.edit'), eval("edit_admin_#{node._type.downcase}_path(node)")) if node._type.eql?('Page')
       ret << ' | ' << (link_to t('admin.new_page'), new_admin_page_path(:parent_id => node.id), :class => 'new_page') if node._type.eql?('Page')
       ret << ' | ' << (link_to t('admin.new_link'), new_admin_link_path(:parent_id => node.id), :class => 'new_link') if node._type.eql?('Page')
       ret << ' | ' << (link_to t(:delete), eval("delete_admin_#{node._type.downcase}_path(node, :authenticity_token => form_authenticity_token)"), :confirm => t('sure?'), :class => 'delete')

app/models/design/theme.rb

@@ -2,8 +2,6 @@ class Theme < Stylesheet
   embedded_in :design
   field :name
   
-  has_many :pages
-  
   before_save :set_name
   
   protected

app/models/page.rb

@@ -1,11 +1,11 @@
 class Page < Item
   
   field :content
+  field :theme_id
   
   validates_presence_of :design_id
   
   belongs_to :design
-  belongs_to :theme
   has_many :page_parts
   
   def create_parts

app/views/admin/pages/_edit.html.erb

@@ -0,0 +1,10 @@
+<%= flash_messages %>
+
+<h1><%= t('admin.editing_page') %></h1>
+
+<%= form_for @item, :url => admin_page_path(@item) do |f| %>
+  <%= render :partial => "form", :locals => { :f => f } %>
+  <p>
+    <%= f.submit t('update') %> <%= link_back %>
+  </p>
+<% end %>

app/views/admin/pages/_form.html.erb

@@ -16,13 +16,13 @@
 
 <p>
 	<%= t('admin.design_name') %>
-	<%= f.select :design_id, @designs.collect { |d| [d.title, d.id] }, :selected => @default_design.id %>
+	<%#= f.select :design_id, @designs.collect { |d| [d.title, d.id] }, :selected => @default_design.id %>
+	<%= f.select :design_id, @designs.collect { |d| [d.title, d.id] } %>
 </p>
 <p>
-	<%= @item.design.inspect %>
 	<%= t('admin.theme') %>
-	<%= f.select :theme_id, @default_design.themes.collect { |t| [t.name, t.id] }, :selected => @default_design.themes.detect {|t| t.file_filename.to_s == 'default.css' }.id %>
-	<%#= f.collection_select :design, @designs, :id, :title %>
+	<%#= f.select :theme_id, @default_design.themes.collect { |t| [t.name, t.id] }, :selected => @default_design.themes.detect {|t| t.file_filename.to_s == 'default.css' }.id %>
+	<%= f.select :theme_id, @design.themes.collect { |t| [t.name, t.id] }%>
 </p>
 
 <p>

app/views/admin/pages/edit.html.erb

@@ -0,0 +1,5 @@
+<% content_for :sidebar do %>
+    <%= render 'admin/items/site_map_left_bar' %>
+<% end -%>
+
+<%= render 'edit' %>

app/views/admin/pages/edit.js.erb

@@ -0,0 +1 @@
+$('#panel_main').html("<%= escape_javascript(render 'edit') %>");

app/views/admin/pages/update.js.erb

@@ -0,0 +1,2 @@
+$('#panel_main').html("<%= escape_javascript(render(:partial => 'admin/pages/show')) %>");
+history.pushState(null, document.title, "<%= escape_javascript(admin_page_url(@item)) %>");