Download as
Browse Code »

Commit 38212a9ce6c4aaaddfa075b9cfceda0d0f1c2e36

Authored by Victor Costa
Committed by Rodrigo Souto
1 parent 2e32b7c4
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

api: update hits for article children

Showing 3 changed files with 12 additions and 0 deletions   Show diff stats
lib/noosfero/api/entities.rb
  Diff comments   View file @38212a9
... ... @@ -68,6 +68,7 @@ module Noosfero
68 68 expose :votes_against
69 69 expose :setting
70 70 expose :position
  71 + expose :hits
71 72 end
72 73  
73 74 class Article < ArticleBase
... ...
lib/noosfero/api/v1/articles.rb
  Diff comments   View file @38212a9
... ... @@ -40,6 +40,7 @@ module Noosfero
40 40 articles = articles.joins('left join votes on articles.id=votes.voteable_id').group('articles.id').reorder('sum(coalesce(votes.vote, 0)) DESC')
41 41 end
42 42  
  43 + Article.hit(articles)
43 44 present articles, :with => Entities::Article, :fields => params[:fields]
44 45 end
45 46  
... ...
test/unit/api/articles_test.rb
  Diff comments   View file @38212a9
... ... @@ -473,4 +473,14 @@ class ArticlesTest &lt; ActiveSupport::TestCase
473 473 assert_equal 'SuggestArticle', json['type']
474 474 end
475 475  
  476 + should 'update hit attribute of article children' do
  477 + a1 = fast_create(Article, :profile_id => user.person.id)
  478 + a2 = fast_create(Article, :parent_id => a1.id, :profile_id => user.person.id)
  479 + a3 = fast_create(Article, :parent_id => a1.id, :profile_id => user.person.id)
  480 + get "/api/v1/articles/#{a1.id}/children?#{params.to_query}"
  481 + json = JSON.parse(last_response.body)
  482 + assert_equal [1, 1], json['articles'].map { |a| a['hits']}
  483 + assert_equal [0, 1, 1], [a1.reload.hits, a2.reload.hits, a3.reload.hits]
  484 + end
  485 +
476 486 end
... ...