2 files changed, 18 insertions, 2 deletions

diff --git a/accounts/imap/imapsettings.cpp b/accounts/imap/imapsettings.cpp
index 6e45b197..fcef8971 100644
--- a/accounts/imap/imapsettings.cpp
+++ b/accounts/imap/imapsettings.cpp
@@ -222,7 +222,7 @@ void ImapSettings::save()
 
 void ImapSettings::remove()
 {
-    if (mIdentifier.isEmpty()) {
+    if (mMailtransportIdentifier.isEmpty()) {
         qWarning() << "We're missing an identifier";
     } else {
         Sink::ApplicationDomain::SinkResource mailTransportResource("", mMailtransportIdentifier, 0, QSharedPointer<Sink::ApplicationDomain::MemoryBufferAdaptor>::create());
@@ -231,14 +231,22 @@ void ImapSettings::remove()
             qWarning() << "Error while removing resource: " << errorMessage;
         })
         .exec();
+    }
 
+    if (mIdentifier.isEmpty()) {
+        qWarning() << "We're missing an identifier";
+    } else {
         Sink::ApplicationDomain::SinkResource resource("", mIdentifier, 0, QSharedPointer<Sink::ApplicationDomain::MemoryBufferAdaptor>::create());
         Sink::Store::remove(resource).then<void>([]() {},
         [](int errorCode, const QString &errorMessage) {
             qWarning() << "Error while removing resource: " << errorMessage;
         })
         .exec();
+    }
 
+    if (mAccountIdentifier.isEmpty()) {
+        qWarning() << "We're missing an identifier";
+    } else {
         Sink::ApplicationDomain::SinkAccount account("", mAccountIdentifier, 0, QSharedPointer<Sink::ApplicationDomain::MemoryBufferAdaptor>::create());
         Sink::Store::remove(account).then<void>([]() {},
         [](int errorCode, const QString &errorMessage) {
diff --git a/accounts/maildir/maildirsettings.cpp b/accounts/maildir/maildirsettings.cpp
index 5125bbe7..5c1fb177 100644
--- a/accounts/maildir/maildirsettings.cpp
+++ b/accounts/maildir/maildirsettings.cpp
@@ -232,7 +232,7 @@ void MaildirSettings::save()
 
 void MaildirSettings::remove()
 {
-    if (mIdentifier.isEmpty()) {
+    if (mMailtransportIdentifier.isEmpty()) {
         qWarning() << "We're missing an identifier";
     } else {
         Sink::ApplicationDomain::SinkResource mailTransportResource("", mMailtransportIdentifier, 0, QSharedPointer<Sink::ApplicationDomain::MemoryBufferAdaptor>::create());
@@ -241,14 +241,22 @@ void MaildirSettings::remove()
             qWarning() << "Error while removing resource: " << errorMessage;
         })
         .exec();
+    }
 
+    if (mIdentifier.isEmpty()) {
+        qWarning() << "We're missing an identifier";
+    } else {
         Sink::ApplicationDomain::SinkResource resource("", mIdentifier, 0, QSharedPointer<Sink::ApplicationDomain::MemoryBufferAdaptor>::create());
         Sink::Store::remove(resource).then<void>([]() {},
         [](int errorCode, const QString &errorMessage) {
             qWarning() << "Error while removing resource: " << errorMessage;
         })
         .exec();
+    }
 
+    if (mAccountIdentifier.isEmpty()) {
+        qWarning() << "We're missing an identifier";
+    } else {
         Sink::ApplicationDomain::SinkAccount account("", mAccountIdentifier, 0, QSharedPointer<Sink::ApplicationDomain::MemoryBufferAdaptor>::create());
         Sink::Store::remove(account).then<void>([]() {},
         [](int errorCode, const QString &errorMessage) {