From 7a07f16351069a980f66ebfeca10c589c11aa527 Mon Sep 17 00:00:00 2001
From: Felix Kehrer <felix.kehrer@gmail.com>
Date: Thu, 14 Jun 2018 21:03:30 +0200
Subject: Remove all logging from EmployeeServiceImpl #27033

---
 .../einsatzverwaltung/service/EmployeeServiceImpl.java         | 10 ----------
 1 file changed, 10 deletions(-)

(limited to 'src/main/java/at')

diff --git a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/EmployeeServiceImpl.java b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/EmployeeServiceImpl.java
index 81261ee..31b5acd 100644
--- a/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/EmployeeServiceImpl.java
+++ b/src/main/java/at/ac/tuwien/sepm/assignment/groupphase/einsatzverwaltung/service/EmployeeServiceImpl.java
@@ -8,15 +8,11 @@ import at.ac.tuwien.sepm.assignment.groupphase.exception.InvalidEmployeeExceptio
 import at.ac.tuwien.sepm.assignment.groupphase.exception.PersistenceException;
 import at.ac.tuwien.sepm.assignment.groupphase.exception.ServiceException;
 import java.util.Set;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import org.springframework.stereotype.Service;
 
 @Service
 public class EmployeeServiceImpl implements EmployeeService {
 
-    private static final Logger LOG = LoggerFactory.getLogger(EmployeeServiceImpl.class);
-
     private final EmployeeDAO employeePersistence;
 
     public EmployeeServiceImpl(EmployeeDAO employeePersistence) {
@@ -30,8 +26,6 @@ public class EmployeeServiceImpl implements EmployeeService {
         try {
             return employeePersistence.add(employee);
         } catch (PersistenceException e) {
-            // TODO: Do not practice "log and rethrow"! (feedback)
-            LOG.error("PersistenceException while adding Employee. Message: {}", e.getMessage());
             throw new ServiceException(e);
         }
     }
@@ -44,9 +38,6 @@ public class EmployeeServiceImpl implements EmployeeService {
             employeePersistence.update(employee);
             return employee;
         } catch (ElementNotFoundException | PersistenceException e) {
-            LOG.error(
-                    "ElementNotFoundException or PersistenceException while updating Employee. Message: {}",
-                    e.getMessage());
             throw new ServiceException(e);
         }
     }
@@ -57,7 +48,6 @@ public class EmployeeServiceImpl implements EmployeeService {
         try {
             return employeePersistence.list();
         } catch (PersistenceException e) {
-            LOG.error("PersistenceException while listing Employees. Message: {}", e.getMessage());
             throw new ServiceException(e);
         }
     }
-- 
cgit v1.2.3-70-g09d2