Stan

Stan,

I think I see the cause of your problems.

removelink.onclick=function(){remove(parname,removelink.id);}

This is an event handler function. It does not have access to these
variables (parname and removelink). The only reason it does anything at all
is because these variables are global, so it always picks up the last one
that was stored, hence your problem. You need to ensure the function always
picks us the current ones that you are working on. To do that, use the
'new Function()' constructor instead of the inline function, and write the
string names in as strings (remove any linebreaks):
removelink.onclick=new Function('remove(\''+parname+'\',\''+removelink.id+'\');}
I have documented the use of this constructor on:
http://www.howtocreate.co.uk/tutorials/javascript/functions

This assumes that you don't put ' or \ or linebreaks in the strings. If you
do, you will need to convert them into valid strings, and that will get a
lot messier. So much so that I have built a utility to convert characters
like these into valid strings:
http://www.howtocreate.co.uk/tutorials/jsexamples/syntax/prepareInline.html

I am assuming that since this is for yourself, you will be able to remember
this and not put these characters in the strings.

A couple of other points:

removelink.setAttribute('ID',fivalue);
use 'id', not 'ID' - this should not make much difference it this current
code, but it is the correct way to do it :)

addedfield = document.createTextNode(fivalue+' \n');
the linebreak will not have any effect. remember, this is a HTML text node,
inside a paragraph. in HTML, all whitespace is displayed as a space. If you
want this to appear as a linebreak, either use <pre> instead of <p>, or
append a <br> onto the end of each (you will need to create a new 'br'
element node).


Hope this helps solve your problem

Mark 'Tarquin' Wilton-Jones - author of http://www.howtocreate.co.uk/