smithjd added a comment.

  Technically, this check isn't actually needed, though it does prevent the 
user from entering a path more than once. At first glance this looks like it 
should work:
  
    if (folder.startsWith(path))
  
  This doesn't prevent the user from also re-specifying valid paths.

REPOSITORY
  R293 Baloo

REVISION DETAIL
  https://phabricator.kde.org/D15583

To: ngraham, #baloo, #dolphin, bruns, #frameworks
Cc: smithjd, anthonyfieroni, kde-frameworks-devel, ashaposhnikov, michaelh, 
astippich, spoorun, ngraham, bruns, abrahams

Reply via email to