Jan Hackel wrote: > Here is the patch. If there are no objections I will check it in. Test-cases > run through, short ttw-testing showed also no problems. > > The patch will in essence do some moving of redundant code and configuration > from QuillsEnabled and Quills package to quills.app.
I just skimmed over the patch file and if I understand correctly, it's largely about moving a load of the syndication code from Products.Quills to quills.app. To the extent that Products.QuillsEnabled uses the same code, then this is clearly desirable. I'm slightly wary about having the new syndication.zcml file included in quills/app/configure.zcml. It seems like a more flexible setup if Products/Quills[Enabled]/configure.zcml include them directly themselves so that it is possible to disable syndication for an installation if so desired. Tim _______________________________________________ quills-dev mailing list [email protected] http://lists.etria.com/cgi-bin/mailman/listinfo/quills-dev
