OpenLayers OpenLayers

Ticket #1613 (new feature)

Opened 1 month ago

Last modified 3 weeks ago

enhance Format.GML to parse gml:boundedBy

Reported by: bartvde Assigned to: tschaub
Priority: critical Milestone: 2.7 Release
Component: Format.GML Version: 2.6
Keywords: Cc:
State: Needs More Work

Description

This is e.g. handy when you want to zoom to the full featureCollection.

Attachments

ticket1613.patch (3.5 kB) - added by bartvde on 07/10/08 03:49:43.
ticket1613.2.patch (3.6 kB) - added by bartvde on 07/28/08 10:49:03.
updates patch which does not fail when <gml:null>missing</gml:null> is in the GML

Change History

07/08/08 03:35:15 changed by bartvde

A problem though is that the read function currently returns an array of features, so there is not really "space" to put an extra return value. If it would return an object with an array of features, there wouldn't be an issue. But changing this will also influence Layer.GML and other code that currently uses Format.GML, so I don't know if this is acceptable.

07/08/08 08:18:57 changed by crschmidt

The way we've addressed this type of requirement is to require additional options to be passed to read which say something like 'return some different type of object': so if 'read' gets a 'returnBbox: true' option, then read can return something other than a list of features.

07/08/08 08:39:14 changed by bartvde

Thanks Chris, I'll take that into account when creating the patch.

07/10/08 03:49:43 changed by bartvde

  • attachment ticket1613.patch added.

07/10/08 03:50:27 changed by bartvde

  • state set to Review.

07/10/08 09:28:52 changed by crschmidt

Tim: I think you had mentioned at one point you were working on an improved GML parser. (One that didn't use so much slow getElementsByTagNameNS.) Is that the case?

07/11/08 02:23:07 changed by bartvde

  • state changed from Review to Needs More Work.

On the mailing list Tim pointed to this:

http://trac.openlayers.org/browser/sandbox/topp/almanac/lib/OpenLayers/Format/GML/Base.js

I'll try and update the patch to avoid getElementsByTagNameNS for the added functionality.

07/28/08 10:49:03 changed by bartvde

  • attachment ticket1613.2.patch added.

updates patch which does not fail when <gml:null>missing</gml:null> is in the GML

07/28/08 14:48:32 changed by euzuro

  • priority changed from minor to critical.