Format/EncodedPolyline: Change default factor to 1e5

This matches the factors in the other methods and the default of the
original algorithm. Having this set at 1 before can be considered
as a bug.
This commit is contained in:
Tobias Bieniek
2013-05-18 15:24:57 +02:00
committed by Bart van den Eijnden
parent 4399ebcb04
commit d1b00824fc
2 changed files with 5 additions and 5 deletions
+2 -2
View File
@@ -103,7 +103,7 @@ OpenLayers.Format.EncodedPolyline = OpenLayers.Class(OpenLayers.Format, {
* coordinates. * coordinates.
*/ */
decode: function(encoded, dims, opt_factor) { decode: function(encoded, dims, opt_factor) {
var factor = opt_factor || 1; var factor = opt_factor || 1e5;
var flatPoints = this.decodeDeltas(encoded, dims, factor); var flatPoints = this.decodeDeltas(encoded, dims, factor);
var flatPointsLength = flatPoints.length; var flatPointsLength = flatPoints.length;
@@ -179,7 +179,7 @@ OpenLayers.Format.EncodedPolyline = OpenLayers.Class(OpenLayers.Format, {
* {String} An encoded string * {String} An encoded string
*/ */
encode: function (points, dims, opt_factor) { encode: function (points, dims, opt_factor) {
var factor = opt_factor || 1; var factor = opt_factor || 1e5;
var flatPoints = []; var flatPoints = [];
var pointsLength = points.length; var pointsLength = points.length;
+3 -3
View File
@@ -106,8 +106,8 @@
for (i in decodedPoints) { for (i in decodedPoints) {
var point = basePoints[i]; var point = basePoints[i];
var decodedPoint = decodedPoints[i]; var decodedPoint = decodedPoints[i];
t.eq(point[0], decodedPoint[0]); t.eq(parseInt(decodedPoint[0] * 1e5), point[0]);
t.eq(point[1], decodedPoint[1]); t.eq(parseInt(decodedPoint[1] * 1e5), point[1]);
} }
} }
@@ -141,7 +141,7 @@
var format = new OpenLayers.Format.EncodedPolyline(); var format = new OpenLayers.Format.EncodedPolyline();
t.eq(format.encode(basePoints, 2), encoded); t.eq(format.encode(basePoints, 2, 1), encoded);
} }
function test_encodeDeltas_returns_expected_value(t) { function test_encodeDeltas_returns_expected_value(t) {