[ZCM] [ZC] 2298/ 6 Comment "WebDAV.MOVE: Events not sent"
"Collector: Zope Bugs, Features, and Patches ..." <[email protected]> Sun, 08 Apr 2007 16:26:08 -0400
| Newsgroups | gmane.comp.web.zope.devel.collector-monitor |
|---|---|
| Message-ID | <[email protected]> |
Issue #2298 Update (Comment) "WebDAV.MOVE: Events not sent" Status Accepted, Zope/bug medium To followup, visit: http://www.zope.org/Collectors/Zope/2298 ============================================================== = Comment - Entry #6 by shh on Apr 8, 2007 4:26 pm Split off a) below into a separate issue: #2307. ________________________________________ = Edit - Entry #5 by shh on Mar 24, 2007 10:59 am Changes: edited transcript ________________________________________ = Comment - Entry #4 by shh on Mar 24, 2007 10:57 am Fixed on 2.9 branch in r73501. Fixed on 2.10 branch in r73500. Fixed on trunk in r73499. Note: Tests live in OFS.tests as they are closely related to tests for copy/paste, cut/paste, etc. Two oddities remain: a) ObjectCopiedEvent is (apparently) not forwarded to sublocations. I have no idea if this is intentional. b) Event handlers for sublocations are called *before* the handler for the container. While this is ok for ObjectRemovedEvent, for Add/Move/Clone I would have expected sublocations to be notified *after* the container. Again, no idea if this is intentional. Look at OFS/tests/testCopySupportEvents.py to see what I mean. ________________________________________ = Edit - Entry #3 by shh on Mar 24, 2007 9:49 am Changes: edited transcript ________________________________________ = Edit - Entry #2 by shh on Mar 24, 2007 9:48 am Changes: submitter email, revised description ________________________________________ = Request - Entry #1 by shh on Mar 24, 2007 9:45 am Status: Pending => Accepted Supporters added: shh This is kind of a follow-up to #2169. Looks like webdav.Resource.COPY and webdav.Resource.MOVE were overlooked when adding events in Zope 2.9. webdav.Resource.COPY does not fire ObjectCopiedEvent, and webdav.Resource.MOVE doesn't fire any relevant event at all. I intend to tackle this before Sunday's deadline. ==============================================================