You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "SebbASF (JIRA)" <ji...@codehaus.org> on 2011/02/11 22:10:23 UTC
[jira] Created: (MPIR-221) Fix for MPIR-171 generates unnecessary
and confusing warnings for non-integral time-zones
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
[jira] Closed: (MPIR-221) Fix for MPIR-171 generates unnecessary
and confusing warnings for non-integral time-zones
Posted by "Herve Boutemy (JIRA)" <ji...@codehaus.org>.
[ http://jira.codehaus.org/browse/MPIR-221?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Herve Boutemy closed MPIR-221.
------------------------------
Resolution: Fixed
Fix Version/s: 2.4
Assignee: Herve Boutemy
improvement done in [r1098769|http://svn.apache.org/viewvc?rev=1098769&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
> Assignee: Herve Boutemy
> Fix For: 2.4
>
>
> 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
[jira] Commented: (MPIR-221) Fix for MPIR-171 generates unnecessary
and confusing warnings for non-integral time-zones
Posted by "Herve Boutemy (JIRA)" <ji...@codehaus.org>.
[ 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