Commit 0f153592 authored by Toni Asensi Esteve's avatar Toni Asensi Esteve

Modified bookmark paths can be used later

Make it possible that modified bookmark paths can be used later. This
avoids Krusader users seeing the effects described in the "Editing
a local path in Bookmark Manager breaks a bookmark" bug report
(https://bugs.kde.org/show_bug.cgi?id=393320), though it would
be better to solve that upstream frameworks-kbookmarks bug.

Revision: !21
parent 4a6a7cdd
......@@ -411,6 +411,16 @@ void KrBookmarkHandler::buildMenu(KrBookmark *parent, QMenu *menu, int depth)
menu->addSeparator();
continue;
}
QUrl urlToSet = bm->url();
if (!urlToSet.isEmpty() && urlToSet.isRelative()) {
// Make it possible that the url can be used later by Krusader.
// This avoids users seeing the effects described in the "Editing a local path in Bookmark
// Manager breaks a bookmark" bug report (https://bugs.kde.org/show_bug.cgi?id=393320),
// though it would be better to solve that upstream frameworks-kbookmarks bug
bm->setURL(QUrl::fromUserInput(urlToSet.toString(), QString(), QUrl::AssumeLocalFile));
}
menu->addAction(bm);
CONNECT_BM(bm);
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment