Fix issue elpaso#32 variable 'url' referenced before assignement when…

… using google API
Flo45 · Aug 2, 2019 · a617f61 · a617f61
1 parent 820d61c
commit a617f61
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/.gitignore b/.gitignore
@@ -2,6 +2,7 @@
 __pycache__/
 *.py[cod]
 *$py.class
+*.pyc
 
 # C extensions
 *.so
@@ -97,3 +98,6 @@ ENV/
 
 # mypy
 .mypy_cache/
+
+# pyCharm stuff
+.idea/
diff --git a/geocoders.py b/geocoders.py
@@ -64,14 +64,14 @@ def __init__(self, api_key=None):
         self.api_key = api_key
 
     def geocode(self, address):
+        try:
+            url = self.url.format(**{'address': address.decode('utf8')})
 
-        if self.api_key is not None and self.api_key.replace(' ', '') != '':
-            url += self.url + '&key=' + self.api_key
-        else:
-            url = self.url
+            if self.api_key is not None and self.api_key.replace(' ', '') != '':
+                url += self.url + '&key=' + self.api_key
+            else:
+                url = self.url
 
-        try: 
-            url = url.format(**{'address': address.decode('utf8')})
             logMessage(url)
             results = json.loads(NAM.request(url, blocking=True)[1].decode('utf8'))['results']
             return [(rec['formatted_address'], (rec['geometry']['location']['lng'], rec['geometry']['location']['lat'])) for rec in results]