From 3651f277d91ab55103eb87e55a9e0fabc7e436ff Mon Sep 17 00:00:00 2001
From: tpylak <tpylak>
Date: Mon, 6 Sep 2010 11:31:44 +0000
Subject: [PATCH] SE-298 minor dynamix: logging

SVN: 17721
---
 .../cisd/openbis/dss/etl/HCSImageCheckList.java    | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/screening/source/java/ch/systemsx/cisd/openbis/dss/etl/HCSImageCheckList.java b/screening/source/java/ch/systemsx/cisd/openbis/dss/etl/HCSImageCheckList.java
index e03a7b1337d..1808f7d316e 100644
--- a/screening/source/java/ch/systemsx/cisd/openbis/dss/etl/HCSImageCheckList.java
+++ b/screening/source/java/ch/systemsx/cisd/openbis/dss/etl/HCSImageCheckList.java
@@ -21,7 +21,11 @@ import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 
+import org.apache.log4j.Logger;
+
 import ch.systemsx.cisd.bds.hcs.Geometry;
+import ch.systemsx.cisd.common.logging.LogCategory;
+import ch.systemsx.cisd.common.logging.LogFactory;
 import ch.systemsx.cisd.common.utilities.AbstractHashable;
 import ch.systemsx.cisd.openbis.plugin.screening.shared.dto.PlateDimension;
 
@@ -36,6 +40,8 @@ import ch.systemsx.cisd.openbis.plugin.screening.shared.dto.PlateDimension;
  */
 public final class HCSImageCheckList
 {
+    private static final Logger operationLog = LogFactory.getLogger(LogCategory.OPERATION,
+            HCSImageCheckList.class);
 
     private final Map<FullLocation, Check> imageMap;
 
@@ -79,7 +85,8 @@ public final class HCSImageCheckList
     public final void checkOff(AcquiredPlateImage image)
     {
         assert image != null : "Unspecified image.";
-        final Check check = imageMap.get(createLocation(image));
+        FullLocation location = createLocation(image);
+        final Check check = imageMap.get(location);
         if (check == null)
         {
             throw new IllegalArgumentException("Invalid channel/well/tile: " + image);
@@ -89,6 +96,11 @@ public final class HCSImageCheckList
         {
             throw new IllegalArgumentException("Image already handled: " + image);
         }
+        if (operationLog.isDebugEnabled())
+        {
+            operationLog.debug("Checking location " + location
+                    + (timepointOrNull == null ? "" : " timepoint " + timepointOrNull));
+        }
         check.checkOff(timepointOrNull);
     }
 
-- 
GitLab