From 4df684c50ecb5239bcad659cfbaf708de03fefca Mon Sep 17 00:00:00 2001
From: Dane Springmeyer <dane@dbsgeo.com>
Date: Fri, 6 Nov 2009 20:56:34 +0000
Subject: [PATCH] PointSymbolizer serialization was missing 'opacity' and
 'allow_overlap'

---
 CHANGELOG        |  2 ++
 src/save_map.cpp | 10 ++++++++++
 2 files changed, 12 insertions(+)

diff --git a/CHANGELOG b/CHANGELOG
index 6eb2aaf32..0e57400e4 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -15,6 +15,8 @@ For a complete change history, see the SVN log.
 Mapnik 0.6.2 Release
 --------------------
 
+- XML: added missing serialization of PointSymbolizer 'opacity' and 'allow_overlap' attributes (r1358)
+
 - PointDatasource: fixed problem with missing geometries (#402)
 
 - PostGIS: Add a 'persist_connection' option (default true), that when false will release 
diff --git a/src/save_map.cpp b/src/save_map.cpp
index 3f93cc1ca..176653297 100644
--- a/src/save_map.cpp
+++ b/src/save_map.cpp
@@ -54,6 +54,16 @@ namespace mapnik
                         ptree::value_type("PointSymbolizer", ptree()))->second;
 
                 add_image_attributes( sym_node, sym );
+
+                point_symbolizer dfl;
+                if (sym.get_allow_overlap() != dfl.get_allow_overlap() || explicit_defaults_ )
+                {
+                    set_attr( sym_node, "allow_overlap", sym.get_allow_overlap() );
+                }
+                if ( sym.get_opacity() != dfl.get_opacity() || explicit_defaults_ )
+                {
+                    set_attr( sym_node, "opacity", sym.get_opacity() );
+                }
             }
 
             void operator () ( const line_symbolizer & sym )