[Zope3-dev] opinion requested from people working towards therelease: doctest inaccuracy for zope.app.form.utility

[Gary Poster]

On Mar 4, 2005, at 6:58 AM, Garrett Smith wrote:

> Gary Poster wrote:

<snip>

>> In the next to last example, if the doctest description were correct,
>> context.foo should be 'Foo', I believe.  Does anyone disagree?  If
>> not, should I
>
> Based on the docs, right.

Cool.

>> (a) check this in,
>> (b) change the XXX to a TODO and check it in,
>> (c) change the test description to actually describe the current
>> behavior, or
>> (d) try to change the code to match the description?
>>
>> If (d) is your response, then please also give me a fallback option
>> (a, b, or c) if I don't have time.  As I write in my XXX, I have a
>> workaround already.
>>
>> Opinions?  I particularly want to hear from the people working towards
>> the next release, like Stephan.
>
> I think the documentation can be changed. We can debate the behavior,
> but it's benign as far as the current edit form -- if any error occurs,
> the transaction is aborted.

Cool.  OK, will do (unless we see dissension in the ranks before I get 
a chance to check it in).

> I don't think we need to make form utils
> worry about transactional behavior.

good enough

Thanks!

Gary