[Zope-PTK] Debugging PTK II

[Shane Hathaway]

Fabio Forno wrote:
> I've started using and debugging the new PTK from the CVS. I will send
> bugs and my corrections to this ML when I find some.

Thanks for your help.  With the new PTK in the repository for three
days now, I was getting concerned that nobody would try it out.  It's
pretty stable--even more so than the old PTK, IMHO.

Of course, there were several logistical problems this weekend.  First,
I didn't finish checking in everything until an hour after the
announcement.  Second, Zope 2.2.1b1 has a bug that prevents importing
most zexp's that contain products--Wizard.zexp and DemoPortal.zexp
included.  As of this morning it is fixed.  However, the anonymous CVS
repository was not in sync with the main repository, so messages
regarding Zope CVS changes went out but the changes were inaccessible. 
That got fixed (partially) this afternoon.

So (all of you out there) if you tried to download and install the new
PTK this weekend and couldn't get it to work, please try again--it
should be fixed now.

> - In the register method of the Portal ZClass I've added this test at
> the beginning, as it was trying to check the password also when the
> validate_email property was set to true. The absence of this test caused
> an exception as password and confirm where not defined
> 
> (snip)

Definitely an oversight.  Thanks, Fabio!

> - In the roster method, in the generic interface, I've modifyed the user
> listing line to get the Home Url of the member:
> 
> <a href="<dtml-var "portal_membership.getHomeUrl(id)">">&dtml-id;</a>

I've expanded your patch slightly to include the case where the member
has no home URL.

 <dtml-let homeUrl="portal_membership.getHomeUrl(id)">
  <dtml-if homeUrl>
   <a href="&dtml-homeUrl;">&dtml-id;</a>
  <dtml-else>
   &dtml-id;
  </dtml-if>
 </dtml-let>

Shane