D14859: Don't try to call transaction documentUrl with a 0 id

[Jaime Torres Amate]

jtamate created this revision.
jtamate added a reviewer: Frameworks.
Herald added projects: Frameworks, Baloo.
Herald added subscribers: Baloo, kde-frameworks-devel.
jtamate requested review of this revision.

REVISION SUMMARY
  In Transaction::documentUrl(quint64 id) const, there is
  Q_ASSERT(id > 0);
  so don't even try to call it with a 0 id.

TEST PLAN
  Executing balooshow -i /home/user/* when baloo is creating the index after deleting .local/share/baloo
  before: crash (assert)
  after: balooshow ends normally.

REPOSITORY
  R293 Baloo

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

AFFECTED FILES
  src/tools/balooshow/main.cpp

To: jtamate, #frameworks
Cc: kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180815/fd433084/attachment-0001.html>