blog: add cover image for blog

(ActionItem2849) Signed-off-by: Marcos Ramos <ms.ramos@outlook.com Signed-off-by: Valéssio Brito <contato@valessiobrito.com.br> Signed-off-by: Victor Hugo Alves de Carvalho <victorhugodf.ac@gmail.com>

blog: add cover image for blog
(ActionItem2849) Signed-off-by: Marcos Ramos <ms.ramos@outlook.com Signed-off-by: Valéssio Brito <contato@valessiobrito.com.br> Signed-off-by: Victor Hugo Alves de Carvalho <victorhugodf.ac@gmail.com>
Marcos Ramos
1 parent 15717094
Showing 8 changed files with 97 additions and 0 deletions Show diff stats
app/controllers/my_profile/cms_controller.rb
app/views/cms/_blog.rhtml
app/views/content_viewer/_article_toolbar.rhtml
features/blog.feature
features/step_definitions/web_steps.rb
public/stylesheets/application.css
test/functional/cms_controller_test.rb
test/unit/blog_test.rb
@@ -73,6 +73,7 @@ class CmsController &lt; MyProfileController
     refuse_blocks
     record_coming
     if request.post?
+      @article.image = nil if params[:remove_image] == 'true'
       @article.last_changed_by = user
       if @article.update_attributes(params[:article])
         if !continue
@@ -54,6 +54,15 @@
 <%= labelled_form_field(_('Description:'), text_area(:article, :body, :rows => 10)) %>
+<% f.fields_for :image_builder, @article.image do |i| %>
+  <%= file_field_or_thumbnail(_('Cover image:'), @article.image, i)%>
+  <%= _("Max size: %s (.jpg, .gif, .png)")% Image.max_size.to_humanreadable %>
+<% end %>
+
+<% unless @article.image.nil? %>
+  <%= labelled_check_box(_('Remove cover image'),'remove_image',true,false)%>
+<% end %>
+
 <%= labelled_form_field(_('How to display posts:'), f.select(:visualization_format, [ [ _('Full post'), 'full'], [ _('First paragraph'), 'short'] ])) %>
 <%= labelled_form_field(_('Posts per page:'), f.select(:posts_per_page, Blog.posts_per_page_options)) %>
@@ -52,6 +52,9 @@
   </div>
   <div id="article-header">
+    <% if @page.blog? and !@page.image.nil? %>
+      <div class="blog-cover"><%= image_tag(@page.image.public_filename())%></div>
+    <% end %>
     <%= link_to(image_tag('icons-mime/rss-feed.png'), @page.feed.url, :class => 'blog-feed-link') if @page.has_posts? && @page.feed %>
     <%= article_title(@page, :no_link => true) %>
     <%= article_translations(@page) %>
@@ -15,6 +15,7 @@ Feature: blog
     And I follow "Create blog"
     Then I should see "My Blog"
     When I fill in "Title" with "My Blog"
+    And I fill in "Address" with "my-blog"
     And I press "Save"
     And I go to joaosilva's control panel
     Then I should see "Configure blog"
@@ -24,6 +25,7 @@ Feature: blog
     And I follow "Create blog"
     Then I should see "My Blog"
     When I fill in "Title" with "My Blog"
+    And I fill in "Address" with "my-blog"
     And I press "Save"
     Then I should be on /joaosilva/my-blog
@@ -33,6 +35,7 @@ Feature: blog
     And I follow "New content"
     When I follow "Blog"
     And I fill in "Title" with "Blog from cms"
+    And I fill in "Address" with "blog-from-cms"
     And I press "Save"
     Then I should be on /joaosilva/blog-from-cms
@@ -42,12 +45,14 @@ Feature: blog
     And I follow "New content"
     And I follow "Blog"
     And I fill in "Title" with "Blog One"
+    And I fill in "Address" with "blog-one"
     And I press "Save"
     Then I go to joaosilva's control panel
     And I follow "Manage Content"
     And I follow "New content"
     And I follow "Blog"
     And I fill in "Title" with "Blog Two"
+    And I fill in "Address" with "blog-two"
     And I press "Save"
     Then I should not see "error"
     And I should be on /joaosilva/blog-two
@@ -109,3 +114,24 @@ Feature: blog
     And I follow "New content"
     When I follow "Blog"
     Then I should see "Tag list"
