X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=eclipse%2Fruntime%2Forg.argeo.slc.client.core%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fslc%2Fclient%2Fcontentprovider%2FResultDetailContentProvider.java;h=f04ba5672c04a95a4697b1456cf4f46bd24ab98c;hb=9c11f2907a62ba5eeb60aa75f868e0ae7197f7d5;hp=36c96326647b246a9b91dca78377ce814e6562d4;hpb=a857d591b8658b2f94ec9398b94e9447f130eb9d;p=gpl%2Fargeo-slc.git 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 36c963266..f04ba5672 100644 --- 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 @@ -5,7 +5,6 @@ import java.util.SortedMap; import org.apache.commons.logging.Log; import org.apache.commons.logging.LogFactory; -import org.argeo.eclipse.ui.TreeObject; import org.argeo.eclipse.ui.TreeParent; import org.argeo.slc.core.structure.tree.TreeSPath; import org.argeo.slc.core.test.tree.PartSubList; @@ -25,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; } @@ -41,8 +42,8 @@ public class ResultDetailContentProvider implements ITreeContentProvider { } public Object getParent(Object node) { - if (node instanceof TreeObject) { - return ((TreeObject) node).getParent(); + if (node instanceof TreeParent) { + return ((TreeParent) node).getParent(); } return null; } @@ -119,10 +120,10 @@ public class ResultDetailContentProvider implements ITreeContentProvider { if (node instanceof ResultTreeParent) { ResultTreeParent rtp = (ResultTreeParent) node; // we dig the tree recursivly - for (TreeObject to : rtp.getChildren()) + for (Object to : rtp.getChildren()) setIsPassed((StatusAware) to); // we set is passed - for (TreeObject to : rtp.getChildren()) { + for (Object to : rtp.getChildren()) { if (!((StatusAware) to).isPassed()) { rtp.isPassed = false; return; @@ -158,7 +159,7 @@ public class ResultDetailContentProvider implements ITreeContentProvider { } } - public class ResultTreeObject extends TreeObject implements StatusAware { + public class ResultTreeObject extends TreeParent implements StatusAware { public ResultTreeObject(String name) { super(name); @@ -184,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; }