Download as
Browse Code »

Commit 9cd6c80322778bf38a3873fd76e6707a8667cbad

Authored by AntonioTerceiro
1 parent cab263df
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

ActionItem519: comment form must post to article URL 

git-svn-id: https://svn.colivre.coop.br/svn/noosfero/trunk@2324 3f533792-8f58-4932-b0fe-aaf55b0a4547
Showing 2 changed files with 10 additions and 1 deletions   Show diff stats
app/views/content_viewer/_comment_form.rhtml
  Diff comments   View file @9cd6c80
@@ -17,7 +17,7 @@ @@ -17,7 +17,7 @@
17 17
18 <h4><%= _('Post a comment') %></h4> 18 <h4><%= _('Post a comment') %></h4>
19 19
20 -<% form_tag( {}, { :id => comment_form_id } ) do %> 20 +<% form_tag( @page.url, { :id => comment_form_id } ) do %>
21 21
22 <% unless logged_in? %> 22 <% unless logged_in? %>
23 23
test/functional/content_viewer_controller_test.rb
  Diff comments   View file @9cd6c80
@@ -226,6 +226,15 @@ class ContentViewerControllerTest &lt; Test::Unit::TestCase @@ -226,6 +226,15 @@ class ContentViewerControllerTest &lt; Test::Unit::TestCase
226 assert_no_tag :tag => 'strong', :attributes => { :id => 'html_test_comment' } 226 assert_no_tag :tag => 'strong', :attributes => { :id => 'html_test_comment' }
227 end 227 end
228 228
  229 + should "point to article's url in comment form" do
  230 + page = profile.articles.create!(:name => 'myarticle', :body => 'the body of the text')
  231 + Article.any_instance.stubs(:url).returns('http://www.mysite.com/person/article')
  232 +
  233 + get :view_page, :profile => profile.identifier, :page => [ 'myarticle' ]
  234 +
  235 + assert_tag :tag => 'form', :attributes => { :id => /^comment_form/, :action => 'http://www.mysite.com/person/article' }
  236 + end
  237 +
229 should "display current article's tags" do 238 should "display current article's tags" do
230 page = profile.articles.create!(:name => 'myarticle', :body => 'test article', :tag_list => 'tag1, tag2') 239 page = profile.articles.create!(:name => 'myarticle', :body => 'test article', :tag_list => 'tag1, tag2')
231 240