[Lada-commits] [PATCH 6 of 6] Removed verbose logging

Wald Commits scm-commit at wald.intevation.org
Wed Aug 31 16:24:14 CEST 2016


# HG changeset patch
# User Raimund Renkert <raimund.renkert at intevation.de>
# Date 1472653444 -7200
# Branch schema-update
# Node ID 4e0a9c7a8f236bd0de50a0919889e3dd7b4b5bd3
# Parent  4d95cc7f0a43447a0c1d01c84b15cd48052010c4
Removed verbose logging.

diff -r 4d95cc7f0a43 -r 4e0a9c7a8f23 src/main/java/de/intevation/lada/rest/StatusService.java
--- a/src/main/java/de/intevation/lada/rest/StatusService.java	Wed Aug 31 15:52:15 2016 +0200
+++ b/src/main/java/de/intevation/lada/rest/StatusService.java	Wed Aug 31 16:24:04 2016 +0200
@@ -91,9 +91,6 @@
 @RequestScoped
 public class StatusService {
 
-    @Inject
-    private Logger logger = Logger.getLogger(StatusService.class);
-
     /**
      * The data repository granting read/write access.
      */
@@ -284,23 +281,12 @@
             // status_kombi
             // 1. The old 'status_wert' is 'rückfrage'
             //    User has 'funktion' 1 for the given mstId
-            logger.debug("old: " + oldKombi.getStatusStufe().getStufe());
-            logger.debug("new: " + newKombi.getStatusStufe().getStufe());
-            logger.debug("mstold: " + oldStatus.getMstId());
-            logger.debug("mstnew: " + status.getMstId());
-            logger.debug("user: ");
-            for (int i = 0; i < userInfo.getMessstellen().size(); i++) {
-                logger.debug(userInfo.getMessstellen().get(i));
-            }
             if (oldKombi.getStatusWert().getId() == 4) {
-                logger.debug("old status is rueckfrage");
                 if (userInfo.getFunktionenForMst(status.getMstId()).contains(1)) {
                     // Set the new status.
-                    logger.debug("user is allowed to set the status");
                     return setNewStatus(status, newKombi, messung, request);
                 }
                 else {
-                    logger.debug("not allowed");
                     // Not allowed.
                     return new Response(false, 699, null);
                 }
@@ -315,14 +301,12 @@
                 // a) user wants to reset the current status
                 //    'status wert' == 8
                 if (newKombi.getStatusWert().getId() == 8) {
-                    logger.debug("reset the status");
                     return authorization.filter(
                         request,
                         resetStatus(status, oldStatus, messung),
                         StatusProtokoll.class);
                 }
                 // b) update the status by the setting the new one.
-                logger.debug("set edited status");
                 return setNewStatus(status, newKombi, messung, request);
             }
             // 3. user wants to advance to the next 'status_stufe'
@@ -332,11 +316,9 @@
                 newKombi.getStatusStufe().getId() ==
                     oldKombi.getStatusStufe().getId() + 1) {
                 // Set the next status
-                logger.debug("set next status");
                 return setNewStatus(status, newKombi, messung, request);
             }
         }
-        logger.debug("something other....");
         return new Response(false, 699, null);
     }
 
@@ -502,8 +484,6 @@
             }
             StatusProtokoll copy = new StatusProtokoll();
             StatusProtokoll orig = proto.get(ndx);
-            logger.debug(orig.getStatusKombi());
-            logger.debug(orig.getMstId());
             copy.setDatum(new Timestamp(new Date().getTime()));
             copy.setMstId(orig.getMstId());
             copy.setMessungsId(orig.getMessungsId());


More information about the Lada-commits mailing list