Changeset 5972

Timestamp:

02/02/08 11:32:19 (1 year ago)

Author:

tcoulter

Message:

If you don't move the mouse, and get to a point where the code hits finalize() or cancel(), the handler will pass a null point to the renderer. Fixed this to suppress errors.

Files:

• sandbox/topp/nymap/lib/OpenLayers/Handler/Point.js (modified) (3 diffs)

sandbox/topp/nymap/lib/OpenLayers/Handler/Point.js

@@ -112 +112 @@
-            this.map.addLayer(this.layer);
-        }
+        
+        this.createFeature();
-
-        return true;
@@ -160 +162 @@
-     */
-    finalize: function() {
+        this.cleanup("done");
+    },
+
+    /**
+     * APIMethod: cancel
+     * Finish the geometry and call the "cancel" callback.
+     */
+    cancel: function() {
+        this.cleanup("cancel");
+    },
+    
+    cleanup: function(callback) {
-        this.layer.eraseFeatures(this.point);
-        this.mouseDown = false;
-        this.lastDown = null;
-        this.lastUp = null;
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
-        this.destroyFeature();
-    },
@@ -258 +259 @@
-     */
-    mousemove: function (evt) {
-        if (this.point == null) {
-            this.createFeature();
-        }
-        
-        var lonlat = this.map.getLonLatFromPixel(evt.xy);
-        this.point.geometry.x = lonlat.lon;