Merge lp:~stolowski/unity-lens-applications/unity-lens-applications.formatting-fixes into lp:unity-lens-applications

[Paweł Stołowski]

Formatting fixes.

[lp:~stolowski/unity-lens-applications/unity-lens-applications.formatting-fixes]

313. By Paweł Stołowski on 2012-08-08

Merged previews branch.

314. By Paweł Stołowski on 2012-08-08

Some more formatting fixes.

315. By Paweł Stołowski on 2012-08-21

Merged trunk.

Unmerged revisions

=== modified file 'configure.ac'
--- configure.ac	2012-04-27 07:39:53 +0000
+++ configure.ac	2012-08-08 16:35:21 +0000
@@ -103,9 +103,9 @@
 # look for dbus service dir
 #############################################
 if test "x$with_localinstall" = "xyes"; then
-	DBUSSERVICEDIR="${datadir}/dbus-1/services/"
+        DBUSSERVICEDIR="${datadir}/dbus-1/services/"
 else
-	DBUSSERVICEDIR=`$PKG_CONFIG --variable=session_bus_services_dir dbus-1`
+        DBUSSERVICEDIR=`$PKG_CONFIG --variable=session_bus_services_dir dbus-1`
 fi
 AC_SUBST(DBUSSERVICEDIR)
 
@@ -140,7 +140,7 @@
 
   Prefix         : ${prefix}
   Sysconfdir     : ${sysconfdir}
-  
+
   Local install  : ${with_localinstall}
 
   Extra CFlags   : ${CPPFLAGS} $MAINTAINER_CFLAGS

=== modified file 'src/Makefile.am'
--- src/Makefile.am	2012-01-24 11:30:08 +0000
+++ src/Makefile.am	2012-08-08 16:35:21 +0000
@@ -19,7 +19,7 @@
   -I$(srcdir) \
   -g
 
-# Note that we require the GLIB_2_22 flag for Valac because libzeitgeist
+# Note that we require glib 2.26 because libzeitgeist
 # expects us to use g_ptr_array_unref instead of g_ptr_array_free.
 unity_applications_daemon_VALAFLAGS = \
   -C \
@@ -35,7 +35,7 @@
   --pkg libgnome-menu \
   --pkg unity-package-search \
   --pkg unity-ratings-db \
-  --define=GLIB_2_22 \
+  --target-glib=2.26 \
   $(MAINTAINER_VALAFLAGS)
 
 unity_package_search_libs = -lxapian -lstdc++
@@ -54,10 +54,13 @@
   app-watcher.vala \
   config.vala \
   daemon.vala \
+  launcher-client.vala \
   main.vala \
   runner.vala \
   schemas.vala \
   utils.vala \
+  aptd-client.vala \
+  software-center-data-provider.vala \
   $(NULL)
 
 unity-package-search.o : $(srcdir)/unity-package-search.cc $(srcdir)/unity-package-search.h

=== added file 'src/aptd-client.vala'
--- src/aptd-client.vala	1970-01-01 00:00:00 +0000
+++ src/aptd-client.vala	2012-08-08 16:35:21 +0000
@@ -0,0 +1,97 @@
+/*
+ * Copyright (C) 2012 Canonical Ltd
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 3 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ * Authored by Pawel Stolowski <[e-mail]>
+ */
+
+namespace Unity.ApplicationsLens {
+  
+  static const string APTD_DBUS_NAME = "org.debian.apt";
+  static const string APTD_DBUS_PATH = "/org/debian/apt";
+
+  /**
+   * Expose a subset of org.debian.apt interfaces -- only what's needed by applications lens.
+   */
+  [DBus (name = "org.debian.apt")]
+  public interface AptdService: GLib.Object
+  {
+    public abstract async string install_packages (string[] packages) throws IOError;
+    public abstract async string remove_packages (string[] packages) throws IOError;
+    public abstract async void quit () throws IOError;
+  }
+  
+  [DBus (name = "org.debian.apt.transaction")]
+  public interface AptdTransactionService: GLib.Object
+  {
+    public abstract void run () throws IOError;
+    public abstract void simulate () throws IOError;
+    public abstract void cancel () throws IOError;
+    public signal void finished (string exit_state);
+  }
+  
+  public class AptdProxy: GLib.Object
+  {
+    public void connect_to_aptd () throws IOError
+    {
+      _aptd_service = Bus.get_proxy_sync (BusType.SYSTEM, APTD_DBUS_NAME, APTD_DBUS_PATH);
+    }
+
+    public async string install_packages (string[] packages)
+    {
+      string res = yield _aptd_service.install_packages (packages);
+      return res;
+    }
+
+    public async string remove_packages (string[] packages)
+    {
+      string res = yield _aptd_service.remove_packages (packages);
+      return res;
+    }
+
+    public async void quit ()
+    {
+        yield _aptd_service.quit ();
+    }
+
+    private AptdService _aptd_service;
+  }
+
+  public class AptdTransactionProxy: GLib.Object
+  {
+    public signal void finished (string transaction_id);
+
+    public void connect_to_aptd (string transaction_id) throws IOError
+    {
+      _aptd_service = Bus.get_proxy_sync (BusType.SYSTEM, APTD_DBUS_NAME, transaction_id);
+      _aptd_service.finished.connect ((exit_state) =>
+        {
+          debug ("aptd transaction finished: %s\n", exit_state);
+          finished (transaction_id);
+        });
+    }
+
+    public void simulate ()
+    {
+      _aptd_service.simulate ();
+    }
+
+    public void run ()
+    {
+      _aptd_service.run ();
+    }
+
+    private AptdTransactionService _aptd_service;
+  }
+}

=== modified file 'src/daemon.vala'
--- src/daemon.vala	2012-04-23 13:58:42 +0000
+++ src/daemon.vala	2012-08-08 16:35:21 +0000
@@ -27,6 +27,7 @@
 namespace Unity.ApplicationsLens {
 
   const string ICON_PATH = Config.DATADIR + "/icons/unity-icon-theme/places/svg/";
+  const string ICON_APP_INSTALL_PATH = Config.DATADIR + "/app-install/icons";
 
   public class Daemon : GLib.Object
   {
@@ -41,13 +42,33 @@
      * the Xapian index from the Software Center */
     private Unity.Package.Searcher? pkgsearcher;
     public Unity.Package.Searcher appsearcher;
-    
+
     /* Read the app ratings dumped by the Software Center */
     private Unity.Ratings.Database? ratings;
 
     private Unity.Lens lens;
     private Unity.Scope scope;
-    
+
+    /* Support aptd dbus interface; created when application install/remove was requested by preview action */
+    private AptdProxy aptdclient;
+    private AptdTransactionProxy aptd_transaction;
+
+    /* Maps demangled names to mangled names expected by S-C xapian DB (e.g. kde4-KCharSelect.desktop -> kde4__KCharSelect.desktop).
+       There are very few apps that need this and we only store mappings when needed, so it takes very little memory.
+     */
+    private HashTable<string, string> mangled_desktop_ids;
+
+    /* Used for adding launcher icon on app installation from the preview */
+    private LauncherProxy launcherservice;
+
+    /* Desktop file & icon name for unity-install:// install candidate displayed in last preview; we store
+     * them here to avoid extra query for app details if app install action is activated */
+    private string preview_installable_desktop_file;
+    private string preview_installable_icon_file;
+
+    /* SoftwareCenter data provider used for app preview details */
+    private SoftwareCenterDataProviderProxy sc_data_provider;
+
     private Unity.ApplicationsLens.Runner runner;
 
     /* Keep references to the FilterOptions for sources */
@@ -60,14 +81,14 @@
 
     private Gee.List<string> image_extensions;
     private HashTable<string,Icon> file_icon_cache;
-    
+
     /* Monitor the favorite apps in the launcher, so we can filter them
      * out of the results for Recent Apps */
     private Unity.LauncherFavorites favorite_apps;
     private AppWatcher app_watcher;
 
     private PtrArray zg_templates;
-    
+
     /* Gnome menu structure - also used to check whether apps are installed */
     private uint app_menu_changed_reindex_timeout = 0;
     private GMenu.Tree app_menu = null;
@@ -95,7 +116,7 @@
       monitor.events_inserted.connect (mark_dirty);
       monitor.events_deleted.connect (mark_dirty);
       log.install_monitor (monitor);
-      
+
       popularity_map = new HashMap<string, int> ();
       popularities_dirty = true;
       // refresh the popularities every now and then
@@ -111,17 +132,17 @@
 
       pkgsearcher = new Unity.Package.Searcher ();
       if (pkgsearcher == null)
-        {
-          critical ("Failed to load Software Center index. 'Apps Available for Download' will not be listed");
-        }
-      
+      {
+        critical ("Failed to load Software Center index. 'Apps Available for Download' will not be listed");
+      }
+
       try {
         ratings = new Unity.Ratings.Database ();
       } catch (FileError e) {
         warning (e.message);
         ratings = null;
       }
-      
+
       /* Image file extensions in order of popularity */
       image_extensions = new Gee.ArrayList<string> ();
       image_extensions.add ("png");
@@ -135,6 +156,7 @@
       build_app_menu_index ();
 
       file_icon_cache = new HashTable<string,Icon>(str_hash, str_equal);
+      mangled_desktop_ids = new HashTable<string, string>(str_hash, str_equal);
 
       scope = new Unity.Scope ("/com/canonical/unity/scope/applications");
       //scope.icon = @"$(Config.PREFIX)/share/unity/themes/applications.png";
@@ -158,7 +180,7 @@
       {
         dispatch_search (lens_search, search_type, cancellable);
       });
-      
+
       /* Re-do the search if the filters changed */
       scope.filters_changed.connect (() =>
       {
@@ -172,13 +194,14 @@
       });
 
       scope.activate_uri.connect (activate);
+      scope.preview_uri.connect (preview);
 
       /* Listen for changes in the installed applications */
       AppInfoManager.get_default().changed.connect (mark_dirty);
-      
+
       /* Now start the RunEntry */
       runner = new Unity.ApplicationsLens.Runner (this);
-      
+
       try {
         uri_regex = new Regex ("^[a-z]+:.+$");
         mountable_regex = new Regex ("((ftp|ssh|sftp|smb|dav)://).+");
@@ -186,14 +209,16 @@
         uri_regex = null;
         critical ("Failed to compile URI regex. URL launching will be disabled");
       }
-      
+
       favorite_apps = Unity.LauncherFavorites.get_default ();
       favorite_apps.changed.connect(mark_dirty);
 
       app_watcher = new AppWatcher ();
       app_watcher.running_applications_changed.connect (mark_dirty);
 
-      
+      aptdclient = new AptdProxy ();
+      launcherservice = new LauncherProxy ();
+
       lens = new Unity.Lens ("/com/canonical/unity/lens/applications", "applications");
       lens.search_hint = _("Search Applications");
       lens.visible = true;
@@ -253,9 +278,9 @@
       var cat = new Unity.Category (_("Recently Used"),
                                     new FileIcon (icon_dir.get_child ("group-recent.svg")));
       categories.append (cat);
-      
+
       cat = new Unity.Category (_("Recent Apps"),
-                                    new FileIcon (icon_dir.get_child ("group-apps.svg")));
+                                new FileIcon (icon_dir.get_child ("group-apps.svg")));
       categories.append (cat);
 
       cat = new Unity.Category (_("Installed"),
@@ -265,11 +290,11 @@
       cat = new Unity.Category (_("Apps Available for Download"),
                                 new FileIcon (icon_dir.get_child ("group-downloads.svg")));
       categories.append (cat);
-      
+
       cat = new Unity.Category (_("Applications"),
                                 new FileIcon (icon_dir.get_child ("group-apps.svg")));
       categories.append (cat);
-      
+
       lens.categories = categories;
     }
 
