Message string for SMTP server connection error is incorrect. File: composeMsgs.properties, key: smtpSendRefused

RESOLVED FIXED in Thunderbird 40.0

Status

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: iacchi, Assigned: sshagarwal)

Tracking

({regression})

unspecified
Thunderbird 40.0
regression

Thunderbird Tracking Flags

(thunderbird40 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

4 years ago
In composeMsgs.properties, the key smtpSendRefused shows the text "The message could not be sent because connecting to Outgoing server (SMTP) %S failed. The server may be unavailable or is refusing Outgoing server (SMTP) connections. Please verify that your Outgoing server (SMTP) settings are correct and try again."

The previous instance of the string only reported "SMTP server" instead of "Outgoing server (SMTP)". This change is ok for the first and the last part of the sentence, but "is refusing Outgoing server (SMTP) connections" makes no sense, it should be "is refusing SMTP connections".

Updated

4 years ago
Blocks: 952493
(Assignee)

Comment 1

4 years ago
Created attachment 8590430 [details] [diff] [review]
Patch v1
Attachment #8590430 - Flags: review?(iann_bugzilla)
(Assignee)

Comment 2

4 years ago
Comment on attachment 8590430 [details] [diff] [review]
Patch v1

Sorry it has issues.
Attachment #8590430 - Attachment is obsolete: true
Attachment #8590430 - Flags: review?(iann_bugzilla)
(Assignee)

Comment 3

4 years ago
Created attachment 8590441 [details] [diff] [review]
Patch v1.1

I forgot to change the string key. This patch fixes the issue.

Thanks.
Attachment #8590441 - Flags: review?(iann_bugzilla)
(Assignee)

Updated

4 years ago
Component: MailNews: Composition → Composition
Product: SeaMonkey → MailNews Core
Assignee: nobody → syshagarwal
Keywords: regression

Updated

4 years ago
Attachment #8590441 - Flags: review?(iann_bugzilla) → review+

Updated

4 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 4

4 years ago
Thanks.
Keywords: checkin-needed

Comment 5

4 years ago
Comment on attachment 8590441 [details] [diff] [review]
Patch v1.1

a=me for SeaMonkey CLOSED TREE
https://hg.mozilla.org/comm-central/rev/a195e2c4d205
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
status-thunderbird40: --- → fixed
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 40.0
You need to log in before you can comment on or make changes to this bug.