230108 - Deleting Account gives incorrect error message "Failed to remove this account."

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Account Configuration, defect)

Description

[Ray Booysen]

Build:2004010408

If an account is deleted, a message box pops up saying that the account deletion
failed.  However, the account is deleted.

Comment 1

[Ray Booysen]

Attachment: Screenshot showing Message Box

Comment 2

[Henrik Lynggaard Hansen]

*** Bug 227188 has been marked as a duplicate of this bug. ***

Comment 3

[Henrik Lynggaard Hansen]

Changing to All/all since the dupe was on linux and this was on windows

Comment 4

[Stefan Borggraefe]

*** Bug 227188 has been marked as a duplicate of this bug. ***

Comment 5

[Stefan Borggraefe]

*** Bug 228235 has been marked as a duplicate of this bug. ***

Comment 6

[Stefan Borggraefe]

The statement accountManager.removeAccount(account);
in accountManager.js always throws an exception.

This works in Mozilla 1.6a. The only change to removeAccount() after the release
of Mozilla 1.6a was made via bug 218825. So I suspect this bug is a regression
caused by bug 218825. What do you think, Neil?

Comment 7

[[e-mail]]

The JavaScript Debugger says that when you remove an account its item is removed
from the tree, therefore deselecting it... this means that the account selection
code is triggered when there isn't a selected account... the actual error occurs
on line 716 of AccountManager.js when it tries to compare the old and new value.

There are actually two further errors, the first error is that when you remove
an account, and you had the "main" page selected, the account manager won't load
the main page for any account until you first load one of the other pages; the
second error is that after deleting an account the account manager doesn't
scroll back to the very top.

Comment 8

[José Jeria]

The account is not completely removed, in the mail folder of your profile, there
is a folder left with the name of the pop adress.

Comment 9

[Stefan Borggraefe]

José: That's Bug 128744. This bug is just about the alert.

Comment 10

[Stefan Borggraefe]

Attachment: Proposed fix.

This fixes the bug by checking if there is currently something selected. If not
onAccountClick() returns early. A call of setSelection(null, null) after the
account removal ensures that onAccountClick() is called again with a valid
selection. Thus the other problems mentioned by Neil disappear, too.

I furthermore removed the workaround for bug 51546, because it's not necessary
anymore (most likely because bug 92366 was fixed) and renamed the variable
result to currentSelection to reflect its purpose.

I would let Seth do the sr to see if he's ok with the removal of the workaround
for bug 51546.

What I still don't really understand is why this worked before...

Comment 11

[[e-mail]]

Attachment: Includes my other two nits

This is just Stefan's patch with a couple of tweaks to fix my other nits.

Comment 12

[Stefan Borggraefe]

Checked in by Neil! Thanks for that and the updated patch! Marking FIXED. :-)