Revert "Revert "Clean code and add 'name' in theme.rb""

This reverts commit 00813c9060.

app/controllers/admin/pages_controller.rb

@@ -21,14 +21,11 @@ class Admin::PagesController < ApplicationController
     @item = Page.new
     @item.is_published = true
     @item.parent_id = @parent_item.id rescue nil
+    @designs = Design.all.entries
+    @default_design = @designs.detect {|d| d.title.to_s == 'Bob' }
   end
 
   def edit
-    @item = Page.find(params[:id])
-    # @item.content = parse_content(@item.content, {:locale => 'show'})
-    @item.content = parse_page_code(@item)
-    @i18n_variable = @item.i18n_variable
-    @code = params[:code]
   end
 
   def create
@@ -51,19 +48,6 @@ class Admin::PagesController < ApplicationController
   end
 
   def update
-    @item = Page.find(params[:id])
-    parse_content_update_code(@item, params[:page][:content])
-    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,7 +12,6 @@ module Admin::ItemHelper
       ret << "<ul>"
       ret << "<li>"
       ret << (link_to node.name, dest)
-      ret << ' | ' << (link_to t('admin.code'), eval("edit_admin_#{node._type.downcase}_path(node, :code => true)")) 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

@@ -1,3 +1,15 @@
 class Theme < Stylesheet
   embedded_in :design
+  field :name
+  
+  has_many :pages
+  
+  before_save :set_name
+  
+  protected
+  
+  def set_name
+    self.name = self.file_filename.split('.')[0]
+  end
+  
 end

app/models/page.rb

@@ -1,11 +1,11 @@
 class Page < Item
   
   field :content
-  field :design_name
   
-  validates_presence_of :design_name, :design_id
+  validates_presence_of :design_id
   
   belongs_to :design
+  belongs_to :theme
   has_many :page_parts
   
   def create_parts
@@ -27,15 +27,4 @@ class Page < Item
     Design.find(design_id)
   end
   
-  protected
-  
-  def setup_default_value
-    super
-    
-    if self.design_id
-      self.design_name = get_design.title
-    end
-    
-  end
-  
 end

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

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

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

@@ -15,8 +15,14 @@
 <% end %>
 
 <p>
-<%= f.label :design_id, t('admin.design_name') %>
-<%= f.select :design_id, Design.all.map{ |l| [l.title, l.id] } %>
+	<%= t('admin.design_name') %>
+	<%= f.select :design_id, @designs.collect { |d| [d.title, d.id] }, :selected => @default_design.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 %>
 </p>
 
 <p>

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

@@ -1,3 +0,0 @@
-<%= f.error_messages %>
-
-<%= f.text_area :content %>

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

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

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

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

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

@@ -2,4 +2,4 @@
 	<%= render 'admin/items/site_map_left_bar' %>
 <% end -%>
 
-<%= render @code ? 'show_code' : 'show' %>
+<%= render 'show' %>

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

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