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

git-svn-id: http://svn.openlayers.org/trunk/openlayers@2899 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf

lib/OpenLayers/BaseTypes.js

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

lib/OpenLayers/Geometry/LinearRing.js

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

tests/Geometry/test_LinearRing.html

@@ -90,6 +90,13 @@
         t.eq(ring.getArea(), 100, "getArea works lovely");
     }
 
+    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>
 </head>