From b699afa57f7b9a6e03815558ac89f64922719186 Mon Sep 17 00:00:00 2001
From: Bart van den Eijnden <bartvde@opengeo.org>
Date: Wed, 29 Feb 2012 10:57:08 +0100
Subject: [PATCH] make vendorOptions an object instead of an array, which makes
 it more convenient to prevent duplicate vendorOptions

---
 lib/OpenLayers/Format/SLD/v1_0_0_GeoServer.js | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/lib/OpenLayers/Format/SLD/v1_0_0_GeoServer.js b/lib/OpenLayers/Format/SLD/v1_0_0_GeoServer.js
index 49596be1c4..bee661311b 100644
--- a/lib/OpenLayers/Format/SLD/v1_0_0_GeoServer.js
+++ b/lib/OpenLayers/Format/SLD/v1_0_0_GeoServer.js
@@ -58,12 +58,9 @@ OpenLayers.Format.SLD.v1_0_0_GeoServer = OpenLayers.Class(
             },
             "VendorOption": function(node, obj) {
                 if (!obj.vendorOptions) {
-                    obj.vendorOptions = [];
+                    obj.vendorOptions = {};
                 }
-                obj.vendorOptions.push({
-                    name: node.getAttribute("name"),
-                    value: this.getChildValue(node)
-                });
+                obj.vendorOptions[node.getAttribute("name")] = this.getChildValue(node);
             }
         }, OpenLayers.Format.SLD.v1_0_0.prototype.readers["sld"])
     }, OpenLayers.Format.SLD.v1_0_0.prototype.readers),
@@ -130,8 +127,11 @@ OpenLayers.Format.SLD.v1_0_0_GeoServer = OpenLayers.Class(
     addVendorOptions: function(node, symbolizer) {
         var options = symbolizer.vendorOptions;
         if (options) {
-            for (var i=0, ii=options.length; i<ii; ++i) {
-                this.writeNode("VendorOption", options[i], node);
+            for (var key in symbolizer.vendorOptions) {
+                this.writeNode("VendorOption", {
+                    name: key, 
+                    value: symbolizer.vendorOptions[key]
+                }, node);
             }
         }
         return node;