diff options
author | Julien HENRY <julien.henry@sonarsource.com> | 2015-03-16 15:42:43 +0100 |
---|---|---|
committer | Julien HENRY <julien.henry@sonarsource.com> | 2015-03-16 15:55:22 +0100 |
commit | 07018b840638267e75af62f13c34ee67a691f988 (patch) | |
tree | 3cdc2d73e36110e00916acbc2c71a1456aef5a61 /sonar-batch/src/main | |
parent | 3ca885f38cd9fee24961846c15a4ef5fa200796e (diff) | |
download | sonarqube-07018b840638267e75af62f13c34ee67a691f988.tar.gz sonarqube-07018b840638267e75af62f13c34ee67a691f988.zip |
SONAR-6262 Fix NPE in link publisher on Views
Diffstat (limited to 'sonar-batch/src/main')
-rw-r--r-- | sonar-batch/src/main/java/org/sonar/batch/report/ComponentsPublisher.java | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/sonar-batch/src/main/java/org/sonar/batch/report/ComponentsPublisher.java b/sonar-batch/src/main/java/org/sonar/batch/report/ComponentsPublisher.java index 186f4368d6e..06f5998bb7d 100644 --- a/sonar-batch/src/main/java/org/sonar/batch/report/ComponentsPublisher.java +++ b/sonar-batch/src/main/java/org/sonar/batch/report/ComponentsPublisher.java @@ -91,7 +91,7 @@ public class ComponentsPublisher implements ReportPublisher { for (BatchResource child : batchResource.children()) { builder.addChildRefs(child.batchId()); } - if (ResourceUtils.isProject(r)) { + if (isRealProjectOrModule(r)) { ProjectDefinition def = getProjectDefinition(reactor, r.getKey()); ComponentLink.Builder linkBuilder = ComponentLink.newBuilder(); @@ -108,6 +108,11 @@ public class ComponentsPublisher implements ReportPublisher { } } + // Exclude views + private static boolean isRealProjectOrModule(Resource r) { + return ResourceUtils.isProject(r) && !ResourceUtils.isView(r) && !ResourceUtils.isSubview(r); + } + private ProjectDefinition getProjectDefinition(ProjectReactor reactor, String keyWithBranch) { for (ProjectDefinition p : reactor.getProjects()) { if (keyWithBranch.equals(p.getKeyWithBranch())) { |