TemplateTool: py3

product/ERP5/Tool/TemplateTool.py

@@ -346,7 +346,9 @@ class TemplateTool (BaseTool):
       try:
         os.close(tempid) # Close the opened fd as soon as possible.
         file_path, headers = urlretrieve(url, temppath)
-        if re.search(r'<title>.*Revision \d+:', open(file_path, 'r').read()):
+        with open(file_path, 'rb') as f:
+          content = f.read()
+        if re.search(br'<title>.*Revision \d+:', content):
           # this looks like a subversion repository, try to check it out
           LOG('ERP5', INFO, 'TemplateTool doing a svn checkout of %s' % url)
           return self._download_svn(url, bt_id)
@@ -714,7 +716,7 @@ class TemplateTool (BaseTool):
       """
         Get the list of repositories.
       """
-      return self.repository_dict.keys()
+      return list(self.repository_dict.keys())
 
     security.declarePublic( 'decodeRepositoryBusinessTemplateUid' )
     def decodeRepositoryBusinessTemplateUid(self, uid):
@@ -924,7 +926,7 @@ class TemplateTool (BaseTool):
 
       # Calculate the reverse dependency graph
       reverse_dependency_dict = {}
-      for bt_id, dependency_id_list in dependency_dict.items():
+      for bt_id, dependency_id_list in list(dependency_dict.items()):
         update_dependency_id_list = []
         for dependency_id in dependency_id_list:
 
@@ -1127,7 +1129,7 @@ class TemplateTool (BaseTool):
             e = int(e)
           except ValueError:
             # ASCII code is one byte, so this produces negative.
-            e = struct.unpack('b', e)[0] - 0x200
+            e = struct.unpack('b', e.encode())[0] - 0x200
         except IndexError:
           e = 0
         return e