Etria Lists

On 23.09.2007, at 13:26, Tim Hicks wrote:

> 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?

i dunno ;-)

i just merged it... remove it if it's superfluous...

>
> Tim
>

Tom Lazar
http://tomster.org