Re: Little bug in CPSLDAPSetup profile.

Georges Racinet <[email protected]>
Newsgroups gmane.comp.web.zope.cps.devel
Message-ID <[email protected]>
On Dec 4, 2006, at 12:33 PM, JoseLuis de la Rosa Triviño wrote:

> Hello,
>
>
> This issue is related to the problem that Jesus Martin was dealing  
> with.
>
>
> They were using CPSLDAPSetup and the problem was that the home  
> folder of the members were not created even if "Enable private  
> area" was true.
>
>
> The fact is that, in logged_in.py, the value of "first_time" should  
> be "2000/01/01" so that the member area is created, but in the  
> configuration file of the members directory in CPSLDAPSetup  
> (members.xml) the default expression for "last_login_time" is None.

I see. Since I like to nitpick, I'd rather change logged_in.py, but  
this would contradict the policy of keeping backwards compatibility,  
sadly.
Strange that we didn't see it before. On the other hand, logged_in.py  
is one of the most often customized scripts in our customer projects,  
so that might explain.

>  I think that the solution would be to use the expression:
>  python:{'homeless': False, 'last_login_time': '2000/01/01' }
'last_login_time' is handled by a CPS DateTime Field, so this should  
rather be a DateTime instance (some conversions probably apply, but  
I'd rather be explicit).

>
> instead of:
> python:{'homeless': False, 'last_login_time': None}
>
> in the definition profile of the members directory in CPSLDAPSetup.
>
> If you agree I will create the ticket and commit the solution.

I do!
Go for it, but please wait till 3.4.3 tags are completed (will happen  
within a few hours), and use the 3.4.4 milestone.

Thanks a lot for report and solution.

---------
Georges Racinet,   Nuxeo SAS
Open Source Enterprise Content Management (ECM)
Web: http://www.nuxeo.com/ and http://www.nuxeo.org/ - Tel: +33 1 40  
33 79 87



_______________________________________________
cps-devel mailing list
http://lists.nuxeo.com/mailman/listinfo/cps-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.