radiant-templates-extension 1.0.0

data/lib/templates/page_extensions.rb

@@ -0,0 +1,43 @@
+module Templates::PageExtensions
+
+  def self.included(base)
+    base.class_eval do
+      before_save :match_with_template
+
+      def allowed_children
+        [default_child, Template.all].flatten
+      end
+
+    end
+  end
+
+  def match_with_template
+    unless self.template.blank?
+      self.class_name = self.template.page_class_name
+      self.layout_id = self.template.layout_id
+    end
+  end
+
+  def template_name
+    template ? template.name : "Blank Page"
+  end
+
+  def is_a_blank_page?
+    ! template_id?
+  end
+
+  # Meta-programming methods to determine if page is built on a specific template
+  # e.g. @page.is_a_press_release? assuming you have "Press Release" template defined.
+  begin
+    Template.all.each do |unique_template|
+      class_eval <<-CODE, __FILE__, __LINE__
+        def is_a_#{unique_template.name.titleize.scan(/\w+/).join.underscore}?
+          template_id? && template.name == "#{unique_template.name}"
+        end
+      CODE
+    end
+  rescue
+    # This won't run before initial migrations, which create the Template table
+  end
+
+end

data/lib/templates/tags.rb

@@ -0,0 +1,43 @@
+module Templates::Tags
+  include Radiant::Taggable
+  
+  tag 'boolean_part' do |tag|
+    tag.expand
+  end
+  
+  desc %{
+    Renders its contents if the selected part contains the text "true".  When
+    used with check_box part-types in a template, can turn on and off sections
+    of content.  The 'part' attribute is required.
+    
+    *Usage*:
+    
+    <pre><code><r:boolean_part:if part="display_sidebar">Sidebar content</r:boolean_part:if></code></pre>
+  }
+  tag 'boolean_part:if' do |tag|
+    part_name = tag.attr['part']
+    raise StandardTags::TagError.new("`boolean_part:if' tag requires a 'part' attribute") unless part_name
+    part = tag.locals.page.part(part_name)
+    # result = part.nil? ? "" : part.content.strip
+    result = part.nil? ? "" : (dev?(tag.globals.page.request) && defined?(ConcurrentDraftExtension) ? part.draft_content.strip : part.content.strip)
+    tag.expand if result =~ /t|true|1/i
+  end
+  
+  desc %{
+    Renders its contents if the selected part does not contain the text "true".  When
+    used with check_box part-types in a template, can turn on and off sections
+    of content.  The 'part' attribute is required.
+    
+    *Usage*:
+    
+    <pre><code><r:boolean_part:unless part="display_sidebar">Sidebar content</r:boolean_part:unless></code></pre>
+  }
+  tag 'boolean_part:unless' do |tag|
+    part_name = tag.attr['part']
+    raise StandardTags::TagError.new("`boolean_part:unless' tag requires a 'part' attribute") unless part_name
+    part = tag.locals.page.part(part_name)
+    # result = part.nil? ? "" : part.content.strip
+    result = part.nil? ? "" : (dev?(tag.globals.page.request) && defined?(ConcurrentDraftExtension) ? part.draft_content.strip : part.content.strip)
+    tag.expand unless result =~ /t|true|1/i
+  end
+end

data/spec/controllers/admin_page_controller_extensions_spec.rb

