changeset 8325:02782ed6c8de

Reduce misspelled, superfluous logging.
author "Tom Gottfried <tom@intevation.de>"
date Thu, 25 Sep 2014 12:21:47 +0200 (2014-09-25)
parents d6c03537970e
children f0b78f7498ae
files artifacts/src/main/java/org/dive4elements/river/collections/AttributeWriter.java
diffstat 1 files changed, 3 insertions(+), 14 deletions(-) [+]
line wrap: on
line diff
--- a/artifacts/src/main/java/org/dive4elements/river/collections/AttributeWriter.java	Thu Sep 25 11:00:31 2014 +0200
+++ b/artifacts/src/main/java/org/dive4elements/river/collections/AttributeWriter.java	Thu Sep 25 12:21:47 2014 +0200
@@ -142,13 +142,6 @@
     ) {
         List<String> compatFacets = this.compatibilities.get(outputName);
 
-        if (log.isDebugEnabled() && compatFacets != null) {
-            log.debug("Compabitle Facets:");
-            for (String compatible: compatFacets) {
-                log.debug(   "- " + compatible);
-            }
-        }
-
         try {
             writeFacets(outputName, newOutFacets, oldOutFacets, compatFacets);
         }
@@ -191,16 +184,12 @@
         for (Facet fac: newFacets) {
             ManagedFacet facet = (ManagedFacet) fac;
 
-            if (debug) {
-                log.debug("Try to add Facet: " + facet.getName());
-            }
-
             String bondage = facet.getBoundToOut();
             if (bondage != null && bondage.equals(outputName)) {
                 log.debug("Adding bound facet regardless of compatibility: " +
                     facet.getName());
             } else if (!compatibleFacets.contains(facet.getName())) {
-                log.debug("Have incompatible facet, skip: " + facet.getName());
+                log.debug("Skip incompatible facet " + facet.getName());
                 continue;
             } else if (facet.getBoundToOut() != null &&
                     !facet.getBoundToOut().equals(outputName)) {
@@ -208,8 +197,8 @@
                         " because it is bound to: " + facet.getBoundToOut());
                 continue;
             } else {
-                log.debug("Facet is bound to: " + facet.getBoundToOut());
-                log.debug("Have compatible facet: " + facet.getName());
+                log.debug("Compatible facet " + facet.getName() +
+                    " is bound to: " + facet.getBoundToOut());
             }
 
             ManagedFacet picked = pickFacet(facet, oldFacets);

http://dive4elements.wald.intevation.org