From a6978d55a84c71f82745d138a4aca651f0140c77 Mon Sep 17 00:00:00 2001
From: Susana Montes Pedreira <smontes@wirelessgalicia.com>
Date: Tue, 22 Feb 2011 13:21:59 +0100
Subject: [PATCH] [Bug #847] Fix bug

check if the position is inside range before using it.

FEA : ItEr71S04BugFixing
---
 .../navalplanner/web/tree/TreeController.java    | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/navalplanner-webapp/src/main/java/org/navalplanner/web/tree/TreeController.java b/navalplanner-webapp/src/main/java/org/navalplanner/web/tree/TreeController.java
index 4aa7e5415..20d72e013 100644
--- a/navalplanner-webapp/src/main/java/org/navalplanner/web/tree/TreeController.java
+++ b/navalplanner-webapp/src/main/java/org/navalplanner/web/tree/TreeController.java
@@ -378,7 +378,9 @@ public abstract class TreeController<T extends ITreeNode<T>> extends
                     Navigation navigation, Treerow treerow) {
                 List<InputElement> boxes = getNavigableElements(treerow);
                 int position = boxes.indexOf(inputElement);
-
+                if (position > boxes.size() - 1) {
+                    return;
+                }
                 switch (navigation) {
                 case UP:
                     focusGoUp(treerow, position);
@@ -518,11 +520,13 @@ public abstract class TreeController<T extends ITreeNode<T>> extends
             private void focusCorrectBox(Treerow treerow, int position,
                     Navigation whereIfDisabled) {
                 List<InputElement> boxes = getNavigableElements(treerow);
-
-                if (boxes.get(position).isDisabled()) {
-                    moveFocusTo(boxes.get(position), whereIfDisabled, treerow);
-                } else {
-                    boxes.get(position).focus();
+                if (position < boxes.size() - 1) {
+                    if (boxes.get(position).isDisabled()) {
+                        moveFocusTo(boxes.get(position), whereIfDisabled,
+                                treerow);
+                    } else {
+                        boxes.get(position).focus();
+                    }
                 }
             }