Refactor metadef ORM classes to use to_dict instead of as_dict

The Metadef ORM base dictionary class should replace as_dict with to_dict
to be consistent with other Glance ORM classes.

Change-Id: I1550bd519f100cdffcf5046e9838cf192d4aac50
Closes-Bug: 1370058

glance/db/sqlalchemy/metadef_api/namespace.py

@@ -92,7 +92,7 @@ def _get(context, namespace_id, session):
                                         id=namespace_id)
 
     # Make sure they are allowed to view it.
-    if not _is_namespace_visible(context, namespace_rec.as_dict()):
+    if not _is_namespace_visible(context, namespace_rec.to_dict()):
         msg = ("Forbidding request, metadata definition namespace=%s"
                " is not visible.") % namespace_rec.namespace
         LOG.debug(msg)
@@ -116,7 +116,7 @@ def _get_by_name(context, name, session):
         raise exc.MetadefNamespaceNotFound(namespace_name=name)
 
     # Make sure they are allowed to view it.
-    if not _is_namespace_visible(context, namespace_rec.as_dict()):
+    if not _is_namespace_visible(context, namespace_rec.to_dict()):
         msg = ("Forbidding request, metadata definition namespace=%s"
                " not visible." % name)
         LOG.debug(msg)
@@ -210,13 +210,13 @@ def get_all(context, session, marker=None, limit=None,
         namespaces = _get_all(
             context, session, filters, marker, limit, sort_key, sort_dir)
 
-    return map(lambda ns: ns.as_dict(), namespaces)
+    return map(lambda ns: ns.to_dict(), namespaces)
 
 
 def get(context, name, session):
     """Get a namespace by name, raise if not found"""
     namespace_rec = _get_by_name(context, name, session)
-    return namespace_rec.as_dict()
+    return namespace_rec.to_dict()
 
 
 def create(context, values, session):
@@ -235,7 +235,7 @@ def create(context, values, session):
         raise exc.MetadefDuplicateNamespace(
             namespace_name=namespace_name)
 
-    return namespace.as_dict()
+    return namespace.to_dict()
 
 
 def update(context, namespace_id, values, session):
@@ -257,7 +257,7 @@ def update(context, namespace_id, values, session):
                 % values['namespace'])
         raise exc.MetadefDuplicateNamespace(emsg)
 
-    return namespace_rec.as_dict()
+    return namespace_rec.to_dict()
 
 
 def delete(context, name, session):
@@ -277,7 +277,7 @@ def delete(context, name, session):
         else:
             raise e
 
-    return namespace_rec.as_dict()
+    return namespace_rec.to_dict()
 
 
 def delete_cascade(context, name, session):
@@ -304,4 +304,4 @@ def delete_cascade(context, name, session):
             else:
                 raise e
 
-    return namespace_rec.as_dict()
+    return namespace_rec.to_dict()

glance/db/sqlalchemy/metadef_api/object.py

@@ -66,7 +66,7 @@ def get_all(context, namespace_name, session):
 
     md_objects_list = []
     for obj in md_objects:
-        md_objects_list.append(obj.as_dict())
+        md_objects_list.append(obj.to_dict())
     return md_objects_list
 
 
@@ -88,13 +88,13 @@ def create(context, namespace_name, values, session):
         raise exc.MetadefDuplicateObject(
             object_name=md_object.name, namespace_name=namespace_name)
 
-    return md_object.as_dict()
+    return md_object.to_dict()
 
 
 def get(context, namespace_name, name, session):
     md_object = _get_by_name(context, namespace_name, name, session)
 
-    return md_object.as_dict()
+    return md_object.to_dict()
 
 
 def update(context, namespace_name, object_id, values, session):
@@ -119,7 +119,7 @@ def update(context, namespace_name, object_id, values, session):
                 % {'name': md_object.name, 'namespace_name': namespace_name})
         raise exc.MetadefDuplicateObject(emsg)
 
-    return md_object.as_dict()
+    return md_object.to_dict()
 
 
 def delete(context, namespace_name, object_name, session):
