Revision: 33
Author: cphillip
Date: 2006-03-31 11:21:53 -0800 (Fri, 31 Mar 2006)
ViewCVS: http://svn.sourceforge.net/wnmap/?rev=33&view=rev
Log Message:
-----------
Fix a bug with renaming that would leave markers[] improperly structured.
* web/js/nodemap.js:
(renameMarker): Be sure to base64 decode the old node name to use in indexing
of markers and to delete that old location so it no longer
shows up in for (i in foo) constructs.
Modified Paths:
--------------
trunk/src/web/js/nodemap.js
Modified: trunk/src/web/js/nodemap.js
===================================================================
--- trunk/src/web/js/nodemap.js 2006-03-21 05:52:55 UTC (rev 32)
+++ trunk/src/web/js/nodemap.js 2006-03-31 19:21:53 UTC (rev 33)
@@ -241,24 +241,25 @@
return prompt ('Enter a new name for this node:', name);
}
-function renameMarker (oldName, newName)
+function renameMarker (oldB64Name, newName)
{
+ var oldName = decode64 (oldB64Name);
+
if ( markers[newName] != null ) {
alert("A marker named '" + newName + "' already exists!");
return 0;
}
- var marker = getMarker (oldName);
+ var marker = markers [oldName];
marker.name = newName;
markers [newName] = marker;
markers [oldName] = null;
+ delete markers[oldName];
populateMap ();
}
-
-
// some helper functions
// based on code from
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.
|
