[ 
http://jira.codehaus.org/browse/MPIR-221?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=265485#action_265485
 ] 

Herve Boutemy commented on MPIR-221:
------------------------------------

enhanced message in [r1098766|http://svn.apache.org/viewvc?rev=1098766&view=rev]

> Fix for MPIR-171 generates unnecessary and confusing warnings for 
> non-integral time-zones
> -----------------------------------------------------------------------------------------
>
>                 Key: MPIR-221
>                 URL: http://jira.codehaus.org/browse/MPIR-221
>             Project: Maven 2.x Project Info Reports Plugin
>          Issue Type: Bug
>    Affects Versions: 2.3.1
>            Reporter: SebbASF
>
> The change for MPIR-171 added a check to see if the timezone is numeric. If 
> not, it reports a warning. 
> However, the check fails to allow for non-integral timezones - e.g. 5.5 - and 
> the warning message is incorrect, as it does not show the correct information.
> For example, the following setting:
>     <developer>
>       <timezone>+5.5</timezone>
>     </developer>
> results in the warning:
> [WARNING] The time zone '-2147483648' for the developer 'Name' is not a 
> recognised time zone, use a number in the range -12 to +14 instead of.
> The message should read something like:
> [WARNING] The time zone '+5.5' for the developer 'Name' is not a recognised 
> time zone, use a number in the range -12 to +14 or a valid long TimeZone id.
> except that it should not be generated for that particular timezone, as +5.5 
> is as valid as +5 (which does not generate a warning).
> To summarise:
> 1) the warning message is not helpful, as it does not show the correct 
> timezone.
> 2) the warning should not be generated for fractional timezones.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to