From dbdf9a3df8605687682bf200ae719711f898918d Mon Sep 17 00:00:00 2001 From: Schuyler Erle Date: Fri, 9 Mar 2007 21:54:41 +0000 Subject: [PATCH] Switch to sequential ID numbering. Fixes #226. git-svn-id: http://svn.openlayers.org/trunk/openlayers@2645 dc9f47b5-9b13-0410-9fdd-eb0c1a62fdaf --- lib/OpenLayers/Util.js | 5 ++++- tests/test_Util.html | 8 +++++++- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/lib/OpenLayers/Util.js b/lib/OpenLayers/Util.js index 43f034ef8c..630fc90572 100644 --- a/lib/OpenLayers/Util.js +++ b/lib/OpenLayers/Util.js @@ -622,6 +622,8 @@ OpenLayers.Util.getArgs = function(url) { return args; } +OpenLayers.Util.lastSeqID = 0; + /** * @param {String} prefix String to prefix random id. If null, default * is "id_" @@ -633,7 +635,8 @@ OpenLayers.Util.createUniqueID = function(prefix) { if (prefix == null) { prefix = "id_"; } - return prefix + Math.round(Math.random() * 10000); + OpenLayers.Util.lastSeqID += 1; + return prefix + OpenLayers.Util.lastSeqID; }; /** Constant inches per unit diff --git a/tests/test_Util.html b/tests/test_Util.html index d8b0f650fb..907ee3e104 100644 --- a/tests/test_Util.html +++ b/tests/test_Util.html @@ -563,7 +563,13 @@ } - + function test_Util_createUniqueIDSeq(t) { + t.plan(1); + OpenLayers.Util.lastSeqID = 0; + OpenLayers.Util.createDiv(); + OpenLayers.Util.createDiv(); + t.eq(OpenLayers.Util.createDiv().id, "OpenLayersDiv3", "Div created is sequential, starting at lastSeqID in Util."); + } // -->