Details
-
Type:
Bug
-
Status: Closed
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 3.2
-
Fix Version/s: EE-3.3.0.GA, 4.0.BETA, 4.0
-
Component/s: ACE-Components, Sample Apps
-
Labels:None
-
Environment:ICEfaces3 trunk rev # 31740
-
Assignee Priority:P1
-
Affects:Sample App./Tutorial
Description
if dragging and dropping nodes starting from the second node, so that each node becomes the child of an upper node (see attached screen shot), at one certain drag movement, a java.lang.IndexOutOfBoundsException is thrown in the tomcat log.
24-Oct-2012 3:06:00 PM com.sun.faces.context.PartialViewContextImpl processPartial
INFO: java.lang.IndexOutOfBoundsException: Index: 2, Size: 2
java.lang.IndexOutOfBoundsException: Index: 2, Size: 2
at java.util.ArrayList.RangeCheck(ArrayList.java:547)
at java.util.ArrayList.get(ArrayList.java:322)
at org.icefaces.samples.showcase.example.ace.tree.LocationNodeImpl.getChildAt(LocationNodeImpl.java:69)
at org.icefaces.ace.model.tree.ListNodeDataModel.navToChild(ListNodeDataModel.java:111)
at org.icefaces.ace.model.tree.ListNodeDataModel.navToKey(ListNodeDataModel.java:92)
at org.icefaces.ace.model.tree.ListNodeDataModel.navToKey(ListNodeDataModel.java:8)
at org.icefaces.ace.component.tree.Tree.setNodeToKey(Tree.java:159)
at org.icefaces.ace.component.tree.Tree.setKey(Tree.java:127)
at org.icefaces.ace.component.tree.TreeDecoder.decodeReordering(TreeDecoder.java:83)
at org.icefaces.ace.component.tree.TreeDecoder.decode(TreeDecoder.java:70)
at org.icefaces.ace.component.tree.TreeRenderer$1.<init>(TreeRenderer.java:63)
at org.icefaces.ace.component.tree.TreeRenderer.decode(TreeRenderer.java:62)
at javax.faces.component.UIComponentBase.decode(UIComponentBase.java:787)
at org.icefaces.ace.component.tree.Tree.processDecodes(Tree.java:301)
at com.sun.faces.context.PartialViewContextImpl$PhaseAwareVisitCallback.visit(PartialViewContextImpl.java:506)
at com.sun.faces.component.visit.PartialVisitContext.invokeVisitCallback(PartialVisitContext.java:183)
at org.icefaces.ace.component.tree.Tree.visitTree(Tree.java:390)
at javax.faces.component.UIComponent.visitTree(UIComponent.java:1601)
at javax.faces.component.UIForm.visitTree(UIForm.java:344)
Activity
I saw this once yesterday in Showcase myself, after I reloaded the page after playing with tree selections. Could not reproduce again this morning when I tried a similar action.
The steps from the description may not be complete; I am getting this exception once in a while when dragging /dropping nodes on this page, however could not identify the exact sequence of steps to reproduce it.
I also got this exception in the ace:tree regression test application.
| Repository | Revision | Date | User | Message |
| ICEsoft Public SVN Repository | #31754 | Thu Oct 25 11:11:15 MDT 2012 | nils.lundquist | |
| Files Changed | ||||
MODIFY
/icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/tree/tree.js
|
The JS drag logic has border case bugs that can cause stack traces of this sort by submitting incorrect indexes. However these bugs are very difficult to reproduce in the cases we've observered to the point of not being a concern to regular operation.
Should anyone discover a test that reproduces these bugs with ease this issue should be reopened to profile the erroneous logic and see if there is a way we can avoid them.
| Field | Original Value | New Value |
|---|---|---|
| Reporter | Migration [ remote ] | Carmen Cristurean [ ccristurean ] |
| Attachment | tree-dragDrop-IndexOutOfBoundException.png [ 14911 ] |
| Attachment | tree-dragDrop-IndexOutOfBoundException.png [ 14912 ] |
| Assignee | Nils Lundquist [ nils.lundquist ] | |
| Fix Version/s | 3.2 [ 10338 ] | |
| Assignee Priority | P1 [ 10010 ] |
| Status | Open [ 1 ] | Resolved [ 5 ] |
| Resolution | Cannot Reproduce [ 5 ] |
I saw this once yesterday in Showcase myself, after I reloaded the page after playing with tree selections. Could not reproduce again this morning when I tried a similar action.
| Status | Resolved [ 5 ] | Closed [ 6 ] |
| Assignee Priority | P1 [ 10010 ] |
We found consistent steps on how to reproduce this error. Because of this we have reopened this JIRA
Icefaces3 trunk revision# 34972/35014/ ICEfaces 3.2
Server:tomcat7
Browser: FF3.6/20, GoogleChrome26
Tree
> Overview Page:
Server Internal error popup and server side error when selecting a province and dragging the node to the right into the selection box. See attached screen shot.
May 14, 2013 11:02:24 AM com.sun.faces.lifecycle.ApplyRequestValuesPhase execute
WARNING: Index: 3, Size: 1
java.lang.IndexOutOfBoundsException: Index: 3, Size: 1
at java.util.ArrayList.add(ArrayList.java:367)
at org.icefaces.ace.model.tree.ListNodeDataModel.insert(ListNodeDataModel.java:198)
at org.icefaces.ace.model.tree.ListNodeDataModel.insert(ListNodeDataModel.java:24)
at org.icefaces.ace.component.tree.Tree.insertNode(Tree.java:98)
at org.icefaces.ace.component.tree.TreeDecoder.decodeReordering(TreeDecoder.java:86)
at org.icefaces.ace.component.tree.TreeDecoder.decode(TreeDecoder.java:66)
at org.icefaces.ace.component.tree.TreeRenderer$1.<init>(TreeRenderer.java:64)
at org.icefaces.ace.component.tree.TreeRenderer.decode(TreeRenderer.java:63)
at javax.faces.component.UIComponentBase.decode(UIComponentBase.java:793)
at org.icefaces.ace.component.tree.Tree.processDecodes(Tree.java:299)
at com.sun.faces.context.PartialViewContextImpl$PhaseAwareVisitCallback.visit(PartialViewContextImpl.java:504)
at com.sun.faces.component.visit.PartialVisitContext.invokeVisitCallback(PartialVisitContext.java:183)
This is a partial listing of the error.
| Resolution | Cannot Reproduce [ 5 ] | |
| Status | Closed [ 6 ] | Reopened [ 4 ] |
| Affects | Sample App./Tutorial [ 10001 ] | |
| Assignee Priority | P1 [ 10010 ] |
| Fix Version/s | EE-3.3.0.GA [ 10572 ] | |
| Fix Version/s | 3.4 [ 10770 ] | |
| Fix Version/s | 3.2 [ 10338 ] |
| Attachment | treeServerError.PNG [ 15881 ] |
| Security | Private [ 10001 ] |
Just to augment the instructions above, for me, I saw this if I first did a successful Drag and Drop operation within the left-hand tree. Then tried to move a tree node to the right-hand list caused the error.
| Repository | Revision | Date | User | Message |
| ICEsoft Public SVN Repository | #35534 | Wed May 22 22:29:52 MDT 2013 | nils.lundquist | |
| Files Changed | ||||
|
MODIFY
/icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/tree/tree.js
|
Revision #35534
Committed by nils.lundquist
4 minutes ago
ICE-8694 - Use ui.item parent when we've confirmed that our reorder drop is within the same parent. the drop point referenced by 'this' is unreliable in this context, and ui.item parent is assured to have the correct 'new' drop parent.
| Status | Reopened [ 4 ] | Resolved [ 5 ] |
| Resolution | Fixed [ 1 ] |
| Comment |
[ This is still not really a reliable case. After 4 attempts I was able to get the error reproducing - after which I was easily able to continue to cause errors in the bugged state. However, upon restarting my server it took another 6 or 7 refreshes before I could reproduce the error. Perhaps some browser or configuration is more susceptible. ] |
Verified with icefaces3/trunk revision# 35543 in Firefox21, Chrome26, IE8.
| Fix Version/s | 4.0 [ 11382 ] |
| Status | Resolved [ 5 ] | Closed [ 6 ] |
This error would occur if the tree traversal logic had a bug or if the client side tree component had produced incorrect indexes due to a bug or race condition. Without a precise test case there is no way to determine the cause of this error.