On Thu, May 08, 2008 at 01:39:41PM -0400, Josh Johnson (jjmojojjmojo) wrote:
> I'm looking at the code for OrderedMultiSelectWidget, in  
> zope.app.form.browser.itemswidgets (line 535). 
> This code doesn't make sense to me:
> if hasattr(self.context.context, self.context.__name__):
>     available_values = self.context.get(self.context.context)

self is a widget.
self.context is a field.
self.context.context is whatever the field was bound to.

This code works fine in an *edit* form, where all fields are bound to the
object you're editing.  It won't work in any other kind of form where
you may want to specify the initial values yourself (with
setUpDataWidgets or widget.setRenderedValue).  It's possible that the
hasattr check was intended to make the other kinds of forms appear to
work, until you stumbled upon an unfortunate name clash.

I don't know how to fix it, though.

> else:
>    available_values = []
> From what I can tell, this code is grabbing anything on the current  
> content/view that's named the same as the name of the vocabulary  
> provider. This bit me earlier with this error:
>  Module zope.app.form.browser.itemswidgets, line 547, in choices
> TypeError: iterable argument required
> I had a named utility that was mapped to a function called 'activities'  
> which provides IVocabularyFactory. I had a browser form page that had a  
> field in it that used that named utility. The form class had a method on  
> it called 'activities'.  Through some pdb digging, I was able to figure  
> out that it was grabbing the view method and not the function. It makes  
> sense looking at the code, but why would it work that way? Am I looking  
> at this wrong, or is this a bug?
> My setup :
> Plone 3.0.6 (according to Plone control pannel.. it's supposed to be 3.1.1)
> CMF 2.1.1
> Zope (Zope 2.10.5-final, python 2.4.3, linux2)
> Python 2.4.3 (#2, Mar 7 2008, 01:58:20) [GCC 4.0.3 (Ubuntu 4.0.3-1ubuntu5)]
> PIL 1.1.5
> Five 1.5.6
> I can put together some example code to reproduce the issue, if necessary.

I think a reproducible testcase attached to a bug in
http://bugs.launcpad.net/zope3 could help a bit.

Marius Gedminas
Never be afraid to tell the world who you are.
                -- Anonymous

Attachment: signature.asc
Description: Digital signature

Zope-Dev maillist  -  Zope-Dev@zope.org
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://mail.zope.org/mailman/listinfo/zope )

Reply via email to