Merge lp:~kamstrup/libunity/search-really-finished into lp:libunity

[Mikkel Kamstrup Erlandsen]

Fixes attached bug across alt-f2, u-pf, u-p-a, and any other place implementation using search.finished() in one fell swoop. See commit log for details.

[Didier Roche-Tolomelli (didrocks)]

oh! this long standing issue was a Mikkel's bug zomg /o\

Excellent, works as discussed on IRC! approved :)

review: Approve

=== modified file 'src/unity-place.vala'
--- src/unity-place.vala	2011-03-21 10:57:23 +0000
+++ src/unity-place.vala	2011-04-06 14:42:19 +0000
@@ -255,7 +255,12 @@
       try {
         Variant vhints = hints;
         var params = new Variant ("(us@a{ss})", section, search, vhints);
-        var conn = Bus.get_sync(BusType.SESSION);      
+        var conn = Bus.get_sync(BusType.SESSION);
+        
+        /* Make sure we push any changes to the models *before* sending
+         * the SearchFinished signal */
+        parent_entry.flush_all_models ();
+        
         conn.emit_signal (null,                                 /* dest name */
                           parent_entry.dbus_path,               /* orig path */
                           "com.canonical.Unity.PlaceEntry",     /* iface */
@@ -482,6 +487,34 @@
       return info.hints.size ();
     }
 
+    public void flush_all_models ()
+    {
+      /* Important note: If the revision queues of the models are empty
+       *                 these calls are just no-ops */
+      if (sections_model is Dee.SharedModel)
+        {
+          (sections_model as Dee.SharedModel).flush_revision_queue ();
+        }
+      
+      if (entry_renderer_info.results_model is Dee.SharedModel)
+        {
+          (entry_renderer_info.results_model as Dee.SharedModel).flush_revision_queue ();
+        }
+      if (entry_renderer_info.groups_model is Dee.SharedModel)
+        {
+          (entry_renderer_info.groups_model as Dee.SharedModel).flush_revision_queue ();
+        }
+      
+      if (global_renderer_info.results_model is Dee.SharedModel)
+        {
+          (entry_renderer_info.results_model as Dee.SharedModel).flush_revision_queue ();
+        }
+      if (global_renderer_info.groups_model is Dee.SharedModel)
+        {
+          (global_renderer_info.groups_model as Dee.SharedModel).flush_revision_queue ();
+        }
+    }
+
     /*
      * Internal API
      */