ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #256 from bartvde/vendoroptions

Browse Source 
make vendorOptions an object instead of an array (r=@ahocevar)
This commit is contained in:
Bart van den Eijnden
2012-02-29 02:11:41 -08:00
parent 557d7ab0d7 b699afa57f
commit 2955302650
1 changed files with 7 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/OpenLayers/Format/SLD/v1_0_0_GeoServer.js
+7 -7
View File
@@ -58,12 +58,9 @@ OpenLayers.Format.SLD.v1_0_0_GeoServer = OpenLayers.Class(
}, },
"VendorOption": function(node, obj) { "VendorOption": function(node, obj) {
if (!obj.vendorOptions) { if (!obj.vendorOptions) {
obj.vendorOptions = []; obj.vendorOptions = {};
} }
obj.vendorOptions.push({ obj.vendorOptions[node.getAttribute("name")] = this.getChildValue(node);
name: node.getAttribute("name"),
value: this.getChildValue(node)
});
} }
}, OpenLayers.Format.SLD.v1_0_0.prototype.readers["sld"]) }, OpenLayers.Format.SLD.v1_0_0.prototype.readers["sld"])
}, OpenLayers.Format.SLD.v1_0_0.prototype.readers), }, 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) { addVendorOptions: function(node, symbolizer) {
var options = symbolizer.vendorOptions; var options = symbolizer.vendorOptions;
if (options) { if (options) {
for (var i=0, ii=options.length; i<ii; ++i) { for (var key in symbolizer.vendorOptions) {
this.writeNode("VendorOption", options[i], node); this.writeNode("VendorOption", {
name: key,
value: symbolizer.vendorOptions[key]
}, node);
} }
} }
return node; return node;