Ticket #1548: 1548_r1790_A0.patch

tests/Rule.html

@@ -13 +13 @@
-        t.eq(rule.foo, "bar", "constructor sets options correctly"); 
-        t.eq(typeof rule.evaluate, "function", "rule has an evaluate function"); 
-    }
+    
+    function test_Rule_getContext(t) { 
+        t.plan(2);
+        var rule, options;
+        
+        var feature = {
+            attributes: {
+                'dude': 'hello'
+            },
+            'foobar': 'world'
+        }
+        
+        rule = new OpenLayers.Rule();
+        var context = rule.getContext(feature);
+        t.eq(context.dude, "hello", "value returned by getContext is correct"
+            + " if no context is specified"); 
+        
+        var options = {
+            context: function(feature){
+                return feature;
+            }
+        };
+        rule = new OpenLayers.Rule(options);
+        var context = rule.getContext(feature);
+        t.eq(context.foobar, "world", "value returned by getContext is correct"
+            + " if a context is given in constructor options"); 
+    }
-
-    function test_Rule_destroy(t) {
-        t.plan(1);

lib/OpenLayers/StyleMap.js

@@ -130 +130 @@
-     *                rules for
-     * symbolizers  - {Object} Hash of symbolizers, keyed by the desired
-     *                property values 
+     * context      - {Object} An optional object with properties that
+     *                symbolizers' property values should be evaluated
+     *                against. If no context is specified, feature.attributes
+     *                will be used
-     */
-
+, context
-        var rules = [];
-        for (var value in symbolizers) {
-            rules.push(new OpenLayers.Rule({
-                symbolizer: symbolizers[value],
+                context: context,
-                filter: new OpenLayers.Filter.Comparison({
-                    type: OpenLayers.Filter.Comparison.EQUAL_TO,
-                    property: property,

lib/OpenLayers/Rule.js

@@ -173 +173 @@
-        if (!context) {
-            context = feature.attributes || feature.data;
-        }
+        if (typeof this.context == "function") {
+            context = this.context(feature);
+        }
-        return context;
-    },
-        

examples/styles-unique.html

@@ -47 +47 @@
-            
-            layer.addFeatures(features);
-            map.addLayer(layer);
+            
+            // create 20 random features with a random state property.
+            var features = new Array(20);
+            var states = [
+                OpenLayers.State.UNKNOWN,
+                OpenLayers.State.UPDATE,
+                OpenLayers.State.DELETE,
+                OpenLayers.State.INSERT
+            ];
+            for (var i=0; i<20; i++) {
+                features[i] = new OpenLayers.Feature.Vector(
+                    new OpenLayers.Geometry.Point(Math.random()*360-180, Math.random()*180-90)
+                );
+                features[i].state = states[parseInt(Math.random()*4)];
+            }
+            
+            var context = function(feature) {
+                return feature;
+            }
+            var styleMap = new OpenLayers.StyleMap();
+            
+            // create a lookup table with different symbolizers for the diffent
+            // state values
+            var lookup = {
+                "Unknown": {fillColor: "green"},
+                "Update": {fillColor: "blue"},
+                "Delete": {fillColor: "red"},
+                "Insert": {fillColor: "orange"}
+            }
+            
+            styleMap.addUniqueValueRules("default", "state", lookup, context);
+            layer = new OpenLayers.Layer.Vector('Points', {
+                styleMap: styleMap
+            });
+            
+            layer.addFeatures(features);
+            map.addLayer(layer);
-        }
-    </script>
-  </head>
@@ -56 +93 @@
-    <div id="tags"></div>
-
-    <p id="shortdesc">
-
+
+
+
+
+
-    </p>
-
-    <div id="map" class="smallmap"></div>