Changeset 7216 for trunk/openlayers

Timestamp:

05/19/08 09:34:35 (3 months ago)

Author:

pgiraud

Message:

context can now be given as argument in StyleMap.addUniqueValueRules()
this, for example, allows user to access to properties that are not part of the feature attributes.
r=elemoine (closes #1548)

Files:

• trunk/openlayers/examples/styles-unique.html (modified) (2 diffs)
• trunk/openlayers/lib/OpenLayers/Rule.js (modified) (1 diff)
• trunk/openlayers/lib/OpenLayers/StyleMap.js (modified) (1 diff)
• trunk/openlayers/tests/Rule.html (modified) (1 diff)

trunk/openlayers/examples/styles-unique.html

@@ -48 +48 @@
-            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 = {};
+            lookup[OpenLayers.State.UNKNOWN] = {fillColor: "green"};
+            lookup[OpenLayers.State.UPDATE] = {fillColor: "green"};
+            lookup[OpenLayers.State.DELETE] = {fillColor: "red"};
+            lookup[OpenLayers.State.INSERT] = {fillColor: "orange"};
+
+            styleMap.addUniqueValueRules("default", "state", lookup, context);
+            layer = new OpenLayers.Layer.Vector('Points', {
+                styleMap: styleMap
+            });
+            
+            layer.addFeatures(features);
+            map.addLayer(layer);
-        }
-    </script>
@@ -57 +93 @@
-
-    <p id="shortdesc">
-
+
+
+
+
+
-    </p>
-

trunk/openlayers/lib/OpenLayers/Rule.js

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

trunk/openlayers/lib/OpenLayers/StyleMap.js

@@ -131 +131 @@
-     * 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,

trunk/openlayers/tests/Rule.html

@@ -14 +14 @@
-        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) {