server: Remove useless update events
authorJens Georg <jensg@openismus.com>
Thu, 10 Jan 2013 10:20:26 +0000 (11:20 +0100)
committerJens Georg <jensg@openismus.com>
Thu, 10 Jan 2013 15:49:00 +0000 (16:49 +0100)
src/librygel-server/rygel-trackable-container.vala
src/plugins/media-export/rygel-media-export-harvester.vala
src/plugins/media-export/rygel-media-export-harvesting-task.vala

index 566de68..e22feb4 100644 (file)
@@ -61,6 +61,9 @@ public interface Rygel.TrackableContainer : Rygel.MediaContainer {
 
         this.updated (object, ObjectEventType.DELETED);
         this.total_deleted_child_count++;
+
+        // FIXME: Check if modification of child_count should lead to
+        // LastChange event.
         this.updated ();
     }
 
index f0a0a0f..07f93d3 100644 (file)
@@ -245,11 +245,6 @@ internal class Rygel.MediaExport.Harvester : GLib.Object {
 
                 object = parent;
             }
-
-            if (parent != null) {
-                parent.updated ();
-                cache.save_container (parent);
-            }
         } catch (Error error) {
             warning (_("Error removing object from database: %s"),
                      error.message);
index 037cc5b..c3c8c52 100644 (file)
@@ -289,10 +289,6 @@ public class Rygel.MediaExport.HarvestingTask : Rygel.StateMachine,
                                             this.flag);
                 } catch (Error error) {};
             }
-            parent.updated (parent);
-            try {
-                this.cache.save_container (parent);
-            } catch (Error error) { }
 
             this.completed ();
         }
@@ -378,11 +374,7 @@ public class Rygel.MediaExport.HarvestingTask : Rygel.StateMachine,
             var container = this.containers.peek_head ();
             try {
                 var cache = MediaCache.get_default ();
-                if (cache.get_child_count (container.id) > 0) {
-                    var head = this.containers.peek_head ();
-                    head.updated (head);
-                    cache.save_container (head);
-                } else {
+                if (cache.get_child_count (container.id) == 0) {
                     var parent = container.parent as TrackableContainer;
                     parent.remove_child_tracked.begin (container, () => {
                         try {