[Commented] (MENFORCER-301) banDuplicatePomDependencyVersions does not check managementDependencies

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[Commented] (MENFORCER-301) banDuplicatePomDependencyVersions does not check managementDependencies

JIRA jira@apache.org

    [ https://issues.apache.org/jira/browse/MENFORCER-301?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16439161#comment-16439161 ]

ASF GitHub Bot commented on MENFORCER-301:
------------------------------------------

kudrevatykh opened a new pull request #33: MENFORCER-301 check dependencyManagement
URL: https://github.com/apache/maven-enforcer/pull/33
 
 
   really check dependencyManagement section

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[hidden email]


> banDuplicatePomDependencyVersions does not check managementDependencies
> -----------------------------------------------------------------------
>
>                 Key: MENFORCER-301
>                 URL: https://issues.apache.org/jira/browse/MENFORCER-301
>             Project: Maven Enforcer Plugin
>          Issue Type: Bug
>          Components: Standard Rules
>    Affects Versions: 3.0.0-M1
>            Reporter: Alexander Kudrevatykh
>            Priority: Major
>
> MENFORCER-152 added rule for ban duplicate depndencies, but dependencyManagement section does not checked correctly
> codeĀ  {code:java} if ( model.getDependencyManagement() != null )
> {
>     List<Dependency> managementDependencies = model.getDependencies();
>     Map<String, Integer> duplicateManagementDependencies = validateDependencies( managementDependencies );{code} should be written as
> {code}
> if ( model.getDependencyManagement() != null )
> {
>     List<Dependency> managementDependencies = model.getDependencyManagement().getDependencies();
>     Map<String, Integer> duplicateManagementDependencies = validateDependencies( managementDependencies );{code}
> and same fix should be applied to {noformat}profiles{noformat} checking code



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)