search

camptocamp / cgxp

22 stars 22 forks source link

WFS GetFeature with GetFeature plugin #740

Closed rbovard closed 10 years ago

rbovard commented 10 years ago

After upgrading to 1.4.3, the GetFeature plugin return not the same highlighted geometry depending if it's a WMS GetFeatureInfo or a WFS GetFeature on MultiLineString layers.

My data and MapFile doesn't evolved between those two releases, it's why I ask if something changed in the mechanism of the GetFeature plugin?

rbovard commented 10 years ago

Can someone else confirm or infirm this? @kalbermattenm @gnerred

Thanks :)

kalbermattenm commented 10 years ago

SITN is running on the master branch of CGXP, and I can't reproduce this issue... Actually our instances used 1.4.2 and went straight to master...

Maybe @gnerred might be able to help you...

yjacolin commented 10 years ago

It seems that the problem should come from your project as nothing change on this but we didn't investigate more on this.

yjacolin commented 10 years ago

The problem seems occur only on multiline geometries.

rbovard commented 10 years ago

Thanks for your answers.

@yjacolin Yes it's what I saw, our dev and demo instances are in 1.4.3 and our prod in 1.4.2 and the data (and MapFile) are the same.

I wanted to know if it's a bug specific to our project or if it happens with all 1.4.3 versions before more investigate.

So @gnerred if you can take a look, it will be nice, thanks.

yjacolin commented 10 years ago

FYI, same issue on GeoMapfish demo. I don't have access to the server via ssh at this moment so I can't get the current release but it smells a bug in CGXP. I will ask to @sbrunner to look into this.

gnerred commented 10 years ago

@rbovard. I have the same configuration as SITN. Prod is on 1.4.2, and dev is on master. I don't have the 1.4.3 to test sorry...

rbovard commented 10 years ago

@yjacolin Thanks for the update, I'll wait on @sbrunner

@gnerred No worries, since it's confirmed on another instance than ours it's ok :)

rbovard commented 10 years ago

Any news on this one?

Thanks.

yjacolin commented 10 years ago

@sbrunner is on vacation. More information next week, sorry.

sbrunner commented 10 years ago

@yjacolin demo don't have MultiLineString than I'm not able to see the issue on it ...

yjacolin commented 10 years ago

@sbrunner

hum, so are you sure this bug is link to a MultiLineString?

Check http://mapfish-geoportal.demo-camptocamp.com/demo/wsgi/short/Nhce and click on the green line.

Y.

sbrunner commented 10 years ago

It's a LineString, not a MultiLineString ! And it don't also works in WMS/GetFeatureInfo because the type in the mapfile is wrong ...