Re: [Zope-CMF] member area / home folder changes

2013-08-05 Thread Charlie Clark

Am 05.08.2013, 15:38 Uhr, schrieb yuppie :


Last time I checked the syndication views had some issues. Two things I
remember:



- The folder syndication form seems to allow enabling folder syndication
if portal syndication is disabled. This is confusing.



- syndication.pt exists but is not used.


Okay, I've still got the e-mails. I just remember seeing that you'd  
already fixed some of the stuff and wasn't sure what was still  
outstanding. I will have time this month maybe at the upcoming sprint in  
Halle.


Charlie
--
Charlie Clark
Managing Director
Clark Consulting & Research
German Office
Kronenstr. 27a
Düsseldorf
D- 40217
Tel: +49-211-600-3657
Mobile: +49-178-782-6226
___
Zope-CMF maillist  -  Zope-CMF@zope.org
https://mail.zope.org/mailman/listinfo/zope-cmf

See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests


Re: [Zope-CMF] member area / home folder changes

2013-08-05 Thread yuppie
Hi Charlie!


Charlie Clark wrote:
> Okay. Is there any of my stuff outstanding that you haven't already
> fixed for me?

Last time I checked the syndication views had some issues. Two things I
remember:

- The folder syndication form seems to allow enabling folder syndication
if portal syndication is disabled. This is confusing.

- syndication.pt exists but is not used.


Cheers,

Yuppie

___
Zope-CMF maillist  -  Zope-CMF@zope.org
https://mail.zope.org/mailman/listinfo/zope-cmf

See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests


Re: [Zope-CMF] member area / home folder changes

2013-08-05 Thread Charlie Clark

Am 05.08.2013, 12:26 Uhr, schrieb yuppie :


No objections. I'd just like to do some small polishing before we create
2.3 branches.


Okay. Is there any of my stuff outstanding that you haven't already fixed  
for me?


Charlie
--
Charlie Clark
Managing Director
Clark Consulting & Research
German Office
Kronenstr. 27a
Düsseldorf
D- 40217
Tel: +49-211-600-3657
Mobile: +49-178-782-6226
___
Zope-CMF maillist  -  Zope-CMF@zope.org
https://mail.zope.org/mailman/listinfo/zope-cmf

See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests


Re: [Zope-CMF] member area / home folder changes

2013-08-05 Thread yuppie
Hi Charlie!


Charlie Clark wrote:
> Am 01.08.2013, 11:46 Uhr, schrieb yuppie :
>> First a few words about names: CMF uses sometimes 'member area' and
>> sometimes 'home folder'. IMembershipTool has 'getHomeFolder' and
>> 'getHomeUrl' methods as well as 'createMemberArea' and
>> 'deleteMemberArea' methods. If there is a difference between the two
>> names, 'home folder' is just the folder and 'member area' the folder
>> including all subobjects. In my proposal I proposed to add portal types
>> named 'Members' and 'MemberArea', but I now changed this to 'Members
>> Folder' and 'Home Folder'. Hope that's ok.
> 
> I think this is clearer: users are interested in their own or others
> (home) folder. The Members Folder is really only of interest to admins.
> Do the new types have any special functions or attributes? Or are they
> purely semantic? You mention a proposal - did that go to the list and I
> missed it? Or did you put something up on launchpad?

I didn't write a full proposal. Just mentioned the planed changes in
this mail:
https://mail.zope.org/pipermail/zope-cmf/2013-July/030754.html

- 'Home Folder' is a special folder type that uses the normal Portal
Folder class, but a different factory that replaces parts of the
old createMemberArea() code.

- 'Members Folder' is a special folder type that uses the normal Portal
Folder class, but limits sub-items to 'Home Folder' objects and has a
special default view that replaces the 'index_html' DTML Method
currently added to the members folder.

>> 'createMemberArea' now uses separate factories for creating member
>> areas. This allows to use the same method in CMFCore and CMFDefault. The
>> MembershipTool in CMFDefault no longer has a customized version of
>> 'createMemberArea'.
> 
> I'm not sure what the separate factories are for "member areas and…"?
> But it certainly makes sense to remove a customisation in CMFDefault.

I meant the actual factories are now separated from the rest of the
createMemberArea code. The new factories implementing
zope.component.interfaces.IFactory are now looked up and used by
createMemberArea.

> The only thing I have here is that changes should probably come in a new
> release. I think we've (well, you've) done most of the work for moving
> from TTW and we can look to faster releases than in the past because of
> the improved test coverage.

