You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "Jens Mühlenhoff (JIRA)" <ji...@codehaus.org> on 2009/02/26 18:05:19 UTC
[jira] Issue Comment Edited: (MRELEASE-417) generated
ReleaseDescriptor contains buggy method: getDependencyDevelopmentVersion
and getDependencyReleaseVersion
[ http://jira.codehaus.org/browse/MRELEASE-417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=167184#action_167184 ]
Jens Mühlenhoff edited comment on MRELEASE-417 at 2/26/09 11:03 AM:
--------------------------------------------------------------------
Sorry, I cant edit my bug, it belongs to the prepare component. VErsion of maven is 2.0.10.
This could be a fix, but I don't know how to tell it maven to use it during code generation:
{{noformat}
public String getDependencyDevelopmentVersion( String artifactConflictId )
{
if( resolvedSnapshotDependencies != null ) {
java.util.Map versionMap = ( java.util.Map ) resolvedSnapshotDependencies.get( artifactConflictId );
if( versionMap != null ) {
return ( String ) versionMap.get( DEVELOPMENT_KEY );
}
}
return null;
}
public String getDependencyReleaseVersion( String artifactConflictId )
{
if ( ( resolvedSnapshotDependencies != null ) {
java.util.Map versionMap = ( java.util.Map ) resolvedSnapshotDependencies.get( artifactConflictId );
if( versionMap != null ) {
return ( String ) versionMap.get( RELEASE_KEY );
}
}
return null;
}
{{noformat}
was (Author: jens.muehlenhoff):
This could be a fix, but I don't know how to tell it maven to use it during code generation:
public String getDependencyDevelopmentVersion( String artifactConflictId )
{
if( resolvedSnapshotDependencies != null ) {
java.util.Map versionMap = ( java.util.Map ) resolvedSnapshotDependencies.get( artifactConflictId );
if( versionMap != null ) {
return ( String ) versionMap.get( DEVELOPMENT_KEY );
}
}
return null;
}
public String getDependencyReleaseVersion( String artifactConflictId )
{
if ( ( resolvedSnapshotDependencies != null ) {
java.util.Map versionMap = ( java.util.Map ) resolvedSnapshotDependencies.get( artifactConflictId );
if( versionMap != null ) {
return ( String ) versionMap.get( RELEASE_KEY );
}
}
return null;
}
> generated ReleaseDescriptor contains buggy method: getDependencyDevelopmentVersion and getDependencyReleaseVersion
> ------------------------------------------------------------------------------------------------------------------
>
> Key: MRELEASE-417
> URL: http://jira.codehaus.org/browse/MRELEASE-417
> Project: Maven 2.x Release Plugin
> Issue Type: Bug
> Reporter: Jens Mühlenhoff
>
> The following two methods will always return null if the attribute resolvedSnapshotDependencies is not null.
> Second you can avoid extra lookups by doing a get and check if the value is null (without calling containsKey before).
> /**
> * Retrieve the release version for the resolved snapshot dependency.
> *
> */
> public String getDependencyDevelopmentVersion( String artifactConflictId )
> {
> if ( ( resolvedSnapshotDependencies == null ) && ( resolvedSnapshotDependencies.containsKey( artifactConflictId ) ) )
> {
> java.util.Map versionMap = ( java.util.Map ) resolvedSnapshotDependencies.get( artifactConflictId );
> return ( String ) versionMap.get( DEVELOPMENT_KEY );
> }
> return null;
> }
> /**
> * Retrieve the release version for the resolved snapshot dependency.
> *
> */
> public String getDependencyReleaseVersion( String artifactConflictId )
> {
> if ( ( resolvedSnapshotDependencies == null ) && ( resolvedSnapshotDependencies.containsKey( artifactConflictId ) ) )
> {
> java.util.Map versionMap = ( java.util.Map ) resolvedSnapshotDependencies.get( artifactConflictId );
> return ( String ) versionMap.get( RELEASE_KEY );
> }
> return null;
> }
--
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