[libgovirt] Introduce auxiliary function ovirt_sub_collection_new_from_resource()

[Christophe Fergeau <teuf src gnome org>]

commit 38156b5390f85fc041d738ce3f9f80b074b88380
Author: Eduardo Lima (Etrunko) <etrunko redhat com>
Date:   Thu May 18 16:45:37 2017 -0300

    Introduce auxiliary function ovirt_sub_collection_new_from_resource()
    
    This function eliminates duplication of code used to create a
    subcollection from a resource. Users are updated accordingly.
    
    Signed-off-by: Eduardo Lima (Etrunko) <etrunko redhat com>

 govirt/ovirt-api.c                |   50 +++++++++++++-----------------------
 govirt/ovirt-collection-private.h |    5 +++
 govirt/ovirt-collection.c         |   15 +++++++++++
 govirt/ovirt-storage-domain.c     |   18 ++++---------
 govirt/ovirt-vm.c                 |   18 ++++---------
 5 files changed, 50 insertions(+), 56 deletions(-)
---
diff --git a/govirt/ovirt-api.c b/govirt/ovirt-api.c
index 001ee42..37c0935 100644
--- a/govirt/ovirt-api.c
+++ b/govirt/ovirt-api.c
@@ -123,18 +123,14 @@ OvirtApi *ovirt_api_new(void)
  */
 OvirtCollection *ovirt_api_get_vms(OvirtApi *api)
 {
-    const char *href;
-
     g_return_val_if_fail(OVIRT_IS_API(api), NULL);
 
-    if (api->priv->vms != NULL)
-        return api->priv->vms;
-
-    href = ovirt_resource_get_sub_collection(OVIRT_RESOURCE(api), "vms");
-    if (href == NULL)
-        return NULL;
-
-    api->priv->vms = ovirt_collection_new(href, "vms", OVIRT_TYPE_VM, "vm");
+    if (api->priv->vms == NULL)
+        api->priv->vms = ovirt_sub_collection_new_from_resource(OVIRT_RESOURCE(api),
+                                                                "vms",
+                                                                "vms",
+                                                                OVIRT_TYPE_VM,
+                                                                "vm");
 
     return api->priv->vms;
 }
@@ -151,18 +147,14 @@ OvirtCollection *ovirt_api_get_vms(OvirtApi *api)
  */
 OvirtCollection *ovirt_api_get_vm_pools(OvirtApi *api)
 {
-    const char *href;
-
     g_return_val_if_fail(OVIRT_IS_API(api), NULL);
 
-    if (api->priv->vm_pools != NULL)
-        return api->priv->vm_pools;
-
-    href = ovirt_resource_get_sub_collection(OVIRT_RESOURCE(api), "vmpools");
-    if (href == NULL)
-        return NULL;
-
-    api->priv->vm_pools = ovirt_collection_new(href, "vmpools", OVIRT_TYPE_VM_POOL, "vmpool");
+    if (api->priv->vm_pools == NULL)
+        api->priv->vm_pools = ovirt_sub_collection_new_from_resource(OVIRT_RESOURCE(api),
+                                                                     "vmpools",
+                                                                     "vmpools",
+                                                                     OVIRT_TYPE_VM_POOL,
+                                                                     "vmpool");
 
     return api->priv->vm_pools;
 }
@@ -180,20 +172,14 @@ OvirtCollection *ovirt_api_get_vm_pools(OvirtApi *api)
  */
 OvirtCollection *ovirt_api_get_storage_domains(OvirtApi *api)
 {
-    const char *href;
-
     g_return_val_if_fail(OVIRT_IS_API(api), NULL);
 
-    if (api->priv->storage_domains != NULL)
-        return api->priv->storage_domains;
-
-    href = ovirt_resource_get_sub_collection(OVIRT_RESOURCE(api), "storagedomains");
-    if (href == NULL)
-        return NULL;
-
-    api->priv->storage_domains = ovirt_collection_new(href, "storage_domains",
-                                                      OVIRT_TYPE_STORAGE_DOMAIN,
-                                                      "storage_domain");
+    if (api->priv->storage_domains == NULL)
+        api->priv->storage_domains = ovirt_sub_collection_new_from_resource(OVIRT_RESOURCE(api),
+                                                                            "storagedomains",
+                                                                            "storage_domains",
+                                                                            OVIRT_TYPE_STORAGE_DOMAIN,
+                                                                            "storage_domain");
 
     return api->priv->storage_domains;
 }
