Ticket #1955 (closed feature: fixed)

Opened 4 years ago

Last modified 4 years ago

KML Format does not support ExtendedData

Reported by: crschmidt Owned by: tschaub
Priority: minor Milestone: 2.8 Release
Component: Format.KML Version: 2.7
Keywords: Cc:
State: Complete

Description

The KML format does not have support for KML ExtendedData.

Attachments

kml-ed.patch Download (4.5 KB) - added by crschmidt 4 years ago.

Change History

Changed 4 years ago by crschmidt

Changed 4 years ago by crschmidt

  • state set to Review

Changed 4 years ago by tschaub

  • status changed from new to assigned

Changed 4 years ago by tschaub

  • state changed from Review to Commit

Looks like the patch does what it says (based on tests).

My only minor suggestion would be to get in the habit of caching the length property for NodeLists. Checking length on node lists is more expensive than arrays, and there is no reason to do it for each child node.

Changed 4 years ago by crschmidt

  • status changed from assigned to closed
  • state changed from Commit to Complete
  • resolution set to fixed

(In [9089]) Add ExtendedData support to KML. r=tschaub, patch mine + his comments, thx to rdewit for initial help/pointers, (Closes #1955)

Note: See TracTickets for help on using tickets.