@@ -0,0 +1,61 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+# We only have this spec because setting up an isolated view spec would be too
+# complicated.  Instead we have a quasi-integration test.
+describe Admin::PagesController, "edit view controlled by templates" do
+  integrate_views
+  dataset :users_and_pages, :templates
+  before :each do
+    @page = pages(:home)
+    @page.template = templates(:sample)
+    @page.stub!(:children).and_return([])
+    login_as :designer
+  end
+  
+  def do_get
+    get :edit, :id => @page.id
+  end
+
+  it "should include a hidden field containing the body/structure" do
+    do_get
+    response.should have_tag('input', :attributes => {'type' => 'hidden', 'name' => 'part[0][name]', 'value' => 'body'})
+    response.should have_tag('input', :attributes => {'type' => 'hidden', 'name' => 'part[0][content]'})
+  end
+  
+  it "should display a template part with style and class attributes" do
+    do_get
+    response.should have_tag('input', :attributes => {'type' => 'hidden', 'name' => 'part[1][name]', 'value' => 'Part 1'})
+    response.should have_tag('textarea', :attributes => {'name' => 'part[0][content]', 'class' => 'wysiwyg', 'style' => 'width: 100%'})
+  end
+  
+  it "should display a textarea template part" do
+    do_get
+    response.should have_tag('input', :attributes => {'type' => 'hidden', 'name' => 'part[1][name]', 'value' => 'extended'})
+    response.should have_tag('textarea', :attributes => {'name' => 'part[1][content]', 'class' => 'plaintext'})
+  end
+  
+  it "should display a radio_button template part" do
+    do_get
+    response.should have_tag('input', :attributes => {'type' => 'hidden', 'name' => 'part[2][name]', 'value' => 'featured?'})
+    response.should have_tag('input', :attributes => {'type' => 'radio_button', 'name' => 'part[2][content]', 'value' => 'true'})
+    response.should have_tag('input', :attributes => {'type' => 'hidden', 'name' => 'part[2][content]', 'value' => 'false'}) 
+  end
+  
+  it "should display a hidden template part" do
+    do_get
+    response.should have_tag('input', :attributes => {'type' => 'hidden', 'name' => 'part[3][name]', 'value' => 'Feature image'})
+    response.should have_tag('input', :attributes => {'type' => 'hidden', 'name' => 'part[3][content]'})
+  end
+  
+  it "should display a text field template part" do
+    do_get
+    response.should have_tag('input', :attributes => {'type' => 'hidden', 'name' => 'part[4][name]', 'value' => 'Tagline'})
+    response.should have_tag('input', :attributes => {'type' => 'text', 'name' => 'part[4][content]'})
+  end
+  
+  it "should override the class name field when the template has one specified" do
+    @page.template = templates(:another)
+    do_get
+    response.should have_tag('input', :attributes => {'name' => 'page[class_name]', 'value' => 'FileNotFoundPage'})
+  end
+end

data/spec/controllers/part_types_controller_spec.rb

@@ -0,0 +1,89 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe Admin::PartTypesController do
+  dataset :users, :part_types
+
+  before :each do
+    login_as :admin
+  end
+
+  it "should be a ResourceController" do
+    controller.should be_kind_of(Admin::ResourceController)
+  end
+
+  it "should handle PartTypes" do
+    controller.class.model_class.should == PartType
+  end
+
+  describe "show" do
+    it "should redirect to the edit action" do
+      get :show, :id => 1
+      response.should redirect_to(edit_admin_part_type_path(params[:id]))
+    end
+
+    it "should show xml when format is xml" do
+      part_type = PartType.first
+      get :show, :id => part_type.id, :format => "xml"
+      response.body.should == part_type.to_xml
+    end
+  end
+
+  describe "with invalid page id" do
+    [:edit, :remove].each do |action|
+      before do
+        @parameters = {:id => 999}
+      end
+      it "should redirect the #{action} action to the index action" do
+        get action, @parameters
+        response.should redirect_to(admin_part_types_path)
+      end
+      it "should say that the 'Part Type could not be found.' after the #{action} action" do
+        get action, @parameters
+        flash[:notice].should == 'Part Type could not be found.'
+      end
+    end
+    it 'should redirect the update action to the index action' do
+      put :update, @parameters
+      response.should redirect_to(admin_part_types_path)
+    end
+    it "should say that the 'Part Type could not be found.' after the update action" do
+      put :update, @parameters
+      flash[:notice].should == 'Part Type could not be found.'
+    end
+    it 'should redirect the destroy action to the index action' do
+      delete :destroy, @parameters
+      response.should redirect_to(admin_part_types_path)
+    end
+    it "should say that the 'Part Type could not be found.' after the destroy action" do
+      delete :destroy, @parameters
+      flash[:notice].should == 'Part Type could not be found.'
+    end
+  end
+
+  { :get => [:index, :new, :edit, :remove],
+    :post => [:create],
+    :put => [:update],
+    :delete => [:destroy] }.each do |method, actions|
+    actions.each do |action|
+      it "should require login to access the #{action} action" do
+        logout
+        lambda { send(method, action).should require_login }
+      end
+
+      it "should allow access to admins for the #{action} action" do
+        lambda {
+          send(method, action, :id => part_type_id(:one_line))
+        }.should restrict_access(:allow => [users(:admin)],
+                                 :url => '/admin/pages')
+      end
+
+      it "should deny non-admins for the #{action} action" do
+        lambda {
+          send(method, action, :id => part_type_id(:plaintext))
+        }.should restrict_access(:deny => [users(:non_admin), users(:existing)],
+                                 :url => '/admin/pages')
+      end
+    end
+  end
+
+end