diff --git a/govirt/ovirt-collection-private.h b/govirt/ovirt-collection-private.h
index 5bc294f..d955fc6 100644
--- a/govirt/ovirt-collection-private.h
+++ b/govirt/ovirt-collection-private.h
@@ -41,6 +41,11 @@ OvirtCollection *ovirt_collection_new_from_xml(RestXmlNode *root_node,
                                                GType resource_type,
                                                const char *resource_name,
                                                GError **error);
+OvirtCollection *ovirt_sub_collection_new_from_resource(OvirtResource *resource,
+                                                        const char *href,
+                                                        const char *collection_name,
+                                                        GType resource_type,
+                                                        const char *resource_name);
 
 G_END_DECLS
 
diff --git a/govirt/ovirt-collection.c b/govirt/ovirt-collection.c
index a3b0f3f..6ec1c6e 100644
--- a/govirt/ovirt-collection.c
+++ b/govirt/ovirt-collection.c
@@ -344,6 +344,21 @@ OvirtCollection *ovirt_collection_new_from_xml(RestXmlNode *root_node,
 }
 
 
+OvirtCollection *ovirt_sub_collection_new_from_resource(OvirtResource *resource,
+                                                        const char *href,
+                                                        const char *collection_name,
+                                                        GType resource_type,
+                                                        const char *resource_name)
+{
+    const char *link = ovirt_resource_get_sub_collection(resource, href);
+
+    if (link == NULL)
+        return NULL;
+
+    return ovirt_collection_new(link, collection_name, resource_type, resource_name);
+}
+
+
 /**
  * ovirt_collection_fetch:
  * @collection: a #OvirtCollection
diff --git a/govirt/ovirt-storage-domain.c b/govirt/ovirt-storage-domain.c
index 0582203..38c4a62 100644
--- a/govirt/ovirt-storage-domain.c
+++ b/govirt/ovirt-storage-domain.c
@@ -317,20 +317,14 @@ OvirtStorageDomain *ovirt_storage_domain_new(void)
  */
 OvirtCollection *ovirt_storage_domain_get_files(OvirtStorageDomain *domain)
 {
-    const char *href;
-
     g_return_val_if_fail(OVIRT_IS_STORAGE_DOMAIN(domain), NULL);
 
-    if (domain->priv->files != NULL)
-        return domain->priv->files;
-
-    href = ovirt_resource_get_sub_collection(OVIRT_RESOURCE(domain), "files");
-    if (href == NULL)
-        return NULL;
-
-    domain->priv->files = ovirt_collection_new(href, "files",
-                                               OVIRT_TYPE_RESOURCE,
-                                               "file");
+    if (domain->priv->files == NULL)
+        domain->priv->files = ovirt_sub_collection_new_from_resource(OVIRT_RESOURCE(domain),
+                                                                     "files",
+                                                                     "files",
+                                                                     OVIRT_TYPE_RESOURCE,
+                                                                     "file");
 
     return domain->priv->files;
 }
diff --git a/govirt/ovirt-vm.c b/govirt/ovirt-vm.c
index 9a07c2f..3d64649 100644
--- a/govirt/ovirt-vm.c
+++ b/govirt/ovirt-vm.c
@@ -329,20 +329,14 @@ gboolean ovirt_vm_refresh_finish(OvirtVm *vm,
  */
 OvirtCollection *ovirt_vm_get_cdroms(OvirtVm *vm)
 {
-    const char *href;
-
     g_return_val_if_fail(OVIRT_IS_VM(vm), NULL);
 
-    if (vm->priv->cdroms != NULL)
-        return vm->priv->cdroms;
-
-    href = ovirt_resource_get_sub_collection(OVIRT_RESOURCE(vm), "cdroms");
-    if (href == NULL)
-        return NULL;
-
-    vm->priv->cdroms =  ovirt_collection_new(href, "cdroms",
-                                             OVIRT_TYPE_CDROM,
-                                             "cdrom");
+    if (vm->priv->cdroms == NULL)
+        vm->priv->cdroms = ovirt_sub_collection_new_from_resource(OVIRT_RESOURCE(vm),
+                                                                  "cdroms",
+                                                                  "cdroms",
+                                                                  OVIRT_TYPE_CDROM,
+                                                                  "cdrom");
 
     return vm->priv->cdroms;
 }