gnome-shell r163 - in trunk: js/ui src

[marinaz svn gnome org]

Author: marinaz
Date: Thu Jan 22 21:28:19 2009
New Revision: 163
URL: http://svn.gnome.org/viewvc/gnome-shell?rev=163&view=rev

Log:
shell-global.c: remove a log message about missing 'Categories' key in the desktop file, as this key is not required

main.js: set desktop environment to "GNOME" so that GIO can determine correctly which applications should be shown in the menus.

appDisplay.js: filter out applications that should not be shown in the menus, check if get_executable() for GAppInfo is not null before using it   


Modified:
   trunk/js/ui/appDisplay.js
   trunk/js/ui/main.js
   trunk/src/shell-global.c

Modified: trunk/js/ui/appDisplay.js
==============================================================================
--- trunk/js/ui/appDisplay.js	(original)
+++ trunk/js/ui/appDisplay.js	Thu Jan 22 21:28:19 2009
@@ -142,6 +142,10 @@
         let apps = Gio.app_info_get_all();
         for (let i = 0; i < apps.length; i++) {
             let appInfo = apps[i];
+            
+            if (!appInfo.should_show())
+                continue;
+            
             let appId = appInfo.get_id();
             this._allItems[appId] = appInfo;
             // [] is returned if we could not get the categories or the list of categories was empty
@@ -195,10 +199,14 @@
                 return true;
         }
 
-        let exec = itemInfo.get_executable().toLowerCase();
-        if (exec.indexOf(search) >= 0)
-            return true;
-        
+        if (itemInfo.get_executable() == null) {
+            log("Missing an executable for " + itemInfo.get_name());
+        } else {
+            let exec = itemInfo.get_executable().toLowerCase();
+            if (exec.indexOf(search) >= 0)
+                return true;
+        }
+
         // we expect this._categories.hasOwnProperty(itemInfo.get_id()) to always be true here     
         let categories = this._categories[itemInfo.get_id()]; 
         for (let i = 0; i < categories.length; i++) {

Modified: trunk/js/ui/main.js
==============================================================================
--- trunk/js/ui/main.js	(original)
+++ trunk/js/ui/main.js	Thu Jan 22 21:28:19 2009
@@ -2,6 +2,7 @@
 
 const Clutter = imports.gi.Clutter;
 const Shell = imports.gi.Shell;
+const Gio = imports.gi.Gio;
 const Signals = imports.signals;
 const Mainloop = imports.mainloop;
 const Tweener = imports.tweener.tweener;
@@ -104,7 +105,9 @@
 
 function start() {
     let global = Shell.Global.get();
-    
+
+    Gio.DesktopAppInfo.set_desktop_env("GNOME");
+
     global.grab_dbus_service();
     global.start_task_panel();
 

Modified: trunk/src/shell-global.c
==============================================================================
--- trunk/src/shell-global.c	(original)
+++ trunk/src/shell-global.c	Thu Jan 22 21:28:19 2009
@@ -378,7 +378,7 @@
                                                  &error);
         if (error != NULL)
           {
-            g_warning ("Error when getting 'Categories' from a key file %s: %s", desktop_file_name, error->message);
+            // "Categories" is not a required key in the desktop files, so it's ok if we didn't find it
             g_clear_error (&error);
           } 
       }