diff options
author | Julien Lancelot <julien.lancelot@sonarsource.com> | 2014-01-30 15:08:13 +0100 |
---|---|---|
committer | Julien Lancelot <julien.lancelot@sonarsource.com> | 2014-01-30 15:08:23 +0100 |
commit | 51c4b4c460628f1ed4c3661aa770b8635f531b14 (patch) | |
tree | b8b7d118a014c700b495947f4b7b822627039dc8 /sonar-core | |
parent | b88008865c3fc1ffd0ec950408b42c901ec43e29 (diff) | |
download | sonarqube-51c4b4c460628f1ed4c3661aa770b8635f531b14.tar.gz sonarqube-51c4b4c460628f1ed4c3661aa770b8635f531b14.zip |
SONAR-4818 Setting a percentage in the metric value of a measure filter breaks the search
Diffstat (limited to 'sonar-core')
-rw-r--r-- | sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterEngine.java | 3 | ||||
-rw-r--r-- | sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterResult.java | 2 |
2 files changed, 4 insertions, 1 deletions
diff --git a/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterEngine.java b/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterEngine.java index 540e450d38b..577d1af6f20 100644 --- a/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterEngine.java +++ b/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterEngine.java @@ -60,6 +60,9 @@ public class MeasureFilterEngine implements ServerComponent { List<MeasureFilterRow> rows = executor.execute(filter, context); result.setRows(rows); + } catch (NumberFormatException e) { + result.setError(MeasureFilterResult.Error.VALUE_SHOULD_BE_A_NUMBER); + LOG.error("Value selected for the metric should be a number: " + context); } catch (Exception e) { result.setError(MeasureFilterResult.Error.UNKNOWN); LOG.error("Fail to execute measure filter: " + context, e); diff --git a/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterResult.java b/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterResult.java index 63f42c790f9..100c6d9e8a3 100644 --- a/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterResult.java +++ b/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterResult.java @@ -26,7 +26,7 @@ import java.util.List; public class MeasureFilterResult { public static enum Error { - UNKNOWN + UNKNOWN, VALUE_SHOULD_BE_A_NUMBER } private List<MeasureFilterRow> rows = null; |