OpenLayers OpenLayers

Ticket #973 (closed bug: fixed)

Opened 8 months ago

Last modified 8 months ago

Format.GeoRSS doesn't use Format.XML serializer

Reported by: crschmidt Assigned to:
Priority: minor Milestone: 2.5 Release
Component: Format.GeoRSS Version: 2.4
Keywords: Cc:
State:

Description

The format.georss doesn't use the Format.XML serializer. I'd already rewritten a huge chunk of this code so that it worked correctly -- the attached patch is faster, cleaner, and will work in IE.

Note that if this patch is too big, we can instead create a tiny patch to just fix that aspect of the georss format, but it's not particularly useful as it is, so I'd like to get this in.

Attachments

georss.patch (19.1 kB) - added by crschmidt on 09/14/07 10:08:11.
georss.2.patch (22.3 kB) - added by tschaub on 09/14/07 18:39:53.
read/write georss

Change History

09/14/07 10:08:11 changed by crschmidt

  • attachment georss.patch added.

09/14/07 18:39:53 changed by tschaub

  • attachment georss.2.patch added.

read/write georss

09/14/07 18:41:01 changed by tschaub

check this one - I removed the Util.getNodes functions - and deprecated trim() calls - reworked a few things - added to vector-formats.html

all tests pass in IE/FF

09/14/07 18:45:59 changed by crschmidt

Looks good to me. Shall I commit?

09/14/07 18:47:09 changed by tschaub

  • keywords changed from review to commit.

commit away!

09/14/07 18:50:33 changed by crschmidt

  • keywords deleted.
  • status changed from new to closed.
  • resolution set to fixed.

(In [4305]) Format.GeoRSS didn't use Format.XML serializer, nor did it support reading anything other than points. Both of these are resolved with the new GeoRSS support, which supports all of GeoRSS simple. Includes tests and significant review from Senor Schaub, and addition to the examples/vector-formats.html file. (Closes #973)