Clean and add some comments to the content providers

[gpl/argeo-slc.git] / eclipse / runtime / org.argeo.slc.client.core / src / main / java / org / argeo / slc / client / contentprovider / ResultDetailContentProvider.java

diff --git a/eclipse/runtime/org.argeo.slc.client.core/src/main/java/org/argeo/slc/client/contentprovider/ResultDetailContentProvider.java b/eclipse/runtime/org.argeo.slc.client.core/src/main/java/org/argeo/slc/client/contentprovider/ResultDetailContentProvider.java
index 10cf6a619a591eae55f18c39907d10db105d9daf..f04ba5672c04a95a4697b1456cf4f46bd24ab98c 100644 (file)
--- a/eclipse/runtime/org.argeo.slc.client.core/src/main/java/org/argeo/slc/client/contentprovider/ResultDetailContentProvider.java
+++ b/eclipse/runtime/org.argeo.slc.client.core/src/main/java/org/argeo/slc/client/contentprovider/ResultDetailContentProvider.java
@@ -24,7 +24,9 @@ public class ResultDetailContentProvider implements ITreeContentProvider {
 
        public Object[] getChildren(Object parent) {
                if (parent instanceof TreeTestResult) {
-                       log.error("We should not reach this point.");
+                       if (log.isErrorEnabled())
+                               log.error("Trying to get children for parent object of type TreeTestResult. "
+                                               + "We should not reach this point.");
                        return null;
                }
 
@@ -183,6 +185,7 @@ public class ResultDetailContentProvider implements ITreeContentProvider {
 
                public ResultPartNode(String label, Integer status, String message) {
                        super(label);
+                       log.debug("Creating new leaf with label : " + label);
                        handleStatus(status);
                        this.message = message;
                }