ramido / choosel

Automatically exported from code.google.com/p/choosel
0 stars 0 forks source link

Graph Expansion failing when adding multiple items with relationships #36

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

1. In Choosel Example, open up a graph view, and add all of the graph test 
elements to it.  This will result in no errors being fired and no elements 
being added to the view.
2. This will also occur if you add all of the graph Test elements to a list, 
create a selection of more than one of them, and then adding the selection to 
the graph view.
3. To see what should occur, add all of the graph test items to a list, and 
then add them one by one to the Graph view.  All of the items, labeled 1 to 25, 
are linked to the ones 1 above and 1 below them.

Original issue reported on code.google.com by patrickc...@gmail.com on 28 Jul 2010 at 9:26

GoogleCodeExporter commented 9 years ago
The problem happens when adding more than 1 resource item at the same time, 
when there is a potential relationship between those resource items.

Very likely this is due to the fact that AbstractAutomaticGraphNodeExpander is 
based on single resources and uses the contains and containsResourceWithURI 
methods of the view callback - these return true already if the resource item 
has not been added to the viewcontentdisplay yet.

Original comment by Lars.Grammel on 30 Jul 2010 at 6:49

GoogleCodeExporter commented 9 years ago

Original comment by Lars.Grammel on 3 Aug 2010 at 1:41

GoogleCodeExporter commented 9 years ago

Original comment by Lars.Grammel on 3 Aug 2010 at 1:41

GoogleCodeExporter commented 9 years ago
I committed a fix for this problem. Please verify.

Original comment by Lars.Grammel on 3 Aug 2010 at 4:26

GoogleCodeExporter commented 9 years ago

Original comment by Lars.Grammel on 5 Aug 2010 at 6:23