not break shit

maximebedard · Mar 26, 2017 · 2719d23 · 2719d23
1 parent 733ff5d
commit 2719d23
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 7 deletions.
diff --git a/app/controllers/videos_controller.rb b/app/controllers/videos_controller.rb
@@ -6,7 +6,7 @@ def index
 
   def show
     @video = Video.find(params[:id])
-    @feedback = Ordinateur.redis.hget("likes:#{current_user.id}")
+    @video.increment_view_count(current_user)
   end
 
   def new
@@ -20,7 +20,6 @@ def create
   end
 
   def like
-    Ordinateur.redis.hset("likes:#{current_user.id}", !!params[:feedback])
   end
 
   def update

diff --git a/app/models/video.rb b/app/models/video.rb
@@ -7,8 +7,11 @@ def public_url
     ""
   end
 
-  def views_count
-    # shitty but works
-    Ordinateur.redis.increment("views:#{@video.id}:#{current_user.id}")
+  def view_count
+    @view_count ||= Ordinateur.redis.hgetall("views:#{id}").sum
+  end
+
+  def increment_view_count(current_user)
+    Ordinateur.redis.incr("views:#{id}:#{current_user.id}")
   end
 end
diff --git a/app/views/videos/show.html.erb b/app/views/videos/show.html.erb
@@ -7,8 +7,7 @@
   <%= link_to nil, "👍" %>
   <%= link_to nil, "👎" %>
 
-  Views: <%= @video.views_count %>
+  Views: <%= @video.view_count %>
 
   <%= render(@video.comments) %>
-  <%= render "add_comment", comment: Comment.new %>
 </div>