only call cache_sweeper on certain methods, use alias_method on callb…

…ack methods
keavy · Jun 30, 2011 · 512d4a1 · 512d4a1
1 parent bb08948
commit 512d4a1
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 22 deletions.
diff --git a/app/controllers/admin/events_controller.rb b/app/controllers/admin/events_controller.rb
@@ -1,7 +1,7 @@
 class Admin::EventsController < AdminAreaController
   inherit_resources
 
-  cache_sweeper :events_sweeper
+  cache_sweeper :events_sweeper, :only => [:create, :update, :destroy]
 
   def edit
     resource.admin_submitted = true

diff --git a/app/controllers/events_controller.rb b/app/controllers/events_controller.rb
@@ -3,7 +3,7 @@ class EventsController < ApplicationController
   before_filter :authenticate, :only => [:edit, :update] unless Rails.env == 'development'
   before_filter :check_authorized, :only => [:edit, :update] unless Rails.env == 'development'
 
-  cache_sweeper :events_sweeper
+  cache_sweeper :events_sweeper, :only => [:create, :update, :destroy]
 
   layout :select_layout
 

diff --git a/app/controllers/happenings_controller.rb b/app/controllers/happenings_controller.rb
@@ -5,7 +5,7 @@ class HappeningsController < ApplicationController
   before_filter :authenticate unless Rails.env == 'development'
   before_filter :check_authorized unless Rails.env == 'development'
 
-  cache_sweeper :events_sweeper
+  cache_sweeper :events_sweeper, :only => [:create, :update, :destroy]
 
   def index
     if @event.happenings.blank?

diff --git a/app/sweepers/events_sweeper.rb b/app/sweepers/events_sweeper.rb
@@ -1,23 +1,13 @@
 class EventsSweeper < ActionController::Caching::Sweeper
-  observe Event
-  observe Happening
+  observe Event, Happening
 
-  def after_create(record)
-    expire_cache(record)
+  def expire_cached_content(record)
+    expire_fragment(:fragment => 'events/attend')
+    expire_fragment(:fragment => 'events/speak')
+    expire_fragment(:fragment => 'events/archive')
   end
 
-  def after_update(record)
-    expire_cache(record)
-  end
-
-  def after_destroy(record)
-    expire_cache(record)
-  end
-
-  private
-  def expire_cache(record)
-    expire_fragment 'events/attend'
-    expire_fragment 'events/speak'
-    expire_fragment 'events/archive'
-  end
-end
+  alias_method :after_save, :expire_cached_content
+  alias_method :after_destroy, :expire_cached_content
+  alias_method :after_update, :expire_cached_content
+end