By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 2326

Summary: Delete double Cervisia menu entry in konqueror 'Open With' menu
Product: TDE Reporter: Darrell <darrella>
Component: tdesdkAssignee: Timothy Pearson <kb9vqf>
Status: CONFIRMED ---    
Severity: normal CC: bugwatch, darrella, michele.calgaro
Priority: P5    
Version: R14.0.x [Trinity]   
Hardware: Other   
OS: Linux   
Compiler Version: TDE Version String:
Application Version: Application Name:
Bug Depends on:    
Bug Blocks: 2968    
Attachments: Patch to remove double Cervisia menu entry

Description Darrell 2015-01-25 18:26:23 CST
Created attachment 2434 [details]
Patch to remove double Cervisia menu entry

When using the konqueror context menu on a folder, the 'Open With' menu option displays Cervisia twice.

The attached proposed patch removes the duplicate menu entry but has the effect of also removing Cervisia from the 'Preview In' menu option.
Comment 1 Michele Calgaro 2015-01-25 18:29:44 CST
Uhm... only one Cervisia entry here.
Comment 2 Darrell 2015-01-25 21:17:01 CST
I spent a few hours digging into this bug. I now can replicate this bug repeatedly.

* Start with a fresh profile.
* Open konqueror.
* Select tree view.
* Save the profile.
* Exit TDE.

* Start TDE.
* Start konqueror.
* "Right-click" on a folder.
* Select the 'Open With' menu option.
* Verify only one instance of Cervisia.

* "Right-click" on a folder.
* From the pop-up context menu, select Properties.
* In the Properties dialog, select the Type icon (wrench icon).
* Select the Embedding tab.
* Move only tree view to the top of the list.
* Select the OK button.
* Close the Properties dialog.

* Exit TDE.
* Start TDE.
* Start konqueror.
* "Right-click" on a folder.
* Select the 'Open With' menu option.
* Verify two instances of Cervisia.

I can offset the problem by MANUALLY editing the user's profilerc file and reducing the entries of [inode/directory] to just three entries:

===================================
[inode/directory - 1]
AllowAsDefault=true
Application=konq_treeview.desktop
GenericServiceType=KParts/ReadOnlyPart
Preference=11
ServiceType=inode/directory

[inode/directory - 2]
AllowAsDefault=true
Application=konq_detailedlistview.desktop
GenericServiceType=KParts/ReadOnlyPart
Preference=10
ServiceType=inode/directory

[inode/directory - 3]
AllowAsDefault=true
Application=konq_iconview.desktop
GenericServiceType=KParts/ReadOnlyPart
Preference=7
ServiceType=inode/directory

===================================

Notice the number of [inode/directory] entries is significantly less than what is produced by TDE when modifying the embedding order in the dialog.

I have seen this bug for years. I am reasonably certain the bug is traceable to the KDE 3 days.
Comment 3 Michele Calgaro 2015-01-25 22:29:01 CST
I will try the same sequence of step tomorrow, then let you know.
Comment 4 Michele Calgaro 2015-01-30 01:58:44 CST
Confirmed. The following sequence is enough to reproduce it.

* Start konqueror.
* "Right-click" on a folder.
* Select the 'Open With' menu option -> Verify only one instance of Cervisia.
* "Right-click" on a folder.
* From the pop-up context menu, select Properties.
* In the Properties dialog, select the Type icon (wrench icon).
* Select the Embedding tab.
* Move only tree view to the top of the list.
* Select the OK button.
* Close the Properties dialog.
* "Right-click" on a folder.
* Select the 'Open With' menu option -> Verify two instances of Cervisia.

This looks like a bug somewhere. I suggest the proposed patch *NOT* to be pushed since it takes away some other functionality. Instead proper inspection of the problem should be performed before doing any change
Comment 5 Darrell 2015-01-30 07:18:34 CST
>This looks like a bug somewhere. I suggest the proposed patch *NOT* to be pushed
I agree. A bug and the patch only masks the bug.
Comment 6 Michele Calgaro 2015-01-30 20:58:25 CST
Comment on attachment 2434 [details]
Patch to remove double Cervisia menu entry

Marked as obsolete since we have agreed not to push it.