@@ -281,7 +306,7 @@
       {
         var filter = new CheckOptionFilter ("type", _("Type"));
         filter.sort_type = Unity.OptionsFilter.SortType.DISPLAY_NAME;
-        
+
         filter.add_option ("accessories", _("Accessories"));
         filter.add_option ("education", _("Education"));
         filter.add_option ("game", _("Games"));
@@ -321,38 +346,37 @@
         return usc_apps_option.active;
       return true;
     }
-  
+
     /* Load xdg menu info and build a Xapian index over it.
      * Do throttled re-index if the menu changes */
     private bool build_app_menu_index ()
-    {            
+    {
       if (app_menu == null)
-        {
-          debug ("Building initial application menu");
+      {
+        debug ("Building initial application menu");
         
-          /* We need INCLUDE_NODISPLAY to employ proper de-duping between
-           * the Installed and Availabale categorys. If a NoDisplay app is installed,
-           * eg. Evince, it wont otherwise be in the menu index, only in the
-           * S-C index - thus show up in the Available category */
-          app_menu = GMenu.Tree.lookup ("unity-lens-applications.menu",
-                                        GMenu.TreeFlags.INCLUDE_NODISPLAY);
-          
-          app_menu.add_monitor ((menu) => {
+        /* We need INCLUDE_NODISPLAY to employ proper de-duping between
+         * the Installed and Availabale categorys. If a NoDisplay app is installed,
+         * eg. Evince, it wont otherwise be in the menu index, only in the
+         * S-C index - thus show up in the Available category */
+        app_menu = GMenu.Tree.lookup ("unity-lens-applications.menu",
+                                      GMenu.TreeFlags.INCLUDE_NODISPLAY);
+
+        app_menu.add_monitor ((menu) => {
             /* Reschedule the timeout if we already have one. The menu tree triggers
              * many change events during app installation. This way we wait the full
              * delay *after* the last change event has triggered */
             if (app_menu_changed_reindex_timeout != 0)
               Source.remove (app_menu_changed_reindex_timeout);
-            
-            app_menu_changed_reindex_timeout =
-                                  Timeout.add_seconds (5, build_app_menu_index_and_result_models);
+
+            app_menu_changed_reindex_timeout = Timeout.add_seconds (5, build_app_menu_index_and_result_models);
           });
-        }
-      
+      }
+
       debug ("Indexing application menu");
       appsearcher = new Unity.Package.Searcher.for_menu (app_menu);
       app_menu_changed_reindex_timeout = 0;
-      
+
       return false;
     }
 
@@ -363,7 +387,7 @@
     private bool build_app_menu_index_and_result_models ()
     {
       build_app_menu_index ();
-      
+
       mark_dirty ();
 
       return false;
@@ -391,15 +415,15 @@
                                              OptionsFilter? options)
     {
       string s = search_string.strip ();
-      
+
       if (!s.has_suffix ("*") && s != "")
         s = s + "*";
-      
+
       if (s != "")
         s = @"app:($s)";
       else
         return extract_type_query (options);
-      
+
       if (options == null || !options.filtering)
         return s;
       else
@@ -448,13 +472,13 @@
        * to finish, to prevent flicker. */
 
       debug ("Searching for: %s", search.search_string);
-      
+
       var filter = scope.get_filter ("type") as OptionsFilter;
 
       string pkg_search_string = prepare_pkg_search_string (search, filter);
 
       bool has_search = !Utils.is_search_empty (search);
-      
+
       Timer timer = new Timer ();
 
       var transaction = new Dee.Transaction (model);
@@ -513,8 +537,7 @@
           // no need to bother
           return;
         } catch (GLib.Error e) {
-          warning ("Error performing search '%s': %s",
-                   search.search_string, e.message);
+          warning ("Error performing search '%s': %s", search.search_string, e.message);
         }
       }
 
@@ -556,7 +579,7 @@
 
       search.finished ();
     }
-    
+
     private async void update_global_search (Unity.LensSearch search,
                                              Cancellable cancellable)
     {
@@ -564,17 +587,17 @@
        * In global search, with a non-empty search string, we collate all
        * hits under one Applications category
        */
-      
+
       if (Utils.is_search_empty (search))
-        {
-          yield update_global_without_search (search, cancellable);
-          return;
-        }
-      
+      {
+        yield update_global_without_search (search, cancellable);
+        return;
+      }
+
       var model = search.results_model;
-    
+
       model.clear ();
-      
+
       var search_string = prepare_pkg_search_string (search, null);
       Set<string> installed_uris = new HashSet<string> ();
       Set<string> available_uris = new HashSet<string> ();
@@ -585,18 +608,18 @@
       resort_pkg_search_results (appresults);
       add_pkg_search_result (appresults, installed_uris, available_uris, model,
                              Category.APPLICATIONS);
-      
+
       timer.stop ();
       debug ("Global search listed %i Installed apps in %fms for query: %s",
              appresults.num_hits, timer.elapsed ()*1000, search_string);
-      
+
       /* Allow new searches once we enter an idle again.
        * We don't do it directly from here as that could mean we start
        * changing the model even before we had flushed out current changes
        */
       search.finished ();
     }
-    
+
     private async void update_global_without_search (Unity.LensSearch search,
                                                      Cancellable cancellable)
     {
@@ -605,7 +628,7 @@
        * Excluding apps with icons in the launcher (be they running or faves)
        */
       var model = search.results_model;      
-      
+
       Timer timer = new Timer ();
 
       if (local_apps_active () && display_recent_apps)
@@ -678,103 +701,122 @@
 
       return result == null ? "NOT category:XYZ" : "(%s)".printf (result);
     }
-    
+
     private string prepare_pkg_search_string (Unity.LensSearch search,
                                               OptionsFilter? options)
     {
       if (Utils.is_search_empty (search))
-        {
-          if (options == null || !options.filtering)
-            return "type:Application";
-          else
-            return "type:Application AND " + extract_type_query (options);
-        }
+      {
+        if (options == null || !options.filtering)
+          return "type:Application";
+        else
+          return "type:Application AND " + extract_type_query (options);
+      }
       else
+      {
+        var s = search.search_string;
+
+        s = s.strip ();
+
+        /* The Xapian query parser seems to handle hyphens in a special way,
+         * namely that it forces the joined tokens into a phrase query
+         * no matter if it appears as the last word in a query and we have
+         * the PARTIAL flag set on the query parser. This makes 'd-f' not
+         * match 'd-feet' etc. */
+        s = s.delimit ("-", ' ');
+
+        if (options == null || !options.filtering)
+          return "type:Application AND " + s;
+        else
+          return "type:Application AND %s AND %s".printf (extract_type_query (options), s);
+      }
+    }
+
+    /**
+     * Find app icon in DATADIR/app-install/icons, based on package name
+     * trying all supported image extensions.
+     */
+    public string find_app_install_icon_path (string pkgname)
+    {
+      string icon = @"$ICON_APP_INSTALL_PATH/$preview_installable_icon_file.";
+      string[] exts = {"png", "xpm"};
+      foreach (string ext in exts)
+      {
+        string path = icon + ext;
+        if (FileUtils.test (path, FileTest.EXISTS))
         {
-          var s = search.search_string;
-
-          s = s.strip ();
-          
-          /* The Xapian query parser seems to handle hyphens in a special way,
-           * namely that it forces the joined tokens into a phrase query
-           * no matter if it appears as the last word in a query and we have
-           * the PARTIAL flag set on the query parser. This makes 'd-f' not
-           * match 'd-feet' etc. */
-          s = s.delimit ("-", ' ');
-
-          if (options == null || !options.filtering)
-            return "type:Application AND " + s;
-          else
-            return "type:Application AND %s AND %s".printf (extract_type_query (options), s);
+          return path;
         }
+      }
+      return "";
     }
-    
+
     public Icon find_pkg_icon (Unity.Package.PackageInfo pkginfo)
     {
       string desktop_id = Path.get_basename (pkginfo.desktop_file);
       bool installed = AppInfoManager.get_default().lookup (desktop_id) != null;
-      
+
       /* If the app is already installed we should be able to pull the
        * icon from the theme */
       if (installed)
         return new ThemedIcon (pkginfo.icon);
-      
+
       /* App is not installed - we need to find the right icon in the bowels
        * of the software center */
       if (pkginfo.icon.has_prefix ("/"))
-        {
-          return new FileIcon (File.new_for_path (pkginfo.icon));
-        }
+      {
+        return new FileIcon (File.new_for_path (pkginfo.icon));
+      }
       else
-        {
-          Icon icon = file_icon_cache.lookup (pkginfo.icon);
-          
-          if (icon != null)
-            return icon;
-          
-          /* If the icon name contains a . it probably already have a
-           * type postfix - so test icon name directly */
-          string path;
-          if ("." in pkginfo.icon)
-            {
-              path = @"$(Config.DATADIR)/app-install/icons/$(pkginfo.icon)";
-              if (FileUtils.test (path, FileTest.EXISTS))
-                {
-                  icon = new FileIcon (File.new_for_path (path));
-                  file_icon_cache.insert (pkginfo.icon, icon);
-                  return icon;
-                }
-              /* Try also software center cache dir */
-              path = Path.build_filename (Environment.get_user_cache_dir (),
-                                          "software-center",
-                                          "icons",
-                                          pkginfo.icon);
-              if (FileUtils.test (path, FileTest.EXISTS))
-                {
-                  icon = new FileIcon (File.new_for_path (path));
-                  file_icon_cache.insert (pkginfo.icon, icon);
-                  return icon;
-                }
-            }
-          
-          /* Now try appending all the image extensions we know */
-          foreach (var ext in image_extensions)
-          {
-            path = @"$(Config.DATADIR)/app-install/icons/$(pkginfo.icon).$(ext)";
-            if (FileUtils.test (path, FileTest.EXISTS))
-              {
-                /* Got it! Cache the icon path and return the icon */
-                icon = new FileIcon (File.new_for_path (path));
-                file_icon_cache.insert (pkginfo.icon, icon);
-                return icon;
-              }
-          }
-        }
-      
+      {
+        Icon icon = file_icon_cache.lookup (pkginfo.icon);
+
+        if (icon != null)
+          return icon;
+
+        /* If the icon name contains a . it probably already have a
+         * type postfix - so test icon name directly */
+        string path;
+        if ("." in pkginfo.icon)
+        {
+          path = @"$(Config.DATADIR)/app-install/icons/$(pkginfo.icon)";
+          if (FileUtils.test (path, FileTest.EXISTS))
+          {
+            icon = new FileIcon (File.new_for_path (path));
+            file_icon_cache.insert (pkginfo.icon, icon);
+            return icon;
+          }
+          /* Try also software center cache dir */
+          path = Path.build_filename (Environment.get_user_cache_dir (),
+                                      "software-center",
+                                      "icons",
+                                      pkginfo.icon);
+          if (FileUtils.test (path, FileTest.EXISTS))
+          {
+            icon = new FileIcon (File.new_for_path (path));
+            file_icon_cache.insert (pkginfo.icon, icon);
+            return icon;
+          }
+        }
+
+        /* Now try appending all the image extensions we know */
+        foreach (var ext in image_extensions)
+        {
+          path = @"$(Config.DATADIR)/app-install/icons/$(pkginfo.icon).$(ext)";
+          if (FileUtils.test (path, FileTest.EXISTS))
+          {
+            /* Got it! Cache the icon path and return the icon */
+            icon = new FileIcon (File.new_for_path (path));
+            file_icon_cache.insert (pkginfo.icon, icon);
+            return icon;
+          }
+        }
+      }
+
       /* Cache the fact that we couldn't find this icon */
       var icon = new ThemedIcon ("applications-other");
       file_icon_cache.insert (pkginfo.icon, icon);
-      
+
       return icon;
     }
 
@@ -791,7 +833,14 @@
         desktop_id = desktop_id[colon_pos+1:desktop_id.length];
         /* well it's still not real desktop_id, S-C converts slashes to "__"
          * if the desktop file is in /usr/share/applications */
-        desktop_id = desktop_id.replace ("__", "-");
+        string demangled_desktop_id = desktop_id.replace ("__", "-");
+
+        // store demangled name -> mangled name mapping
+        if (desktop_id != demangled_desktop_id)
+        {
+          mangled_desktop_ids.replace (demangled_desktop_id, desktop_id);
+        }
+        desktop_id = demangled_desktop_id;
       }
 
       return desktop_id;
