[Geoserver-devel] ReferencedEnvelope patch incoming

GeoServer GeoServer Developer
1

Having chased the ReferencedEnvelope problems out of GeoTools it is time for me to confront them in the GeoServer codebase.

Currently I am testing WMS, any time we work with bounds we need to take this regression (2D and 3D bounds do not mix) into account.

As am example in GetMap we have:

mapContent.getViewport().setBounds(new ReferencedEnvelope(envelope, mapcrs));

This fat out fails when, and needs to be replaced with the factory method:

mapContent.getViewport().setBounds(ReferencedEnvelope.reference(envelope, mapcrs));

So expect an annoying patch to review this weekend, I am traveling next week but will try and answer email.

–
Jody Garnett

2

Oh a couple small bits of progress to report:

  • Can now add a layer, and get WFS output
  • Layer preview and Streaming renderer are tripping up (on IRC Andrea indicates that StreamingRenderer is asking for 2D geometry, and the DataStore is ignoring the hint)

–
Jody Garnett

On Saturday, 1 December 2012 at 11:50 PM, Jody Garnett wrote:

Having chased the ReferencedEnvelope problems out of GeoTools it is time for me to confront them in the GeoServer codebase.

Currently I am testing WMS, any time we work with bounds we need to take this regression (2D and 3D bounds do not mix) into account.

As am example in GetMap we have:

mapContent.getViewport().setBounds(new ReferencedEnvelope(envelope, mapcrs));

This fat out fails when, and needs to be replaced with the factory method:

mapContent.getViewport().setBounds(ReferencedEnvelope.reference(envelope, mapcrs));

So expect an annoying patch to review this weekend, I am traveling next week but will try and answer email.

–
Jody Garnett

3

First batch of work is ready:

The allows you to configure a layer with 3D content (oracle and property file have been tested), and I have been able to verify WFS functionality.
The addition of EnvelopePanel is a user interface change, and while it is simple and straight forward, another set of eyes checking it over would be appreciated.

The next batch of work involves letting WMS in on the party, work is available no the following branch: