diff flys-client/src/main/java/de/intevation/flys/client/server/LoadArtifactServiceImpl.java @ 1367:ab8eb2f544f2

Replaced stdout and stderr logging with log4j loggers in server classes. flys-client/trunk@3069 c6561f87-3c4e-4783-a992-168aeb5c3f6f
author Ingo Weinzierl <ingo.weinzierl@intevation.de>
date Tue, 25 Oct 2011 12:31:15 +0000
parents a3c04eb326f3
children 0d3bbe4240b1
line wrap: on
line diff
--- a/flys-client/src/main/java/de/intevation/flys/client/server/LoadArtifactServiceImpl.java	Tue Oct 25 11:07:14 2011 +0000
+++ b/flys-client/src/main/java/de/intevation/flys/client/server/LoadArtifactServiceImpl.java	Tue Oct 25 12:31:15 2011 +0000
@@ -3,6 +3,8 @@
 import java.util.ArrayList;
 import java.util.HashMap;
 
+import org.apache.log4j.Logger;
+
 import de.intevation.flys.client.shared.exceptions.ServerException;
 import de.intevation.flys.client.shared.model.Artifact;
 import de.intevation.flys.client.shared.model.Collection;
@@ -20,6 +22,10 @@
 extends      ArtifactServiceImpl
 implements   LoadArtifactService
 {
+    private static final Logger logger =
+        Logger.getLogger(LoadArtifactServiceImpl.class);
+
+
     public static final String ERROR_LOAD_ARTIFACT = "error_load_artifact";
 
 
@@ -43,7 +49,7 @@
         String         locale
     )
     throws ServerException {
-        System.out.println(
+        logger.info(
             "LoadArtifactServiceImpl.load: " + recom.getMasterArtifact());
 
         // 1) Clone the Artifact specified in >>recom<<
@@ -51,12 +57,12 @@
             url, locale, factory, recom);
 
         if (clone != null) {
-            System.out.println("Successfully create Artifact Clone. Add now!");
+            logger.debug("Successfully create Artifact Clone. Add now!");
             Collection c = CollectionHelper.addArtifact(
                 parent, clone, url, locale);
 
             if (c != null) {
-                System.out.println("Successfully added Clone to Collection.");
+                logger.debug("Successfully added Clone to Collection.");
 
                 return clone;
             }
@@ -89,7 +95,7 @@
         String           locale
     )
     throws ServerException {
-        System.out.println("LoadArtifactServiceImpl.loadMany");
+        logger.debug("LoadArtifactServiceImpl.loadMany");
 
         ArrayList<Artifact> artifacts = new ArrayList<Artifact>();
         HashMap<Recommendation, Artifact> cloneMap =
@@ -103,7 +109,7 @@
             Artifact prevClone = cloneMap.get(recom);
             if (prevClone != null) {
                 // Already cloned a recommendation like this.
-                System.out.println("LoadArtifactServiceImpl: Avoid reclones, "
+                logger.debug("LoadArtifactServiceImpl: Avoid reclones, "
                     + "clone already exists.");
                 artifacts.add(prevClone);
             }
@@ -113,13 +119,13 @@
                     ? factory
                     : recom.getFactory();
 
-                System.out.println("One will be cloned with : " + realFactory);
+                logger.debug("One will be cloned with : " + realFactory);
 
                 Artifact clone = ArtifactHelper.createArtifact(
                     url, locale, realFactory, recom);
         
                 if (clone != null) {
-                    System.out.println("LoadArtifactServiceImple: Successfully "
+                    logger.debug("LoadArtifactServiceImple: Successfully "
                         + "loaded Artifact Clone.");
                     Collection c = CollectionHelper.addArtifact(
                         parent, clone, url, locale);

http://dive4elements.wald.intevation.org