D7502: Fix for QStandardPaths porting (D7418)

Christian Ehrlicher noreply at phabricator.kde.org
Thu Aug 24 07:12:20 UTC 2017


chehrlic created this revision.
Restricted Application added a reviewer: KDE Games.
Restricted Application added a subscriber: KDE Games.

REVISION SUMMARY
  Since QStandardPaths::locate() only returns a valid filename when
  
    the file exists, we have to make sure to not pass an empty 
    filename to KConfig when trying to restore / save a puzzle

TEST PLAN
  1. remove ~/.local/share/palapeli (very important!)
    1. Start a puzzle, quit palapeli
    2. Start palapeli and proceed with the last puzzle -> puzzle is loaded correctly
    3. Try to restart puuzle (after finished or within) -> all pieces are placed randomly

REPOSITORY
  R423 Palapeli

BRANCH
  qstandarddirs

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

AFFECTED FILES
  src/engine/gameplay.cpp
  src/engine/gameplay.h

To: chehrlic, #kde_games
Cc: #kde_games
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-games-devel/attachments/20170824/32a1c7c3/attachment.html>


More information about the kde-games-devel mailing list