problemas con document.getElementById

[OverCrash]

Buenas!!

Tengo dos selects múltiples donde el primero tiene un cierto numero de campos y que al pulsar sobre el botón de añadir, se pasa el valor seleccionado al segundo select, a la vez que también una opción de mover (eliminar) los campos del segundo select y colocarlos de nuevo en el primero.

el código de la función es este:

Código:

function AddToSecondList()
{
	var fl = document.getElementById("firstlist");
	var sl = document.getElementById("secondlist");    
	for (i = 0; i < fl.options.length; i++)
	{
		if(fl.options[i].selected)
		{
			sl.add(fl.options[i],null);
			
		}
	}
	return true;
}

function DeleteSecondListItem()
{
	var fl = document.getElementById("firstlist");	
	var sl = document.getElementById("secondlist");    
	for (i = 0; i < sl.options.length; i++)
	{
		if(sl.options[i].selected)
		{
			fl.add(sl.options[i],null);
		}
	}
	return true;
}

...
...
...

<input type="button" value="afegir" onclick="AddToSecondList();" />
<input type="button" value="eliminar" onclick="DeleteSecondListItem();" />

Esto me funciona bien en iExplorer 7 y 8, y en Firefox, pero en iexplorer 6, me da un error al presionar sobre el botón de añadir/quitar de "Tipo Incorrecto"....

Sabéis donde puede estar el error??

Gracias

[KiraSakuya]

Esto de los navegadores es una locura, mira a ver si poniendo en vez de

fl.add(sl.options[i],null);

que tienes dentro del if, te funciona con esto otro

fl.options.add(sl.options[i],null);

[OverCrash]

noooooo :((

haciendo estas modificaciones que me comentas, no funciona en ningun navagador, retornandome un mensaje de error "no coinciden los tipos"

[KiraSakuya]

Perdona, es que a estas horas no estoy muy atento, par lo que te dije primero debes crear la opcion

var nuevaOpcion = new Option("Texto","Valor");
document.getElementById("tuComboBox").options.add( nuevaOpcion);

donde texto y valor será lo que recojas del otro combo, a ver si te funciona :)

[OverCrash]

no te acabo de entender... me lo podrias explicar mas detalladamente? vaaaaa que ya son las 12h!!! jejeje

[KiraSakuya]

jaja claro
a ver tu cojes un elemento del combobox1 y lo quieres introducir en el 2, para ello, debes crear una nueva variable de tipo option y para ello utilizas la sentencia siguiente:

var nuevaOpcion(text,value) en tu caso text seria igual a sl.options[i], y el valor seria igual a null tal y como lo indicas.

Una vez creado el objeto se le añade al combobox donde deseas almacenarlo y para ello está lo que en un primer momento te dije:

document.getElementById(tuSelect).options.add(nuev aOpcion);

Donde tuSelect será el objeto de tipo select (supongo que sera una lista) donde quieres añadir el nuevo objeto de tipo option y nuevaOpcion será la opcion que has definido anteriormente, en este caso la dejo con el mismo nombre para ver si asi lo ves mejor :)

No se si me he expresado bien ahora, si eso no tengo problema en seguir intentandolo

[OverCrash]

Vale!! hemos conseguido algo....
para que al traspasar los campos del select 1 al select 2 se eliminen del primero (y vicerversa), como?
y para que los valores del segundo select queden "TODOS" seleccionados?

el codigo ahora esta asi:

Código:

if(fl.options[i].selected){
var nuevaOpcion = new Option(fl.options[i].text,fl.options[i].value); 
document.getElementById("secondlist").options.add(nuevaOpcion);	
}

[KiraSakuya]

A ver, comencemos por lo sencillo, si quieres ir añadiendo elementos y a su vez estos queden seleccionados lo que debes hacer es añadir un par de parametros donde pones (y digo un par porque dejas uno en blanco, que no se te olvide :D ):


var nuevaOpcion = new Option(fl.options[i].text,fl.options[i].value);


De tal forma que quede asi:


var nuevaOpcion = new Option(fl.options[i].text,fl.options[i].value,"","selected");


En cuanto a borra un elemento de un combobox se hace igualando la opción correspondiente a null, de tal forma que quedaria asi:


document.getElementById("secondlist").options[indiceSeleccionado]=null;


Esto ultimo iria en el bucle e indice seleccionado sería el mismo que el del option del que tomas los valores.

Si no entiendes algo ya sabes :) solo tienes que preguntar :D

[OverCrash]

Eres el mejor!!! 100% funcionando.

Os dejo el codigo de como ha quedado, por si a alguno le puede ser útil.

Código:

function AddToSecondList(){
	var fl = document.getElementById("firstlist");
	var sl = document.getElementById("secondlist");    
	for (i = 0; i < fl.options.length; i++){
		if(fl.options[i].selected){
			var nuevaOpcion = new Option(fl.options[i].text,fl.options[i].value,null,"selected"); 
			document.getElementById("secondlist").options.add(nuevaOpcion);	
			document.getElementById("firstlist").options[i] = null;
		}
	}
	return true;
}

function DeleteSecondListItem(){
	var fl = document.getElementById("firstlist");	
	var sl = document.getElementById("secondlist");    
	for (i = 0; i < sl.options.length; i++){
		if(sl.options[i].selected){
			var nuevaOpcion = new Option(sl.options[i].text,fsl.options[i].value,null,"selected"); 
			document.getElementById("firstlist").options.add(nuevaOpcion);	
			document.getElementById("secondlist").options[i] = null;
		}
	}
	return true;
}

Saludos y muchas gracias!!!

[KiraSakuya]

Me alegra que siendo el primer usuario al que ayudo todo haya salido bien :D espero que pueda ayudar a mucha mas gente ;)

Un saludo :)