script funzionante in parte

[Bartjsp]

Ho realizzato questo script per verificare se i campi login e password sono stati inseriti, ma non funziona:

<script type="text/javascript">
<!--
	function ctrlAutenticazione()
	{
		var f = document.log;
		var login = f.login.value;
		var pswd = f.password.value;

		if((login == "") || (login == "undefined"))
		{
			alert("Inserisci il nome utente.");
			login.focus();
			return false;
		} else if ((pswd == "") || (pswd == "undefined")) {
			alert("Inserisci la password.");
			pswd.focus();
			return false;
		}
	}
//-->
</script>

<form onSubmit="return ctrlAutenticazione();" action="checkLogin.php" method="post" name="log">
  Utente <input class="box_login" type="text" name="login">
  Password <input class="box_login" type="password" name="password">
  <input type="image" src="login.gif" name="log">
  <input type="image" src="exit.gif" name="annulla" onClick="window.close();">
</form>

se non inserisco nulla e clicco subito sul pulsante login, javascript risponde che non è stato inserita un login e fin qui tutto ok ma subito dopo mi reindirizza alla pagina dell'action. :confuso:

Ho controllato un altro script sempre per il controllo dei campi e tutto funziona, non capisco dove sia qui l'errore, sto uscendo pazzo.

[Xinod]

l' errore e' qui, in entrambi i casi cerchi di dare un focus a un value

var pswd = f.password.value;
...
pswd.focus();

se invece facessi

var pswd = f.password;
...
} else if ((pswd.value == "") || (pswd.value == "undefined")) {
...
pswd.focus();
return false;

dovrebbe funzionare
ciao

[Bartjsp]

Non è cambiato nulla, tral'altro tra il codice scritto da me e quello tuo non c'è molta differenza, cambia solo la posizione della proprietà.