data/spec/controllers/templates_controller_spec.rb

@@ -0,0 +1,322 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+# Re-raise errors caught by the controller.
+Admin::TemplatesController.class_eval { def rescue_action(e) raise e end }
+
+describe Admin::TemplatesController do
+  dataset :users
+
+  before :each do
+    login_as :designer
+  end
+
+  it "should require login for all actions" do
+    logout
+    lambda { get :index }.should                    require_login
+    lambda { get :new }.should                      require_login
+    lambda { get :edit, :id => 1 }.should           require_login
+    lambda { post :create }.should                  require_login
+    lambda { put :update, :id => 1 }.should         require_login
+    lambda { delete :destroy, :id => 1 }.should     require_login
+    lambda { post :move_higher, :id => 1}.should    require_login
+    lambda { post :move_lower, :id => 1}.should     require_login
+    lambda { post :move_to_top, :id => 1}.should    require_login
+    lambda { post :move_to_bottom, :id => 1}.should require_login
+  end
+
+  describe "Access control" do
+    before :each do
+      @template = mock_model(Template, :save => true, :update_attributes => true,
+                                  :destroy => true, :move_to_top => true, :move_to_bottom => true,
+                                  :move_higher => true, :move_lower => true)
+      Template.stub!(:find).and_return(@template)
+      Template.stub!(:new).and_return(@template)
+    end
+    
+    [:admin, :designer].each do |user|
+      describe "#{user} user" do
+        before :each do
+          login_as user
+        end
+
+        def redirects_to_index
+          response.should be_redirect
+          response.should redirect_to(admin_templates_path)
+        end
+
+        it 'should have access to the index action' do
+          get :index
+          response.should be_success
+        end
+
+        it 'should have access to the new action' do
+          get :new
+          response.should be_success
+        end
+
+        it 'should have access to the create action' do
+          post :create
+          redirects_to_index
+        end
+
+        it 'should have access to the edit action' do
+          get :edit, :id => 1
+          response.should be_success
+        end
+
+        it "show should redirect to the edit action" do
+          get :show, :id => 1
+          response.should redirect_to(edit_admin_template_path(params[:id]))
+        end
+
+        it 'should have access to the update action' do
+          put :update, :id => 1
+          redirects_to_index
+        end
+
+        it 'should have access to the destroy action' do
+          delete :destroy, :id => 1
+          redirects_to_index
+        end
+
+        [:move_higher, :move_lower, :move_to_top, :move_to_bottom].each do |action|
+          it "should have access to the #{action} action" do
+            post action, :id => 1
+            redirects_to_index
+          end
+        end
+      end
+    end
+
+    [:existing, :non_admin].each do |user|
+      describe "#{user} user" do
+        before :each do
+          login_as user
+        end
+
+        def redirects_to_pages
+          response.should be_redirect
+          response.should redirect_to(admin_pages_path)
+          flash[:error].should == 'You must have designer privileges to perform this action.'
+        end
+
+        it 'should not have access to the index action' do
+          get :index
+        end
+
+        it 'should not have access to the new action' do
+          get :new
+        end
+
+        it 'should not have access to the create action' do
+          post :create
+        end
+
+        it 'should not have access to the edit action' do
+          get :edit, :id => 1
+        end
+
+        it 'should not have access to the update action' do
+          put :update, :id => 1
+        end
+
+        it 'should not have access to the destroy action' do
+          delete :destroy, :id => 1
+        end
+
+        [:move_higher, :move_lower, :move_to_top, :move_to_bottom].each do |action|
+          it "should not have access to the #{action} action" do
+            post action, :id => 1
+          end
+        end
+
+        after :each do
+          redirects_to_pages
+        end
+      end
+    end
+  end
+  
+  describe "GET to /admin/templates" do
+    before :each do
+      @template = mock_model(Template)
+      Template.should_receive(:find).with(:all).and_return([@template])
+    end
+    def do_get
+      get :index
+    end
+    
+    it "should load all templates" do
+      do_get
+      assigns[:content_templates].should == [@template]
+    end
+    
+    it "should render the index template" do
+      do_get
+      response.should render_template('index')
+    end
+  end
+  
+  describe "GET to /admin/templates/new" do
+    def do_get
+      get :new
+    end
+    
+    it "should load a new template" do
+      do_get
+      assigns[:content_template].should_not be_nil
+      assigns[:content_template].should be_new_record
+    end
+    
+    it "should render the new template" do
+      do_get
+      response.should render_template('new')
+    end
+  end
+  
+  describe "GET to /admin/templates/1/edit" do
+    before :each do
+      @template = mock_model(Template)
+      Template.should_receive(:find).with('1').and_return(@template)
+    end
+    
+    def do_get
+      get :edit, :id => '1'
+    end
+    
+    it "should load the existing template" do
+      do_get
+      assigns[:content_template].should == @template
+    end
+    
+    it "should render the edit template" do
+      do_get
+      response.should render_template('edit')
+    end
+  end
+  
+  describe "POST to /admin/templates" do
+    before :each do
+      @template = mock_model(Template)
+      Template.should_receive(:new).and_return(@template)
+    end
+    
+    def do_post
+      post :create
+    end
+    
+    describe "success path" do
+      before :each do
+        @template.should_receive(:save).and_return(true)
+      end
+      
+      it "should redirect to the index" do
+        do_post
+        response.should redirect_to('/admin/templates')
+      end
+    end
+    
+    describe "failure path" do
+      before :each do
+        @template.should_receive(:save).and_return(false)
+      end
+      
+      it "should add a flash error message" do
+        do_post
+        flash[:error].should be
+      end
+      
+      it "should render the new template again" do
+        do_post
+        response.should render_template('new')
+      end
+    end
+  end
+  
+  describe "PUT to /admin/templates/1" do
+    before :each do
+      @template = mock_model(Template)
+      Template.should_receive(:find).with('1').and_return(@template)
+    end
+    
+    def do_put
+      put :update, :id => '1'
+    end
+    
+    describe "success path" do
+      before :each do
+        @template.should_receive(:update_attributes).and_return(true)
+      end
+      
+      it "should redirect to the index" do
+        do_put
+        response.should redirect_to('/admin/templates')
+      end
+    end
+    
+    describe "failure path" do
+      before :each do
+        @template.should_receive(:update_attributes).and_return(false)
+      end
+      
+      it "should add a flash error message" do
+        do_put
+        flash[:error].should be
+      end
+      
+      it "should render the edit template again" do
+        do_put
+        response.should render_template('edit')
+      end
+    end
+  end
+  
+  describe "DELETE to /admin/templates/1" do
+    before :each do
+      @template = mock_model(Template, :destroy => true)
+      Template.should_receive(:find).with('1').and_return(@template)
+    end
+    
+    def do_delete
+      delete :destroy, :id => '1'
+    end
+    
+    it "should destroy the template" do
+      @template.should_receive(:destroy).and_return(true)
+      do_delete
+    end
+    
+    it "should redirect to the index" do
+      do_delete
+      response.should redirect_to('/admin/templates')
+    end
+  end
+  
+  [:move_to_top, :move_to_bottom, :move_higher, :move_lower].each do |action|
+    describe "POST to /admin/templates/1/#{action}" do
+      before :each do
+        @template = mock_model(Template, action => true)
+        Template.should_receive(:find).with('1').and_return(@template)
+      end
+      
+      define_method :do_post do
+        post action, :id => '1'
+      end
+      
+      it "should call #{action} on the template" do
+        @template.should_receive(action).and_return(true)
+        do_post
+      end
+      
+      it "should redirect back to the index" do
+        do_post
+        response.should redirect_to('/admin/templates')
+      end
+      
+      it "should register that templates are being reordered" do
+        Template.should_receive(:reordering)
+        do_post
+      end
+    end
+  end
+end