> On June 7, 2012, 2:36 p.m., Sebastian Trueg wrote: > > The problem with this fix is that it breaks tree listings. That was the > > original reason for the code you comment out. So a real fix would look > > different. Sadly I do not have much time for Nepomuk these days... so I > > would accept the patch if it contained a big comment.
Is there a user visible way to see a tree listing that would be broken? If there is something I could test I could perhaps cook up the real fix. Thanks - Simeon ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105158/#review14470 ----------------------------------------------------------- On June 6, 2012, 2:13 a.m., Simeon Bird wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105158/ > ----------------------------------------------------------- > > (Updated June 6, 2012, 2:13 a.m.) > > > Review request for Dolphin, Nepomuk, Peter Penz, and Sebastian Trueg. > > > Description > ------- > > This patch attempts to fix bug 293111. > > The problem is that the nepomuk kioslave gives files a targetUrl > understandable by non-KDE applications (ie, their real path on the > filesystem). > > However, it does not give this to directories, and it only does the fancy > stuff on the query folder, not any subfolders. The subfolders are opened with > the kioslave base listdir class. The easy way to fix this is to set > UDS_TARGET_URL for director a "real" url, which is what the attached patch > does. > > > This addresses bug 293111. > http://bugs.kde.org/show_bug.cgi?id=293111 > > > Diffs > ----- > > nepomuk/kioslaves/search/searchfolder.cpp 8088c46 > > Diff: http://git.reviewboard.kde.org/r/105158/diff/ > > > Testing > ------- > > I compiled and ran it on my machine with some search results. Not tested on > search results that are not local files. > > > Thanks, > > Simeon Bird > >
_______________________________________________ Nepomuk mailing list [email protected] https://mail.kde.org/mailman/listinfo/nepomuk
