consul/nairobi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #385 from AyuntamientoMadrid/author-cache

Browse Source 
bugfix: adds cache for authors of debate/comment
This commit is contained in:
Raimond Garcia
2015-09-06 22:37:49 +02:00
parent 4ec93b9ed9 bf18da3faa
commit 5bc4f048da
3 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
app/helpers/cache_keys_helper.rb
View File

@@ -1,11 +1,11 @@
module CacheKeysHelper module CacheKeysHelper
def locale_and_user_status def locale_and_user_status(authorable=nil)
@cache_key_user ||= calculate_user_status @cache_key_user ||= calculate_user_status(authorable)
"#{I18n.locale}/#{@cache_key_user}" "#{I18n.locale}/#{@cache_key_user}"
end end
def calculate_user_status def calculate_user_status(authorable=nil)
user_status = "user" user_status = "user"
if user_signed_in? if user_signed_in?
@@ -14,6 +14,7 @@ module CacheKeysHelper
user_status += ":org" if current_user.organization? user_status += ":org" if current_user.organization?
user_status += ":admin" if current_user.administrator? user_status += ":admin" if current_user.administrator?
user_status += ":moderator" if current_user.moderator? user_status += ":moderator" if current_user.moderator?
user_status += ":author" if authorable && authorable.author == current_user
else else
user_status += ":visitor" user_status += ":visitor"
end end

2
app/views/comments/_comment.html.erb
View File

@@ -1,4 +1,4 @@
<% cache [locale_and_user_status, comment, @commentable, comment.author, (@comment_flags[comment.id] if @comment_flags)] do %> <% cache [locale_and_user_status(comment), comment, @commentable, comment.author, (@comment_flags[comment.id] if @comment_flags)] do %>
<div class="row"> <div class="row">
<div id="<%= dom_id(comment) %>" class="comment small-12 column"> <div id="<%= dom_id(comment) %>" class="comment small-12 column">

2
app/views/debates/show.html.erb
View File

@@ -1,4 +1,4 @@
<% cache [locale_and_user_status, @debate, @debate.author, Flag.flagged?(current_user, @debate)] do %> <% cache [locale_and_user_status(@debate), @debate, @debate.author, Flag.flagged?(current_user, @debate)] do %>
<section class="debate-show"> <section class="debate-show">
<div id="<%= dom_id(@debate) %>" class="row"> <div id="<%= dom_id(@debate) %>" class="row">
<div class="small-12 medium-9 column"> <div class="small-12 medium-9 column">