croogo/croogo — pull request #86 by rchavik (2012-07-10 14:34:56+00:00)
Title: updateAll() -> afterSave issue

since 1.3, updateAll() triggers afterSave callback, but afterSave() acts on individual record.

This patch does the following:
- detect presence of id via the typical updateAll() used for Nodes/admin_process, then set the Model->id before dispatching event
- remove implicit call to afterSave, otherwise it will get called twice
- if IDs cannot be detected, the Model.afterSave event will not be called since it does not make sense without any id.

Comments ?