Summary: | Copy To/Move To should discard duplicate entries in recent folder list | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | Michal Ziabkowski <mziab> |
Component: | general | Assignee: | David Faure <faure> |
Status: | RESOLVED FIXED | ||
Severity: | wishlist | CC: | faure, finex |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
Michal Ziabkowski
2008-12-15 16:29:04 UTC
I'm surprised; in my tests when developing the feature this didn't happen, and the code still checks for the url being in the list before appending it... Are you sure the exact same url occurs twice? Can you check the konquerorrc config file, group kuick-copy (or kuick-move if it's about moving)? Does it occur twice there? Yes, after selecting /tmp as the destination 3 times I get this in konquerorrc: [kuick-copy] Paths[$e]=/tmp,/tmp,/tmp Just noticed something: this only happens when using the "Root Folder" submenu. Couldn't reproduce it with "Browse..." or "Home Folder". Does it happens only using konqueror or even in dolphin? Both. It hit me first in Dolphin, but I reproduced it in Konqueror as well. Logical, since the code is shared :-) I can reproduce this indeed. Not when selecting the "recent entry", but when using the Root Folder -> tmp entry. Debugging it now. Found the reason, the code wants to add //tmp, which doesn't match the existing /tmp ;-) SVN commit 897797 by dfaure: Fix duplicated entries in the recent folder list of the Copy To / Move To submenus, was due to /tmp != //tmp. BUG: 177858 M +4 -1 konq_copytomenu.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=897797 |