35799 - SMTP Editor, deleting a server fails

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Account Configuration, defect, P1)

Description

[Ninoschka Baca]

Build 2000-04-13-09M16: NT4, Linux 6.0, Mac 8.5.1

Overview: Deleting a server from the SMTP Editor fails.

Steps to reproduce:
1. Open Account Settings
2. Select the "Outgoing SMTP Server" panel
3. Select the "Advanced" button
4. Select a server and select the Delete button

Actual Results: Nothing happens and the server is not removed from the SMTP 
Editor.
Expected Results: The server should be removed immediately.

Comment 1

[selmer (gone)]

Not beta2 stopper, marking M17.

Comment 2

[Alec Flett]

I have this fixed in my tree - I implemented it to make debugging 31787 and
41030 easier.

Without this users can create SMTP servers, but not delete them, which means
they could end up with a giant list of servers.

Fix is low-risk - basically just hooking up the delete button to the backend.
The backend has already been tested by Par's component testing effort.

..so I think it's a good nsbeta2 candidate.

Comment 3

[Alec Flett]

(oh, and of course _I_ have been testing (and using) the backend API) for a few
days now, in addition to Par's effort)

Comment 4

[leger]

Putting on [nsbeta2+][will be minus on 6/15] radar.

Comment 5

[Alec Flett]

fix is in.
A note when testing - I have made it impossible to delete the default SMTP
server, which forces the user to always have one SMTP server.

Comment 6

[Ninoschka Baca]

Build 2000-06-26-08M17: NT4, Linux 6.0, Mac 9.04
Verified Fixed. SMTP Servers can now be deleted