put that back in place.
authormadcoder <madcoder>
Thu, 11 May 2006 21:11:50 +0000 (21:11 +0000)
committermadcoder <madcoder>
Thu, 11 May 2006 21:11:50 +0000 (21:11 +0000)
remote/bugzilla.py

index d04752b4396136cb73d27eed747619f8d65a43c6..4f1f4a5deb793b618f7c052383fb6de286692c3b 100644 (file)
@@ -101,7 +101,17 @@ class OldRemoteBugzilla(RemoteBts):
         RemoteBts.__init__(self, cnf, bugre, urifmt)
 
     def __getClosingStatus(self):
-        return self._cnf['closing']
+        if 'closing' in self._cnf:
+            return self._cnf['closing']
+
+        config = urllib.urlopen(self._cnf['uri'] + '/config.cgi')
+        for l in config.readlines():
+            if l.startswith('var status_closed'):
+                s = l[l.find('['):].strip('[] ,;\r\t\n')
+                self._cnf['closing'] = [x.strip("' ") for x in s.split(',')]
+                config.close()
+                return self._cnf['closing']
+
 
     def isClosing(self, status, resolution):
         return resolution != 'WONTFIX' and status in self.__getClosingStatus()
@@ -126,21 +136,6 @@ class RemoteBugzilla(OldRemoteBugzilla):
     def __init__(self, cnf):
         OldRemoteBugzilla.__init__(self, cnf)
 
-    def __getClosingStatus(self):
-        if 'closing' in self._cnf:
-            return self._cnf['closing']
-
-        config = urllib.urlopen(self._cnf['uri'] + '/config.cgi')
-        for l in config.readlines():
-            if l.startswith('var status_closed'):
-                s = l[l.find('['):].strip('[] ,;\r\t\n')
-                self._cnf['closing'] = [x.strip("' ") for x in s.split(',')]
-                config.close()
-                return self._cnf['closing']
-
-    def isClosing(self, status, resolution):
-        return resolution != 'WONTFIX' and status in self.__getClosingStatus()
-
     def enqueue(self, btsbug, fwd):
         self._queue.append((btsbug, self.extractBugid(fwd)))