@@ -129,7 +129,7 @@ def delete(context, namespace_name, object_name, session):
     session.delete(md_object)
     session.flush()
 
-    return md_object.as_dict()
+    return md_object.to_dict()
 
 
 def delete_namespace_content(context, namespace_id, session):

glance/db/sqlalchemy/metadef_api/property.py

@@ -68,7 +68,7 @@ def get(context, namespace_name, name, session):
     """get a property; raise if ns not found/visible or property not found"""
 
     property_rec = _get_by_name(context, namespace_name, name, session)
-    return property_rec.as_dict()
+    return property_rec.to_dict()
 
 
 def get_all(context, namespace_name, session):
@@ -79,7 +79,7 @@ def get_all(context, namespace_name, session):
 
     properties_list = []
     for prop in properties:
-        properties_list.append(prop.as_dict())
+        properties_list.append(prop.to_dict())
     return properties_list
 
 
@@ -104,7 +104,7 @@ def create(context, namespace_name, values, session):
             property_name=property_rec.name,
             namespace_name=namespace_name)
 
-    return property_rec.as_dict()
+    return property_rec.to_dict()
 
 
 def update(context, namespace_name, property_id, values, session):
@@ -131,7 +131,7 @@ def update(context, namespace_name, property_id, values, session):
                    'namespace_name': namespace_name})
         raise exc.MetadefDuplicateProperty(emsg)
 
-    return property_rec.as_dict()
+    return property_rec.to_dict()
 
 
 def delete(context, namespace_name, property_name, session):
@@ -141,7 +141,7 @@ def delete(context, namespace_name, property_name, session):
         session.delete(property_rec)
         session.flush()
 
-    return property_rec.as_dict()
+    return property_rec.to_dict()
 
 
 def delete_namespace_content(context, namespace_id, session):

glance/db/sqlalchemy/metadef_api/resource_type.py

@@ -37,7 +37,7 @@ def get(context, name, session):
         LOG.debug(msg)
         raise exc.MetadefResourceTypeNotFound(resource_type_name=name)
 
-    return resource_type.as_dict()
+    return resource_type.to_dict()
 
 
 def get_all(context, session):
@@ -48,7 +48,7 @@ def get_all(context, session):
 
     resource_types_list = []
     for rt in resource_types:
-        resource_types_list.append(rt.as_dict())
+        resource_types_list.append(rt.to_dict())
 
     return resource_types_list
 
@@ -69,7 +69,7 @@ def create(context, values, session):
         raise exc.MetadefDuplicateResourceType(
             resource_type_name=resource_type.name)
 
-    return resource_type.as_dict()
+    return resource_type.to_dict()
 
 
 def update(context, values, session):
@@ -81,7 +81,7 @@ def update(context, values, session):
     db_rec.update(values.copy())
     db_rec.save(session=session)
 
-    return db_rec.as_dict()
+    return db_rec.to_dict()
 
 
 def delete(context, name, session):
@@ -108,4 +108,4 @@ def delete(context, name, session):
         else:
             raise e
 
-    return db_rec.as_dict()
+    return db_rec.to_dict()

glance/db/sqlalchemy/metadef_api/resource_type_association.py

@@ -105,7 +105,7 @@ def _create_association(
             resource_type_name=resource_type_name,
             namespace_name=namespace_name)
 
-    return namespace_resource_type_rec.as_dict()
+    return namespace_resource_type_rec.to_dict()
 
 
 def _delete(context, namespace_name, resource_type_name,
@@ -117,7 +117,7 @@ def _delete(context, namespace_name, resource_type_name,
     session.delete(db_rec)
     session.flush()
 
-    return db_rec.as_dict()
+    return db_rec.to_dict()
 
 
 def get(context, namespace_name, resource_type_name, session):

glance/db/sqlalchemy/models_metadef.py

@@ -34,7 +34,7 @@
 class DictionaryBase(models.ModelBase):
     metadata = None
 
-    def as_dict(self):
+    def to_dict(self):
         d = {}
         for c in self.__table__.columns:
             d[c.name] = self[c.name]