No objections. I'd just like to do some small polishing before we create
2.3 branches.


Cheers,

Yuppie

___
Zope-CMF maillist  -  Zope-CMF@zope.org
https://mail.zope.org/mailman/listinfo/zope-cmf

See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests


Re: [Zope-CMF] member area / home folder changes

2013-08-05 Thread Charlie Clark

Hi Yuppie,

(checks that he is writing to the list)

Am 01.08.2013, 11:46 Uhr, schrieb yuppie :


Hi!


Just want to explain the changes I made on CMF trunk.

First a few words about names: CMF uses sometimes 'member area' and
sometimes 'home folder'. IMembershipTool has 'getHomeFolder' and
'getHomeUrl' methods as well as 'createMemberArea' and
'deleteMemberArea' methods. If there is a difference between the two
names, 'home folder' is just the folder and 'member area' the folder
including all subobjects. In my proposal I proposed to add portal types
named 'Members' and 'MemberArea', but I now changed this to 'Members
Folder' and 'Home Folder'. Hope that's ok.


I think this is clearer: users are interested in their own or others  
(home) folder. The Members Folder is really only of interest to admins. Do  
the new types have any special functions or attributes? Or are they purely  
semantic? You mention a proposal - did that go to the list and I missed  
it? Or did you put something up on launchpad?



'createMemberArea' now uses separate factories for creating member
areas. This allows to use the same method in CMFCore and CMFDefault. The
MembershipTool in CMFDefault no longer has a customized version of
'createMemberArea'.


I'm not sure what the separate factories are for "member areas and…"? But  
it certainly makes sense to remove a customisation in CMFDefault.



For backwards compatibility I added two factories that are used if no
'Home Folder' portal type exists. 'cmf.folder.home.bbb1' is compatible
with the old CMFCore code, 'cmf.folder.home.bbb2' with the old
CMFDefault code. In that case the portal type of new home folders is
'Folder' as before.



The recommended factory is 'cmf.folder.home'. It no longer supports the
'createMemberContent' hook. It is now recommended to use a customized
factory instead. The two new portal types 'Members Folder' and 'Home
Folder' allow to customize factories and behavior.



For existing sites you can either switch to the new behavior by using
the two upgrade steps or just keep the old behavior.


The only thing I have here is that changes should probably come in a new  
release. I think we've (well, you've) done most of the work for moving  
from TTW and we can look to faster releases than in the past because of  
the improved test coverage.


Charlie
--
Charlie Clark
Managing Director
Clark Consulting & Research
German Office
Kronenstr. 27a
Düsseldorf
D- 40217
Tel: +49-211-600-3657
Mobile: +49-178-782-6226
___
Zope-CMF maillist  -  Zope-CMF@zope.org
https://mail.zope.org/mailman/listinfo/zope-cmf

See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests


[Zope-CMF] member area / home folder changes

2013-08-01 Thread yuppie
Hi!


Just want to explain the changes I made on CMF trunk.

First a few words about names: CMF uses sometimes 'member area' and
sometimes 'home folder'. IMembershipTool has 'getHomeFolder' and
'getHomeUrl' methods as well as 'createMemberArea' and
'deleteMemberArea' methods. If there is a difference between the two
names, 'home folder' is just the folder and 'member area' the folder
including all subobjects. In my proposal I proposed to add portal types
named 'Members' and 'MemberArea', but I now changed this to 'Members
Folder' and 'Home Folder'. Hope that's ok.

'createMemberArea' now uses separate factories for creating member
areas. This allows to use the same method in CMFCore and CMFDefault. The
MembershipTool in CMFDefault no longer has a customized version of
'createMemberArea'.

For backwards compatibility I added two factories that are used if no
'Home Folder' portal type exists. 'cmf.folder.home.bbb1' is compatible
with the old CMFCore code, 'cmf.folder.home.bbb2' with the old
CMFDefault code. In that case the portal type of new home folders is
'Folder' as before.

The recommended factory is 'cmf.folder.home'. It no longer supports the
'createMemberContent' hook. It is now recommended to use a customized
factory instead. The two new portal types 'Members Folder' and 'Home
Folder' allow to customize factories and behavior.

For existing sites you can either switch to the new behavior by using
the two upgrade steps or just keep the old behavior.


Cheers,

Yuppie
___
Zope-CMF maillist  -  Zope-CMF@zope.org
https://mail.zope.org/mailman/listinfo/zope-cmf

See https://bugs.launchpad.net/zope-cmf/ for bug reports and feature requests