+
+  Scenario: do not display the "clear cover image" when there is no uploaded image
+    Given the following blogs
+      | owner     | name    |
+      | joaosilva | My Blog |
+    And I go to joaosilva's control panel
+    And I follow "Configure blog"
+    Then I should not see "Delete cover image"
+
+  # the step for attaching a file on the input only works with capybara 1.1.2, but it requires rails 1.9.3
+  @selenium-fixme
+  Scenario: display cover image after uploading an image as the blog cover
+    Given the following blogs
+      | owner     | name    |
+      | joaosilva | My Blog |
+    And I go to joaosilva's control panel
+    And I follow "Configure blog"
+    And I attach the file "public/images/rails.png" to "Uploaded data"
+    And I press "Save"
+    When I am on /joaosilva/my-blog
+    Then there should be a div with class "blog-cover"
@@ -243,3 +243,7 @@ Then /^display &quot;([^\&quot;]*)&quot;$/ do |element|
   #   * https://github.com/jnicklas/capybara/issues/76
   evaluate_script("jQuery('#{element}').show() && false;")
 end
+
+Then /^there should be a div with class "([^"]*)"$/ do |klass|
+  should have_selector('div', :class => klass)
+end
@@ -932,6 +932,18 @@ code input {
 #article-actions a{
   white-space: nowrap;
 }
+
+#article-header .blog-cover {
+  overflow: hidden;
+  display: block;
+  width: 100%;
+  max-height: 170px;
+}
+
+#article-header .blog-cover img{
+  width: 100%;
+}
+
 #article-tags {
   font-size: 10px;
   text-align: right;
@@ -6441,3 +6453,4 @@ li.profile-activity-item.upload_image .activity-gallery-images-count-1 img {
   right: 2px;
   z-index: 1;
 }
+
@@ -900,6 +900,15 @@ class CmsControllerTest &lt; ActionController::TestCase
     assert_no_tag :tag => 'a', :attributes => { :href => "/myprofile/#{profile.identifier}/cms/edit/#{profile.blog.feed.id}" }
   end
