[z3-five] Re: [z3-checkins] r5244 - z3/Five/trunk

[Martijn Faassen]

philikon at codespeak.net wrote:
> Author: philikon
> Date: Tue Jun 22 20:57:11 2004
> New Revision: 5244
> 
> Modified:
>    z3/Five/trunk/browser.py
> Log:
> Put an XXX should go XXX comment on __call__.
> 
> 
> Modified: z3/Five/trunk/browser.py
> ==============================================================================
> --- z3/Five/trunk/browser.py	(original)
> +++ z3/Five/trunk/browser.py	Tue Jun 22 20:57:11 2004
> @@ -27,6 +27,11 @@
>  
>      def __call__(self, *args, **kw):
>          # XXX this is definitely not the way Zope 3 does it..
> +
> +        # XXX and it is only needed for tests so that they can all
> +        # views directly. ZPublisher will either find a __call__ on a
> +        # view class or use __browswer_default__ from
> +        # ViewMixinForAttributes
>          if hasattr(self, 'index'):
>              attr = 'index'
>          else:

Hey,

Great work on fixing this issue!

What about the case where in TALES you want to say something like:

views['someview'](foo, bar, baz)

Is that a legitimate use case?

Regards,

Martijn