geotools-gt2-users

[Geotools-gt2-users] GeoTools RC1 and ArcSDE NStrings

[jericks <JE...@co...>]

I was testing an application that uses the GeoTools ArcSDE module to insert
into an ArcSDE FeatureClass and I think I discovered a bug.  For columns of
type NString, you have to use SeRow.setNString instead of SeRow.setString.
ArcSdeFeatureWriter tries to use SeRow.setString for all string types (see
line 653).

else if (colType == SeColumnDefinition.TYPE_STRING
                    || colType == SeColumnDefinition.TYPE_NSTRING
                    || colType == SeColumnDefinition.TYPE_CLOB
                    || colType == SeColumnDefinition.TYPE_NCLOB) {
                convertedValue = Converters.convert(convertedValue,
String.class);
                row.setString(index, (String) convertedValue);

Could this be fixed for the 2.5 release?

Thanks,
Jared
-- 
View this message in context: http://www.nabble.com/GeoTools-RC1-and-ArcSDE-NStrings-tp19336564p19336564.html
Sent from the geotools-gt2-users mailing list archive at Nabble.com.

Re: [Geotools-gt2-users] GeoTools RC1 and ArcSDE NStrings

[Jody G. <jga...@re...>]

Can you create a patch for the arcsde module maintainer to review? The 
user community able to test arcsde (and willing to talk about it) is 
quite small - a lot of work has gone into the ArcSDE datastore and I am 
sure the maintainer will be interested.
Jody

jericks wrote:
> I was testing an application that uses the GeoTools ArcSDE module to insert
> into an ArcSDE FeatureClass and I think I discovered a bug.  For columns of
> type NString, you have to use SeRow.setNString instead of SeRow.setString.
> ArcSdeFeatureWriter tries to use SeRow.setString for all string types (see
> line 653).
>
> else if (colType == SeColumnDefinition.TYPE_STRING
>                     || colType == SeColumnDefinition.TYPE_NSTRING
>                     || colType == SeColumnDefinition.TYPE_CLOB
>                     || colType == SeColumnDefinition.TYPE_NCLOB) {
>                 convertedValue = Converters.convert(convertedValue,
> String.class);
>                 row.setString(index, (String) convertedValue);
>
> Could this be fixed for the 2.5 release?
>
> Thanks,
> Jared
>   

Re: [Geotools-gt2-users] GeoTools RC1 and ArcSDE NStrings

[Gabriel R. <gr...@op...>]

Jared, please file a bug report at jira.codehaus.org for the geotools arcsde 
plugin so the issue does not ends up forgotten on inbox hell and you'll be 
automattically notified when its fixed.

Thanks for the catch,

Gabriel
On Monday 08 September 2008 02:20:49 pm Jody Garnett wrote:
> Can you create a patch for the arcsde module maintainer to review? The
> user community able to test arcsde (and willing to talk about it) is
> quite small - a lot of work has gone into the ArcSDE datastore and I am
> sure the maintainer will be interested.
> Jody
>
> jericks wrote:
> > I was testing an application that uses the GeoTools ArcSDE module to
> > insert into an ArcSDE FeatureClass and I think I discovered a bug.  For
> > columns of type NString, you have to use SeRow.setNString instead of
> > SeRow.setString. ArcSdeFeatureWriter tries to use SeRow.setString for all
> > string types (see line 653).
> >
> > else if (colType == SeColumnDefinition.TYPE_STRING
> >
> >                     || colType == SeColumnDefinition.TYPE_NSTRING
> >                     || colType == SeColumnDefinition.TYPE_CLOB
> >                     || colType == SeColumnDefinition.TYPE_NCLOB) {
> >
> >                 convertedValue = Converters.convert(convertedValue,
> > String.class);
> >                 row.setString(index, (String) convertedValue);
> >
> > Could this be fixed for the 2.5 release?
> >
> > Thanks,
> > Jared
>
> -------------------------------------------------------------------------
> This SF.Net email is sponsored by the Moblin Your Move Developer's
> challenge Build the coolest Linux based applications with Moblin SDK & win
> great prizes Grand prize is a trip for two to an Open Source event anywhere
> in the world http://moblin-contest.org/redirect.php?banner_id=100&url=/
> _______________________________________________
> Geotools-gt2-users mailing list
> Geo...@li...
> https://lists.sourceforge.net/lists/listinfo/geotools-gt2-users