Closed Bug 99133 Opened 24 years ago Closed 21 years ago

Acct Wiz, ignore blank spaces after an email address or server name

Categories

(SeaMonkey :: MailNews: Account Configuration, defect)

Product:
SeaMonkey
Component:
MailNews: Account Configuration
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla1.4beta

People

(Reporter: nbaca, Assigned: Stefan.Borggraefe)

References

Details

(Whiteboard: [adt3])

Attachments

(1 file, 2 obsolete files)

Hopefully even better patch ;-)
5.26 KB, patch
Bienvenu
: review+
mscott
: superreview+
Details | Diff | Splinter Review
Branch build 2001-09-06-05: WinMe Branch build 2001-09-07-03: Mac 9.04 haven't tried linux yet. Overview: In the account wizard's dialog, if an entry is followed by a blank space then ignore the blank space. Steps to reproduce: Scenario 1: 1. Open the Account Wizard 2. Select the ISP or email provider radio button 3. In the Identity dialog enter an email address followed by a blank space ("qatest20@netscape.com ") 4. Select the Next button Actual Results: It reports "Please enter a valid email" Expected Results: It should ignore the trailing space and continue through the Account Wizard. address". Scenario 2: 1. Open the Account Wizard 2. Select the ISP or email provider radio button 3. In the Identity dialog enter a User Name and Email address, Next 4. In the Server dialog enter a Server Name followed by a blank space ("server1 ") Actual Results: It reports "Enter a valid host name" Expected Results: It should ignore the trailing space and continue through the Account Wizard.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.8
Target Milestone: mozilla0.9.8 → mozilla1.0.1
is bug 124170 relevant to dup here? (or at all valid... it's about leading space
*** Bug 163392 has been marked as a duplicate of this bug. ***
OS: Windows ME → All
Attached patch patch (obsolete) — — Splinter Review
rather trivial, please r/sr/etc.
Attached patch actual patch (obsolete) — — Splinter Review
eh. attached wrogn version, sorry. patch removes both leading and trailing whitespace from address and server fields. Bug 124170 is not affected -- it was not even previously possible to successfully enter leading whitespace to smtp server in the acct wizard.
Attachment #95819 - Attachment is obsolete: true
Marking nsbeta1 because this can be very confusing to a user since they don't know to check for a blank space. Note: Problems also occur if an address is created in the address book followed by a blank space. Try sending a message with the "problem" address and it fails.
Keywords: nsbeta1
Mail triage team: nsbeta1+/adt3
Keywords: nsbeta1 → nsbeta1+
Whiteboard: [adt3]
re-assigning nsbeta1+ bugs
Assignee: racham → sspitzer
Status: ASSIGNED → NEW
Target Milestone: mozilla1.0.1 → mozilla1.4beta
*** Bug 154194 has been marked as a duplicate of this bug. ***
Assignee: sspitzer → Stefan.Borggraefe
Attachment #95823 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #157851 - Flags: review?(bienvenu)
Attachment #157851 - Flags: review?(bienvenu) → review+
Attachment #157851 - Flags: superreview?(mscott)
Attachment #157851 - Flags: superreview?(mscott) → superreview+
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: