[
http://jira.codehaus.org/browse/MRELEASE-417?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=167184#action_167184
]
Dennis Lundberg edited comment on MRELEASE-417 at 2/9/10 12:40 PM:
-------------------------------------------------------------------
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:
{code:java}
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;
}
{code}
was (Author: jens.muehlenhoff):
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}
> 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
> Components: prepare
> 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
