DataMapper

Best I can tell, after hooks are left in the dark as to what has transpired. Specifically, for CRUD operations, after hooks always fire regardless of the result, and there appears no way to obtain the result. I suggest either:

1) not firing after create,update,destroy,save is the result is false.

2) having the result of create,update,destroy,save exposed through created?,updated?,destroyed?,saved?

We are currently having to bypass the aspect love and do stuff like

... dm::resource ...

def save
  result = super
  if result
    Activity.log(self)
    Searcher.update(self)
  end 
  result
end

def destroy
  result = super
  if result
    Activity.log(self)
    Searcher.remove(self)
  end 
  result
end

but would rather do something like:

... dm::resource ...

after :save do
  # this would never get called if save == false
  Activity.log(self)
  Searcher.update(self)
end

or

... dm::resource ...

after :save do
  return unless saved?
  Activity.log(self)
  Searcher.update(self)
end

The astute may recommend using transactions and before hooks, but that would require the hooks to be attached to both after create (need the id) and before update, instead of the nice clean, DRY after update.

I can submit a patch implementing your preferred approach, if any.

thanks -- troy