OBJECTID field is always added

[jkerr5]

Even if the metadata specifies an idField, it looks like the OBJECTID field is still added to the list of fields.

I think I have traced it back to here in the code

FeatureServer/src/field/index.js

Line 27 in 4515bed

if (!_.find(responseFields, {'name': 'OBJECTID'})) responseFields.push({name: 'OBJECTID'})

It should be looking for the field that the metadata says is the idField

[rgwozdz]

No, we actually made a conscious decision to include both the field specified by idField as well as add an OBJECTID field (release 2.11.0 from this PR) . We observed some buggy behavior with some ArcGIS clients when the OBJECTID field was missing and had the most reliable behavior when it was always present - so we opted for Koop/FeatureServer to include both the idField and OBJECTID.

[jkerr5]

What if clients pass in a where clause with OBJECTID? The provider would need to have special logic looking for the OBJECTID column to map it to the ID column. This seems like a behavior that pass-through providers should be able to control as it is really changing what columns the client think are available.

…

On Sun, Oct 14, 2018, 4:24 AM Rich Gwozdz ***@***.***> wrote: No, we actually made a conscious decision to include both the field specified by idField as well as add an OBJECTID field (release 2.11.0 <https://github.com/koopjs/FeatureServer/releases/tag/v2.11.0> from this PR <#96>) . We observed some buggy behavior with some ArcGIS clients when the OBJECTID field was missing and had the most reliable behavior when it was always present - so we opted for Koop/FeatureServer to include both the idField and OBJECTID. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#127 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAP8AtS5ZCy4z2OFhY8yVv4kTSzy_aeZks5ukqBwgaJpZM4Xa5Km> .

[rgwozdz]

Yup, you've identified a flaw that needs to be addressed. At minimum we need Winnow to be processing WHEREs with OBJECTIDs properly when idField determines its value. While addressing this, we might be able to take a deeper dive and see if we can avoid the workaround noted above.

[rgwozdz]

@jkerr5 - this is now fixed in FeatureServer 2.16.3 and Winnow 1.16.5. If you set an idField in metadata, that will be used as the objectIdField in ArcGIS clients; a duplicate field named OBJECTID is no longer added.
Thanks again for bring this back to my attention.

[jkerr5]

Thanks Rich!

…

On Wed, Nov 28, 2018, 9:03 PM Rich Gwozdz ***@***.***> wrote: Closed #127 <#127>. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#127 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAP8AshGsnnJRQgDoaeBrIg3CGFG2KXNks5uzuv-gaJpZM4Xa5Km> .