Roman Klesel wrote:
for id in self.manage_targets.keys():
title = self.manage_targets[id]['name']
self.tgt_folder=Folder()
This is dangerous...
self._setObject(id,self.tgt_folder)
This is silly...
Why not just:
self.manage_addFolder(id,title)
grinder_home = self.manage_targets['gr_sys']['grinder_home']
host_name = self.getHostname(grinder_home)
Okay, what type of object is 'self' here, where did you take this code from?
for id in self.objectIds('Folder'):
location_sar = self.manage_targets[id]['location_sar']
phy_mem = self.manage_targets[id]['phy_mem']
system = self.__getitem__(id)
why not just:
system=self[id]
?
Also, this will result in system not being acquisition wrapped in some
circumstances, what you really want is:
system = getattr(self,id)
...but it really depends on what 'self' is, I'm not sure you have that
right.
cheers,
Chris
--
Simplistix - Content Management, Zope & Python Consulting
- http://www.simplistix.co.uk
_______________________________________________
Zope maillist - [email protected]
http://mail.zope.org/mailman/listinfo/zope
** No cross posts or HTML encoding! **
(Related lists -
http://mail.zope.org/mailman/listinfo/zope-announce
http://mail.zope.org/mailman/listinfo/zope-dev )
