ravisangar / smartgwt

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

ListGrid.getEditCol() and ListGrid.getEditRow : bad return type #110

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. call getEditCol on a listGrid
2. If there is a currently edited cell, it works and return the column/row
index of the edited cell.
3. If there is no edited cell, we have an exception

Expected primitive type int; actual value was null

The doc claimed that those methods return null is not cell is being edited.
But the return is set to be an int, which is a primitive.

I think the return should be -1 if no cell is currently being edited. And
the doc should be up to date to reflect that change of return.

Maybe other methods should be created to return editedRow primary key,
edited column field name or id, etc..

What version of the product are you using? On what operating system?

last build (10 fev)

Original issue reported on code.google.com by ricard.i...@gmail.com on 10 Feb 2009 at 9:22

GoogleCodeExporter commented 9 years ago

Original comment by sanjiv.j...@gmail.com on 10 Feb 2009 at 9:53

GoogleCodeExporter commented 9 years ago
getEditCol, getEditRow return -1 if no cell currently being edited.

Original comment by sanjiv.j...@gmail.com on 11 Feb 2009 at 11:10

GoogleCodeExporter commented 9 years ago
Broken again in latest releases.

Original comment by piemm...@googlemail.com on 31 Jul 2013 at 10:24

GoogleCodeExporter commented 9 years ago
Issue will be resolved in tonight's nightly build of 4.1d. 

Original comment by smartgwt...@gmail.com on 6 Aug 2013 at 2:58