@@ -824,7 +873,7 @@
         return rel_b - rel_a; // we want higher relevancy first
       });
     }
-    
+
     private void add_pkg_search_result (Unity.Package.SearchResult results,
                                         Set<string> installed_uris,
                                         Set<string> available_uris,
@@ -838,29 +887,29 @@
 
       foreach (var pkginfo in results.results)
       {
-      	if (pkginfo.desktop_file == null)
+        if (pkginfo.desktop_file == null)
           continue;
-        
+
         string desktop_id = extract_desktop_id (pkginfo.desktop_file,
-          category == Category.AVAILABLE);
+                                                category == Category.AVAILABLE);
         string full_path;
 
         AppInfo? app = appmanager.lookup (desktop_id);
         full_path = appmanager.get_path (desktop_id);
-        
+
         /* De-dupe by 'application://foo.desktop' URI. Also note that we need
          * to de-dupe before we chuck out NoDisplay app infos, otherwise they'd
          * show up from alternate sources */
         string uri = @"application://$(desktop_id)";
         if (uri in installed_uris || uri in available_uris)
           continue;
-        
+
         /* Extract basic metadata and register de-dupe keys */
         string display_name;
         string comment;
         switch (category)
         {
-          case Category.INSTALLED:
+        case Category.INSTALLED:
           case Category.APPLICATIONS:
             installed_uris.add (uri);
             display_name = app.get_display_name ();
@@ -874,52 +923,52 @@
           default:
             warning (@"Illegal category for package search $(category)");
             continue;
-        } 
-        
+        }
+
         /* We can only chuck out NoDisplay and OnlyShowIn app infos after
          * we have registered a de-dupe key for them - which is done in the
          * switch block above) */
         if (app != null && !app.should_show ())
           continue;
-                
+
         if (category == Category.AVAILABLE)
+        {
+          /* If we have an available item, which is not a dupe, but is
+           * installed anyway, we weed it out here, because it's probably
+           * left out from the Installed section because of some rule in the
+           * .menu file */
+          if (app != null)
+            continue;
+
+          /* Filter by app rating in Software Center if enabled */
+          if (ratings != null && ratings_filter.rating > 0.00001)
           {
-            /* If we have an available item, which is not a dupe, but is
-             * installed anyway, we weed it out here, because it's probably
-             * left out from the Installed section because of some rule in the
-             * .menu file */
-            if (app != null)
+            Unity.Ratings.Result result;
+            if (ratings.query (pkginfo.package_name, out result))
+            {
+              if (result.average_rating < ratings_filter.rating * 5 - 0.2)
+                continue;
+            }
+            else
               continue;
-            
-            /* Filter by app rating in Software Center if enabled */
-            if (ratings != null && ratings_filter.rating > 0.00001)
-              {
-                Unity.Ratings.Result result;
-                if (ratings.query (pkginfo.package_name, out result))
-                  {
-                    if (result.average_rating < ratings_filter.rating * 5 - 0.2)
-                      continue;
-                  }
-                else
-                  continue;
-              }
-            
-            /* Apps that are not installed, ie. in the Available category
-             * use the 'unity-install://pkgname/Full App Name' URI scheme,
-             * but only use that after we've de-duped the results.
-             * But only change the URI *after* we've de-duped the results! */
-            uri = @"unity-install://$(pkginfo.package_name)/$(pkginfo.application_name)";
-            available_uris.add (uri);            
           }
-        
+
+          /* Apps that are not installed, ie. in the Available category
+           * use the 'unity-install://pkgname/Full App Name' URI scheme,
+           * but only use that after we've de-duped the results.
+           * But only change the URI *after* we've de-duped the results! */
+          uri = @"unity-install://$(pkginfo.package_name)/$(pkginfo.application_name)";
+          available_uris.add (uri);            
+        }
+
         Icon icon = find_pkg_icon (pkginfo);
