[Zope-PTK] can't add loginmanager below root

[Shane Hathaway]

Ok folks,

I thought I had made it clear that this problem has been solved.  I have
sent a patch to Ty (about two weeks ago) as well as made the change in
the LoginManager stored in the PTK repository.  I hope this will be
understood this time.

To solve this problem, instead of setting _owner on the class
definition, I moved the _owner attribute to the LoginManager
*instance*.  That way Zope's owner system can delete the attribute if it
feels the need to do so, allowing acquisition to take over.

Kevin and everyone, download LoginManager from the PTK CVS repository. 
The fix is there.  In fact, I also have the solution for the DemoPortal
problem, but have been too heavily involved in another project to
distribute the changes yet.  Also, I have updated the Wizard classes
with the ability to provide back buttons and customized form
validation.  It was needed for the other project.

Shane

"Phillip J. Eby" wrote:
> 
> Ty ran into this himself the other day; don't know if he's tracked down the
> source of it yet.  It may have something to do with LoginManager being
> owned by the UnownableOwner.
> 
> At 06:31 PM 6/23/00 -0400, Kevin Dangoor wrote:
> >This is Zope 2.2, of course:
> >
> >Try this:
> >
> >Login as a Manager (not the superuser)
> >Add a Folder
> >Try to add a LoginManager
> >
> >likely you'll receive an error (KeyError for _owner)...
> >
> >  File /home/kid/Zope2/lib/python/Products/LoginManager/LoginManager.py,
> >line 231, in manage_addLoginManager
> >  File /home/kid/Zope2/lib/python/OFS/ObjectManager.py, line 249, in
> >_setObject
> >    (Object: ElementWithAttributes)
> >  File /home/kid/Zope2/lib/python/AccessControl/Owned.py, line 267, in
> >manage_fixupOwnershipAfterAdd
> >    (Object: SheetProviderContainer)
> >  File /home/kid/Zope2/lib/python/AccessControl/Owned.py, line 174, in
> >changeOwnership
> >    (Object: SheetProviderContainer)
> >KeyError: (see above)
> >
> >I'm not sure whether this is LM's fault or something with the current access
> >control. I'm guessing the LM I'm trying to add *should* be owned by my user
> >ID, since I am logged in as a Manager and not superuser...
> >
> >Ideas?
> >Kevin
> >
> >
> >_______________________________________________
> >Zope-PTK maillist  -  Zope-PTK@zope.org
> >http://lists.zope.org/mailman/listinfo/zope-ptk
> >
> >See http://www.zope.org/Products/PTK/Tracker for bug reports and feature
> requests
> >
> >
> >
> 
> _______________________________________________
> Zope-PTK maillist  -  Zope-PTK@zope.org
> http://lists.zope.org/mailman/listinfo/zope-ptk
> 
> See http://www.zope.org/Products/PTK/Tracker for bug reports and feature requests