Re: [Zope-dev] transaction.doom() and ZPublisher

2008-07-17 Thread Dieter Maurer
Brian Sutherland wrote at 2008-7-13 12:41 +0200: >On Sun, Jul 13, 2008 at 09:05:16AM +0200, Dieter Maurer wrote: >> Andreas Jung wrote at 2008-7-12 07:17 +0200: >> > ... >> >What do you mean by "higher level"? I think that the check within the >> >ZPublisher is the highest and right place. >> > >

Re: [Zope-dev] transaction.doom() and ZPublisher

2008-07-13 Thread Brian Sutherland
On Sat, Jul 12, 2008 at 07:17:31AM +0200, Andreas Jung wrote: > > > --On 10. Juli 2008 17:09:36 +0200 Brian Sutherland > <[EMAIL PROTECTED]> wrote: > >> On Thu, Jul 10, 2008 at 12:12:06AM -0400, Paul Winkler wrote: >>> Hi, > > >> >> I havn't investigated properly, but it may be necessary to do the

Re: [Zope-dev] transaction.doom() and ZPublisher

2008-07-13 Thread Brian Sutherland
On Sun, Jul 13, 2008 at 09:05:16AM +0200, Dieter Maurer wrote: > Andreas Jung wrote at 2008-7-12 07:17 +0200: > > ... > >What do you mean by "higher level"? I think that the check within the > >ZPublisher is the highest and right place. > > > >> Code running > >> after the commit() expects a new

Re: [Zope-dev] transaction.doom() and ZPublisher

2008-07-13 Thread Dieter Maurer
Andreas Jung wrote at 2008-7-12 07:17 +0200: > ... >What do you mean by "higher level"? I think that the check within the >ZPublisher is the highest and right place. > >> Code running >> after the commit() expects a new transaction and now will not get that. > >You refer to code executed as part

Re: [Zope-dev] transaction.doom() and ZPublisher

2008-07-11 Thread Andreas Jung
--On 10. Juli 2008 17:09:36 +0200 Brian Sutherland <[EMAIL PROTECTED]> wrote: On Thu, Jul 10, 2008 at 12:12:06AM -0400, Paul Winkler wrote: Hi, I havn't investigated properly, but it may be necessary to do the isDoomed() check at a higher level where you can abort. What do you mean b

Re: [Zope-dev] transaction.doom() and ZPublisher

2008-07-10 Thread Brian Sutherland
On Thu, Jul 10, 2008 at 12:12:06AM -0400, Paul Winkler wrote: > Hi, > > I noticed that Zope 2.11 includes a recent version of the transaction > module including the transaction.doom() method. But I don't see any > check for it in ZPublisher. > > So, if any code calls transaction.doom(), the publ

[Zope-dev] transaction.doom() and ZPublisher

2008-07-09 Thread Paul Winkler
Hi, I noticed that Zope 2.11 includes a recent version of the transaction module including the transaction.doom() method. But I don't see any check for it in ZPublisher. So, if any code calls transaction.doom(), the publisher will raise a user-visible exception when it tries to call commit(). T