]> git.0d.be Git - empathy.git/blobdiff - libempathy-gtk/empathy-contact-list-view.c
Merge branch 'people-nearby-fake-group-613558'
[empathy.git] / libempathy-gtk / empathy-contact-list-view.c
index 3e1464654c9696333fc1fd8689b8815d145946d2..05823435a04727f9b9d42f501b5477156e38a733 100644 (file)
@@ -65,6 +65,7 @@ typedef struct {
        EmpathyContactListFeatureFlags  list_features;
        EmpathyContactFeatureFlags      contact_features;
        GtkWidget                      *tooltip_widget;
+       GtkTargetList                  *file_targets;
 } EmpathyContactListViewPriv;
 
 typedef struct {
@@ -93,6 +94,7 @@ enum DndDragType {
 };
 
 static const GtkTargetEntry drag_types_dest[] = {
+       { "text/path-list",  0, DND_DRAG_TYPE_URI_LIST },
        { "text/uri-list",   0, DND_DRAG_TYPE_URI_LIST },
        { "text/contact-id", 0, DND_DRAG_TYPE_CONTACT_ID },
        { "text/plain",      0, DND_DRAG_TYPE_STRING },
@@ -100,6 +102,7 @@ static const GtkTargetEntry drag_types_dest[] = {
 };
 
 static const GtkTargetEntry drag_types_dest_file[] = {
+       { "text/path-list",  0, DND_DRAG_TYPE_URI_LIST },
        { "text/uri-list",   0, DND_DRAG_TYPE_URI_LIST },
 };
 
@@ -154,6 +157,11 @@ contact_list_view_query_tooltip_cb (EmpathyContactListView *view,
        }
        running++;
 
+       /* Don't show the tooltip if there's already a popup menu */
+       if (gtk_menu_get_for_attach_widget (GTK_WIDGET (view)) != NULL) {
+               goto OUT;
+       }
+
        if (!gtk_tree_view_get_tooltip_context (GTK_TREE_VIEW (view), &x, &y,
                                                keyboard_mode,
                                                &model, &path, &iter)) {
@@ -174,6 +182,8 @@ contact_list_view_query_tooltip_cb (EmpathyContactListView *view,
                priv->tooltip_widget = empathy_contact_widget_new (contact,
                        EMPATHY_CONTACT_WIDGET_FOR_TOOLTIP |
                        EMPATHY_CONTACT_WIDGET_SHOW_LOCATION);
+               gtk_container_set_border_width (
+                       GTK_CONTAINER (priv->tooltip_widget), 8);
                g_object_ref (priv->tooltip_widget);
                g_signal_connect (priv->tooltip_widget, "destroy",
                                  G_CALLBACK (contact_list_view_tooltip_destroy_cb),
@@ -230,6 +240,21 @@ contact_list_view_drag_got_contact (EmpathyTpContactFactory *factory,
                data->old_group, data->new_group);
 
        list = empathy_contact_list_store_get_list_iface (priv->store);
+
+       if (!tp_strdiff (data->new_group, EMPATHY_CONTACT_LIST_STORE_FAVORITE)) {
+               /* Mark contact as favourite */
+               empathy_contact_list_add_to_favourites (list, contact);
+               return;
+       }
+
+       if (!tp_strdiff (data->old_group, EMPATHY_CONTACT_LIST_STORE_FAVORITE)) {
+               /* Remove contact as favourite */
+               empathy_contact_list_remove_from_favourites (list, contact);
+               /* Don't try to remove it */
+               g_free (data->old_group);
+               data->old_group = NULL;
+       }
+
        if (data->new_group) {
                empathy_contact_list_add_to_group (list, contact, data->new_group);
        }
@@ -238,6 +263,27 @@ contact_list_view_drag_got_contact (EmpathyTpContactFactory *factory,
        }
 }
 
+static gboolean
+group_can_be_modified (const gchar *name,
+                      gboolean is_fake_group,
+                      gboolean adding)
+{
+       /* Real groups can always be modified */
+       if (!is_fake_group)
+               return TRUE;
+
+       /* The favorite fake group can be modified so users can
+        * add/remove favorites using DnD */
+       if (!tp_strdiff (name, EMPATHY_CONTACT_LIST_STORE_FAVORITE))
+               return TRUE;
+
+       /* We can remove contacts from the 'ungrouped' fake group */
+       if (!adding && !tp_strdiff (name, EMPATHY_CONTACT_LIST_STORE_UNGROUPED))
+               return TRUE;
+
+       return FALSE;
+}
+
 static gboolean
 contact_list_view_contact_drag_received (GtkWidget         *view,
                                         GdkDragContext    *context,
@@ -258,23 +304,30 @@ contact_list_view_contact_drag_received (GtkWidget         *view,
        gchar         *new_group = NULL;
        gchar         *old_group = NULL;
        gboolean       success = TRUE;
+       gboolean       new_group_is_fake, old_group_is_fake = TRUE;
 
        priv = GET_PRIV (view);
 
        sel_data = (const gchar *) gtk_selection_data_get_data (selection);
        new_group = empathy_contact_list_store_get_parent_group (model,
-                                                                path, NULL);
+                                                                path, NULL, &new_group_is_fake);
+
+       if (!group_can_be_modified (new_group, new_group_is_fake, TRUE))
+               return FALSE;
 
        /* Get source group information. */
        if (priv->drag_row) {
                source_path = gtk_tree_row_reference_get_path (priv->drag_row);
                if (source_path) {
                        old_group = empathy_contact_list_store_get_parent_group (
-                                                                                model, source_path, NULL);
+                                                                                model, source_path, NULL, &old_group_is_fake);
                        gtk_tree_path_free (source_path);
                }
        }
 
+       if (!group_can_be_modified (old_group, old_group_is_fake, FALSE))
+               return FALSE;
+
        if (!tp_strdiff (old_group, new_group)) {
                g_free (new_group);
                g_free (old_group);
@@ -417,7 +470,6 @@ contact_list_view_drag_motion (GtkWidget      *widget,
 {
        EmpathyContactListViewPriv *priv;
        GtkTreeModel               *model;
-       static GtkTargetList  *file_targets = NULL;
        GdkAtom                target;
        GtkTreeIter            iter;
        static DragMotionData *dm = NULL;
@@ -430,11 +482,6 @@ contact_list_view_drag_motion (GtkWidget      *widget,
        priv = GET_PRIV (EMPATHY_CONTACT_LIST_VIEW (widget));
        model = gtk_tree_view_get_model (GTK_TREE_VIEW (widget));
 
-       if (file_targets == NULL) {
-               file_targets = gtk_target_list_new (drag_types_dest_file,
-                                                   G_N_ELEMENTS (drag_types_dest_file));
-       }
-
        is_row = gtk_tree_view_get_path_at_pos (GTK_TREE_VIEW (widget),
                                                x,
                                                y,
@@ -453,14 +500,24 @@ contact_list_view_drag_motion (GtkWidget      *widget,
        }
 
        if (path == NULL) {
-               gdk_drag_status (context, 0, time_);
+               /* Coordinates don't point to an actual row, so make sure the pointer
+                  and highlighting don't indicate that a drag is possible.
+                */
+               gdk_drag_status (context, GDK_ACTION_DEFAULT, time_);
                gtk_tree_view_set_drag_dest_row (GTK_TREE_VIEW (widget), NULL, 0);
                return FALSE;
        }
-       target = gtk_drag_dest_find_target (widget, context, file_targets);
+       target = gtk_drag_dest_find_target (widget, context, priv->file_targets);
        gtk_tree_model_get_iter (model, &iter, path);
 
        if (target == GDK_NONE) {
+               /* If target == GDK_NONE, then we don't have a target that can be
+                  dropped on a contact.  This means a contact drag.  If we're
+                  pointing to a group, highlight it.  Otherwise, if the contact
+                  we're pointing to is in a group, highlight that.  Otherwise,
+                  set the drag position to before the first row for a drag into
+                  the "non-group" at the top.
+                */
                GtkTreeIter  group_iter;
                gboolean     is_group;
                GtkTreePath *group_path;
@@ -493,15 +550,21 @@ contact_list_view_drag_motion (GtkWidget      *widget,
                }
        }
        else {
+               /* This is a file drag, and it can only be dropped on contacts,
+                  not groups.
+                */
                EmpathyContact *contact;
                gtk_tree_model_get (model, &iter,
                                    EMPATHY_CONTACT_LIST_STORE_COL_CONTACT, &contact,
                                    -1);
-               if (contact) {
+               if (contact != NULL &&
+                   empathy_contact_is_online (contact) &&
+                   (empathy_contact_get_capabilities (contact) & EMPATHY_CAPABILITIES_FT)) {
                        gdk_drag_status (context, GDK_ACTION_COPY, time_);
                        gtk_tree_view_set_drag_dest_row (GTK_TREE_VIEW (widget),
                                                         path,
                                                         GTK_TREE_VIEW_DROP_INTO_OR_BEFORE);
+                       g_object_unref (contact);
                }
                else {
                        gdk_drag_status (context, 0, time_);
@@ -666,10 +729,16 @@ contact_list_view_popup_menu_idle_cb (gpointer user_data)
        }
 
        if (menu) {
+               g_signal_connect (menu, "deactivate",
+                                 G_CALLBACK (gtk_menu_detach), NULL);
+               gtk_menu_attach_to_widget (GTK_MENU (menu),
+                                          GTK_WIDGET (data->view), NULL);
                gtk_widget_show (menu);
                gtk_menu_popup (GTK_MENU (menu),
                                NULL, NULL, NULL, NULL,
                                data->button, data->time);
+               g_object_ref_sink (menu);
+               g_object_unref (menu);
        }
 
        g_slice_free (MenuPopupData, data);
@@ -741,54 +810,55 @@ contact_list_view_row_activated (GtkTreeView       *view,
 }
 
 static void
-contact_list_start_voip_call (EmpathyCellRendererActivatable *cell,
+contact_list_view_call_activated_cb (
+    EmpathyCellRendererActivatable *cell,
     const gchar                    *path_string,
-    EmpathyContactListView         *view,
-    gboolean with_video)
+    EmpathyContactListView         *view)
 {
-       EmpathyContactListViewPriv *priv = GET_PRIV (view);
-       GtkTreeModel               *model;
-       GtkTreeIter                 iter;
-       EmpathyContact             *contact;
-
-       if (!(priv->contact_features & EMPATHY_CONTACT_FEATURE_CALL)) {
-               return;
-       }
+       GtkWidget *menu;
+       GtkTreeModel *model;
+       GtkTreeIter iter;
+       EmpathyContact *contact;
+       GdkEventButton *event;
+       GtkMenuShell *shell;
+       GtkWidget *item;
 
        model = gtk_tree_view_get_model (GTK_TREE_VIEW (view));
-       if (!gtk_tree_model_get_iter_from_string (model, &iter, path_string)) {
+       if (!gtk_tree_model_get_iter_from_string (model, &iter, path_string))
                return;
-       }
 
        gtk_tree_model_get (model, &iter,
                            EMPATHY_CONTACT_LIST_STORE_COL_CONTACT, &contact,
                            -1);
+       if (contact == NULL)
+               return;
 
-       if (contact) {
-               EmpathyCallFactory *factory;
-               factory = empathy_call_factory_get ();
-               empathy_call_factory_new_call_with_streams (factory, contact,
-                       TRUE, with_video);
-               g_object_unref (contact);
-       }
-}
-
-static void
-contact_list_view_video_call_activated_cb (
-    EmpathyCellRendererActivatable *cell,
-    const gchar                    *path_string,
-    EmpathyContactListView         *view)
-{
-  contact_list_start_voip_call (cell, path_string, view, TRUE);
-}
+       event = (GdkEventButton *) gtk_get_current_event ();
 
+       menu = gtk_menu_new ();
+       shell = GTK_MENU_SHELL (menu);
+
+       /* audio */
+       item = empathy_contact_audio_call_menu_item_new (contact);
+       gtk_menu_shell_append (shell, item);
+       gtk_widget_show (item);
+
+       /* video */
+       item = empathy_contact_video_call_menu_item_new (contact);
+       gtk_menu_shell_append (shell, item);
+       gtk_widget_show (item);
+
+       g_signal_connect (menu, "deactivate",
+                         G_CALLBACK (gtk_menu_detach), NULL);
+       gtk_menu_attach_to_widget (GTK_MENU (menu),
+                                  GTK_WIDGET (view), NULL);
+       gtk_widget_show (menu);
+       gtk_menu_popup (GTK_MENU (menu), NULL, NULL, NULL, NULL,
+                       event->button, event->time);
+       g_object_ref_sink (menu);
+       g_object_unref (menu);
 
-static void
-contact_list_view_audio_call_activated_cb (EmpathyCellRendererActivatable *cell,
-                                    const gchar                    *path_string,
-                                    EmpathyContactListView         *view)
-{
-  contact_list_start_voip_call (cell, path_string, view, FALSE);
+       g_object_unref (contact);
 }
 
 static void
@@ -831,54 +901,70 @@ contact_list_view_pixbuf_cell_data_func (GtkTreeViewColumn     *tree_column,
                                         GtkTreeIter           *iter,
                                         EmpathyContactListView *view)
 {
-       gchar    *icon_name;
-       gboolean  is_group;
-       gboolean  is_active;
+       GdkPixbuf *pixbuf;
+       gboolean   is_group;
+       gboolean   is_active;
 
        gtk_tree_model_get (model, iter,
                            EMPATHY_CONTACT_LIST_STORE_COL_IS_GROUP, &is_group,
                            EMPATHY_CONTACT_LIST_STORE_COL_IS_ACTIVE, &is_active,
-                           EMPATHY_CONTACT_LIST_STORE_COL_ICON_STATUS, &icon_name,
+                           EMPATHY_CONTACT_LIST_STORE_COL_ICON_STATUS, &pixbuf,
                            -1);
 
        g_object_set (cell,
                      "visible", !is_group,
-                     "icon-name", icon_name,
+                     "pixbuf", pixbuf,
                      NULL);
 
-       g_free (icon_name);
+       if (pixbuf != NULL) {
+               g_object_unref (pixbuf);
+       }
 
        contact_list_view_cell_set_background (view, cell, is_group, is_active);
 }
 
 static void
-contact_list_view_audio_call_cell_data_func (
-                                      GtkTreeViewColumn      *tree_column,
-                                      GtkCellRenderer        *cell,
-                                      GtkTreeModel           *model,
-                                      GtkTreeIter            *iter,
-                                      EmpathyContactListView *view)
+contact_list_view_group_icon_cell_data_func (GtkTreeViewColumn     *tree_column,
+                                            GtkCellRenderer       *cell,
+                                            GtkTreeModel          *model,
+                                            GtkTreeIter           *iter,
+                                            EmpathyContactListView *view)
 {
+       GdkPixbuf *pixbuf = NULL;
        gboolean is_group;
-       gboolean is_active;
-       gboolean can_voip;
+       gchar *name;
 
        gtk_tree_model_get (model, iter,
                            EMPATHY_CONTACT_LIST_STORE_COL_IS_GROUP, &is_group,
-                           EMPATHY_CONTACT_LIST_STORE_COL_IS_ACTIVE, &is_active,
-                           EMPATHY_CONTACT_LIST_STORE_COL_CAN_AUDIO_CALL, &can_voip,
+                           EMPATHY_CONTACT_LIST_STORE_COL_NAME, &name,
                            -1);
 
+       if (!is_group)
+               goto out;
+
+       if (!tp_strdiff (name, EMPATHY_CONTACT_LIST_STORE_FAVORITE)) {
+               pixbuf = empathy_pixbuf_from_icon_name ("emblem-favorite",
+                       GTK_ICON_SIZE_MENU);
+       }
+       else if (!tp_strdiff (name, EMPATHY_CONTACT_LIST_STORE_PEOPLE_NEARBY)) {
+               pixbuf = empathy_pixbuf_from_icon_name ("im-local-xmpp",
+                       GTK_ICON_SIZE_MENU);
+       }
+
+out:
        g_object_set (cell,
-                     "visible", !is_group && can_voip,
-                     "icon-name", EMPATHY_IMAGE_VOIP,
+                     "visible", pixbuf != NULL,
+                     "pixbuf", pixbuf,
                      NULL);
 
-       contact_list_view_cell_set_background (view, cell, is_group, is_active);
+       if (pixbuf != NULL)
+               g_object_unref (pixbuf);
+
+       g_free (name);
 }
 
 static void
-contact_list_view_video_call_cell_data_func (
+contact_list_view_audio_call_cell_data_func (
                                       GtkTreeViewColumn      *tree_column,
                                       GtkCellRenderer        *cell,
                                       GtkTreeModel           *model,
@@ -887,23 +973,23 @@ contact_list_view_video_call_cell_data_func (
 {
        gboolean is_group;
        gboolean is_active;
-       gboolean can_voip;
+       gboolean can_audio, can_video;
 
        gtk_tree_model_get (model, iter,
                            EMPATHY_CONTACT_LIST_STORE_COL_IS_GROUP, &is_group,
                            EMPATHY_CONTACT_LIST_STORE_COL_IS_ACTIVE, &is_active,
-                           EMPATHY_CONTACT_LIST_STORE_COL_CAN_VIDEO_CALL, &can_voip,
+                           EMPATHY_CONTACT_LIST_STORE_COL_CAN_AUDIO_CALL, &can_audio,
+                           EMPATHY_CONTACT_LIST_STORE_COL_CAN_VIDEO_CALL, &can_video,
                            -1);
 
        g_object_set (cell,
-                     "visible", !is_group && can_voip,
-                     "icon-name", EMPATHY_IMAGE_VIDEO_CALL,
+                     "visible", !is_group && (can_audio || can_video),
+                     "icon-name", can_video? EMPATHY_IMAGE_VIDEO_CALL : EMPATHY_IMAGE_VOIP,
                      NULL);
 
        contact_list_view_cell_set_background (view, cell, is_group, is_active);
 }
 
-
 static void
 contact_list_view_avatar_cell_data_func (GtkTreeViewColumn     *tree_column,
                                         GtkCellRenderer       *cell,
@@ -944,22 +1030,12 @@ contact_list_view_text_cell_data_func (GtkTreeViewColumn     *tree_column,
 {
        gboolean is_group;
        gboolean is_active;
-       gboolean show_status;
-       gchar *name;
 
        gtk_tree_model_get (model, iter,
                            EMPATHY_CONTACT_LIST_STORE_COL_IS_GROUP, &is_group,
                            EMPATHY_CONTACT_LIST_STORE_COL_IS_ACTIVE, &is_active,
-                           EMPATHY_CONTACT_LIST_STORE_COL_STATUS_VISIBLE, &show_status,
-                           EMPATHY_CONTACT_LIST_STORE_COL_NAME, &name,
                            -1);
 
-       g_object_set (cell,
-                     "show-status", show_status,
-                     "text", name,
-                     NULL);
-       g_free (name);
-
        contact_list_view_cell_set_background (view, cell, is_group, is_active);
 }
 
@@ -1114,6 +1190,22 @@ contact_list_view_setup (EmpathyContactListView *view)
                      "visible", FALSE,
                      NULL);
 
+       /* Group icon */
+       cell = gtk_cell_renderer_pixbuf_new ();
+       gtk_tree_view_column_pack_start (col, cell, FALSE);
+       gtk_tree_view_column_set_cell_data_func (
+               col, cell,
+               (GtkTreeCellDataFunc) contact_list_view_group_icon_cell_data_func,
+               view, NULL);
+
+       g_object_set (cell,
+                     "xpad", 0,
+                     "ypad", 0,
+                     "visible", FALSE,
+                     "width", 16,
+                     "height", 16,
+                     NULL);
+
        /* Name */
        cell = empathy_cell_renderer_text_new ();
        gtk_tree_view_column_pack_start (col, cell, TRUE);
@@ -1124,10 +1216,16 @@ contact_list_view_setup (EmpathyContactListView *view)
 
        gtk_tree_view_column_add_attribute (col, cell,
                                            "name", EMPATHY_CONTACT_LIST_STORE_COL_NAME);
+       gtk_tree_view_column_add_attribute (col, cell,
+                                           "text", EMPATHY_CONTACT_LIST_STORE_COL_NAME);
+       gtk_tree_view_column_add_attribute (col, cell,
+                                           "presence-type", EMPATHY_CONTACT_LIST_STORE_COL_PRESENCE_TYPE);
        gtk_tree_view_column_add_attribute (col, cell,
                                            "status", EMPATHY_CONTACT_LIST_STORE_COL_STATUS);
        gtk_tree_view_column_add_attribute (col, cell,
                                            "is_group", EMPATHY_CONTACT_LIST_STORE_COL_IS_GROUP);
+       gtk_tree_view_column_add_attribute (col, cell,
+                                           "compact", EMPATHY_CONTACT_LIST_STORE_COL_COMPACT);
 
        /* Audio Call Icon */
        cell = empathy_cell_renderer_activatable_new ();
@@ -1142,23 +1240,7 @@ contact_list_view_setup (EmpathyContactListView *view)
                      NULL);
 
        g_signal_connect (cell, "path-activated",
-                         G_CALLBACK (contact_list_view_audio_call_activated_cb),
-                         view);
-
-       /* Video Call Icon */
-       cell = empathy_cell_renderer_activatable_new ();
-       gtk_tree_view_column_pack_start (col, cell, FALSE);
-       gtk_tree_view_column_set_cell_data_func (
-               col, cell,
-               (GtkTreeCellDataFunc) contact_list_view_video_call_cell_data_func,
-               view, NULL);
-
-       g_object_set (cell,
-                     "visible", FALSE,
-                     NULL);
-
-       g_signal_connect (cell, "path-activated",
-                         G_CALLBACK (contact_list_view_video_call_activated_cb),
+                         G_CALLBACK (contact_list_view_call_activated_cb),
                          view);
 
        /* Avatar */
@@ -1252,6 +1334,9 @@ contact_list_view_finalize (GObject *object)
        if (priv->tooltip_widget) {
                gtk_widget_destroy (priv->tooltip_widget);
        }
+       if (priv->file_targets) {
+               gtk_target_list_unref (priv->file_targets);
+       }
 
        G_OBJECT_CLASS (empathy_contact_list_view_parent_class)->finalize (object);
 }
@@ -1382,6 +1467,10 @@ empathy_contact_list_view_init (EmpathyContactListView *view)
                                              empathy_contact_list_store_row_separator_func,
                                              NULL, NULL);
 
+       /* Set up drag target lists. */
+       priv->file_targets = gtk_target_list_new (drag_types_dest_file,
+                                                 G_N_ELEMENTS (drag_types_dest_file));
+
        /* Connect to tree view signals rather than override. */
        g_signal_connect (view, "button-press-event",
                          G_CALLBACK (contact_list_view_button_press_event_cb),
@@ -1465,7 +1554,8 @@ empathy_contact_list_view_get_flags (EmpathyContactListView *view)
 }
 
 gchar *
-empathy_contact_list_view_get_selected_group (EmpathyContactListView *view)
+empathy_contact_list_view_get_selected_group (EmpathyContactListView *view,
+                                             gboolean *is_fake_group)
 {
        EmpathyContactListViewPriv *priv;
        GtkTreeSelection          *selection;
@@ -1473,6 +1563,7 @@ empathy_contact_list_view_get_selected_group (EmpathyContactListView *view)
        GtkTreeModel              *model;
        gboolean                   is_group;
        gchar                     *name;
+       gboolean                   fake;
 
        g_return_val_if_fail (EMPATHY_IS_CONTACT_LIST_VIEW (view), NULL);
 
@@ -1486,6 +1577,7 @@ empathy_contact_list_view_get_selected_group (EmpathyContactListView *view)
        gtk_tree_model_get (model, &iter,
                            EMPATHY_CONTACT_LIST_STORE_COL_IS_GROUP, &is_group,
                            EMPATHY_CONTACT_LIST_STORE_COL_NAME, &name,
+                           EMPATHY_CONTACT_LIST_STORE_COL_IS_FAKE_GROUP, &fake,
                            -1);
 
        if (!is_group) {
@@ -1493,6 +1585,9 @@ empathy_contact_list_view_get_selected_group (EmpathyContactListView *view)
                return NULL;
        }
 
+       if (is_fake_group != NULL)
+               *is_fake_group = fake;
+
        return name;
 }
 
@@ -1529,7 +1624,7 @@ contact_list_view_group_remove_activate_cb (GtkMenuItem            *menuitem,
        EmpathyContactListViewPriv *priv = GET_PRIV (view);
        gchar                      *group;
 
-       group = empathy_contact_list_view_get_selected_group (view);
+       group = empathy_contact_list_view_get_selected_group (view, NULL);
        if (group) {
                gchar     *text;
                GtkWindow *parent;
@@ -1557,6 +1652,7 @@ empathy_contact_list_view_get_group_menu (EmpathyContactListView *view)
        GtkWidget                  *menu;
        GtkWidget                  *item;
        GtkWidget                  *image;
+       gboolean                   is_fake_group;
 
        g_return_val_if_fail (EMPATHY_IS_CONTACT_LIST_VIEW (view), NULL);
 
@@ -1565,8 +1661,9 @@ empathy_contact_list_view_get_group_menu (EmpathyContactListView *view)
                return NULL;
        }
 
-       group = empathy_contact_list_view_get_selected_group (view);
-       if (!group) {
+       group = empathy_contact_list_view_get_selected_group (view, &is_fake_group);
+       if (!group || is_fake_group) {
+               /* We can't alter fake groups */
                return NULL;
        }