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  -  Zope@zope.org
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 )

Reply via email to