RSpecVersion.todict() needs to turn values() into a list()

[sfa.git] / sfa / server / api_versions.py

diff --git a/sfa/server/api_versions.py b/sfa/server/api_versions.py
index 2f1ccf7..9138a74 100644 (file)
--- a/sfa/server/api_versions.py
+++ b/sfa/server/api_versions.py
@@ -32,16 +32,16 @@ class ApiVersions:
         self.xml = XML(source)
 
     def get_versions(self):
-        versions = []
+        versions = {}
         for value in self.xml.todict().values():
             if not value:
                 continue
             if isinstance(value, list):
                 for item in value:
                     if isinstance(item, dict) and \
-                       set(ApiVersions.required_fields).issubset(item.keys()):
-                        api_version = {str(item['version']): item['url']}
-                        versions.append(api_version)
+                       set(ApiVersions.required_fields).issubset(item.keys()) and \
+                       item['version'] != '' and item['url'] != '':
+                        versions[str(item['version'])] = item['url']
         return versions