Any report with a handler and no conclusion is an active report,

therefore we don't need to check if the report is active
This commit is contained in:
Keir Nellyer 2016-10-04 16:20:11 +01:00
parent c462282632
commit afee339e16
1 changed files with 14 additions and 14 deletions

View File

@ -553,22 +553,22 @@ public class ReportManager
reportIds.stream().map(_reportRepository::getReport).collect(Collectors.toList())
).thenCompose(UtilFuture::sequence)
.thenApply(UtilCollections::unboxPresent)
.thenCompose(reports -> UtilFuture.filter(reports, this::isActiveReport)).thenApply(reports ->
{
Report report = null;
int size = reports.size();
.thenApply(reports ->
{
Report report = null;
int size = reports.size();
if (size == 1)
{
report = reports.get(0);
}
else if (size > 1)
{
throw new IllegalStateException("Account is handling multiple reports.");
}
if (size == 1)
{
report = reports.get(0);
}
else if (size > 1)
{
throw new IllegalStateException("Account is handling multiple reports.");
}
return Optional.ofNullable(report);
});
return Optional.ofNullable(report);
});
future.exceptionally(throwable -> Optional.empty());