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

fixed error logging

Browse source
This commit is contained in:
anhefti 2022-11-14 15:23:44 +01:00
parent 0a99979e32
commit 42c0732159
2 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/main/java/ch/ethz/seb/sebserver/webservice/servicelayer/dao/AdditionalAttributesDAO.java
View file

@ -12,6 +12,9 @@ import java.util.Collection;
import java.util.Map;
import java.util.stream.Collectors;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import ch.ethz.seb.sebserver.gbl.api.EntityType;
import ch.ethz.seb.sebserver.gbl.model.EntityKey;
import ch.ethz.seb.sebserver.gbl.util.Result;
@ -23,6 +26,8 @@ import ch.ethz.seb.sebserver.webservice.datalayer.batis.model.AdditionalAttribut
* in a separated data-base table. */
public interface AdditionalAttributesDAO {
Logger log = LoggerFactory.getLogger(AdditionalAttributesDAO.class);
/** Use this to get all additional attribute records for a specific entity.
*
* @param type the entity type
@ -85,8 +90,9 @@ public interface AdditionalAttributesDAO {
return Result.tryCatch(() -> attributes.entrySet()
.stream()
.map(attr -> saveAdditionalAttribute(type, entityId, attr.getKey(), attr.getValue()))
.flatMap(Result::onErrorLogAndSkip)
.map(attr -> saveAdditionalAttribute(type, entityId, attr.getKey(), attr.getValue())
.onError(error -> log.warn("Failed to save additional attribute: ", error.getMessage())))
.flatMap(Result::skipOnError)
.collect(Collectors.toList()));
}

2
src/main/java/ch/ethz/seb/sebserver/webservice/servicelayer/dao/impl/AdditionalAttributesDAOImpl.java
View file

@ -95,7 +95,7 @@ public class AdditionalAttributesDAOImpl implements AdditionalAttributesDAO {
if (value == null) {
throw new IllegalArgumentException(
"value cannot be null. Use delete to delete an additional attribute");
"value cannot be null. Use delete to delete an additional attribute: " + String.valueOf(name));
}
if (log.isDebugEnabled()) {