Christophe de VIENNE schrieb:
> 2008/11/12 Christoph Zwerschke <[EMAIL PROTECTED]>:
>> Christophe de VIENNE schrieb:
>>> Shouldn't line 108 be :
>>>
>>> 108        if identity.anonymous:
>> No, I think current.anonymous is correct; "current" is the identity
>> wrapper. Maybe you were confused because it's named "identity" in the
>> template std vars.
>>
> 
> The thing is that in the other predicates, "identity" is used. Note
> that in this context "identity" is the parameter of the function, not
> the identity module.
> So my understanding is that the predicate shouldn't access the global
> "current" identity but the one given as a parameter.

Yes, this is an inconsistency in the code which I also noticed earlier
but didn't have the time to correct. Can you file a bug ticket, please?

It doesn't matter when you use the standard identity decorator, but it
might lead to subtle bugs when customizing the identity predicates.


Chris

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"TurboGears Trunk" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/turbogears-trunk?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to