[Bug 256970] Failed Account setup leaves new identity

[Tobias Koenig]

https://bugs.kde.org/show_bug.cgi?id=256970





--- Comment #1 from Tobias Koenig <tokoe kde org>  2010-12-28 10:21:16 ---
commit 51b1a0fc97c97fdb4fd2251f9e6a42190784d04c
branch master
Author: Tobias Koenig <tokoe at kde.org>
Date:   Tue Dec 28 10:21:03 2010 +0100

    Remove the identity from IdentityManager on rollback

    We can't use IdentityManager::rollback() here, because in
Identity::create()
    IdentityManager::commit() is called and therefor no transaction available
to
    rollback. So remove the identity explicitly in its own transaction.

    CCBUG: 256970

diff --git a/accountwizard/identity.cpp b/accountwizard/identity.cpp
index 8bb5dc0..d9f6a72 100644
--- a/accountwizard/identity.cpp
+++ b/accountwizard/identity.cpp
@@ -81,8 +81,9 @@ QString Identity::identityName() const

 void Identity::destroy()
 {
+  m_manager->removeIdentity( m_identity->identityName() );
+  m_manager->commit();
   m_identity = 0;
-  m_manager->rollback();
   emit info( i18n( "Identity removed." ) );
 }

-- 
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.