-        
+
         model.append (uri, icon.to_string (),
                       category,"application/x-desktop",
                       display_name != null ? display_name : "",
                       comment != null ? comment : "", 
-		      full_path != null ? "file://" + full_path : "");
-        
+                      full_path != null ? "file://" + full_path : "");
+
         /* Stop if we added the number of items requested */
         n_added++;
         if (max_add > 0 && n_added >= max_add)
@@ -927,6 +976,248 @@
       }
     }
 
+    private async void call_install_packages (string package_name, out string tid)
+    {
+      tid = yield aptdclient.install_packages ({package_name});
+    }
+
+    private async void call_remove_packages (string package_name, out string tid)
+    {
+      tid = yield aptdclient.remove_packages ({package_name});
+    }
+
+    private Unity.ActivationResponse app_preview_launch (string uri)
+    {
+      return activate (uri);
+    }
+
+    /**
+     * Handler for free apps installation.
+     * Triggers package installation via apt-daemon DBus service
+     */
+    private Unity.ActivationResponse app_preview_install (string uri)
+    {
+      if (uri.has_prefix ("unity-install://"))
+      {
+        string app = uri.substring (16); // trim "unity-install://"
+        string[] parts = app.split ("/");
+
+        if (parts.length > 1)
+        {
+          string pkgname = parts[0];
+          string appname = parts[1];
+          aptdclient.connect_to_aptd ();
+          call_install_packages.begin (pkgname, (obj, res) =>
+          {
+            try {
+              string tid;
+              call_install_packages.end (res, out tid);
+              debug ("transaction started: %s, pkg: %s\n", tid, pkgname);
+              aptd_transaction = new AptdTransactionProxy ();
+              aptd_transaction.connect_to_aptd (tid);
+              aptd_transaction.simulate ();
+              aptd_transaction.run ();
+
+              launcherservice.connect_to_launcher ();
+              string desktop_file = preview_installable_desktop_file;
+              string icon = find_app_install_icon_path (preview_installable_icon_file);
+              
+              launcherservice.add_launcher_item_from_position (appname, icon, 0, 0, 32, desktop_file, tid);
+            }
+            catch (IOError e)
+            {
+              warning (@"Package '$(pkgname)' installation failed: $(e.message)");
+            }
+          });
+        }
+        else
+        {
+          warning (@"Bad install uri: '%s'", uri);
+        }
+      }
+      else
+      {
+        warning (@"Can't handle '%s' in app_preview_install handler", uri);
+        return new Unity.ActivationResponse(Unity.HandledType.NOT_HANDLED);
+      }
+      return new Unity.ActivationResponse(Unity.HandledType.SHOW_DASH);
+    }
+
+    private Unity.ActivationResponse app_preview_install_commercial (string uri)
+    {
+      return activate (uri); //this will just launch Software-Center
+    }
+
+    private Unity.ActivationResponse app_preview_buy (string uri)
+    {
+      return activate (uri); //this will just launch Software-Center
+    }
+
+    private Unity.ActivationResponse app_preview_uninstall (string uri)
+    {
+      if (uri.has_prefix ("application://"))
+      {
+        string desktopfile = uri.substring (14); // trim "application://"
+
+        // de-mangle desktop file names back to what S-C expects
+        if (mangled_desktop_ids.contains (desktopfile))
+        {
+          desktopfile = mangled_desktop_ids.get (desktopfile);
+        }
+
+        var pkginfo = pkgsearcher.get_by_desktop_file (desktopfile);
+
+        if (pkginfo != null && pkginfo.package_name != null)
+        {
+          aptdclient.connect_to_aptd ();
+          call_remove_packages.begin (pkginfo.package_name, (obj, res) =>
+          {
+            try {
+              string tid;
+              call_remove_packages.end (res, out tid);
+              debug ("transaction started: %s, pkg: %s\n", tid, pkginfo.package_name);
+              aptd_transaction = new AptdTransactionProxy ();
+              aptd_transaction.connect_to_aptd (tid);
+              aptd_transaction.simulate ();
+              aptd_transaction.run ();
+            }
+            catch (IOError e)
+            {
+              warning (@"Package '$(pkginfo.package_name)' removal failed: $(e.message)");
+            }
+          });
+          return new Unity.ActivationResponse(Unity.HandledType.SHOW_DASH);
+        }
+        else
+        {
+          warning (@"Cannot find package info for $uri");
+        }
+      }
+      warning (@"Can't handle '%s' in app_preview_uninstall handler", uri);
+      return new Unity.ActivationResponse(Unity.HandledType.NOT_HANDLED);
+    }
+
+    public Unity.Preview preview (string uri)
+    {
+      Unity.ApplicationPreview? preview = null;
+
+      string pkgname = "";
+      string appname = "";
+
+      if (uri.has_prefix ("application://") || uri.has_prefix ("unity-install://"))
+      {
+        if (uri.has_prefix ("application://"))
+        {
+          string desktopfile = uri.substring (14); //remove "application://" prefix
+
+          // de-mangle desktop file names back to what S-C expects
+          if (mangled_desktop_ids.contains (desktopfile))
+          {
+            desktopfile = mangled_desktop_ids.get (desktopfile);
+          }
+
+          Unity.Package.PackageInfo pkginfo = pkgsearcher.get_by_desktop_file (desktopfile);
+          if (pkginfo != null)
+          {
+            appname = pkginfo.application_name;
+            pkgname = pkginfo.package_name;
+          }
+        }
+        else // unity-install
+        {
+          string app = uri.substring (16); //remove "unity-install://" prefix
+          string[] parts = app.split ("/");
+          if (parts.length > 1)
+          {
+            pkgname = parts[0];
+            appname = parts[1];
+          }
+        }
+
+        if (pkgname != "")
+        {
+          try {
+            sc_data_provider = new SoftwareCenterDataProviderProxy ();
+            sc_data_provider.connect_to ();
+
+            debug ("Requesting pkg info: %s, %s\n", pkgname, appname);
+            sc_data_provider.get_app_details (appname, pkgname);
+
+            Icon icon = new GLib.ThemedIcon (sc_data_provider.icon);         
+            Icon? screenshot = null;
+
+            if (sc_data_provider.screenshot != null)
+            {
+              File scr_file = File.new_for_uri (sc_data_provider.screenshot);
+              screenshot = new FileIcon (scr_file);
+            }
+
+            preview = new Unity.ApplicationPreview (sc_data_provider.name, sc_data_provider.summary, sc_data_provider.description, icon, screenshot);
+            preview.license = sc_data_provider.license;
+
+            if (ratings !=null)
+            {
+              Unity.Ratings.Result result;
+              ratings.query (pkgname, out result);
+              preview.set_rating (result.average_rating, result.total_rating);
+            }
+
+            preview.add_info (new InfoHint.with_variant ("size", _("Size"), null, new GLib.Variant.int64 (sc_data_provider.size)));
+            preview.add_info (new InfoHint ("hardware-requirements", _("Hardware requirements"), null, sc_data_provider.hardware_requirements));
+            preview.add_info (new InfoHint ("version", _("Version"), null, sc_data_provider.version));
+            preview.add_info (new InfoHint ("website", _("Website"), null, sc_data_provider.website));
+            preview.add_info (new InfoHint ("price", _("Price"), null, sc_data_provider.price));
+
+            if (uri.has_prefix ("unity-install://")) // application needs to be purchased/installed
+            {
+              // uninstalled and not purchased before
+              if (sc_data_provider.pkg_state == SoftwareCenterDataProviderProxy.PackageState.NEEDS_PURCHASE)
+              {
+                var buy_action = new Unity.PreviewAction ("buy", _("Buy"), null);
+                buy_action.activated.connect (app_preview_buy);
+                preview.add_action (buy_action);
+              }
+              else // uninstalled, purchased before
+              {
+                var install_action = new Unity.PreviewAction ("install", _("Install"), null);
+                if (sc_data_provider.price != "")
+                {
+                  install_action.activated.connect (app_preview_install_commercial);
+                }
+                else
+                {
+                  install_action.activated.connect (app_preview_install);
+                }
+                preview.add_action (install_action);
+              }
+            }
+            else // application is already installed
+            {
+              preview.add_info (new InfoHint ("date-installed", _("Installed on"), null, sc_data_provider.installation_date));
+              var launch_action = new Unity.PreviewAction ("launch", _("Launch"), null);
+              launch_action.activated.connect (app_preview_launch);
+              preview.add_action (launch_action);
+              var uninstall_action = new Unity.PreviewAction ("uninstall", _("Uninstall"), null);
+              uninstall_action.activated.connect (app_preview_uninstall);
+              preview.add_action (uninstall_action);
+            }
+
+            preview_installable_desktop_file = sc_data_provider.desktop_file;
+            preview_installable_icon_file = sc_data_provider.icon;
+          }
+          catch (IOError e)
+          {
+            warning ("Failed to get package details for '%s': %s", uri, e.message);
+          }
+        }
+        else
+        {
+          warning ("pksearcher: no data for '%s'", uri);
+        }
+      }
+      return preview;
+    }
+
     /**
      * Override of the default activation handler. The apps lens daemon
      * can handle activation of installable apps using the Software Center
@@ -936,79 +1227,79 @@
       string[] args;
       string exec_or_dir = null;
       if (uri.has_prefix ("unity-install://"))
-        {
-          unowned string pkg = uri.offset (16); // strip off "unity-install://" prefix
-          debug ("Installing: %s", pkg);
-          args = new string[2];
-          args[0] = "software-center";
-          args[1] = pkg;
-        }
+      {
+        unowned string pkg = uri.offset (16); // strip off "unity-install://" prefix
+        debug ("Installing: %s", pkg);
+        args = new string[2];
+        args[0] = "software-center";
+        args[1] = pkg;
+      }
       else if (uri.has_prefix ("unity-runner://"))
-        {
-          string orig;
-          orig = uri.offset (15);
-          if (orig.has_prefix("\\\\"))
-            orig = orig.replace ("\\\\","smb://");
-          if (uri_regex != null && uri_regex.match (orig)) {
-            try {
-              /* this code ensures that a file manager will be used
-               * if uri it's a remote location that should be mounted */
-              if (mountable_regex.match (orig)) {
-                var muris = new GLib.List<string>();
-                muris.prepend (orig);
-                var file_manager = AppInfo.get_default_for_type("inode/directory", true);
-                file_manager.launch_uris(muris,null);
-              } else {
+      {
+        string orig;
+        orig = uri.offset (15);
+        if (orig.has_prefix("\\\\"))
+          orig = orig.replace ("\\\\","smb://");
+        if (uri_regex != null && uri_regex.match (orig)) {
+          try {
+            /* this code ensures that a file manager will be used
+             * if uri it's a remote location that should be mounted */
+            if (mountable_regex.match (orig)) {
+              var muris = new GLib.List<string>();
+              muris.prepend (orig);
+              var file_manager = AppInfo.get_default_for_type("inode/directory", true);
+              file_manager.launch_uris(muris,null);
+            } else {
               AppInfo.launch_default_for_uri (orig, null);
-              }
-            } catch (GLib.Error error) {
-              warning ("Failed to launch URI %s", orig);
-              return new Unity.ActivationResponse(Unity.HandledType.NOT_HANDLED);
             }
-            return new Unity.ActivationResponse(Unity.HandledType.HIDE_DASH);
-
-          } else {
-            exec_or_dir = Utils.subst_tilde (orig);
-            args = exec_or_dir.split (" ", 0);
-            for (int i = 0; i < args.length; i++)
-              args[i] = Utils.subst_tilde (args[i]);
+          } catch (GLib.Error error) {
+            warning ("Failed to launch URI %s", orig);
+            return new Unity.ActivationResponse(Unity.HandledType.NOT_HANDLED);
           }
-          this.runner.add_history (orig);
+          return new Unity.ActivationResponse(Unity.HandledType.HIDE_DASH);
+
+        } else {
+          exec_or_dir = Utils.subst_tilde (orig);
+          args = exec_or_dir.split (" ", 0);
+          for (int i = 0; i < args.length; i++)
+            args[i] = Utils.subst_tilde (args[i]);
         }
+        this.runner.add_history (orig);
+      }
       else
