From 61940db92f3bc8b16b8f7253b29fc7d598fcf0ff Mon Sep 17 00:00:00 2001
From: Dane Springmeyer <dane@dbsgeo.com>
Date: Mon, 8 Nov 2010 19:54:28 +0000
Subject: [PATCH] tweak the upgrade logic so that we don't loose existing
 attributes - allows upgrading a partially compliant xml

---
 utils/upgrade_map_xml/upgrade_map_xml.py | 12 ++----------
 1 file changed, 2 insertions(+), 10 deletions(-)

diff --git a/utils/upgrade_map_xml/upgrade_map_xml.py b/utils/upgrade_map_xml/upgrade_map_xml.py
index a050e6850..4509de84d 100755
--- a/utils/upgrade_map_xml/upgrade_map_xml.py
+++ b/utils/upgrade_map_xml/upgrade_map_xml.py
@@ -60,16 +60,8 @@ def fixup_pointsym(sym):
         sym.attrib.pop('type')
 
 def fixup_sym_attributes(sym):
-    #if not sym.find('CssParameter'):
-    #    return
-    attrib = {}
-    metawriter = sym.attrib.get('meta-writer')
-    if metawriter:
-        attrib['meta-writer'] = metawriter
-    metaoutput = sym.attrib.get('meta-output')
-    if metaoutput:
-        attrib['meta-output'] = metaoutput
-    
+    # copy, so we don't loose after clear()
+    attrib = dict(sym.attrib)
     for css in sym.findall('CssParameter'):
         key = css.attrib.get('name')
         value = css.text