Walidacja pól Opcje

[Largo]

Witam,

Mam oto taki skrypcik:

[HTML] pobierz, plaintext 
<script type='text/javascript'>
function checkFields()
{
	var requiredFields = new Array ( 'game_name', 'game_pass', 'battle_acc', 'reason', 'tid' );
	var errors = "";
 
	for ( var i = 0; i < requiredFields.length; i++ )
	{
		if ( requiredFields[i] == "" )
		{
			errors += '{$this->ipsclass->lang['required_fields']}';
		}
	}
 
	if ( errors != "" )
	{
		alert( errors );
		return false;
	}
	return true;
}
</script>
[HTML] pobierz, plaintext 

Ma on teoretycznie bardzo proste. Ma zdefiniowane pola "name", które są wymagane. Niestety, nie działa jak powinien, nic nie robi, FF nie wyrzuca błędów, nic... Jak to poprawić?

[nospor]

if ( requiredFields[i] == "" )

mala wskazowka:
przeciez pod requiredFields[i] masz tylko nazwe pola a nie pole (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[Largo]

Witaj,

Fakt, sprawdziłem i masz rację. Poprawiłem to tak, ale dalej nie działa:

[HTML] pobierz, plaintext 
<script type='text/javascript'>
function checkFields(form)
{
var requiredFields = new Array ( 'game_name', 'game_pass', 'battle_acc', 'reason', 'tid' );
var errors = "";
 
for ( var i = 0; i < requiredFields.length; i++ )
{
if ( form.getElementsByName(requiredFields[i]).value == "" )
{
errors += '{$this->ipsclass->lang['required_fields']}';
}
}
 
if ( errors != "" )
{
alert( errors );
return false;
}
return true;
}
</script>
[HTML] pobierz, plaintext 

[nospor]

nie trzymaj tam nazwy tylko id.
poźniej do pola dobieraj sie poprzez document.getElementById()
no i dla pol nadaj jeszcze to id

[Largo]

Witaj,

Dziękuje za pomysł. Wszystko dobrze mi teraz działa :-) Ale dlaczego ja o ID nie pomyślałem? :-) Chyba nauka JS mi się od samych podstaw kłania... Mimo to dziękuję.

Pozdrawiam,
Largo

PS. Mam kolejny, ale wyjątkowo dziwny problem! W pętli przejeżdzam po błędach i mam taki kod:

Kod

            alert ( 'Zmienna nazywa się: ' + requiredFieldsErrors[i] );
            errors += "{$this->ipsclass->lang['" + requiredFieldsErrors[i] + "']}\n";

Pierwsza linijka wykonuje się poprawnie ( Pokazuje: Zmienna nazywa się: XX ) i tak 4 razy i poprawne dane daje, a druga część nie. Tutaj chodzi o to, że w $this->ipsclass->lang[ ' ' ] Przekazuje się nazwę zmiennej ( 'no_game' ), tamten skrypt to pokazuje, a ten nie? Dlaczego?

Ten post edytował Largo 2.04.2009, 15:40:19

[nospor]

co to jest:
{$this->ipsclass->lang
?