From c311147fa8c0c0cd827ed8db1d7b0115067472ad Mon Sep 17 00:00:00 2001 From: Godin Date: Wed, 20 Oct 2010 11:46:26 +0000 Subject: [PATCH] SONAR-1808: Fix possible NPE --- .../sonar/plugins/cobertura/api/AbstractCoberturaParser.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/plugins/sonar-cobertura-plugin/src/main/java/org/sonar/plugins/cobertura/api/AbstractCoberturaParser.java b/plugins/sonar-cobertura-plugin/src/main/java/org/sonar/plugins/cobertura/api/AbstractCoberturaParser.java index 109bb353da1..4a4acb20b9c 100644 --- a/plugins/sonar-cobertura-plugin/src/main/java/org/sonar/plugins/cobertura/api/AbstractCoberturaParser.java +++ b/plugins/sonar-cobertura-plugin/src/main/java/org/sonar/plugins/cobertura/api/AbstractCoberturaParser.java @@ -86,8 +86,9 @@ public abstract class AbstractCoberturaParser { String lineId = line.getAttrValue("number"); data.addLine(lineId, (int) parseNumber(line.getAttrValue("hits"), ENGLISH)); + String isBranch = line.getAttrValue("branch"); String text = line.getAttrValue("condition-coverage"); - if (StringUtils.isNotBlank(text)) { + if (StringUtils.equals(isBranch, "true") && StringUtils.isNotBlank(text)) { String[] conditions = StringUtils.split(StringUtils.substringBetween(text, "(", ")"), "/"); data.addConditionLine(lineId, Integer.parseInt(conditions[0]), Integer.parseInt(conditions[1]), StringUtils.substringBefore(text, " ")); } -- 2.39.5