[Zope-CMF] [dev] five.localsitemanager: site manager names

[yuppie]

Hi!


Trying to clean up site creation in CMF, I noticed this issue:

zope.app.component uses a hardcoded '++etc++site' as name, but 
five.localsitemanager's make_site function computes it like this:

     name = 'five'
     path = getattr(obj, 'getPhysicalPath', None)
     if path is not None and callable(path):
         name = '/'.join(path())

So the name is location dependent. Moving the site would require 
updating the name, but there is no event handler that does it.

I see 2 possible ways to fix this:

1.) Add an event handler that updates the name.

2.) Use the same hardcoded name as Zope 3. A customized __repr__ method 
could still show the complete path, at least as long as the active site 
is set accordingly.

Any thoughts? I prefer solution 2.


Cheers,

     Yuppie