sjivan / gwt-ext

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

Bug in casting Date in ColumnConfig #459

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Logger reported this problem (and the fix) on the forum.

Please take a look here for complete info:
http://gwt-ext.com/forum/viewtopic.php?f=8&t=3418

I was able to recreate this problem in Mac hosted mode and Opera browser.

In the ColumnConfig.java, method setRenderer, you have this:

var valJ = (val  == null || val === undefined || val === '') ? null :
$wnd.GwtExt.convertToJavaType(val);

This seems to be not enought for Opera.

This change works well on it, and do not affect other browsers:

var valJ;
if ($wnd.Ext.isDate(val)) {
  valJ =
@com.gwtext.client.util.JavaScriptObjectHelper::toDate(D)(val.getTime());
} else {
  valJ = (val  == null || val === undefined || val === '') ? null :
$wnd.GwtExt.convertToJavaType(val);
}

Thanks for verifying.

Original issue reported on code.google.com by micg...@gmail.com on 28 Nov 2008 at 9:22

GoogleCodeExporter commented 8 years ago
See http://gwt-ext.com/forum/viewtopic.php?f=8&t=3400&p=10932#p10932

Original comment by sanjiv.j...@gmail.com on 2 Dec 2008 at 10:05

GoogleCodeExporter commented 8 years ago

Original comment by sanjiv.j...@gmail.com on 2 Dec 2008 at 10:09

GoogleCodeExporter commented 8 years ago
Hi. Should we expect the fix to be available in the next release?

Original comment by micg...@gmail.com on 16 Dec 2008 at 10:59

GoogleCodeExporter commented 8 years ago
yes, i'll try to put out a nightly build with the fix prior to that as well.

Original comment by sanjiv.j...@gmail.com on 16 Dec 2008 at 1:29

GoogleCodeExporter commented 8 years ago
I see that 2.0.6 has been released, but the release notes do not mention that 
this bug has been fixed -- has it 
been overlooked?  Thanks.

Original comment by bor...@gmail.com on 15 Jan 2009 at 7:17

GoogleCodeExporter commented 8 years ago
Can we get this in next release?

Original comment by airan...@gmail.com on 26 Feb 2009 at 12:56