Hi folks,

I've noticed a new small bug, but that needs some code rewriting. Steps to
reproduce:

1. You need to be disconnected
2. Add a new Jabber account
3. Delete it, when asked about removing it into the Jabber server say
"Remove and unregister". You'll see that the jabber account will have
disappeared.
4. Close the account manager
5. Reopen it and voilà, there's your fresh and pretty Jabber account.

I've tracked some code and I've this:

- kopete/libkopete/kopeteaccountmanager.h

    /**
     * \brief Delete the account and clean the config data
     *
     * This is praticaly called by the account config page when you remove
the account.
     */
    void removeAccount( Account *account );

- kopete/kopete/config/accounts/kopeteaccountconfig.cpp

void KopeteAccountConfig::slotRemoveAccount()
{
    KopeteAccountLVI *lvi = selectedAccount();

    if ( !lvi || !lvi->account() )
        return;

    Kopete::Account *i = lvi->account();
    if ( KMessageBox::warningContinueCancel( this, i18n( "Are you sure you
want to remove the account \"%1\"?", i->accountLabel() ),
        i18n( "Remove Account" ), KGuiItem(i18n( "Remove Account" ),
"editdelete"),
         "askRemoveAccount", KMessageBox::Notify | KMessageBox::Dangerous )
== KMessageBox::Continue )
    {
        Kopete::AccountManager::self()->removeAccount( i );
        delete lvi;
    }
}

Here, "delete lvi;" will think that
Kopete::AccountManager::self()->removeAccount( i ); should have removed the
account, but in Jabber protocol (that deleting it maybe is not as true as we
could think, because you can be disconnected). For that reason I propose
removeAccount( Account *account ); to be bool instead of void, and see if
everything went allright in the server and the account was removed.

Waiting for your comments ;)

Bye,
Rafael Fernández López.