planetfederal / geoserver-exts

Other
31 stars 40 forks source link

Importer attributes to geom transform #7

Closed rmarianski closed 12 years ago

rmarianski commented 12 years ago

@jdeolive , @ischneider : Some importer changes that shouldn't change existing behavior. Care to review?

These are the changes that were required to support the workflow where a user was asked what columns should represent a lat/lng in a csv file. They boil down to:

rmarianski commented 12 years ago

@jdeolive Is the above test what you had in mind?

jdeolive commented 12 years ago

Yup, that looks great.

rmarianski commented 12 years ago

@jdeolive I removed the context parameter, and made the csv indirect tests more thorough by checking the attributes on the first feature that got imported. If it's ok, can you merge?