Author: sletellier Date: 2010-09-29 15:06:14 +0200 (Wed, 29 Sep 2010) New Revision: 2097 Url: http://nuiton.org/repositories/revision/jaxx/2097 Log: Throw illegalArgumentException if node is not found Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/nav/treetable/NavTreeTableBridge.java Modified: trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/nav/treetable/NavTreeTableBridge.java =================================================================== --- trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/nav/treetable/NavTreeTableBridge.java 2010-09-29 12:27:08 UTC (rev 2096) +++ trunk/jaxx-runtime/src/main/java/jaxx/runtime/swing/nav/treetable/NavTreeTableBridge.java 2010-09-29 13:06:14 UTC (rev 2097) @@ -127,6 +127,9 @@ public void removeNodeFromParent(N node) { N parent = node.getParent(); int index = parent.getIndex(node); + if (index == -1) { + throw new IllegalArgumentException("Node " + node + " is not in tree"); + } parent.remove(node); model.getModelSupport().fireChildRemoved(getTreePath(parent), index, node); } @@ -222,11 +225,14 @@ N parent = node.getParent(); if (parent != null) { - int indice = parent.getIndex(node); + int index = parent.getIndex(node); + if (index == -1) { + throw new IllegalArgumentException("Node " + node + " is not in tree"); + } if (log.isDebugEnabled()) { - log.debug("Notify for node : " + node + ", for parent [" + parent + "] child " + indice + " inserted."); + log.debug("Notify for node : " + node + ", for parent [" + parent + "] child " + index + " inserted."); } - nodesWereInserted(parent, new int[]{indice}); + nodesWereInserted(parent, new int[]{index}); notifyChildNodesInserted(node); } }