2009/8/21 Fabio Tranchitella <kob...@kobold.it>: > * 2009-08-21 21:07, Dan Korostelev wrote: >> IXMLRPCPublisher adapters for zope.container - move them to >> zope.container. The IBrowserPublisher adapters that are already there, so >> it won't make things worser. The zope.container package may be refactored >> later to break dependency on zope.publisher though. > > -1, I think this is bad and we shouldn't do it: zope.container has nothing > to do with the publisher, and it shouldn't depend on it. The fact that we > already have the adapters for IBrowserPublisher shouldn't be a reason to > bring more publisher-related stuff over there.
Sorry, I didn't explain correctly. Those things are actually simply two ZCML declarations that register traversers that are already defined in zope.container.traversal and used for browser traversing. See for yourself - zope/app/container/xmlrpc/configure.zcml. I think those zcml declarations should be placed together with browser adapter registrations, so people who will be refactoring zope.container could see them. :) -- WBR, Dan Korostelev _______________________________________________ Zope-Dev maillist - Zope-Dev@zope.org http://mail.zope.org/mailman/listinfo/zope-dev ** No cross posts or HTML encoding! ** (Related lists - http://mail.zope.org/mailman/listinfo/zope-announce http://mail.zope.org/mailman/listinfo/zope )
