Hi there, Christian Zagrodnick wrote: > Any opinion about this?
I guess the documentation was a little bit ahead of the implementation at the time I did this. Though, I think the after commit hooks should be called in the abort() method as well as it might be interesting, as I mentioned it in the documentation, and as well to be consistent with the commit() behavior. Maybe cleaning up the documentation would be simpler if nobody needs this feature right now. Cheers, J. > > On 2007-08-30 15:48:53 +0200, Christian Zagrodnick <[EMAIL PROTECTED]> said: > >> Hi, >> >> the documentation in the _transaction.py reads: >> >> After-commit hook >> ------------------ >> >> Sometimes, applications want to execute code after a transaction is >> committed or aborted. [...] The callable will be called with a Boolean >> value representing the status of the commit operation as first >> argument (true if successfull or false iff aborted) [...] >> >> And indeed in the commit() method it reads: >> >> try: >> self._commitResources() >> self.status = Status.COMMITTED >> except: >> t, v, tb = self._saveAndGetCommitishError() >> self._callAfterCommitHooks(status=False) >> raise t, v, tb >> else: >> if self._manager: >> self._manager.free(self) >> self._synchronizers.map(lambda s: s.afterCompletion(self)) >> self._callAfterCommitHooks(status=True) >> >> >> But in the abort() method the hooks are not called. >> >> Is this intentional? If yes, the documentation should be adjusted to >> reflect that the hooks are *only* called if there *is* a commit and it >> is *failing*. >> >> If not the hooks should be called in abort (my favourite) :) >> >> >> Regards, > > -- Julien Anguenot | Nuxeo R&D (Paris, France) Open Source ECM - http://www.nuxeo.com Nuxeo 5 : http://www.nuxeo.org Mobile: +33 (0) 6 72 57 57 66
Description: OpenPGP digital signature
_______________________________________________ For more information about ZODB, see the ZODB Wiki: http://www.zope.org/Wikis/ZODB/ ZODB-Dev mailing list - ZODB-Dev@zope.org http://mail.zope.org/mailman/listinfo/zodb-dev