From 2e69d681022a86cf7733ff4697ee2a24fbf18266 Mon Sep 17 00:00:00 2001
From: Bart van den Eijnden <bartvde@opengeo.org>
Date: Thu, 23 Feb 2012 15:55:40 +0100
Subject: [PATCH] map fill-opacity in TextSymbolizer to fontOpacity

---
 lib/OpenLayers/Format/SLD/v1.js        | 16 +++++++++++-----
 tests/Format/SLD/v1_0_0_GeoServer.html |  4 +++-
 2 files changed, 14 insertions(+), 6 deletions(-)

diff --git a/lib/OpenLayers/Format/SLD/v1.js b/lib/OpenLayers/Format/SLD/v1.js
index e23a14d995..a6fe969c4a 100644
--- a/lib/OpenLayers/Format/SLD/v1.js
+++ b/lib/OpenLayers/Format/SLD/v1.js
@@ -338,9 +338,14 @@ OpenLayers.Format.SLD.v1 = OpenLayers.Class(OpenLayers.Format.Filter.v1_0_0, {
             "CssParameter": function(node, symbolizer) {
                 var cssProperty = node.getAttribute("name");
                 var symProperty = this.cssMap[cssProperty];
-                // for labels, fill should map to the fontColor
-                if (symbolizer.label && cssProperty === 'fill') {
-                    symProperty = "fontColor";
+                // for labels, fill should map to fontColor and fill-opacity 
+                // to fontOpacity
+                if (symbolizer.label) {
+                    if (cssProperty === 'fill') {
+                        symProperty = "fontColor";
+                    } else if (cssProperty === 'fill-opacity') {
+                        symProperty = "fontOpacity";
+                    }
                 }
                 if(symProperty) {
                     // Limited support for parsing of OGC expressions
@@ -893,9 +898,10 @@ OpenLayers.Format.SLD.v1 = OpenLayers.Class(OpenLayers.Format.Filter.v1_0_0, {
                 }
                 // add in optional Fill
                 if(symbolizer.fontColor != null ||
-                   symbolizer.fillOpacity != null) {
+                   symbolizer.fontOpacity != null) {
                     this.writeNode("Fill", {
-                        fillColor: symbolizer.fontColor
+                        fillColor: symbolizer.fontColor,
+                        fillOpacity: symbolizer.fontOpacity
                     }, node);
                 }
                 return node;
diff --git a/tests/Format/SLD/v1_0_0_GeoServer.html b/tests/Format/SLD/v1_0_0_GeoServer.html
index c396ec61ce..bfe9b3ec4a 100644
--- a/tests/Format/SLD/v1_0_0_GeoServer.html
+++ b/tests/Format/SLD/v1_0_0_GeoServer.html
@@ -71,12 +71,14 @@
             </Font>
             <Fill>
               <CssParameter name="fill">#000000</CssParameter>
+              <CssParameter name="fill-opacity">0.5</CssParameter>
             </Fill>
             <Graphic>
               <Mark>
                 <WellKnownName>square</WellKnownName>
                   <Fill>
-                    <CssParameter name="fill">#59BF34</CssParameter> 
+                    <CssParameter name="fill">#59BF34</CssParameter>
+                    <CssParameter name="fill-opacity">0.8</CssParameter>
                   </Fill>
                   <Stroke>
                     <CssParameter name="stroke">#2D6917</CssParameter>