OpenLayers OpenLayers

Ticket #1406 (closed bug: fixed)

Opened 9 months ago

Last modified 9 months ago

support reprojecting wfs request coordinates

Reported by: crschmidt Assigned to: tschaub
Priority: minor Milestone: 2.6 Release
Component: Layer.WFS Version: 2.5
Keywords: Cc:
State: Complete

Description

Currently, WFS will try to reproject data it receives if the projection differs from the map projection... but it doesn't actually reproject the bounding box it's querying with, so this will never actually work.

Attachments

projected.patch (1.0 kB) - added by crschmidt on 02/29/08 01:41:05.
projected_wfs.patch (3.1 kB) - added by crschmidt on 02/29/08 02:07:38.
wfs_reprojection.patch (6.3 kB) - added by crschmidt on 02/29/08 09:23:01.
wfs_reprojection.2.patch (5.4 kB) - added by tschaub on 02/29/08 13:25:40.
wfs reprojected

Change History

02/29/08 01:41:05 changed by crschmidt

  • attachment projected.patch added.

02/29/08 02:07:19 changed by crschmidt

The WFS layer did not properly reproject its request bounds, despite the GML reader being set up with reprojection parameters. The same was true of the WFS commit, which would not reproject. Fix both of these issues, and add FeatureServer as a datasource in the spherical-mercator example, allowing users to see how to set reprojection parameters on their layers to take advantage of automatic reprojection.

02/29/08 02:07:38 changed by crschmidt

  • attachment projected_wfs.patch added.

02/29/08 02:10:19 changed by crschmidt

This functionality should have gone in with r5828, but didn't. Since that was before our 2.6 feature freeze, and this completes that feature, I'd like to put this into 2.6 if possible. (yeah, yeah, I know, feature creep)

02/29/08 02:36:04 changed by crschmidt

  • state set to Review.

I've just gotten confirmation from another user who has applied this patch that it works for him with spherical mercator layers as well (woot)

02/29/08 04:31:57 changed by tschaub

  • state changed from Review to Needs More Work.

Can you bump this in to a new example? I think it deserves it's own. Also, #1407 is related to (but not a blocker for) this.

02/29/08 09:23:01 changed by crschmidt

  • attachment wfs_reprojection.patch added.

02/29/08 09:23:45 changed by crschmidt

  • state changed from Needs More Work to Review.

Done.

02/29/08 13:25:40 changed by tschaub

  • attachment wfs_reprojection.2.patch added.

wfs reprojected

02/29/08 13:29:33 changed by tschaub

  • state changed from Review to Commit.

Thanks man. Cool example. I should have been clearer above, but my request for kicking the wfs stuff in to a new example was about leaving the spherical mercator example proxy free. I think a lot of people use that example (locally) and a lot of those people would probably be confused by the alerts.

My patch doesn't touch the spherical mercator example or proxy.cgi. The same stuff is available on port 80 there.

I've also added another bit to the example. We're still weak on style examples and I think the more the better. Plus, styleMaps rule.

If you don't like, please commit yours (maybe I'll just tweak examples later). If you agree, please commit mine. Thanks for this nice change.

02/29/08 13:39:37 changed by crschmidt

  • status changed from new to closed.
  • state changed from Commit to Complete.
  • resolution set to fixed.

(In [6422]) Improve WFS reprojection:

  • Transform bbox back to layer projection
  • Transform commits back to layer projection

r=tschaub, (Closes #1406)