[SOLUCIONADO] onclick sencillo no funciona

[juanvc123]

hola, tengo un problema con este onclick que a mi parecer es demasiado sencillo pero por un error desconocido no me funciona!
Aqui esta el input que donde al dar click me llame la función: (este input esta dentro de un form que no tiene action porque no quiero refrescar al pag)

Código:

<input type="button" value="Subir" id="subir" name="subir" onclick="subir()" />

este es el campo text alque refiere la función:

Código:

<input type="text" id="campocedula" class="inputtext"  />

y esta es la función:

Código:

function subir(){
  if(document.getElementById("campocedula").value == "")
  {
     alert('Porfavor digita la cedula');
  }
  else
  {
    alert('llenaste bien');
  }
}

pues no me funciona, espero alguien pueda orientarme hacia la solución.

[chwc]

mira, podes agregarle esto a tu formulario

<form method='post' onsubmit="return subir()">

y que cuando se envie con un input de tipo submit, podes controlar el envio con el retorno de la función
ej:

Código Javascript:

function subir(){
  if(document.getElementById("campocedula").value == "")
  {
     alert('Porfavor digita la cedula');
return false; // no se envia el formulario
  }
  else
  {
   return true; // se envia el formulario normalmente
  }
}
 

un saludo !

[emprear]

@juanvc123
Buen descubrimiento has hecho.
En mis muchos años de hacer forms nunca me habia pasado, Tu función no se ejecuta porque tiene el mismo nombre del id ó name.
cambia el y/o name id ó el nombre de la función.

Saludos

[juanvc123]

Iniciado por emprear

@juanvc123
Buen descubrimiento has hecho.
En mis muchos años de hacer forms nunca me habia pasado, Tu función no se ejecuta porque tiene el mismo nombre del id ó name.
cambia el y/o name id ó el nombre de la función.

Saludos

Es cierto! ese era el problema! gracias por responder!

[juanvc123]

Iniciado por chwc

mira, podes agregarle esto a tu formulario


y que cuando se envie con un input de tipo submit, podes controlar el envio con el retorno de la función
ej:

Código Javascript:

function subir(){
  if(document.getElementById("campocedula").value == "")
  {
     alert('Porfavor digita la cedula');
return false; // no se envia el formulario
  }
  else
  {
   return true; // se envia el formulario normalmente
  }
}
 

un saludo !

Gracias por responder, seria otro metodo.