Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#364 i18n bugs

release_4.0
closed
Other (70)
5
2012-10-10
2005-08-18
No

Hi,

I've noticed some i18n bugs. Here they are:

-
com/puppycrawl/tools/checkstyle/checks/ArrayTypeStyleCheck.java
is not i18ned.
When i18ned, "Array brackets at illegal position." can
be translated in French by "Les crochets du tableau ne
sont pas placés au bon endroit.".

-
com/puppycrawl/tools/checkstyle/checks/design/HideUtilityClassConstructorCheck.java
is not i18ned.
When i18ned, "Utility classes should not have a public
or default constructor." can be translated in French by
"Les classes utilitaires ne doivent pas avoir de
constructeur par défaut ou public.".

-
com/puppycrawl/tools/checkstyle/checks/coding/AvoidInlineConditionalsCheck.java
is i18ned but missing French translation.
The French translation corresponding to the key
"inline.conditional.avoid" is "L'utilisation de
l'opérateur conditionel est prohibée.".

-
com/puppycrawl/tools/checkstyle/checks/design/messages_fr.properties
has a slight error for the "throws.count" key
translation (the verb "est de" is missing).
This should be:
throws.count=Le nombre de clause \"throws\" est de
{0,number,integer}, alors que le maximum autorisé est
de {1,number,integer}.

Discussion

  • Logged In: YES
    user_id=746148

    I've added i18n support for these two check, but have not
    added French messages, since I'm not sure if my editor will
    work well
    with non-ascii symbols :(

    Could you please provide diffs for messages so I could just
    apply them (it;s better to attach diffs as files)

    Thanks in advance.

     
  • Logged In: YES
    user_id=1200314

    Here are the diffs (I put only the keys which message has to
    be updated or added).

    Feel free to ask me if you have any problem with that.

     
  • Logged In: YES
    user_id=1200314

    Oops, forgot to check teh checkbox :o)

     
  • French i18n diffs

     
    Attachments
  • Logged In: YES
    user_id=746148

    I've committed changes to CVS for 4.0.
    Thank you for contribution