The default bug view has changed. See this FAQ.

New account wizard uses "server name" and "hostname"

VERIFIED FIXED in Thunderbird 13.0

Status

MailNews Core
Account Manager
--
minor
VERIFIED FIXED
16 years ago
3 years ago

People

(Reporter: Dan Mellem, Assigned: aceman)

Tracking

({polish})

Trunk
Thunderbird 13.0
polish

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

5.69 KB, patch
standard8
: review+
Ian Neal
: review+
Details | Diff | Splinter Review
(Reporter)

Description

16 years ago
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:0.9.2) Gecko/20010628
BuildID:    2001062815

When a new mail account is created and one of the POP3/IMAP and SMTP server host
names is left blank, an alert "Please enter a valid hostname." is displayed. The
fields in the form are labeled "Server Name." The names should match.

Is this test even necessary? The account could only be used for sending (e.g., I
use the browser occasionally to send but have my work address and the e-mail
address. I don't retrieve mail from it) or receiving. Most people would set up
both servers but, if one is blank, just don't check mail/allow creation.

Reproducible: Always
Steps to Reproduce:
1.In a new profile, go to e-mail.
2.Select ISP, type in your name, etc., and go forward 2x.
3.

Also, if you go back to the "ISP or Newsgroup" selection, anything that was set
is lost.
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows 2000 → All
Hardware: PC → All

Comment 1

16 years ago
I think `server name' is the more understandable of the two by a considerable 
margin.

Comment 2

16 years ago
Taking. What I'll also do is have the first input field which requires a server
name but doesn't yet have one, focused and ready for typing into. This saves the
user wondering which field the dialog is complaining about (it may not be obvious).

On the point about whether these checks are necessary, I'd say they were, at
least until we have sufficient time to code a run-time checking of such servers
when we want to read/send mail and prompt for server names if we don't have them.
Assignee: racham → jg
Keywords: polish

Updated

16 years ago
Status: NEW → ASSIGNED
Priority: -- → P1
Target Milestone: --- → mozilla0.9.7

Comment 3

15 years ago
Also, it requires a server _NAME_ and won't accept an IP number (in 0.9.7/linux,
build id 2001122108). Why? You don't always have "host names", or at least
not always ones that are easier to type than are their numeric counterparts.

Comment 4

15 years ago
The Target Milestone (0.9.7) is passed.  This bug should be retargeted.
If you are no longer interested or able to work on this bug, please reset
the Target Milestone and hand it back to default owner of the component.

Comment 5

15 years ago
this needs to be retargeted

James: any progress?

a quick freetext search on lxr for "Please enter a valid hostname" returns these
4 links. Not sure about the 3rd one as it is an addressbook thing.

http://lxr.mozilla.org/mozilla/source/l10n/langpacks/en-DE/chrome/en-DE/messenger/locale/prefs.properties#33
http://lxr.mozilla.org/mozilla/source/l10n/langpacks/en-GB/chrome/en-GB/messenger/locale/prefs.properties#33
http://lxr.mozilla.org/mozilla/source/mailnews/addrbook/resources/locale/en-US/addressBook.properties#97
http://lxr.mozilla.org/mozilla/source/mailnews/base/prefs/resources/locale/en-US/prefs.properties#43
Keywords: mozilla1.0

Comment 6

15 years ago
Bouncing out, hope someone with more time than I catches.
Assignee: jg → racham
Status: ASSIGNED → NEW

Comment 7

15 years ago
Is all this bug needs is the word hostname changed to server name?
If thats the case, I just did a quick patch that does this.  I'll attach it so
you can see what I've done.

Comment 8

15 years ago
Created attachment 106602 [details] [diff] [review]
quick patch that changes hostname to server name

Here is the quick patch I made.  All it does is change hostname to server name
in the files that basic found minus the addressbook one.  The addressbook one
can be added if you want.
(Reporter)

Comment 9

15 years ago
Yes, that's all that's needed unless it would be better to change the
functionality so a receive-only (or send-only with a reply-to) configuration is
possible.

Comment 10

15 years ago
I'm not quite sure what you mean by "receive-only (or send-only with a reply-to)
configuration".  However, no matter what it is, my limited programming knowledge
probably won't be able to do something like that.  As well, it sounds like it
changes the behaviour of the wizard, which means it should be a different bug.
(Reporter)

Comment 11

15 years ago
Right now the wizard REQUIRES an entry for Incoming Server and Outgoing Server.
Although most users will have an entry for both of these, it's not really required. 

Examples:
If someone leaves the Incoming Server field blank then they will only be able to
send mail from the client. An example would be a webmail user. They can send
e-mail using the proper return address but won't be able to retrieve their mail
using IMAP or POP.
If someone leaves the Outgoing Server blank then they won't be able to send
mail. This could be used just to look at mail in an IMAP mailbox or to retrieve
messages (e.g., cron e-mails) from a receive-only mailbox.

I propose dropping the check altogether and verify that the mail client does
something smart if the field is blank.
Keywords: mozilla1.0
Target Milestone: mozilla0.9.7 → ---

Comment 12

14 years ago
FYI: If the proposal in comment 11 is adopted, I think that should fix Bug
159700 as well.
mass re-assign.
Assignee: racham → sspitzer
Comment on attachment 106602 [details] [diff] [review]
quick patch that changes hostname to server name

This should change the "hostname" to "server name" which is the original issue
in this bug report.
Attachment #106602 - Flags: review?(neil.parkwaycc.co.uk)

Updated

13 years ago
Attachment #106602 - Flags: review?(neil.parkwaycc.co.uk) → review+
Comment on attachment 106602 [details] [diff] [review]
quick patch that changes hostname to server name

review please
Attachment #106602 - Flags: superreview?(mscott)

Updated

13 years ago
Attachment #106602 - Flags: superreview?(mscott) → superreview+
Product: Browser → Seamonkey

Updated

12 years ago
Assignee: sspitzer → mail
Priority: P1 → --
QA Contact: nbaca

Comment 16

8 years ago
This patch has r+ and sr+, however there doesn't seem to be any indication that it was ever checked in. Looking at MXR, both SeaMonkey and Thunderbird still have the "Please enter a valid hostname" wording in various locations.
(In reply to comment #16)
> This patch has r+ and sr+, however there doesn't seem to be any indication that
> it was ever checked in. Looking at MXR, both SeaMonkey and Thunderbird still
> have the "Please enter a valid hostname" wording in various locations.

The patch is at least incorrect now. The string names should be changing to alter localisers to the change in context.

Updated

8 years ago
QA Contact: mailnews-account
Whiteboard: [has draft patch]

Comment 18

7 years ago
Instead of arguing about whether it should be one or the other, why not:

"Server hostname"

Comment 19

7 years ago
That sounds a bit redundant as server=host, and "server name" was already approved. There are also similar occurrences in the shared address book,
which could be changed to "server name" then as well.

I'm adding the MXR query for that string and move this bug to MailNews Core.
Assignee: mail → nobody
Component: MailNews: Account Configuration → Account Manager
Product: SeaMonkey → MailNews Core
QA Contact: mailnews-account → account-manager
Whiteboard: [has draft patch] → [needs new patch]
(Assignee)

Comment 20

5 years ago
This also happens in the Account manager-> SMTP server name.
I'll update the patch.
Assignee: nobody → acelists
Whiteboard: [needs new patch]
(Assignee)

Comment 21

5 years ago
Created attachment 596393 [details] [diff] [review]
patch for account manager

I think the mxr search is bogus. The other places where "hostname" is referenced in an error message use the term "web site" (cookie manager) and "hostname" (LDAP addressbook) in the corresponding form field. So there is no point in changing the error text to "server name" in them.
I attach updated patch that only fixes the account manager.
Attachment #106602 - Attachment is obsolete: true
Attachment #596393 - Flags: review?(mbanner)
(Assignee)

Updated

5 years ago
Attachment #596393 - Flags: review?(iann_bugzilla)

Updated

5 years ago
Attachment #596393 - Flags: review?(iann_bugzilla) → review+

Updated

5 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 22

5 years ago
(In reply to :aceman from comment #21)
> I think the mxr search is bogus. The other places where "hostname" is
> referenced in an error message use the term "web site" (cookie manager) and
> "hostname" (LDAP addressbook) in the corresponding form field. So there is
> no point in changing the error text to "server name" in them.

So technically the cookie manager should also be fixed from calling "hostname" in the error message. But it should be another bug and it probably is in the Firefox/Toolkit.
Anybody interested in filing it?
Attachment #596393 - Flags: review?(mbanner) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
Checked in: http://hg.mozilla.org/comm-central/rev/59710ebe215a
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 13.0
(Assignee)

Updated

5 years ago
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.