On Tuesday, January 21, 2014 11:27:25 AM UTC-5, Annet wrote:
>
>
> That doesn't look like it will work -- the first branch will only execute 
>> if controls is an instance of all three types of widgets, which will never 
>> be true. I think you want:
>>
>> if not isinstance(controls, (INPUT, SELECT, TEXTAREA)): 
>>
>>
> No it didn't work :-)  clicking through some forms made that clear.
> What did work was: if isinstance(controls, basestring):
>

The only problem with that is if you use a "represent" attribute that wraps 
some text in an HTML helper -- it won't be a basestring in that case.
 

> I tried: if not isinstance(controls, (INPUT, SELECT, TEXTAREA)): 
>
> But that adds a empty <p class="form-control-static"></p> to other 
> controls as well.
>

Good point. I guess there are a few other types of objects you'd have to 
account for.

Anthony

-- 
Resources:
- http://web2py.com
- http://web2py.com/book (Documentation)
- http://github.com/web2py/web2py (Source code)
- https://code.google.com/p/web2py/issues/list (Report Issues)
--- 
You received this message because you are subscribed to the Google Groups 
"web2py-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to web2py+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to