search

javierdotnet / gwt-ext

Automatically exported from code.google.com/p/gwt-ext
0 stars 3 forks source link

Some of MultiSelectionModel's 'void' methods return a value #433

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. call MultiSelectionModels select(TreeNode treeNode), select(TreeNode
treeNode, boolean keepExisting), or unselect(TreeNode treeNode)  methods

What is the expected output? What do you see instead?
operation works fine, but logs an error

What version of the product are you using? On what operating system?
2.0.5  on windows vista

Please provide any additional information below.
Solution would be to remove the 'return' since these methods are declared
as void.

Original issue reported on code.google.com by bradley...@gmail.com on 7 Oct 2008 at 2:45

GoogleCodeExporter commented 9 years ago 
Removed the return from the JSNI code.  extjs does return the selected TreeNode 
on 
selection but since it is passed as the first argument, it is really unecessary 
to 
return it back.  unSelect method was really wrong since unselect does not 
return 
anything.

Original comment by mlim1...@gmail.com on 10 Oct 2008 at 7:46