D17879: Ensure StandardDirs::saveDir returns existing dir

Modestas Vainius noreply at phabricator.kde.org
Sun Dec 30 19:46:06 GMT 2018


modax created this revision.
modax added a reviewer: dvratil.
Herald added a project: KDE PIM.
Herald added a subscriber: kde-pim.
modax requested review of this revision.

REVISION SUMMARY
  This is slight adapted version of the code removed in commit
  278a85f8ef9a90b1707ff6f66bd1a95155f93685 <https://phabricator.kde.org/R165:278a85f8ef9a90b1707ff6f66bd1a95155f93685>. I believe it got lost
  unintentionally in that commit as there are lots of places in current
  codebase which relies on StandardDirs::saveDir() returning existing
  directory (just check comments in those places!), e.g.:
  
  - AkonadiServer::startDatabaseProcess()
  - PartStreamer::PartStreamer
  - StorageJanitor::verifyExternalParts()
  - etc.

TEST PLAN
  akonadictl fsck on database with unreferenced external files actually moves
  files to file_lost+found dir now.

REPOSITORY
  R165 Akonadi

BRANCH
  bug/standarddirs-savedir-ensure-created

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

AFFECTED FILES
  src/private/standarddirs.cpp

To: modax, dvratil
Cc: kde-pim, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20181230/93eb39f4/attachment.html>


More information about the kde-pim mailing list