michaelh created this revision.
michaelh added reviewers: dfaure, vhanda, Frameworks.
Restricted Application added a project: Frameworks.
michaelh requested review of this revision.

REVISION SUMMARY
  In debug mode this assert prevents usage of cli tools like baloosearch.
  Replace Q_ASSERT with an if.. clause until the cause for assert failure is 
clear

TEST PLAN
    $baloosearch type:Audio | grep '^$' | wc -l
    17
  
  + 17 lines on stderr "Id  148657949215033349  has no valid filename, ignoring 
it."
  
  baloosearch://?query=type:Audio in dolphin looks normal.

REPOSITORY
  R293 Baloo

BRANCH
  empty-file

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

AFFECTED FILES
  src/engine/documenturldb.cpp

To: michaelh, dfaure, vhanda, #frameworks
Cc: michaelh, ngraham

Reply via email to