Closed Bug 1386587 Opened 2 years ago Closed 2 years ago

Comment and error message for pop3ServerBusy is wrong

Categories

(MailNews Core :: Networking: POP, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 57.0

People

(Reporter: nONoNonO, Assigned: jorgk)

Details

Attachments

(1 file)

In chrome/mailnews/localMsgs.properties both for mail and suite the comment for pop3ServerBusy and the error message itself are both probably wrong. The comment seems to be wrongly copied and should read something like 'Status - pop3 server busy' or 'Status - pop3 folder busy' and the error message itself should possibly be tweaked, since it seems to occur when a local folder is busy and not when a server or an account is busy.
# Status - write error occurred
# LOCALIZATION NOTE (pop3ServerBusy): Do not translate the word "%S" below.
# Place %S where the account name should appear.
pop3ServerBusy=The account %S is being processed. Please wait until processing is complete to get messages.

Well, it doesn't see so wrong to me. So most likely there was a write error on the folder, but we could use:
# Status - pop3 folder busy

What's your suggestion for the message?

CC'ing Richard who might help with a patch.
:frg suggested on IRC that the message might be wrong, maybe he has a suggestion for improving it?
Flags: needinfo?(frgrahl)
> The account %S is being processed.

Maybe if this is not too long: "Server is currently busy. Operation for account %S in progress. Please wait until processing is complete to get messages." If it is indeed a folder busy message probably without the "Server is currently busy".

Acount being processed just doesn't feel right. 

fwiw I never saw the message yet.
Flags: needinfo?(frgrahl)
If there's no suggestion to improve the entity, I propose to only fix the comment, as suggested by Jorg in comment #1.
Attachment #8902208 - Flags: review?(o.e.ekker)
Comment on attachment 8902208 [details] [diff] [review]
1386587-pop3ServerBusy.patch (v1)

Review of attachment 8902208 [details] [diff] [review]:
-----------------------------------------------------------------

I'm not sure if I'm qualified to review, but this looks good to me :-)
Attachment #8902208 - Flags: review?(o.e.ekker) → review+
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/9f258e1db839
Correct comment for pop3ServerBusy in localMsgs.properties. r=nONoNonO
Status: UNCONFIRMED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Thanks for the review. It's better to make sure the reporter is happy than land it without review and rs=comment-only. Besides, in cases like this I can choose my reviewer according to what I deem appropriate ;-)
Target Milestone: --- → Thunderbird 57.0
Assignee: nobody → jorgk
You need to log in before you can comment on or make changes to this bug.