Fixes to various issues reported by pyflakes
authorTzafrir Cohen <tzafrir.cohen@xorcom.com>
Wed, 14 Dec 2016 08:21:25 +0000 (10:21 +0200)
committerTzafrir Cohen <tzafrir.cohen@xorcom.com>
Wed, 21 Dec 2016 09:02:20 +0000 (11:02 +0200)
Pyflake is a python (2) source checker. This patch fixes various
(mostly trivial) errors and warnings it reports.

Change-Id: Ia35c5ac61751b927814cf693994c632c412386ea

build_tools/get_documentation.py
build_tools/post_process_documentation.py
contrib/ast-db-manage/config/versions/4c573e7135bd_fix_tos_field_types.py
contrib/ast-db-manage/config/versions/5139253c0423_make_q_member_uniqueid_autoinc.py
rest-api-templates/asterisk_processor.py
rest-api-templates/swagger_model.py
rest-api-templates/transform.py

index 2ff4cb2..3211896 100644 (file)
@@ -9,11 +9,8 @@ the GNU General Public License Version 2.
 '''
 
 import sys
-import os
 import xml.dom.minidom
 
-from xml.dom.minidom import Element
-
 
 def get_manager_event_method_type(candidate_string):
     if "ast_manager_event_multichan" in candidate_string:
index 5fa1c5b..c1496be 100644 (file)
@@ -9,11 +9,9 @@ the GNU General Public License Version 2.
 '''
 
 import sys
-import os
 import optparse
-import xml.dom.minidom
 
-from xml.dom.minidom import Element, parse
+from xml.dom.minidom import parse
 
 
 def merge_parameter_information(managerEvent):
index 27b498f..17906ae 100755 (executable)
@@ -11,7 +11,6 @@ revision = '4c573e7135bd'
 down_revision = '28887f25a46f'
 
 from alembic import op
-from alembic import context
 import sqlalchemy as sa
 from sqlalchemy.dialects.postgresql import ENUM
 
index 01d4985..72f5ee1 100755 (executable)
@@ -33,7 +33,6 @@ import sqlalchemy as sa
 
 
 def upgrade():
-    context = op.get_context()
     # Was unable to find a way to use op.alter_column() to add the unique
     # index property.
     op.drop_column('queue_members', 'uniqueid')
index 68a6799..9812946 100644 (file)
@@ -20,6 +20,7 @@
 Asterisk RESTful HTTP binding code.
 """
 
+import os
 import re
 
 from swagger_model import *
index c71352a..0885241 100644 (file)
@@ -737,7 +737,7 @@ class ResourceListing(Stringify):
         self.swagger_version = resources_json.get('swaggerVersion')
         if not self.swagger_version in SWAGGER_VERSIONS:
             raise SwaggerError(
-                "Unsupported Swagger version %s" % swagger_version, context)
+                "Unsupported Swagger version %s" % self.swagger_version, context)
 
         validate_required_fields(resources_json, self.required_fields, context)
         self.api_version = resources_json['apiVersion']
index fc12efe..c3a0300 100644 (file)
@@ -52,7 +52,6 @@ class Transform(object):
         dest_exists = os.path.exists(dest_file)
         if dest_exists and not self.overwrite:
             return
-        tmp_file = tempfile.mkstemp()
         with tempfile.NamedTemporaryFile() as out:
             out.write(renderer.render(self.template, model))
             out.flush()