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
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)
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.