api: multiple articles hits

Victor Costa · Rodrigo Souto
1 parent 16d23214
Showing 2 changed files with 15 additions and 0 deletions Show diff stats
app/models/article.rb
test/unit/article_test.rb
@@ -634,6 +634,11 @@ class Article &lt; ActiveRecord::Base
     self.hits += 1
   end
  
+  def self.hit(articles)
+    Article.where(:id => articles.map(&:id)).update_all('hits = hits + 1')
+    articles.each { |a| a.hits += 1 }
+  end
+
   def can_display_hits?
     true
   end
@@ -2154,4 +2154,14 @@ class ArticleTest &lt; ActiveSupport::TestCase
     assert_equivalent [a1,a2], Article.display_filter(nil, user)
   end
  
+  should 'update hit attribute of article array' do
+    a1 = fast_create(Article)
+    a2 = fast_create(Article)
+    a3 = fast_create(Article)
+    Article.hit([a1, a2, a3])
+    Article.hit([a2, a3])
+    assert_equal [1, 2, 2], [a1.hits, a2.hits, a3.hits]
+    assert_equal [1, 2, 2], [a1.reload.hits, a2.reload.hits, a3.reload.hits]
+  end
+
 end
...	...	@@ -634,6 +634,11 @@ class Article < ActiveRecord::Base
634	634	self.hits += 1
635	635	end
636	636
	637	+ def self.hit(articles)
	638	+ Article.where(:id => articles.map(&:id)).update_all('hits = hits + 1')
	639	+ articles.each { \|a\| a.hits += 1 }
	640	+ end
	641	+
637	642	def can_display_hits?
638	643	true
639	644	end
...	...
...	...	@@ -2154,4 +2154,14 @@ class ArticleTest < ActiveSupport::TestCase
2154	2154	assert_equivalent [a1,a2], Article.display_filter(nil, user)
2155	2155	end
2156	2156
	2157	+ should 'update hit attribute of article array' do
	2158	+ a1 = fast_create(Article)
	2159	+ a2 = fast_create(Article)
	2160	+ a3 = fast_create(Article)
	2161	+ Article.hit([a1, a2, a3])
	2162	+ Article.hit([a2, a3])
	2163	+ assert_equal [1, 2, 2], [a1.hits, a2.hits, a3.hits]
	2164	+ assert_equal [1, 2, 2], [a1.reload.hits, a2.reload.hits, a3.reload.hits]
	2165	+ end
	2166	+
2157	2167	end
...	...