-        {
-          /* Activation of standard application:// uris */
-
-          /* Make sure fresh install learns quickly */
-          if (popularity_map.size <= 5) popularities_dirty = true;
-
-          return new Unity.ActivationResponse(Unity.HandledType.NOT_HANDLED);
-        }
+      {
+        /* Activation of standard application:// uris */
+
+        /* Make sure fresh install learns quickly */
+        if (popularity_map.size <= 5) popularities_dirty = true;
+
+        return new Unity.ActivationResponse(Unity.HandledType.NOT_HANDLED);
+      }
 
       if ((exec_or_dir != null) && FileUtils.test (exec_or_dir, FileTest.IS_DIR))
       {
         try {
-            AppInfo.launch_default_for_uri ("file://" + exec_or_dir, null);
+          AppInfo.launch_default_for_uri ("file://" + exec_or_dir, null);
         } catch (GLib.Error err) {
-            warning ("Failed to open current folder '%s' in file manager: %s",
-                     exec_or_dir, err.message);
-            return new Unity.ActivationResponse(Unity.HandledType.NOT_HANDLED);
+          warning ("Failed to open current folder '%s' in file manager: %s",
+                   exec_or_dir, err.message);
+          return new Unity.ActivationResponse(Unity.HandledType.NOT_HANDLED);
         }
       }
       else
       {
-          try {
-            unowned string home_dir = GLib.Environment.get_home_dir ();
-            Process.spawn_async (home_dir, args, null, SpawnFlags.SEARCH_PATH, null, null);
-          } catch (SpawnError e) {
-            warning ("Failed to spawn software-center or direct URI activation '%s': %s",
-                     uri, e.message);
-            return new Unity.ActivationResponse(Unity.HandledType.NOT_HANDLED);
-          }
+        try {
+          unowned string home_dir = GLib.Environment.get_home_dir ();
+          Process.spawn_async (home_dir, args, null, SpawnFlags.SEARCH_PATH, null, null);
+        } catch (SpawnError e) {
+          warning ("Failed to spawn software-center or direct URI activation '%s': %s",
+                   uri, e.message);
+          return new Unity.ActivationResponse(Unity.HandledType.NOT_HANDLED);
+        }
       }
 
       return new Unity.ActivationResponse(Unity.HandledType.HIDE_DASH);
-      
+
     }
 
     /* Appends the subject URIs from a set of Zeitgeist.Events to our Dee.Model
@@ -1026,7 +1317,7 @@
         string? app_uri = null;
         if (ev.num_subjects () > 0)
           app_uri = ev.get_subject (0).get_uri ();
-        
+
         if (app_uri == null)
         {
           warning ("Unexpected event without subject");
@@ -1035,7 +1326,7 @@
 
         /* Assert that we indeed have a known application as actor */
         string desktop_id = Utils.get_desktop_id_for_actor (app_uri);
-        
+
         /* Discard Recently Used apps that are in the launcher */
         if ((category_id == Category.RECENT ||
              category_id == Category.RECENT_APPS) &&
@@ -1049,7 +1340,7 @@
 
         if (app == null)
           continue;
-        
+
         if (!app.should_show ())
           continue;
 
@@ -1073,7 +1364,7 @@
                         app.get_description (), full_uri);
       }
     }
-    
+
   } /* END: class Daemon */
 
 } /* namespace */

=== added file 'src/launcher-client.vala'
--- src/launcher-client.vala	1970-01-01 00:00:00 +0000
+++ src/launcher-client.vala	2012-08-08 16:35:21 +0000
@@ -0,0 +1,45 @@
+/*
+ * Copyright (C) 2012 Canonical Ltd
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 3 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ * Authored by Pawel Stolowski <[e-mail]>
+ */
+
+namespace Unity.ApplicationsLens {
+  
+  static const string LAUNCHER_DBUS_NAME = "com.canonical.Unity.Launcher";
+  static const string LAUNCHER_DBUS_PATH = "/com/canonical/Unity/Launcher";
+
+  [DBus (name = "com.canonical.Unity.Launcher")]
+  public interface LauncherService: GLib.Object
+  {
+	public abstract async void add_launcher_item_from_position (string title, string icon, int icon_x, int icon_y, int icon_size, string desktop_file, string aptdaemon_task) throws IOError;
+  }
+
+  public class LauncherProxy: GLib.Object
+  {
+	public void connect_to_launcher ()
+	{
+	  _launcher_service = Bus.get_proxy_sync (BusType.SESSION, LAUNCHER_DBUS_NAME, LAUNCHER_DBUS_PATH);
+	}
+
+    public async void add_launcher_item_from_position (string title, string icon, int icon_x, int icon_y, int icon_size, string desktop_file, string aptdaemon_task) throws IOError
+    {
+      yield _launcher_service.add_launcher_item_from_position (title, icon, icon_x, icon_y, icon_size, desktop_file, aptdaemon_task);
+    }
+
+	private LauncherService _launcher_service;
+  }
+
+}
\ No newline at end of file

