Review Request 127925: KWalletd migration: fix error handling.

David Faure faure at kde.org
Mon May 23 16:58:29 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127925/
-----------------------------------------------------------

(Updated May 23, 2016, 6:58 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Valentin Rusu.


Changes
-------

Submitted with commit 58a8036790ca962fa6f5645e76b8d1264f4de8db by David Faure to branch master.


Repository: kwallet


Description
-------

After a successful migration, performMigration() returns true,
so the qDebug "failed" was wrong, and the "ok" bool should be set to
true so that isMigrationWizardOk() returns true.

This stops the migration from happening on every single boot.


Diffs
-----

  src/runtime/kwalletd/migrationagent.cpp 217d4a8485863835819d9b2cd3ed08b0829fce7f 

Diff: https://git.reviewboard.kde.org/r/127925/diff/


Testing
-------


Thanks,

David Faure

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160523/d699ef65/attachment.html>


More information about the Kde-frameworks-devel mailing list