Achieve that Krusader searches correctly if the the "Follow links" checkbox is disabled (note: it's the default).
This change is aimed to solve https://bugs.kde.org/show_bug.cgi?id=410120
Details
Details
The test folder mentioned in https://bugs.kde.org/show_bug.cgi?id=410120 can be used to search anything in it (having the "Follow links" checkbox disabled, which is the default).
Diff Detail
Diff Detail
- Repository
- R167 Krusader
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Comment Actions
It works properly now - thanks!
I checked the commit you mentioned in the bug. Before it the checkbox was still ignored but the links were always not followed.
Comment Actions
If you'd like this fix to be included in v2.7.2, please push it to master by Aug 4 and I'll backport to stable. Please don't forget to specify BUG, FIXED, Differential Revision tags in the commit message.
Comment Actions
Ah, FileItem::isDir() can also return true for symlinks. Thanks again for fixing the bug i introduced, Toni!
(QFileInfo::isDir() has the same (documented) behaviour and I guess KFileItem::isDir() as well.)
Comment Actions
If any developer is reading this, he'll probably know that those developments end up being a very complex subject. Thank you for all the source code, Alex!