328175 - ISP files: can't force smtp server creation

Status: RESOLVED FIXED

(Thunderbird :: Account Manager, defect)

Description

[Yann Rouillard]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.1) Gecko/20060210 Ubuntu/dapper Firefox/1.5.0.1
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.1) Gecko/20060210 Ubuntu/dapper Firefox/1.5.0.1

When using isp rdf file to customize Account Wizard, it seems the smtpCreateNewServer option is not taken in account. If a default smtp server exists, it is always used instead.

Reproducible: Always

Steps to Reproduce:
1. Put the isp_example.rdf file attached in the defaults/isp directory
2. launch the wizard, create a standard dummy account (not the Imap test account)
3. re-launch the wizard to create Imap test account
4. go to the server page
Actual Results:  
the server page says the existing smtp server will be used.

Expected Results:  
As we requested to create a new smtp server, we should be allowed to enter a new smtp server.

Comment 1

[Yann Rouillard]

Attachment: ISP example file to test the bug.

Comment 2

[Yann Rouillard]

Attachment: Patch which allows the smtpCreateNewServer option to work

This patch allows the smtp hostname editable field to be shown in the server page if smtpCreateNewServer was set to true in the isp rdf file.

Comment 3

[Yann Rouillard]

This bug is part of a serie of bug I filled about bugs/feature in isp rdf files, 
see:
https://bugzilla.mozilla.org/show_bug.cgi?id=328174
https://bugzilla.mozilla.org/show_bug.cgi?id=328175
https://bugzilla.mozilla.org/show_bug.cgi?id=328178
https://bugzilla.mozilla.org/show_bug.cgi?id=328180

Comment 4

[David :Bienvenu]

Comment on attachment 212727 [details] [diff] [review]
Patch which allows the smtpCreateNewServer option to work

forgot to mark this reviewed.