=== added file 'src/software-center-data-provider.vala'
--- src/software-center-data-provider.vala	1970-01-01 00:00:00 +0000
+++ src/software-center-data-provider.vala	2012-08-08 16:35:21 +0000
@@ -0,0 +1,115 @@
+/*
+ * Copyright (C) 2012 Canonical Ltd
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 3 as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ *
+ * Authored by Pawel Stolowski <[e-mail]>
+ */
+
+namespace Unity.ApplicationsLens {
+  
+  static const string SCDP_DBUS_NAME = "com.ubuntu.SoftwareCenterDataProvider";
+  static const string SCDP_DBUS_PATH = "/com/ubuntu/SoftwareCenterDataProvider";
+
+  [DBus (name = "com.ubuntu.SoftwareCenterDataProvider")]
+  public interface SoftwareCenterDataProviderService: GLib.Object
+  {
+    public abstract HashTable<string, Variant> get_app_details (string appname, string pkgname) throws IOError;
+  }
+  
+  public class SoftwareCenterDataProviderProxy: GLib.Object
+  {
+    public enum PackageState
+    {
+      UNINSTALLED,
+      INSTALLED,
+      NEEDS_PURCHASE,
+      UNKNOWN
+    }
+
+    public string name { get; set; }
+    public string summary { get; set; }
+    public string description { get; set; }
+    public string version { get; set; }
+    public string screenshot { get; set; }
+    public string desktop_file { get; set; }
+    public string license { get; set; }
+    public string icon { get; set; }
+    public string price { get; set; }
+    public PackageState pkg_state { get; set; }
+    public string installation_date { get; set; }
+    public string website { get; set; }
+    public int64 size { get; set; }
+    public string hardware_requirements { get; set; }
+
+	public void connect_to () throws IOError
+	{
+	  _service = Bus.get_proxy_sync (BusType.SESSION, SCDP_DBUS_NAME, SCDP_DBUS_PATH);
+	}
+
+	public HashTable<string, Variant> get_app_details (string appname, string pkgname)
+	{
+	  HashTable<string, Variant> data = _service.get_app_details (appname, pkgname);
+
+      name = data.get ("name").get_string ();
+      summary = data.get ("summary").get_string ();
+      description = data.get ("description").get_string ();
+      version = data.get ("version").get_string ();
+      desktop_file = data.get ("desktop_file").get_string ();
+      license = data.get ("license").get_string ();
+      icon = data.get ("icon_file_name").get_string ();
+      price = data.get ("price").get_string ();
+      installation_date = data.get ("installation_date").get_string ();
+      website = data.get ("website").get_string ();
+      hardware_requirements = data.get ("hardware_requirements").get_string ();
+      size = int64.parse (data.get ("size").get_string ());
+      
+      var state = data.get ("pkg_state").get_string ();
+
+      if (state == "installed")
+      {
+        pkg_state = PackageState.INSTALLED;
+      }
+      else if (state == "uninstalled")
+      {
+        pkg_state = PackageState.UNINSTALLED;
+      }
+      else if (state == "needs_purchase")
+      {
+        pkg_state = PackageState.NEEDS_PURCHASE;
+      }
+      else
+      {
+        pkg_state = PackageState.UNKNOWN;
+      }
+
+      screenshot = null;
+
+      if (data.contains ("screenshots"))
+      {
+        var screenshot_var = data.get ("screenshots").get_child_value (0).lookup_value ("large_image_url", VariantType.STRING);
+        if (screenshot_var != null)
+        {
+          screenshot = screenshot_var.get_string ();
+        }
+      }
+      if (screenshot == null)
+      {
+        screenshot = "";
+      }
+	  return data;
+	}
+
+	private SoftwareCenterDataProviderService _service;
+  }
+}

=== modified file 'src/unity-package-search.cc'
--- src/unity-package-search.cc	2012-03-27 10:49:37 +0000
+++ src/unity-package-search.cc	2012-08-08 16:35:21 +0000
@@ -1,6 +1,6 @@
 /*
  * Copyright (C) 2010 Canonical Ltd
-   *
+ *
  * This program is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
@@ -50,12 +50,15 @@
 #define XAPIAN_VALUE_SUMMARY 177
 #define XAPIAN_VALUE_ARCHIVE_CHANNEL 178
 #define XAPIAN_VALUE_DESKTOP_FILE 179
+#define XAPIAN_VALUE_SCREENSHOT_URLS 185
+#define XAPIAN_VALUE_DESCRIPTION 188
+#define XAPIAN_VALUE_VERSION_INFO 198
 
 #include "unity-package-search.h"
 
 extern "C"
 {
-extern gchar* unity_applications_lens_utils_preprocess_string (const gchar* input);
+  extern gchar* unity_applications_lens_utils_preprocess_string (const gchar* input);
 }
 
 struct _UnityPackageSearcher
@@ -71,17 +74,17 @@
 class LocaleKeyMaker : public Xapian::KeyMaker
 {
   private:
-    Xapian::valueno value;
-  
+  Xapian::valueno value;
+
   public:
-    
+
     LocaleKeyMaker (Xapian::valueno value)
     {
       this->value = value;
     }
-    
+
     virtual ~LocaleKeyMaker() { }
-    
+
     virtual std::string operator() (const Xapian::Document &doc) const
     {
       string val = doc.get_value (value);
@@ -89,40 +92,40 @@
       string col_key_mm = col_key_c;
       g_free (col_key_c);
       return col_key_mm;
-    }    
+    }
 };
 
 /* Do generic searcher setup */
 static void
 init_searcher (UnityPackageSearcher *searcher)
 {
-  // Activate Xapian CJK support 
+  // Activate Xapian CJK support
   setenv("XAPIAN_CJK_NGRAM", "1", 1);
 
-	Xapian::Database db = *searcher->db;
+  Xapian::Database db = *searcher->db;
 
   // Start an enquire session
   Xapian::Enquire *enquire = new Xapian::Enquire (db);
-  //enquire->set_sort_by_value (XAPIAN_VALUE_APPNAME, FALSE);    
+  enquire->set_sort_by_value (XAPIAN_VALUE_APPNAME, FALSE);
   searcher->enquire = enquire;
 
   // Make sure we respect sorting rules for the current locale
   searcher->sorter = new LocaleKeyMaker (XAPIAN_VALUE_APPNAME);
   enquire->set_sort_by_key (searcher->sorter, FALSE);
 
-    // Create query parser
-    Xapian::QueryParser *query_parser = new Xapian::QueryParser ();
-    query_parser->add_prefix ("section", "AE");
-    query_parser->add_prefix ("type", "AT");
-    query_parser->add_prefix ("category", "AC");
-    query_parser->add_prefix ("name", "AA");
-    query_parser->add_prefix ("pkgname", "AP");
-    query_parser->add_prefix ("exec", "XX");
-    query_parser->add_prefix ("keyword", "KW");
-    query_parser->set_default_op (Xapian::Query::OP_AND);
-    query_parser->set_database (db);
-    searcher->query_parser = query_parser;
-  
+  // Create query parser
+  Xapian::QueryParser *query_parser = new Xapian::QueryParser ();
+  query_parser->add_prefix ("section", "AE");
+  query_parser->add_prefix ("type", "AT");
+  query_parser->add_prefix ("category", "AC");
+  query_parser->add_prefix ("name", "AA");
+  query_parser->add_prefix ("pkgname", "AP");
+  query_parser->add_prefix ("exec", "XX");
+  query_parser->add_prefix ("keyword", "KW");
+  query_parser->set_default_op (Xapian::Query::OP_AND);
+  query_parser->set_database (db);
+  searcher->query_parser = query_parser;
+
   // Init random number generator from 32 lowest bits of system time
   searcher->random = g_rand_new_with_seed (g_get_monotonic_time ());
 }
@@ -141,110 +144,103 @@
   gchar                *dum1, *dum2, *dum3;
   gint                  i, len;
 
