Problema con este codigo

[thi]

Hola a todos,

Este codigo me da error si le doy a "Cancelar" en cambio si le doy a Aceptar me lo hace correctamente. No es que me muestre un error sino lo que pasa es que si le doy a "Cancelar" no puedo volver a llamar a la funcion porque no me hace nada si no refresco la ventana, si la refresco entonces ya esta correcto. Así que entiendo que en el else debería poner algo, pero no se el que porque no quiero que haga nada, por cierto tambien hice pruebas sin el else y me pasa lo mismo.

<script language="JavaScript">
var eliminar;
var ventana_eliminar;
function eliminar(codigo_ofer)
{
eliminar = confirm('¿Deseas eliminar esta oferta?');
if(eliminar == true)
{
ventana_eliminar=window.open(codigo_ofer,'name','w idth=0,height=0,left=150,top=10');
alert('La oferta ha sido eliminada correctamente.');
}
else
{

}
}
</script>

[David]

Si no pretendes realizar ninguna acción no es necesario el else. De todos modos, habría que ver dónde llamas a la función para saber si por ahí está el problemna.

[zerokilled]

independientemente de la repuesta, OK o CANCEL, tienes conflicto de identificadores. la funcion puede operar solamente una vez. tienes la declaracion de una funcion que se llama eliminar y en el cuerpo de la funcion utilizas la "variable global" eliminar. aunque parecen entidades diferentes las funciones se podrian considerar tambien como variables. por tanto, en tu codigo sucede que estas sobreescribiendo la funcion eliminar con un boolean. para solucionarlo, puedes preceder el nombre de la variable en el cuerpo de la funcion con var para hacer que esta variable sea local a la funcion.

Código:

var eliminar;
var ventana_eliminar;
function eliminar(codigo_ofer)
{
var eliminar = confirm('¿Deseas eliminar esta oferta?');
if(eliminar == true)
{
ventana_eliminar=window.open(codigo_ofer,'name','w idth=0,height=0,left=150,top=10');
alert('La oferta ha sido eliminada correctamente.');
}
else
{

}
}

[thi]

Gracias por vuestras respuestas,

era un conflicto de nombres.

Un saludo.