Skip to content

Commit 1a863de

Browse files
committed
Use CompareType.AbstractCompareClause to accommodate InClause
1 parent fbb9058 commit 1a863de

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

SivStudies/src/org/labkey/sivstudies/etl/SubjectScopedSelect.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,7 @@ private void processBatch(List<String> subjects, Logger log, PipelineJob job)
150150
final SimpleFilter subjectFilter = new SimpleFilter(FieldKey.fromString(_settings.get(Settings.targetSubjectColumn.name())), subjects, CompareType.IN);
151151
if (_settings.get(Settings.targetAdditionalFilters.name()) != null)
152152
{
153-
List<CompareType.CompareClause> additionalFilters = parseAdditionalFilters(_settings.get(Settings.targetAdditionalFilters.name()));
153+
List<CompareType.AbstractCompareClause> additionalFilters = parseAdditionalFilters(_settings.get(Settings.targetAdditionalFilters.name()));
154154
additionalFilters.forEach(subjectFilter::addCondition);
155155
}
156156

@@ -260,7 +260,7 @@ else if (getMode() == MODE.UPDATE_ONLY)
260260
}
261261
}
262262

263-
private List<CompareType.CompareClause> parseAdditionalFilters(String rawVal)
263+
private List<CompareType.AbstractCompareClause> parseAdditionalFilters(String rawVal)
264264
{
265265
rawVal = StringUtils.trimToNull(rawVal);
266266
if (rawVal == null)
@@ -276,12 +276,12 @@ private List<CompareType.CompareClause> parseAdditionalFilters(String rawVal)
276276
}
277277

278278
return filter.getClauses().stream().map(fc -> {
279-
if (fc instanceof CompareType.CompareClause cc)
279+
if (fc instanceof CompareType.AbstractCompareClause cc)
280280
{
281281
return cc;
282282
}
283283

284-
throw new IllegalStateException("Expected all filters to be instance CompareType.CompareClause, found: " + fc.getClass());
284+
throw new IllegalStateException("Expected all filters to be instance CompareType.AbstractCompareClause, found: " + fc.getClass());
285285
}).toList();
286286
}
287287

@@ -378,8 +378,8 @@ private List<Map<String, Object>> getRowsToImport(List<String> subjects, Logger
378378
sr.addFilter(_settings.get(Settings.dataSourceSubjectColumn.name()), StringUtils.join(subjects, ";"), Filter.Operator.IN);
379379
if (_settings.get(Settings.dataSourceAdditionalFilters.name()) != null)
380380
{
381-
List<CompareType.CompareClause> additionalFilters = parseAdditionalFilters(_settings.get(Settings.dataSourceAdditionalFilters.name()));
382-
for (CompareType.CompareClause f : additionalFilters)
381+
List<CompareType.AbstractCompareClause> additionalFilters = parseAdditionalFilters(_settings.get(Settings.dataSourceAdditionalFilters.name()));
382+
for (CompareType.AbstractCompareClause f : additionalFilters)
383383
{
384384
Object value;
385385
if (f.getParamVals() == null)
@@ -471,7 +471,7 @@ else if (f.getParamVals().length == 1)
471471
final SimpleFilter filter = new SimpleFilter(FieldKey.fromString(_settings.get(Settings.dataSourceSubjectColumn.name())), subjects, CompareType.IN);
472472
if (_settings.get(Settings.dataSourceAdditionalFilters.name()) != null)
473473
{
474-
List<CompareType.CompareClause> additionalFilters = parseAdditionalFilters(_settings.get(Settings.dataSourceAdditionalFilters.name()));
474+
List<CompareType.AbstractCompareClause> additionalFilters = parseAdditionalFilters(_settings.get(Settings.dataSourceAdditionalFilters.name()));
475475
additionalFilters.forEach(filter::addCondition);
476476
}
477477

0 commit comments

Comments
 (0)