From 1a35b393b16b0ea943f2b2c9c8ba58a4839addfa Mon Sep 17 00:00:00 2001
From: AntonioTerceiro <AntonioTerceiro@3f533792-8f58-4932-b0fe-aaf55b0a4547>
Date: Fri, 28 Dec 2007 17:29:07 +0000
Subject: [PATCH] ActionItem24: not offering to create a child of a no-child article

---
 app/views/cms/view.rhtml               |  4 +++-
 test/functional/cms_controller_test.rb | 26 ++++++++++++++++++++++++++
 2 files changed, 29 insertions(+), 1 deletion(-)

diff --git a/app/views/cms/view.rhtml b/app/views/cms/view.rhtml
index 91b16e2..eb7f6a4 100644
--- a/app/views/cms/view.rhtml
+++ b/app/views/cms/view.rhtml
@@ -36,7 +36,9 @@
 
     <% if @article %>
       <% button_bar(:class => 'file-manager-controls') do %>
-        <%= button('new',  _('New subitem'), :action => 'new', :parent_id => @article.id) %>
+        <% if @article.allow_children? %>
+          <%= button('new',  _('New subitem'), :action => 'new', :parent_id => @article.id) %>
+        <% end %>
         <%= button('up', _('Go up one level'), :action => (@article.parent ? 'view' : 'index'), :id => @article.parent) %>
       <% end %>
     <% end %>
diff --git a/test/functional/cms_controller_test.rb b/test/functional/cms_controller_test.rb
index 90f33ff..715a0fb 100644
--- a/test/functional/cms_controller_test.rb
+++ b/test/functional/cms_controller_test.rb
@@ -172,7 +172,33 @@ class CmsControllerTest < Test::Unit::TestCase
     flunk 'pending'
   end
 
+  should 'offer to create children' do
+    Article.any_instance.stubs(:allow_children?).returns(true)
+
+    article = Article.new(:name => 'test')
+    article.profile = profile
+    article.save!
+
+    get :view, :profile => profile.identifier, :id => article.id
+    assert_response :success
+    assert_template 'view'
+    assert_tag :tag => 'a', :attributes => { :href => "/myprofile/#{profile.identifier}/cms/new?parent_id=#{article.id}"}
+  end
+
   should 'not offer to create children if article does not accept them' do
+    Article.any_instance.stubs(:allow_children?).returns(false)
+
+    article = Article.new(:name => 'test')
+    article.profile = profile
+    article.save!
+
+    get :view, :profile => profile.identifier, :id => article.id
+    assert_response :success
+    assert_template 'view'
+    assert_no_tag :tag => 'a', :attributes => { :href => "/myprofile/#{profile.identifier}/cms/new?parent_id=#{article.id}"}
+  end
+
+  should 'not allow to create children of non-child articles' do
     flunk 'pending'
   end
 
--
libgit2 0.21.2