This is an automated email from the git hooks/post-receive script. New commit to branch feature/7591_Correction_migration_entites_d_un_parent_a_un_autre in repository observe. See https://gitlab.nuiton.org/codelutin/observe.git commit e956bf51040420e5b8359b5b0c045cdd4d4d6a81 Author: Samuel Maisonneuve <maisonneuve@codelutin.com> Date: Thu Jul 28 15:44:51 2016 +0200 Fix loop and rename cleanAndPopulateMethod to clearAndPopulate refs #7591 --- .../ird/observe/ui/actions/shared/MoveActivitySeinesUIAction.java | 2 +- .../src/main/java/fr/ird/observe/ui/tree/ObserveTreeHelper.java | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/observe-application-swing/src/main/java/fr/ird/observe/ui/actions/shared/MoveActivitySeinesUIAction.java b/observe-application-swing/src/main/java/fr/ird/observe/ui/actions/shared/MoveActivitySeinesUIAction.java index 260038f..d3024c8 100644 --- a/observe-application-swing/src/main/java/fr/ird/observe/ui/actions/shared/MoveActivitySeinesUIAction.java +++ b/observe-application-swing/src/main/java/fr/ird/observe/ui/actions/shared/MoveActivitySeinesUIAction.java @@ -209,7 +209,7 @@ public class MoveActivitySeinesUIAction extends AbstractUIAction { treeHelper.selectNode(newActivitiesNode); // 2. Let's fully reload newActivitiesNode - treeHelper.cleanAndPopulateNode(newActivitiesNode); + treeHelper.clearAndPopulateNode(newActivitiesNode); // 3. We ensure the path until the newActivitiesNode is expanded // so that the user can see the activities he moved diff --git a/observe-application-swing/src/main/java/fr/ird/observe/ui/tree/ObserveTreeHelper.java b/observe-application-swing/src/main/java/fr/ird/observe/ui/tree/ObserveTreeHelper.java index cbe447f..9622487 100644 --- a/observe-application-swing/src/main/java/fr/ird/observe/ui/tree/ObserveTreeHelper.java +++ b/observe-application-swing/src/main/java/fr/ird/observe/ui/tree/ObserveTreeHelper.java @@ -581,10 +581,10 @@ public class ObserveTreeHelper extends NavTreeHelper<ObserveNode> { node.populateNode(getBridge(), getDataProvider(), true); } - public void cleanAndPopulateNode(ObserveNode node) { + public void clearAndPopulateNode(ObserveNode node) { // 1. We delete all children of the node because populating only *add* child, it never refreshes totally - for (int i=0, l = node.getChildCount(); i < l; i++) { - removeNode(node.getChildAt(i)); + while (node.getChildCount() > 0) { + removeNode(node.getFirstChild()); } // 2. We re-generate node's children -- To stop receiving notification emails like this one, please contact codelutin.com SCM administrator <admin+scm@codelutin.com>.