-	g_return_if_fail (db != NULL);
-	g_return_if_fail (indexer != NULL);
-	g_return_if_fail (item != NULL);
-	
-	switch (gmenu_tree_item_get_type (item))
-	{
-	  case GMENU_TREE_ITEM_INVALID:
-	    return;
-	  case GMENU_TREE_ITEM_DIRECTORY:
-	    /* Recurse into directory */
-	    iter = gmenu_tree_directory_get_contents (GMENU_TREE_DIRECTORY (item));
-	    for (; iter != NULL; iter = iter->next)
-	      {
-	        index_menu_item (db, indexer, GMENU_TREE_ITEM (iter->data));
-	      }
-	    break;
-	  case GMENU_TREE_ITEM_ENTRY:
-	    /* Add this entry to the index */
-	    entry = GMENU_TREE_ENTRY (item);
-	    
-	    /* Store relevant values */
-	    if (gmenu_tree_entry_get_display_name (entry))
-	      doc.add_value (XAPIAN_VALUE_APPNAME,
-  	                   gmenu_tree_entry_get_display_name (entry));
-  	  if (gmenu_tree_entry_get_icon (entry))
-        doc.add_value (XAPIAN_VALUE_ICON,
-                       gmenu_tree_entry_get_icon (entry));
+  g_return_if_fail (db != NULL);
+  g_return_if_fail (indexer != NULL);
+  g_return_if_fail (item != NULL);
+
+  switch (gmenu_tree_item_get_type (item))
+  {
+    case GMENU_TREE_ITEM_INVALID:
+      return;
+    case GMENU_TREE_ITEM_DIRECTORY:
+      /* Recurse into directory */
+      iter = gmenu_tree_directory_get_contents (GMENU_TREE_DIRECTORY (item));
+      for (; iter != NULL; iter = iter->next)
+      {
+        index_menu_item (db, indexer, GMENU_TREE_ITEM (iter->data));
+      }
+      break;
+    case GMENU_TREE_ITEM_ENTRY:
+      /* Add this entry to the index */
+      entry = GMENU_TREE_ENTRY (item);
+
+      /* Store relevant values */
+      if (gmenu_tree_entry_get_display_name (entry))
+        doc.add_value (XAPIAN_VALUE_APPNAME, gmenu_tree_entry_get_display_name (entry));
+      if (gmenu_tree_entry_get_icon (entry))
+        doc.add_value (XAPIAN_VALUE_ICON, gmenu_tree_entry_get_icon (entry));
       if (gmenu_tree_entry_get_desktop_file_id (entry))
-        doc.add_value (XAPIAN_VALUE_DESKTOP_FILE,
-                       gmenu_tree_entry_get_desktop_file_id (entry));
-	    
-	    /* Index full text data */
-	    indexer->set_document(doc);
+        doc.add_value (XAPIAN_VALUE_DESKTOP_FILE, gmenu_tree_entry_get_desktop_file_id (entry));
+      
+      /* Index full text data */
+      indexer->set_document(doc);
       if (gmenu_tree_entry_get_display_name (entry))
       {
-        dum1 = unity_applications_lens_utils_preprocess_string (
-            gmenu_tree_entry_get_display_name (entry));
+        dum1 = unity_applications_lens_utils_preprocess_string (gmenu_tree_entry_get_display_name (entry));
         indexer->index_text(dum1, 5);
         g_free (dum1);
       }
       if (gmenu_tree_entry_get_name (entry))
       {
-        dum1 = unity_applications_lens_utils_preprocess_string (
-            gmenu_tree_entry_get_name (entry));
+        dum1 = unity_applications_lens_utils_preprocess_string (gmenu_tree_entry_get_name (entry));
         indexer->index_text(dum1, 5);
         g_free (dum1);
       }
       if (gmenu_tree_entry_get_comment (entry))
       {
-        dum1 = unity_applications_lens_utils_preprocess_string (
-            gmenu_tree_entry_get_comment (entry));
-        indexer->index_text (dum1, 0);
-        g_free (dum1);
-      }
-	    
-	    /* Index the XDG categories */
-	    appman = unity_app_info_manager_get_default ();
-	    cats = unity_app_info_manager_get_categories (appman, // const return
-	                                                  gmenu_tree_entry_get_desktop_file_id (entry),
-	                                                  &len);
-	    
-	    /* Note: Wine apps and app launchers created with Alacarte commonly
-	     * don't have any category metadata, so they'll only show up under
-	     * All Applications */
-	    for (i = 0; i < len; i++)
-	      {
-	        dum1 = g_ascii_strdown (cats[i], -1);
-	        dum2 = g_strconcat ("AC", dum1, NULL);
-	        doc.add_term (dum2);
-	        g_free (dum1);
-	        g_free (dum2);
-	      }
-	    
-	    /* Index Keywords*/
-	    keywords = unity_app_info_manager_get_keywords (appman, // const return
-	                                                    gmenu_tree_entry_get_desktop_file_id (entry),
-	                                                    &len);
-	    for (i = 0; i < len; i++)
-	      {
-          dum1 = unity_applications_lens_utils_preprocess_string (keywords[i]);
-	        indexer->index_text (dum1, 0);
-	        indexer->index_text (dum1, 0, "KW");
-          g_free (dum1);
-	      }
-	    
-	    
-	    g_object_unref (appman);
-	    
-	    /* Always assume Type=Application for items in a menu... */
-	    doc.add_term ("ATapplication");
-	    
-	    /* Index application names */
-	    dum1 = (gchar *) gmenu_tree_entry_get_display_name (entry); // const
-	    dum2 = g_strconcat ("AA", dum1, NULL);
+        dum1 = unity_applications_lens_utils_preprocess_string (gmenu_tree_entry_get_comment (entry));
+        indexer->index_text (dum1, 0);
+        g_free (dum1);
+      }
+
+      /* Index the XDG categories */
+      appman = unity_app_info_manager_get_default ();
+      cats = unity_app_info_manager_get_categories (appman, // const return
+                                                    gmenu_tree_entry_get_desktop_file_id (entry),
+                                                    &len);
+
+      /* Note: Wine apps and app launchers created with Alacarte commonly
+       * don't have any category metadata, so they'll only show up under
+       * All Applications */
+      for (i = 0; i < len; i++)
+      {
+        dum1 = g_ascii_strdown (cats[i], -1);
+        dum2 = g_strconcat ("AC", dum1, NULL);
+        doc.add_term (dum2);
+        g_free (dum1);
+        g_free (dum2);
+      }
+
+      /* Index Keywords*/
+      keywords = unity_app_info_manager_get_keywords (appman, // const return
+                                                      gmenu_tree_entry_get_desktop_file_id (entry),
+                                                      &len);
+      for (i = 0; i < len; i++)
+      {
+        dum1 = unity_applications_lens_utils_preprocess_string (keywords[i]);
+        indexer->index_text (dum1, 0);
+        indexer->index_text (dum1, 0, "KW");
+        g_free (dum1);
+      }
+
+      g_object_unref (appman);
+
+      /* Always assume Type=Application for items in a menu... */
+      doc.add_term ("ATapplication");
+
+      /* Index application names */
+      dum1 = (gchar *) gmenu_tree_entry_get_display_name (entry); // const
+      dum2 = g_strconcat ("AA", dum1, NULL);
       doc.add_term (dum2);
       g_free (dum2);
-      
+
       dum1 = (gchar *) gmenu_tree_entry_get_name (entry); // const
-	    dum2 = g_strconcat ("AA", dum1, NULL);
+      dum2 = g_strconcat ("AA", dum1, NULL);
       doc.add_term (dum2);
       g_free (dum2);
 
       /* Index executable name, change - in _ for exec */
-      dum1 = g_strdup (gmenu_tree_entry_get_exec (entry)); // alloc      
+      dum1 = g_strdup (gmenu_tree_entry_get_exec (entry)); // alloc
       if (dum1) {
         dum2 = strstr (dum1, " "); // const
         dum2 == NULL ? : *dum2 = '\0'; // const
@@ -256,25 +252,23 @@
         doc.add_term (dum2);
         g_free (dum2);
         dum2 = g_strconcat ("XX", dum1, NULL); // alloc
-        doc.add_term (dum2);        
+        doc.add_term (dum2);
         g_free (dum2);
-        g_free (dum1);        
-      }      
-      
+        g_free (dum1);
+      }
+
       db->add_document(doc);
-	    break;
-	  case GMENU_TREE_ITEM_SEPARATOR:
-	  case GMENU_TREE_ITEM_HEADER:
-	  case GMENU_TREE_ITEM_ALIAS:
-	    break;
-	  default:
-	    g_warning ("Unexpected GMenuTreeItemType %u",
-	               gmenu_tree_item_get_type (item));
-	    return;
+      break;
+    case GMENU_TREE_ITEM_SEPARATOR:
+    case GMENU_TREE_ITEM_HEADER:
+    case GMENU_TREE_ITEM_ALIAS:
+      break;
+    default:
+      g_warning ("Unexpected GMenuTreeItemType %u", gmenu_tree_item_get_type (item));
+      return;
   }
 
   // Add the document to the database.
-  
 }
 
 /* Create a searcher that searches in a menu tree. The menu tree
@@ -289,16 +283,16 @@
   db = new Xapian::WritableDatabase ();
   searcher->db = db;
   searcher->db->add_database (Xapian::InMemory::open ());
-  
+
   init_searcher (searcher);
-  
+
   /* Index the menu recursively */
   Xapian::TermGenerator *indexer = new Xapian::TermGenerator ();
   index_menu_item (db, indexer,
                    GMENU_TREE_ITEM (gmenu_tree_get_root_directory (menu)));
   delete indexer;
   db->flush ();
-  
+
   return searcher;
 }
 
@@ -325,14 +319,14 @@
     agent = g_strdup_printf("%s/software-center/software-center-agent.db",
                             g_get_user_cache_dir());
     if (g_file_test(agent, G_FILE_TEST_IS_DIR))
-       searcher->db->add_database (Xapian::Database (agent));
+      searcher->db->add_database (Xapian::Database (agent));
     g_free(agent);
   } catch(const Xapian::Error &error) {
     cerr << "Error loading agent indexes: "  << error.get_msg() << endl;
   }
 
   init_searcher (searcher);
-
+  
   return searcher;
 }
 
@@ -373,13 +367,17 @@
 static void _free_package_info (gpointer pkg)
 {
   UnityPackageInfo *pkginfo = (UnityPackageInfo*) pkg;
+  unity_package_package_info_free (pkginfo);
+}
 
+void unity_package_package_info_free (UnityPackageInfo *pkginfo)
+{
   g_free (pkginfo->package_name);
   g_free (pkginfo->application_name);
   g_free (pkginfo->desktop_file);
   g_free (pkginfo->icon);
 
-  g_slice_free (UnityPackageInfo, pkg);
+  g_slice_free (UnityPackageInfo, pkginfo);
 }
 
 UnityPackageSearchResult*
@@ -432,7 +430,7 @@
       searcher->enquire->set_sort_by_relevance ();
       break;
   }
-  
+
   result = g_slice_new0 (UnityPackageSearchResult);
   try
   {
@@ -440,29 +438,25 @@
     searcher->enquire->set_collapse_key(XAPIAN_VALUE_DESKTOP_FILE);
     max_hits = (max_hits != 0 ? max_hits : searcher->db->get_doccount ());
     searcher->enquire->set_query(query);
-    Xapian::MSet matches =
-      searcher->enquire->get_mset(0, max_hits);
+    Xapian::MSet matches = searcher->enquire->get_mset(0, max_hits);
 
     // Retrieve the results, note that we build the result->results
     // list in reverse order and then reverse it before we return it
     result->num_hits = matches.get_matches_estimated ();
-    for (Xapian::MSetIterator i = matches.begin();
-         i != matches.end();
-         ++i)
-      {
-        try
-          {
-            Xapian::Document doc = i.get_document();
-            UnityPackageInfo *pkginfo = _pkginfo_from_document (doc);
-            pkginfo->relevancy = i.get_percent ();
-            result->results = g_slist_prepend (result->results, pkginfo);
-          }
-        catch (Xapian::Error e)
-          {
-            g_warning ("Unable to read document from result set: %s",
-                       e.get_msg().c_str());
-          }
-      }
+    for (Xapian::MSetIterator i = matches.begin(); i != matches.end(); ++i)
+    {
+      try
+      {
+        Xapian::Document doc = i.get_document();
+        UnityPackageInfo *pkginfo = _pkginfo_from_document (doc);
+        pkginfo->relevancy = i.get_percent ();
+        result->results = g_slist_prepend (result->results, pkginfo);
+      }
+      catch (Xapian::Error e)
+      {
+        g_warning ("Unable to read document from result set: %s", e.get_msg().c_str());
+      }
+    }
 
     result->results = g_slist_reverse (result->results);
   }
