Bug 197840 - Creating a link to the folder on the desktop ends with ".desktop"
Summary: Creating a link to the folder on the desktop ends with ".desktop"
Status: RESOLVED FIXED
Alias: None
Product: konqueror
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: David Faure
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-06-25 15:25 UTC by Einar Lielmanis
Modified: 2009-08-27 17:20 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Einar Lielmanis 2009-06-25 15:25:29 UTC
Version:           svn:986516 (using Devel)
OS:                Linux
Installed from:    Compiled sources

On the desktop folderview, when creating links to other folders, the names have a needless ".desktop" extension,

rclick on desktop folder,
create new...
Link to location (URL)...
type in e.g ~/Pictures (file name: Pictures)

expected: an icon leading to "~/Pictures", named "Pictures"
got: an icon leading to "~/Pictures" named "Pictures.desktop"
Comment 1 Aaron J. Seigo 2009-06-26 07:03:42 UTC
it also happens in dolphin; it's a bug in libkonq.
Comment 2 Aaron J. Seigo 2009-06-26 07:04:57 UTC
David: is this the right component for libkonq bugs?

in any case, the problem is in KNewMenu::slotActionTriggered(QAction* action).

if ( KDesktopFile::isDesktopFile( entry.templatePath ) ) is true, and so is if ( df.readType() == "Link" ), and so it appends ".desktop" to the resulting target a few lines later.
Comment 3 David Faure 2009-08-27 17:19:46 UTC
Well, that's because it's a desktop file and not a symlink :-)
But yeah, extensions aren't strictly necessary, since we detect files without it too. I found that I added that logic in 2000, (r62771), so things were different back then.

But I think what you really wanted was a symlink. Wish/Bug 66248, basically, no? (although, as discussed there, the problem is that the icon of the symlink cannot be changed)
Comment 4 David Faure 2009-08-27 17:20:55 UTC
SVN commit 1016350 by dfaure:

respect the user's wish and don't append .desktop to the filename. Fix will be in 4.3.1
BUG: 197840


 M  +0 -2      knewmenu.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1016350