diff options
author | Bharat Mediratta <bharat@menalto.com> | 2009-08-02 12:09:00 -0700 |
---|---|---|
committer | Bharat Mediratta <bharat@menalto.com> | 2009-08-02 12:09:00 -0700 |
commit | 7ad0808a117fd1db4e94da8d7763ccca1d69350a (patch) | |
tree | 3102d84609cfb0412cf431c4ea97fb7c35748d71 /modules/comment | |
parent | f034c6c534ad6a07dacc6e64c7e6b43f139c3831 (diff) |
Change the API for getting to the original state of an ORM.
Old API: $obj->original("field_name")
New API: $obj->original()->field_name
This allows us to revert the varous xxx_updated events back to passing
an original ORM as well as the the updated one. This makes for a
cleaner event API.
Old API: comment_updated($comment) { $comment->original("field_name") }
Old API: comment_updated($old, $new) { $old->field_name }
Diffstat (limited to 'modules/comment')
-rw-r--r-- | modules/comment/models/comment.php | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/modules/comment/models/comment.php b/modules/comment/models/comment.php index d052a39c..83d0888a 100644 --- a/modules/comment/models/comment.php +++ b/modules/comment/models/comment.php @@ -64,17 +64,17 @@ class Comment_Model extends ORM { $created = true; } } + $visible_change = $this->original()->state == "published" || $this->state == "published"; parent::save(); if (isset($created)) { module::event("comment_created", $this); } else { - module::event("comment_updated", $this); + module::event("comment_updated", $this->original(), $this); } - // We only notify on the related items if we're making a visible change, which means moving in - // or out of a published state - if ($this->original("state") == "published" || $this->state == "published") { + // We only notify on the related items if we're making a visible change. + if ($visible_change) { module::event("item_related_update", $this->item()); } |