PointSymbolizer serialization was missing 'opacity' and 'allow_overlap'

mapnik · Nov 6, 2009 · 4df684c · 4df684c
1 parent d1f1f69
commit 4df684c
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git 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
@@ -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 )