@@ -470,7 +464,7 @@
   {
     g_warning ("Error running query '%s': %s", search_string, e.get_msg().c_str());
   }
-  
+
   return result;
 }
 
@@ -486,79 +480,78 @@
   g_return_val_if_fail (searcher != NULL, NULL);
 
   result = g_slice_new0 (UnityPackageSearchResult);
-  result->num_hits = n_apps;  
+  result->num_hits = n_apps;
   lastdocid = searcher->db->get_lastdocid ();
-  
+
   /* Since we really just pick random apps we may end up with dupes */
   unique = g_hash_table_new (g_str_hash, g_str_equal);
-  
+
   /* When looking for random apps we check up to twice the number of
    * requested apps in order to try and avoid dupes. This is a sloppy
    * check, but works well enough in practice */
   if (filter_query == NULL)
+  {
+    g_debug ("RANDOM");
+    for (i = 0, n_unique = 0; i < n_apps*2, n_unique < n_apps; i++)
     {
-      g_debug ("RANDOM");
-      for (i = 0, n_unique = 0; i < n_apps*2, n_unique < n_apps; i++)
-        {          
-          Xapian::Document doc;
-          try
-            {
-              doc = searcher->db->get_document (
-                             g_rand_int_range (searcher->random, 1, lastdocid));
-                        UnityPackageInfo *pkginfo = _pkginfo_from_document (doc);
-              if (g_hash_table_lookup_extended (unique, pkginfo->package_name, NULL, NULL))
-                {
-                  _free_package_info (pkginfo);
-                }
-              else
-                {
-                  g_hash_table_insert (unique, pkginfo->package_name, NULL);
-                  result->results = g_slist_prepend (result->results, pkginfo);
-                  n_unique++;
-                }
-            }
-          catch (Xapian::Error e)
-            {
-              g_warning ("Error getting random apps: %s", e.get_msg().c_str());
-              continue;
-            }
-        }
+      Xapian::Document doc;
+      try
+      {
+        doc = searcher->db->get_document (
+          g_rand_int_range (searcher->random, 1, lastdocid));
+        UnityPackageInfo *pkginfo = _pkginfo_from_document (doc);
+        if (g_hash_table_lookup_extended (unique, pkginfo->package_name, NULL, NULL))
+        {
+          _free_package_info (pkginfo);
+        }
+        else
+        {
+          g_hash_table_insert (unique, pkginfo->package_name, NULL);
+          result->results = g_slist_prepend (result->results, pkginfo);
+          n_unique++;
+        }
+      }
+      catch (Xapian::Error e)
+      {
+        g_warning ("Error getting random apps: %s", e.get_msg().c_str());
+        continue;
+      }
     }
+  }
   else
-    {
-      g_debug ("FILTER %s", filter_query);
-      Xapian::Query query;
-      try
-      {
-        query = searcher->query_parser->parse_query (filter_query, QUERY_PARSER_FILTER_FLAGS);
-        searcher->enquire->set_sort_by_relevance ();
-        searcher->enquire->set_query(query);
-        Xapian::MSet matches = searcher->enquire->get_mset(0, searcher->db->get_doccount ());
-        for (i = 0, n_unique = 0; i < n_apps*4, n_unique < n_apps; i++)
-          {
-            docid = g_rand_int_range (searcher->random, 0, matches.size ());
-            Xapian::MSetIterator iter = matches[docid];
-            Xapian::Document doc = iter.get_document ();
-            UnityPackageInfo *pkginfo = _pkginfo_from_document (doc);
-            if (g_hash_table_lookup_extended (unique, pkginfo->package_name, NULL, NULL))
-              {
-                _free_package_info (pkginfo);
-              }
-            else
-              {
-                g_hash_table_insert (unique, pkginfo->package_name, NULL);
-                result->results = g_slist_prepend (result->results, pkginfo);
-                n_unique++;
-              }
-          }
-      }
-      catch (Xapian::Error e)
-      {
-        g_debug ("Error getting random apps for query '%s': %s",
-                 filter_query, e.get_msg().c_str());
-        return g_slice_new0 (UnityPackageSearchResult);
-      }
-    }
+  {
+    g_debug ("FILTER %s", filter_query);
+    Xapian::Query query;
+    try
+    {
+      query = searcher->query_parser->parse_query (filter_query, QUERY_PARSER_FILTER_FLAGS);
+      searcher->enquire->set_sort_by_relevance ();
+      searcher->enquire->set_query(query);
+      Xapian::MSet matches = searcher->enquire->get_mset(0, searcher->db->get_doccount ());
+      for (i = 0, n_unique = 0; i < n_apps*4, n_unique < n_apps; i++)
+      {
+        docid = g_rand_int_range (searcher->random, 0, matches.size ());
+        Xapian::MSetIterator iter = matches[docid];
+        Xapian::Document doc = iter.get_document ();
+        UnityPackageInfo *pkginfo = _pkginfo_from_document (doc);
+        if (g_hash_table_lookup_extended (unique, pkginfo->package_name, NULL, NULL))
+        {
+          _free_package_info (pkginfo);
+        }
+        else
+        {
+          g_hash_table_insert (unique, pkginfo->package_name, NULL);
+          result->results = g_slist_prepend (result->results, pkginfo);
+          n_unique++;
+        }
+      }
+    }
+    catch (Xapian::Error e)
+    {
+      g_debug ("Error getting random apps for query '%s': %s", filter_query, e.get_msg().c_str());
+      return g_slice_new0 (UnityPackageSearchResult);
+    }
+  }
 
   g_hash_table_unref (unique);
 
@@ -566,6 +559,38 @@
   return result;
 }
 
+UnityPackageInfo *
+unity_package_searcher_get_by_desktop_file (UnityPackageSearcher *searcher, const gchar *desktop_file)
+{
+  g_return_val_if_fail (searcher != NULL, NULL);
+
+  UnityPackageInfo *pkginfo = NULL;
+
+  Xapian::PostingIterator it = searcher->db->postlist_begin ("");
+  Xapian::PostingIterator end_it = searcher->db->postlist_end ("");
+
+  const string query = desktop_file;
+
+  while (it != end_it)
+  {
+    Xapian::Document doc = searcher->db->get_document (*it);
+    string value = doc.get_value (XAPIAN_VALUE_DESKTOP_FILE);
+
+    size_t sep = value.find (':');
+    if (sep != string::npos)
+    {
+      if (value.compare (sep+1, value.size() - sep, query) == 0)
+      {
+        pkginfo = _pkginfo_from_document (doc);
+        return pkginfo;
+      }
+    }
+    ++it;
+  }
+
+  return pkginfo;
+}
+
 void
 unity_package_search_result_free (UnityPackageSearchResult *result)
 {
@@ -574,4 +599,3 @@
   g_slist_free_full (result->results, _free_package_info);
   g_slice_free (UnityPackageSearchResult, result);
 }
-

=== modified file 'src/unity-package-search.h'
--- src/unity-package-search.h	2012-03-09 17:44:30 +0000
+++ src/unity-package-search.h	2012-08-08 16:35:21 +0000
@@ -36,23 +36,23 @@
 
 typedef struct
 {
-	GSList *results;
-	gint    num_hits;
+        GSList *results;
+        gint    num_hits;
 } UnityPackageSearchResult;
 
 typedef struct
 {
-	gchar    *package_name;
-	gchar    *application_name;
-	gchar    *desktop_file;
-	gchar    *icon;
-	gint      relevancy;
+        gchar    *package_name;
+        gchar    *application_name;
+        gchar    *desktop_file;
+        gchar    *icon;
+        gint      relevancy;
 } UnityPackageInfo;
 
 #ifdef __cplusplus
 extern "C" {
 #endif
- 
+
 UnityPackageSearcher*     unity_package_searcher_new    ();
 
 UnityPackageSearcher*     unity_package_searcher_new_for_menu    (GMenuTree *menu);
@@ -71,7 +71,10 @@
 
 void                      unity_package_search_result_free (UnityPackageSearchResult *result);
 
+void                      unity_package_package_info_free (UnityPackageInfo *pkginfo);
+
+UnityPackageInfo*         unity_package_searcher_get_by_desktop_file (UnityPackageSearcher *searcher, const gchar *desktop_file);
+
 #ifdef __cplusplus
 }
 #endif
- 

=== modified file 'vapi/unity-package-search.vapi'
--- vapi/unity-package-search.vapi	2012-03-09 17:44:30 +0000
+++ vapi/unity-package-search.vapi	2012-08-08 16:35:21 +0000
@@ -27,6 +27,7 @@
       public Searcher.for_menu(GMenu.Tree menu);
       public SearchResult search (string search_string, uint max_hits, Unity.Package.SearchType search_type, Unity.Package.Sort sort);
       public SearchResult get_random_apps (string? filter_query, uint n_apps);
+      public PackageInfo? get_by_desktop_file (string desktop_file);
     }
     
     [Compact]
@@ -37,7 +38,7 @@
     }
     
     [Compact]
-    [CCode (cname = "UnityPackageInfo", cheader_filename = "unity-package-search.h")]
+    [CCode (cname = "UnityPackageInfo", free_function = "unity_package_package_info_free", cheader_filename = "unity-package-search.h")]
     public class PackageInfo {
       public string package_name;
 	    public string application_name;