After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 302742 - The mail error messages is difficult to understand/translate
The mail error messages is difficult to understand/translate
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.8.x (obsolete)
Other All
: Normal minor
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
: 345045 (view as bug list)
Depends on:
Blocks: 236276
 
 
Reported: 2005-05-02 17:10 UTC by Funda Wang
Modified: 2010-04-01 18:02 UTC
See Also:
GNOME target: ---
GNOME version: 2.9/2.10


Attachments
Fixed (2.26 KB, patch)
2006-08-22 13:08 UTC, ushveen kaur
needs-work Details | Review
Resubmitted. (2.28 KB, patch)
2006-08-23 09:11 UTC, ushveen kaur
needs-work Details | Review

Description Funda Wang 2005-05-02 17:10:20 UTC
Considering following statements:

#: ../mail/mail-errors.xml.h:205
msgid ""
"The following vFolder(s):\n"
"{0}\n"
"Used the now removed folder:\n"
"    \"{1}\"\n"
"And have been updated."

#: ../mail/mail-errors.xml.h:213
msgid ""
"The following filter rule(s):\n"
"{0}\n"
"Used the now removed folder:\n"
"    \"{1}\"\n"
"And have been updated."

What are they talking about? Would it be better if it looks like:

"The {0} vFolder(s) have been updated, due to the {1} folder which it has "
"been using is now removed."
Comment 1 André Klapper 2005-05-03 16:22:19 UTC
very surely this is a duplicate
Comment 2 André Klapper 2006-07-06 21:36:09 UTC
these strings have incorrect capitals, shouldn't have colons, lack ngettext
support and aren't nice to translate.
Comment 3 André Klapper 2006-07-06 21:36:18 UTC
*** Bug 345045 has been marked as a duplicate of this bug. ***
Comment 4 ushveen kaur 2006-08-22 13:08:26 UTC
Created attachment 71360 [details] [review]
Fixed
Comment 5 André Klapper 2006-08-22 13:35:37 UTC
patch needs work, because:

"The "{0}" Search Folder(s) have been updated"
""{0}"" can be a list of folders. i'm not sure about the capitalization here.

"due to the "{1}" folder which it has been using is now removed." is not easy to understand at all, at least in my opinion. i had to read it three times to understand the meaning.

this still lacks ngettext support, which has to be added in the .c file i guess.
Comment 6 ushveen kaur 2006-08-23 09:11:32 UTC
Created attachment 71440 [details] [review]
Resubmitted.

I dont know how to add ngettext support to .xml file......adding to the .c file also does not seem posible in this context.
However,the string is changed to:
The "{0}" Search Folder(s) have been updated due to removal of "{1}" folder, which was used by the search folder.
hope this is clear and easy to understand.
Comment 7 Srinivasa Ragavan 2007-08-23 07:06:12 UTC
Some how I feel that the text has redundant terms. Also the first hunk has been added already.
Comment 8 Matthew Barnes 2008-03-11 00:28:38 UTC
Bumping version to a stable release.
Comment 9 Milan Crha 2010-04-01 18:02:56 UTC
Created commit 125573b in evo master (2.31.1+)