zervo/seb-server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fixed notification confirm from SEB client

Browse source
This commit is contained in:
anhefti 2021-09-16 11:33:25 +02:00
parent c126056959
commit 0546b97b0a
1 changed files with 15 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
src/main/java/ch/ethz/seb/sebserver/webservice/servicelayer/dao/impl/ClientEventDAOImpl.java
View file

@ -210,17 +210,26 @@ public class ClientEventDAOImpl implements ClientEventDAO {
.selectByExample()
.where(ClientEventRecordDynamicSqlSupport.clientConnectionId, isEqualTo(clientConnectionId))
.and(ClientEventRecordDynamicSqlSupport.type, isEqualTo(EventType.NOTIFICATION.id))
.and(
ClientEventRecordDynamicSqlSupport.numericValue,
isEqualTo(new BigDecimal(notificationValueId)))
.build()
.execute();
if (records.size() != 1) {
log.warn("Expected one notification event log but found: ", records.size());
if (log.isDebugEnabled()) {
log.debug("Found notification for clientConnectionId: {} notification: {}",
clientConnectionId,
records);
}
return records.get(0);
return records.stream()
.filter(rec -> {
final BigDecimal numericValue = rec.getNumericValue();
if (numericValue == null) {
return false;
}
return numericValue.longValue() == notificationValueId;
})
.findFirst()
.orElseThrow(() -> new IllegalStateException(
"Failed to find pending notification event for confirm:" + notificationValueId));
})
.flatMap(ClientEventDAOImpl::toClientNotificationModel);