+  should 'remove the image of an article' do
+    blog = Blog.create(:profile_id => profile.id, :name=>'testblog', :image_builder => { :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png')})
+    blog.save!
+    post :edit, :profile => profile.identifier, :id => blog.id, :remove_image => 'true'
+    blog.reload
+
+    assert_nil blog.image
+  end
+
   should 'update feed options by edit blog form' do
     profile.articles << Blog.new(:name => 'Blog for test', :profile => profile)
     post :edit, :profile => profile.identifier, :id => profile.blog.id, :article => { :feed => { :limit => 7 } }
@@ -218,4 +218,36 @@ class BlogTest &lt; ActiveSupport::TestCase
     assert ! blog.empty?
   end
+  should 'set cover image' do
+    profile = fast_create(Profile)
+    blog = Blog.create(:profile_id => profile.id, :name=>'testblog', :image_builder => { :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png')})
+    blog.save!
+    blog.reload
+    assert_equal blog.image(true).filename, 'rails.png'
+  end
+
+  should 'remove cover image' do
+    profile = fast_create(Profile)
+    blog = Blog.create(:profile_id => profile.id, :name=>'testblog', :image_builder => { :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png')})
+    blog.save!
+    blog.reload
+
+    blog.image = nil
+    blog.save!
+    blog.reload
+    assert blog.image.nil?
+  end
+
+  should 'update cover image' do
+    profile = fast_create(Profile)
+    blog = Blog.create(:profile_id => profile.id, :name=>'testblog', :image_builder => { :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png')})
+    blog.save!
+    blog.reload
+
+    blog.image = Image.create!(:uploaded_data => fixture_file_upload('/files/noosfero-network.png', 'image/png'))
+    blog.save!
+    blog.reload
+
+    assert_equal blog.image(true).filename, 'noosfero-network.png'
+  end
 end
	@@ -54,6 +54,15 @@		@@ -54,6 +54,15 @@
54		54
55	<%= labelled_form_field(_('Description:'), text_area(:article, :body, :rows => 10)) %>	55	<%= labelled_form_field(_('Description:'), text_area(:article, :body, :rows => 10)) %>
56		56
		57	+<% f.fields_for :image_builder, @article.image do \|i\| %>
		58	+ <%= file_field_or_thumbnail(_('Cover image:'), @article.image, i)%>
		59	+ <%= _("Max size: %s (.jpg, .gif, .png)")% Image.max_size.to_humanreadable %>
		60	+<% end %>
		61	+
		62	+<% unless @article.image.nil? %>
		63	+ <%= labelled_check_box(_('Remove cover image'),'remove_image',true,false)%>
		64	+<% end %>
		65	+
57	<%= labelled_form_field(_('How to display posts:'), f.select(:visualization_format, [ [ _('Full post'), 'full'], [ _('First paragraph'), 'short'] ])) %>	66	<%= labelled_form_field(_('How to display posts:'), f.select(:visualization_format, [ [ _('Full post'), 'full'], [ _('First paragraph'), 'short'] ])) %>
58		67
59	<%= labelled_form_field(_('Posts per page:'), f.select(:posts_per_page, Blog.posts_per_page_options)) %>	68	<%= labelled_form_field(_('Posts per page:'), f.select(:posts_per_page, Blog.posts_per_page_options)) %>
	@@ -243,3 +243,7 @@ Then /^display "([^\"]*)"$/ do \|element\|		@@ -243,3 +243,7 @@ Then /^display "([^\"]*)"$/ do \|element\|
243	# * https://github.com/jnicklas/capybara/issues/76	243	# * https://github.com/jnicklas/capybara/issues/76
244	evaluate_script("jQuery('#{element}').show() && false;")	244	evaluate_script("jQuery('#{element}').show() && false;")
245	end	245	end
		246	+
		247	+Then /^there should be a div with class "([^"]*)"$/ do \|klass\|
		248	+ should have_selector('div', :class => klass)
		249	+end
	@@ -932,6 +932,18 @@ code input {		@@ -932,6 +932,18 @@ code input {
932	#article-actions a{	932	#article-actions a{
933	white-space: nowrap;	933	white-space: nowrap;
934	}	934	}
		935	+
		936	+#article-header .blog-cover {
		937	+ overflow: hidden;
		938	+ display: block;
		939	+ width: 100%;
		940	+ max-height: 170px;
		941	+}
		942	+
		943	+#article-header .blog-cover img{
		944	+ width: 100%;
		945	+}
		946	+
935	#article-tags {	947	#article-tags {
936	font-size: 10px;	948	font-size: 10px;
937	text-align: right;	949	text-align: right;
	@@ -6441,3 +6453,4 @@ li.profile-activity-item.upload_image .activity-gallery-images-count-1 img {		@@ -6441,3 +6453,4 @@ li.profile-activity-item.upload_image .activity-gallery-images-count-1 img {
6441	right: 2px;	6453	right: 2px;
6442	z-index: 1;	6454	z-index: 1;
6443	}	6455	}
		6456	+
	@@ -900,6 +900,15 @@ class CmsControllerTest < ActionController::TestCase		@@ -900,6 +900,15 @@ class CmsControllerTest < ActionController::TestCase
900	assert_no_tag :tag => 'a', :attributes => { :href => "/myprofile/#{profile.identifier}/cms/edit/#{profile.blog.feed.id}" }	900	assert_no_tag :tag => 'a', :attributes => { :href => "/myprofile/#{profile.identifier}/cms/edit/#{profile.blog.feed.id}" }
901	end	901	end
902		902
		903	+ should 'remove the image of an article' do
		904	+ blog = Blog.create(:profile_id => profile.id, :name=>'testblog', :image_builder => { :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png')})
		905	+ blog.save!
		906	+ post :edit, :profile => profile.identifier, :id => blog.id, :remove_image => 'true'
		907	+ blog.reload
		908	+
		909	+ assert_nil blog.image
		910	+ end
		911	+
903	should 'update feed options by edit blog form' do	912	should 'update feed options by edit blog form' do
904	profile.articles << Blog.new(:name => 'Blog for test', :profile => profile)	913	profile.articles << Blog.new(:name => 'Blog for test', :profile => profile)
905	post :edit, :profile => profile.identifier, :id => profile.blog.id, :article => { :feed => { :limit => 7 } }	914	post :edit, :profile => profile.identifier, :id => profile.blog.id, :article => { :feed => { :limit => 7 } }
	@@ -218,4 +218,36 @@ class BlogTest < ActiveSupport::TestCase		@@ -218,4 +218,36 @@ class BlogTest < ActiveSupport::TestCase
218	assert ! blog.empty?	218	assert ! blog.empty?
219	end	219	end
220		220
		221	+ should 'set cover image' do
		222	+ profile = fast_create(Profile)
		223	+ blog = Blog.create(:profile_id => profile.id, :name=>'testblog', :image_builder => { :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png')})
		224	+ blog.save!
		225	+ blog.reload
		226	+ assert_equal blog.image(true).filename, 'rails.png'
		227	+ end
		228	+
		229	+ should 'remove cover image' do
		230	+ profile = fast_create(Profile)
		231	+ blog = Blog.create(:profile_id => profile.id, :name=>'testblog', :image_builder => { :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png')})
		232	+ blog.save!
		233	+ blog.reload
		234	+
		235	+ blog.image = nil
		236	+ blog.save!
		237	+ blog.reload
		238	+ assert blog.image.nil?
		239	+ end
		240	+
		241	+ should 'update cover image' do
		242	+ profile = fast_create(Profile)
		243	+ blog = Blog.create(:profile_id => profile.id, :name=>'testblog', :image_builder => { :uploaded_data => fixture_file_upload('/files/rails.png', 'image/png')})
		244	+ blog.save!
		245	+ blog.reload
		246	+
		247	+ blog.image = Image.create!(:uploaded_data => fixture_file_upload('/files/noosfero-network.png', 'image/png'))
		248	+ blog.save!
		249	+ blog.reload
		250	+
		251	+ assert_equal blog.image(true).filename, 'noosfero-network.png'
		252	+ end
221	end	253	end