Skip to content.

Etria Lists

 

[quills-dev] [Collective-checkins] r49960 - quills.app/trunk/quills/app/browser

Tim Hicks tim at sitefusion.co.uk
Sun Sep 23 11:26:09 UTC 2007


quills-dev at lists.etria.com wrote:
> Author: tomster
> Date: Sun Sep 23 11:21:58 2007
> New Revision: 49960
> 
> Removed:
>    quills.app/trunk/quills/app/browser/QuillsBaseView.py
> Modified:
>    quills.app/trunk/quills/app/browser/baseview.py
>    quills.app/trunk/quills/app/browser/controllerView.py
> Log:
> merged QuillsBaseView into BaseView (one is enough ;-)

Yeah, I've been thinking that for a while, but never got around to it...

> Modified: quills.app/trunk/quills/app/browser/baseview.py
> ==============================================================================
> --- quills.app/trunk/quills/app/browser/baseview.py	(original)
> +++ quills.app/trunk/quills/app/browser/baseview.py	Sun Sep 23 11:21:58 2007
> @@ -47,4 +47,13 @@
>          entry_absolute_url = getattr(weblogentry, 'absolute_url', None)
>          if entry_absolute_url is None:
>              entry_absolute_url = weblogentry.context.absolute_url
> -        return context.absolute_url() == entry_absolute_url()
> \ No newline at end of file
> +        return context.absolute_url() == entry_absolute_url()
> +
> +    def test(self, value, trueVal, falseVal):
> +        """
> +            helper method, mainly for setting html attributes.
> +        """
> +        if value:
> +            return trueVal
> +        else:
> +            return falseVal

Why do we need this method?  TAL already has "python:test(value,
trueVal, falseVal)", so why reinvent it?

Tim


More information about the quills-dev mailing list