Bug 1259292

Created attachment 1069381 [details]
Screenshot

What's the output of:
gvfs-mime --query video/x-ogm+ogg

Is this a clean installation, or an upgrade?

Clean install of RHEL-7.2-20150824.n.0 compose

[root@wlan-r2s30 ~]# gvfs-mime --query video/x-ogm+ogg
Default application for 'video/x-ogm+ogg': totem.desktop
Registered applications:
	totem.desktop
	org.gnome.Totem.desktop
Recommended applications:
	totem.desktop
	org.gnome.Totem.desktop


Both files are provided by totem-3.14.3-2.el7.x86_64

This is caused by:

commit c6c23140c939c52be137bd64ead1681a81de5292
Author: Ray Strode <rstrode>
Date:   Fri Jun 26 13:05:55 2015 -0400

    Add totem.desktop to keep user mime associates working
    
    Related: #1174589 123541

I'm not sure we want to have this...

So the problem the duplicate file tries to solve is keeping user defined custom mime handlers working.  See bug 1235413 comment 5 part 2

I agree though a duplicate icon in the applications list isn't really acceptable. We also have duplicate icons in the notification list (see bug 1259763 ) 

I think maybe we need to drop the duplicate file an come at the custom mime handler problem a different way.

Also see bug See bug 1259763

I would suggest that we make the code populating the notification panel and the default mime handler combos respect the NoDisplay key. That should solve the problem, and is probably the right thing to do, regardless.

So we can't actually filter on NoDisplay since the Notifications panel depends on being able to show entries with NoDisplay=true (for instance the sharing panel).

I guess we can filter out by duplicate name except that runs into problems when running under a translated desktop.  We'd have to side step GDesktopAppInfo and load the desktop file a second time into a keyfile to get the untranslated names.

We could add a new key X-GNOME-Compat-Entry=true or something and check for that, but then we have to touch all the compat desktop files again, and we have to touch all the code that looks at the files to support the new key.

We could also copy the compat map that gnome-shell has to glib and use it for the mime handlers, and then drop all the compat files, but it could be that there are other things using the old names we aren't specifically aware of yet.

not sure what the best way forward is.

okay so i think the easiest way forward is to put

X-GNOME-UsesNotifications=false in all the compat files to hide them from the notifications list, and then fixes the details panel to filter out NoDisplay applications.

grr, so apparently filtering out NoDisplay was tried 2 years ago and reverted:

https://bugzilla.gnome.org/show_bug.cgi?id=702681

So at this point I'm going to go with a downstream patch to filter out X-GNOME-Compat-Entry=true lines

Is this a right component?

totem is one of the components affected. to minimize paperwork i'm using this bug for all the components

(gtk+, totem, file-roller, nautilus, gedit)

(to be clear, the main fix is in gtk+ but all the compat desktop entries need to be updated as per comment 9 and comment 11)

okay should be all set now.

for the custom key I ended up going with X-RHEL-AliasOf=name-of-new-file instead of X-GNOME-Compat-Entry=true.  This is because of a limitation in the GDesktopAppInfo api that would have required me to load each file twice if I wanted to check the value of a new custom key, but I can use the api without reloading the file if all I want to do is check the existence of a new key.

*** Bug 1259763 has been marked as a duplicate of this bug. ***

"Videos" is no longer listed twice under Music or Video. Verified.

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2015-2227.html