From 93d09d05cf7b0b32e3e16de3b2c2fa2266cf9abd Mon Sep 17 00:00:00 2001
From: Felix Kehrer <felix.kehrer@gmail.com>
Date: Fri, 25 May 2018 22:33:34 +0200
Subject: added logging before throwing ServiceException #27033

---
 .../groupphase/einsatzverwaltung/service/VehicleServiceImpl.java    | 6 ++++++
 1 file changed, 6 insertions(+)

(limited to 'src')

diff --git a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/VehicleServiceImpl.java b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/VehicleServiceImpl.java
index 4010df3..026bca6 100644
--- a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/VehicleServiceImpl.java
+++ b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/VehicleServiceImpl.java
@@ -38,6 +38,7 @@ public class VehicleServiceImpl implements VehicleService {
         try {
             vehiclePersistence.add(vehicle);
         } catch (PersistenceException e) {
+            LOG.error("PersistenceException while adding Vehicle. Message : {}", e.getMessage());
             throw new ServiceException(e);
         }
         return 0;
@@ -48,8 +49,11 @@ public class VehicleServiceImpl implements VehicleService {
         try {
             vehiclePersistence.update(vehicle);
         } catch (ElementNotFoundException e) {
+            LOG.error(
+                    "ElementNotFoundException while updating Vehicle. Message: {}", e.getMessage());
             throw new ServiceException("Element not found");
         } catch (PersistenceException e) {
+            LOG.error("PersistenceException while updating Vehicle. Message: {}", e.getMessage());
             throw new ServiceException(e);
         }
         return vehicle;
@@ -106,6 +110,7 @@ public class VehicleServiceImpl implements VehicleService {
     @Override
     public Set<Vehicle> list(EnumSet<Status> statuses) throws ServiceException {
         if (statuses == null) {
+            LOG.error("Statuses may not be null");
             throw new ServiceException("statuses may not be null");
         }
 
@@ -114,6 +119,7 @@ public class VehicleServiceImpl implements VehicleService {
         try {
             vehicles = vehiclePersistence.list();
         } catch (PersistenceException e) {
+            LOG.error("PersistenceException while listing Vehicles. Message: {}", e.getMessage());
             throw new ServiceException(e);
         }
 
-- 
cgit v1.2.3-70-g09d2