Changeset 2899

Timestamp:

03/26/07 16:55:32 (2 years ago)

Author:

sderle

Message:

Patch from fredj adds toString() method to Geometry.Linearring. Fixes #565. Thanks, Fred.

Files:

• trunk/openlayers/lib/OpenLayers/BaseTypes.js (modified) (1 diff)
• trunk/openlayers/lib/OpenLayers/Geometry/LinearRing.js (modified) (1 diff)
• trunk/openlayers/tests/Geometry/test_LinearRing.html (modified) (1 diff)

trunk/openlayers/lib/OpenLayers/BaseTypes.js

@@ -21 +21 @@
-            if (typeof arguments[i] == "function") {
-                var mixin = arguments[i];
-new mixin(OpenLayers.Class.isPrototype)
+mixin.prototype
-            }
-            OpenLayers.Util.extend(proto, arguments[i]);

trunk/openlayers/lib/OpenLayers/Geometry/LinearRing.js

@@ -109 +109 @@
-    },
-
+
+    /**
+     * @returns The coordinates components as a string
+     * @type String
+     */
+    toString: function() {
+        return this.components.toString();
+    },
+
-    /** @final @type String */
-    CLASS_NAME: "OpenLayers.Geometry.LinearRing"

trunk/openlayers/tests/Geometry/test_LinearRing.html

@@ -91 +91 @@
-    }
-
+    function test_05_LinearRing_toString(t) {
+        t.plan(1);
+        
+        ring = new OpenLayers.Geometry.LinearRing(components);
+        t.eq( ring.toString(), components.toString() + ',' + components[0].toString(), "toString output is ok");
+    }
+
-  // -->
-  </script>