aboutsummaryrefslogtreecommitdiffstats
path: root/sonar-core
diff options
context:
space:
mode:
authorJulien Lancelot <julien.lancelot@sonarsource.com>2014-02-03 09:13:49 +0100
committerJulien Lancelot <julien.lancelot@sonarsource.com>2014-02-03 09:13:49 +0100
commit5f8377f385a665ffe936d7bb16e96aa223bd42d3 (patch)
tree14d72e08a4f1a16eee7e1de2dcdad079a91bfc33 /sonar-core
parentc9b42bec3ba7ab534e3a0b1917c03a3795ef56a4 (diff)
downloadsonarqube-5f8377f385a665ffe936d7bb16e96aa223bd42d3.tar.gz
sonarqube-5f8377f385a665ffe936d7bb16e96aa223bd42d3.zip
SONAR-4921 Remove management of baseId in MeasureFilter
Diffstat (limited to 'sonar-core')
-rw-r--r--sonar-core/src/main/java/org/sonar/core/measure/MeasureFilter.java9
-rw-r--r--sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterExecutor.java3
-rw-r--r--sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterFactory.java4
-rw-r--r--sonar-core/src/test/java/org/sonar/core/measure/MeasureFilterTest.java1
4 files changed, 0 insertions, 17 deletions
diff --git a/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilter.java b/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilter.java
index 0e84c50e090..18fe67ceccb 100644
--- a/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilter.java
+++ b/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilter.java
@@ -65,15 +65,6 @@ public class MeasureFilter {
return this;
}
- public Long getBaseResourceId() {
- return baseResourceId;
- }
-
- public MeasureFilter setBaseResourceId(Long i) {
- this.baseResourceId = i;
- return this;
- }
-
public MeasureFilter setOnBaseResourceChildren(boolean b) {
this.onBaseResourceChildren = b;
return this;
diff --git a/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterExecutor.java b/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterExecutor.java
index a80c621fcd5..fc1efc1e5f9 100644
--- a/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterExecutor.java
+++ b/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterExecutor.java
@@ -76,14 +76,11 @@ public class MeasureFilterExecutor implements ServerComponent {
private void prepareContext(MeasureFilterContext context, MeasureFilter filter, SqlSession session) {
if (filter.getBaseResourceKey() != null) {
context.setBaseSnapshot(resourceDao.getLastSnapshot(filter.getBaseResourceKey(), session));
- } else if (filter.getBaseResourceId() != null) {
- context.setBaseSnapshot(resourceDao.getLastSnapshotByResourceId(filter.getBaseResourceId(), session));
}
}
static boolean isValid(MeasureFilter filter, MeasureFilterContext context) {
boolean valid = Strings.isNullOrEmpty(filter.getBaseResourceKey()) || context.getBaseSnapshot()!=null;
- valid &= filter.getBaseResourceId()==null || context.getBaseSnapshot()!=null;
valid &= !(filter.isOnBaseResourceChildren() && context.getBaseSnapshot() == null);
valid &= !(filter.isOnFavourites() && context.getUserId() == null);
valid &= validateMeasureConditions(filter);
diff --git a/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterFactory.java b/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterFactory.java
index b1a10230726..47021ecf9bd 100644
--- a/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterFactory.java
+++ b/sonar-core/src/main/java/org/sonar/core/measure/MeasureFilterFactory.java
@@ -46,10 +46,6 @@ public class MeasureFilterFactory implements ServerComponent {
MeasureFilter filter = new MeasureFilter();
filter.setBaseResourceKey((String) properties.get("base"));
- String baseId = "baseId";
- if (properties.containsKey(baseId)) {
- filter.setBaseResourceId(Long.valueOf((String) properties.get(baseId)));
- }
filter.setResourceScopes(toList(properties.get("scopes")));
filter.setResourceQualifiers(toList(properties.get("qualifiers")));
filter.setResourceLanguages(toList(properties.get("languages")));
diff --git a/sonar-core/src/test/java/org/sonar/core/measure/MeasureFilterTest.java b/sonar-core/src/test/java/org/sonar/core/measure/MeasureFilterTest.java
index 9ed5b301a43..a80ef5ad755 100644
--- a/sonar-core/src/test/java/org/sonar/core/measure/MeasureFilterTest.java
+++ b/sonar-core/src/test/java/org/sonar/core/measure/MeasureFilterTest.java
@@ -61,6 +61,5 @@ public class MeasureFilterTest {
public void filter_is_not_empty_if_at_least_condition_on_root_resource() {
assertThat(new MeasureFilter().isEmpty()).isTrue();
assertThat(new MeasureFilter().setBaseResourceKey("foo").isEmpty()).isFalse();
- assertThat(new MeasureFilter().setBaseResourceId(123L).isEmpty()).isFalse();
}
}