https://bugs.kde.org/show_bug.cgi?id=375370

Nate Graham <n...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
      Latest Commit|                            |https://commits.kde.org/bal
                   |                            |oo/8a0b8270dcd389db38cf3a4c
                   |                            |20d65d46a9dda5d7
         Resolution|---                         |FIXED
   Version Fixed In|                            |5.51

--- Comment #13 from Nate Graham <n...@kde.org> ---
Git commit 8a0b8270dcd389db38cf3a4c20d65d46a9dda5d7 by Nathaniel Graham.
Committed on 29/09/2018 at 22:59.
Pushed by ngraham into branch 'master'.

Allow removing non-existent folders from include and exclude lists

Summary:
Currently, if you add a folder to Baloo's `includeFolders` or `excludeFolders`
lists, it cannot be removed again if it has been deleted. This patch fixes that
for both `balooctl` and also the KCM.
FIXED-IN: 5.51

Test Plan:
```
mkdir ~/foobar-test
balooctl config add excludeFolders ~/foobar-test
rmdir ~/foobar-test
balooctl config remove excludeFolders ~/foobar-test
```

Without the patch, `balooctl` exits with an error, reports "Path does not
exist", and does not remove the entry.

With the patch, `balooctl` exits cleanly with no error and removes the path.

Works from the KCM, too.

Reviewers: #baloo, bruns

Reviewed By: #baloo, bruns

Subscribers: kde-frameworks-devel

Tags: #frameworks, #baloo

Differential Revision: https://phabricator.kde.org/D15844

M  +3    -24   src/tools/balooctl/configcommand.cpp

https://commits.kde.org/baloo/8a0b8270dcd389db38cf3a4c20d65d46a9dda5d7

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to