[gnome-photos/wip/rishi/collection: 18/47] source: Prevent the ID and name from getting overwritten

[Debarshi Ray <debarshir src gnome org>]

commit ae34f02a06ae81b43183f5f89fbe83c9fac4bf96
Author: Debarshi Ray <debarshir gnome org>
Date:   Thu Jan 11 19:04:54 2018 +0100

    source: Prevent the ID and name from getting overwritten
    
    The "id" and "name" properties of a Source created from a GoaObject are
    set based on attributes of the GoaObject, and are not separately routed
    through GObject's property machinery. Since they are marked as
    construct properties, GObject will set them to their default values,
    which is NULL in both cases, regardless of whether they were already
    set outside of the property machinery.
    
    This hasn't been a problem so far because GObject sorts the construct
    properties alphabetically before setting them, and "id" and "name"
    precede "object". This will break when a "mount" property, analogous to
    "object", for GMount-backed sources in a subsequent commit.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=751212

 src/photos-source.c | 26 ++++++++++++++++++++++----
 1 file changed, 22 insertions(+), 4 deletions(-)
---
diff --git a/src/photos-source.c b/src/photos-source.c
index dec7fe08..89551d8d 100644
--- a/src/photos-source.c
+++ b/src/photos-source.c
@@ -182,12 +182,30 @@ photos_source_set_property (GObject *object, guint prop_id, const GValue *value,
       break;
 
     case PROP_ID:
-      self->id = g_value_dup_string (value);
-      break;
+      {
+        const gchar *id;
+
+        id = g_value_get_string (value);
+        g_return_if_fail (self->id == NULL || id == NULL);
+
+        if (self->id == NULL && id != NULL)
+          self->id = g_strdup (id);
+
+        break;
+      }
 
     case PROP_NAME:
-      self->name = g_value_dup_string (value);
-      break;
+      {
+        const gchar *name;
+
+        name = g_value_get_string (value);
+        g_return_if_fail (self->name == NULL || name == NULL);
+
+        if (self->name == NULL && name != NULL)
+          self->name = g_strdup (name);
+
+        break;
+      }
 
     case PROP_OBJECT:
       {