Browse files

Merge pull request #9402 from senny/9356_update_guides

update the example for `after_commit` in the guides.
  • Loading branch information...
2 parents 05857ec + 74bc0d0 commit aff623254c3e1480eccf15e0f671404838aa327a @rafaelfranca rafaelfranca committed Feb 24, 2013
Showing with 7 additions and 9 deletions.
  1. +7 −9 guides/source/
@@ -342,19 +342,17 @@ By using the `after_commit` callback we can account for this case.
class PictureFile < ActiveRecord::Base
- attr_accessor :delete_file
+ after_commit :delete_picture_file_from_disk, :on => [:destroy]
- after_destroy do |picture_file|
- picture_file.delete_file = picture_file.filepath
- end
- after_commit do |picture_file|
- if picture_file.delete_file && File.exist?(picture_file.delete_file)
- File.delete(picture_file.delete_file)
- picture_file.delete_file = nil
+ def delete_picture_file_from_disk
+ if File.exist?(filepath)
+ File.delete(filepath)
+NOTE: the `:on` option specifies when a callback will be fired. If you
+don't supply the `:on` option the callback will fire for every action.
The `after_commit` and `after_rollback` callbacks are guaranteed to be called for all models created, updated, or destroyed within a transaction block. If any exceptions are raised within one of these callbacks, they will be ignored so that they don't interfere with the other callbacks. As such, if your callback code could raise an exception, you'll need to rescue it and handle it appropriately within the callback.

0 comments on commit aff6232

Please sign in to comment.