[Erp5-report] r34249 luke - /erp5/trunk/utils/erp5.timmy/src/erp5/timmy/timmy.py

nobody at svn.erp5.org nobody at svn.erp5.org
Thu Apr 1 18:02:20 CEST 2010


Author: luke
Date: Thu Apr  1 18:02:18 2010
New Revision: 34249

URL: http://svn.erp5.org?rev=34249&view=rev
Log:
 - defensive if no control script available return error instead of failing

Modified:
    erp5/trunk/utils/erp5.timmy/src/erp5/timmy/timmy.py

Modified: erp5/trunk/utils/erp5.timmy/src/erp5/timmy/timmy.py
URL: http://svn.erp5.org/erp5/trunk/utils/erp5.timmy/src/erp5/timmy/timmy.py?rev=34249&r1=34248&r2=34249&view=diff
==============================================================================
--- erp5/trunk/utils/erp5.timmy/src/erp5/timmy/timmy.py [utf8] (original)
+++ erp5/trunk/utils/erp5.timmy/src/erp5/timmy/timmy.py [utf8] Thu Apr  1 18:02:18 2010
@@ -287,6 +287,8 @@
     self.control_script = os.path.join('var', item_id, 'bin', 'zopectl')
 
   def _zopectl_subprocess(self, action):
+    if not os.path.exists(self.control_script):
+      return ['reportError', 'no control script', '1']
     popen = subprocess.Popen([self.control_script, action],
         stdout=subprocess.PIPE, stderr=subprocess.PIPE)
     return list(popen.communicate()) + [popen.returncode]




More information about the Erp5-report mailing list