diff options
| author | saturneric <[email protected]> | 2025-04-18 17:56:51 +0000 |
|---|---|---|
| committer | saturneric <[email protected]> | 2025-04-18 17:56:51 +0000 |
| commit | a7ea8e6de4bb205d2a3783d5cb9bde8bee0086b9 (patch) | |
| tree | 54838273836a155033d33b246ec4a4200291825b /src/core/function/gpg/GpgUIDOperator.cpp | |
| parent | fix: issues found by testing (diff) | |
| parent | fix: spelling mistake on project config by nightly build (diff) | |
| download | GpgFrontend-a7ea8e6de4bb205d2a3783d5cb9bde8bee0086b9.tar.gz GpgFrontend-a7ea8e6de4bb205d2a3783d5cb9bde8bee0086b9.zip | |
Merge branch 'develop'
Diffstat (limited to 'src/core/function/gpg/GpgUIDOperator.cpp')
| -rw-r--r-- | src/core/function/gpg/GpgUIDOperator.cpp | 19 |
1 files changed, 8 insertions, 11 deletions
diff --git a/src/core/function/gpg/GpgUIDOperator.cpp b/src/core/function/gpg/GpgUIDOperator.cpp index e07b4192..ca056be0 100644 --- a/src/core/function/gpg/GpgUIDOperator.cpp +++ b/src/core/function/gpg/GpgUIDOperator.cpp @@ -29,7 +29,6 @@ #include "GpgUIDOperator.h" #include "core/function/gpg/GpgAutomatonHandler.h" -#include "core/model/GpgData.h" #include "core/utils/GpgUtils.h" namespace GpgFrontend { @@ -65,8 +64,8 @@ auto GpgUIDOperator::DeleteUID(const GpgKeyPtr& key, int uid_index) -> bool { } AutomatonNextStateHandler next_state_handler = [](AutomatonState state, - QString status, - QString args) { + const QString& status, + const QString& args) { auto tokens = args.split(' '); switch (state) { @@ -124,15 +123,14 @@ auto GpgUIDOperator::DeleteUID(const GpgKeyPtr& key, int uid_index) -> bool { return QString("Y"); case GpgAutomatonHandler::kAS_START: case GpgAutomatonHandler::kAS_ERROR: - return QString(""); default: return QString(""); } return QString(""); }; - return GpgAutomatonHandler::GetInstance(GetChannel()) - .DoInteract(key, next_state_handler, action_handler); + auto [err, succ] = auto_.DoInteract(key, next_state_handler, action_handler); + return err == GPG_ERR_NO_ERROR && !succ; } auto GpgUIDOperator::RevokeUID(const GpgKeyPtr& key, int uid_index, @@ -153,8 +151,8 @@ auto GpgUIDOperator::RevokeUID(const GpgKeyPtr& key, int uid_index, reason_text.split('\n', Qt::SkipEmptyParts)); AutomatonNextStateHandler next_state_handler = [](AutomatonState state, - QString status, - QString args) { + const QString& status, + const QString& args) { auto tokens = args.split(' '); switch (state) { @@ -240,15 +238,14 @@ auto GpgUIDOperator::RevokeUID(const GpgKeyPtr& key, int uid_index, return QString("Y"); case GpgAutomatonHandler::kAS_START: case GpgAutomatonHandler::kAS_ERROR: - return QString(""); default: return QString(""); } return QString(""); }; - return GpgAutomatonHandler::GetInstance(GetChannel()) - .DoInteract(key, next_state_handler, action_handler); + auto [err, succ] = auto_.DoInteract(key, next_state_handler, action_handler); + return err == GPG_ERR_NO_ERROR && !succ; } } // namespace GpgFrontend |