Hi.
This is just a small bug that I found. The function *bool
alreadyInList(..)*would consider folders with the same starting name
to be equal.
Please let me know if it's okay to commit it.
- Vishesh Handa
Index: strigiserviceconfig.cpp
===================================================================
--- strigiserviceconfig.cpp (revision 1122801)
+++ strigiserviceconfig.cpp (working copy)
@@ -197,8 +197,10 @@ namespace {
{
bool included = false;
for ( int i = 0; i < folders.count(); ++i ) {
+ // Folders are saved with without a trailing slash. The + '/' makes sure
+ // folders just starting with the same name aren't selected.
if ( f != folders[i].first &&
- f.startsWith( folders[i].first ) )
+ f.startsWith( folders[i].first + '/' ) )
included = folders[i].second;
}
return included == include;
_______________________________________________
Nepomuk mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/nepomuk
