Don't cache when there's no pages

refinery · Jun 22, 2010 · 26b92c1 · 26b92c1
1 parent aefb7c7
commit 26b92c1
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/vendor/plugins/refinery/lib/refinery/application_controller.rb b/vendor/plugins/refinery/lib/refinery/application_controller.rb
@@ -61,8 +61,10 @@ def login?
 
   # get all the pages to be displayed in the site menu.
   def find_pages_for_menu
-    @menu_pages = Rails.cache.read(cache_key = "#{Refinery.base_cache_key}_menu_pages")
-    @menu_pages = Rails.cache.write(cache_key, Page.top_level(include_children=true)) if @menu_pages.nil?
+    if (@menu_pages = Rails.cache.read(cache_key = "#{Refinery.base_cache_key}_menu_pages")).nil?
+      @menu_pages = Page.top_level(include_children = true)
+      Rails.cache.write(cache_key, @menu_pages) if @menu_pages.present?
+    end
   end
 
   # use a different model for the meta information.