[Geoserver-devel] ResourceInfo.getBoundingBox

GeoServer GeoServer Developer
1

Hi Justin,

when you have a chance could you please take a look at my comment on http://jira.codehaus.org/browse/GEOS-2260.

Basically I would like to add a setter for ResourceInfo's bounding box instead of declaring it to be a derived property of the native bounding box, in order to keep ResourceInfo from performing the computation, but would like to hear your opinion first.

cheers,

Gabriel

2

Hi guys,
I'm sending a possible patch I applied on my side, look if can be useful.

On Fri, Oct 3, 2008 at 5:24 PM, Gabriel Roldan <groldan@anonymised.com> wrote:

Hi Justin,

when you have a chance could you please take a look at my comment on
http://jira.codehaus.org/browse/GEOS-2260.

Basically I would like to add a setter for ResourceInfo's bounding box
instead of declaring it to be a derived property of the native bounding box,
in order to keep ResourceInfo from performing the computation, but would
like to hear your opinion first.

cheers,

Gabriel

--
-------------------------------------------------------
Eng. Alessio Fabiani
Vice-President /CTO GeoSolutions S.A.S.
Via Carignoni 51
55041 Camaiore (LU)
Italy

phone: +39 0584983027
fax: +39 0584983027
mob: +39 349 8227000

http://www.geo-solutions.it

-------------------------------------------------------

(attachments)

fix_for_GEOS-2260_and_GEOS-2261.patch (18.5 KB)

3

Hi Gabriel,

I agree... and I like the idea of leaving *Info objects as just data... no processing. Off the top of my head I don't remember what the main motivation was for doing that. I *think* that it was to avoid having to keep the two values in sync. But i don't think i have a strong opinion on this so I am +1 for the patch.

I notice the patch sets the value when the catalog is loaded. Are there any other calls to setNativeBounds()? Which we would need to update to add a supplementary call to setBboundingBox()?

-Justin

Gabriel Roldan wrote:

Hi Justin,

when you have a chance could you please take a look at my comment on http://jira.codehaus.org/browse/GEOS-2260.

Basically I would like to add a setter for ResourceInfo's bounding box instead of declaring it to be a derived property of the native bounding box, in order to keep ResourceInfo from performing the computation, but would like to hear your opinion first.

cheers,

Gabriel

--
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.