Commit 7f1e33c2

mo khan <mo@mokhan.ca>
2014-10-20 03:10:56
move caching out of view and to app controller.
master
1 parent b639f50
Changed files (2)
app
controllers
application_controller.rb
views
layouts
_header.html.erb
app/controllers/application_controller.rb
@@ -23,7 +23,7 @@ class ApplicationController < ActionController::Base
   private
 
   def load_header
-    @categories = Category.all
+    @categories = Rails.cache.fetch(:categories) { Category.all }
   end
 
   def authenticate!
app/views/layouts/_header.html.erb
@@ -14,10 +14,8 @@
               <%= text_field_tag :q, params[:q], class: 'search-query', placeholder: 'Search' %>
             <% end %>
           </li>
-          <% cache [@categories, params[:q]].compact do %>
-            <% @categories.each do |category| %>
-              <li class="<%= "active" if params["category"] == category.slug %>"><%= link_to category.name.pluralize, category_path(category, q: params[:q]) %></li>
-            <% end %>
+          <% @categories.each do |category| %>
+            <li class="<%= "active" if params["category"] == category.slug %>"><%= link_to category.name.pluralize, category_path(category, q: params[:q]) %></li>
           <% end %>
           <li class="<%= "active" if params["controller"] == "profiles" %>"> <%= link_to "Artists", profiles_path(q: params[:q]) %> </li>
           <li class="<%= "active" if params["controller"] == "tutorials" %>"> <%= link_to "Tutorials", tutorials_path(q: params[:q]) %> </li>