D20775: file import: do not use overwritten temorary file

Tobias Klausmann noreply at phabricator.kde.org
Tue Apr 23 21:45:05 BST 2019


tklausmann created this revision.
tklausmann added a reviewer: Umbrello.
Herald added a subscriber: umbrello-devel.
Herald added a project: Umbrello.
tklausmann requested review of this revision.

REVISION SUMMARY
  We checked if the temporary file we created still exists after its creation,
  yet in the meantime we have already copied to it (which overwrites it):
  KIO::FileCopyJob *job = KIO::file_copy(url, dest, -1, KIO::Overwrite);
  
  Instead of the temporary file, check if the regular file we use later on exists.
  
  Bug: 406574

REPOSITORY
  R139 Umbrello

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

AFFECTED FILES
  umbrello/umldoc.cpp

To: tklausmann, #umbrello
Cc: umbrello-devel, vanderbilt, astranieri
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/umbrello-devel/attachments/20190423/0ec310ad/attachment.html>


More information about the umbrello-devel mailing list