[PHP][TPL] przekazywanie zmiennych Opcje

[krystian1991x]

Mam problem z formularzem rejestracji a konkretnie taki ustawiłem sobie żeby sprawdzało czy wszystkie pola zostały wypełnione oraz czy pola z hasłem oraz pola z email są takie same niestety coś tutaj nie gra Bo gdy za pierwszym razem wypełnię formularz to plik się przeładuje ale nie wyskoczy żaden komunikat :/ za drugim razem za to pokazuje mi tak jak by błąd z poprzedniego wypełnienia czyli jak w poprzednim niezgodne było hasło to wtedy wyskakuje mi błąd że pola z hasłem nie są identyczne o co tutaj chodzi ? Podaje kod PHP

[PHP] pobierz, plaintext 
 $login    = mysql_escape_string($_POST['nick']);
 $password = mysql_escape_string($_POST['pw']);
 $rpw      = mysql_escape_string($_POST['rpw']);
 $email    = mysql_escape_string($_POST['email']);
 $remail   = mysql_escape_string($_POST['remail']);
 
 $login_use = mysql_query ("SELECT login FROM user WHERE login='$login'");
 $email_use = mysql_query ("SELECT email FROM user WHERE email='$email'");  
 
 if($_POST['register'] == 1) {
  if($login == "" && $password == "" && $rpw == "" && $email == "" && $remail == "") 
   {
     $error_register = "bad_null";  
   } else{
   if($pw == $rpw) {
 
     $error_register = "bad_pw"; 
                    } else {
 
            if($email != $remail) { $error_register = "bad_email"; 
            } else {
 
            $error_register = "yes";
 
            $zapytanie   = "INSERT INTO `user` (`id`, `login`, `password`, `email`, `avatar`) 
                            VALUES ('', '$login', '".md5($password)."', '$email', '')" ;
            $idzapytania = mysql_query($zapytanie);
                }         }
         }
     }
   $smarty->assign('index', $img);   
   $smarty->assign('error', $error_register);
   $smarty->display('register.tpl');
[PHP] pobierz, plaintext 

[sadistic_son]

Po pierwsze to w linijce 11 wszystkie AND pozamieniaj na OR. Czyli && na ||. Bo wystarczy abyś wpisał cokolwiek w którekolwiek pole i skrypt nie wychwyci że pozostałe pola są puste. A w linijce 15 powinno być raczej != zamiast == , nie sądzisz?

[krystian1991x]

mój błąd ale nadal nie wyświetla błędu za pierwszym razem gdy klikam wyślij tylko za drugim i to błąd z poprzedniego wysyłania ;/ konkretnie wpisuje np 2 odmienne adresy email klikam wyślij strona sie przeładuje zero komunikatu o błędzie oraz o tym czy wysłało wypełniam więc formularz jeszcze raz np tym razem daje niepoprawne pola z hasłem i co ? i wyświetla mi błąd z nieprawidłowym adresem email ;(

na tym forum każdy jest mądry do czasu ktoś mi powiedział coś mądrego że na takich forach ludzie którzy się na czymś znają nie mają czasu na takie rzeczy a ci którzy się najmniej znają pieprzą najwięcej

[krispak]

Sprawdz to czy jest ok..

[PHP] pobierz, plaintext 
if($_POST['register'] == 1)
{
	$login    = mysql_escape_string($_POST['nick']);
	$password = mysql_escape_string($_POST['pw']);
	$rpw      = mysql_escape_string($_POST['rpw']);
	$email    = mysql_escape_string($_POST['email']);
	$remail   = mysql_escape_string($_POST['remail']);
 
	$login_use = mysql_query("SELECT login, email FROM user WHERE login = '$login' OR email = '$email'"); // zamienilem na jedno zapytanie po co dwa?
 
	if($login == "" || $password == "" || $rpw == "" || $email == "" || $remail == "")
	{
		$error_register['bad_null'] = "bad_null";
		$blad = true;
	}
	if($pw != $rpw)
	{
		$error_register['bad_pw'] = "bad_pw"; 
		$blad = true;
        }
        if($email != $remail) 
	{
		$error_register['bad_email'] = "bad_email";
		$blad = true;
        } 
	if($blad)
	{
		$smarty->assign('post', $_POST); // wrzucasz te posty w value aby przy zle wypelnionym nie trzeba bylo wypelniac jeszcze raz wystarczy poprawic
		$smarty->assign('error', $error_register); //errory
	} else {
		$zapytanie   = "INSERT INTO `user` (`id`, `login`, `password`, `email`, `avatar`) VALUES ('', '$login', '".md5($password)."', '$email', '')";
		$idzapytania = mysql_query($zapytanie);
       }
}
 
$smarty->assign('index', $img);   
$smarty->display('register.tpl');
[PHP] pobierz, plaintext 

Bledy w register.tpl wywoluj tak {$error.bad_null}, {$error.bad_pw}, {$error.bad_email}
Mozesz tez dodac Post aby nie wypelniac ponownie w razie bledu, czyli w value ustaw tak jak masz w post np. gdzie wpisujesz nick ustaw value="{$post.nick}"
Pozdrawiam

[krystian1991x]

ja to robię jako wyskakujące okienka "alert" i chyba tak jak napisałeś się nie da bo zobacz jak ja to robię w TPL

[HTML] pobierz, plaintext 
 {if $error.bad_null == bad_null }
<script type="text/javascript">
{literal}
function show_orders()
{
alert("Wypelnij wszystkie pola !");
}
{/literal}
</script>
{/if}
[HTML] pobierz, plaintext 

i tak dalej i tak dalej próbowałem też zamiast stawiać warunku w treści okienka wpieprzyć zmienną ale cóż nie działało :<

[krispak]

Wg mnie zle to robisz. Sprawdzanie w js to jedno a sprawdzanie w php to drugie. Sprawdzanie w js musisz robic przed wyslaniem formularza czyli po kliknieciu zapisz wyskakuje komunikat "Wypelnij wszystkie pola" jezeli jakies sa wymagane a sa puste, ale formularz nie jest wysylany. Dopiero jak przejdzie walidacje w js ze wszystko wypelnione to wysyla..
Chyba ze te alerty wrzuc tutaj..

[PHP] pobierz, plaintext 
if($blad)
{
     echo '<script type="text/javascript">';
     foreach($error_register as $value)
     {
           echo 'alert(\''.$value.'\');';
      }
     echo '</script>';
} else {
     $zapytanie   = "INSERT INTO `user` (`id`, `login`, `password`, `email`, `avatar`) VALUES ('', '$login', '".md5($password)."', '$email', '')";
     $idzapytania = mysql_query($zapytanie);
}
[PHP] pobierz, plaintext 

Ten post edytował krispak 6.09.2010, 22:30:26

[krystian1991x]

zrobiłem to w ten sposób ale i tak nie działa HELP

[HTML] pobierz, plaintext 
<form action="register.php" onsubmit="return validate();" method="post">
 
<div style="padding-left: 100px;">
<table style="width: 377px; height: 220px; background-image: url('templates/images/54.png');">
 
<tr>
<td valign="top">
<table style="color: #FFFFFF; font-family: Tahoma; font-size: 11px; padding-left: 36px;padding-top: 40px;">
<tr><td valign="top" align="right" style="padding-top: 5px;"><b>Login:</b></td><td><input style="background-repeat: no-repeat; background-color: #64a424;padding-left: 4px;background-image: url('templates/images/52.png'); border: 0px; width: 165px; height: 22px;" name="nick" type="text" /></td></tr>
<tr><td valign="top" align="right" style="padding-top: 5px;"><b>Hasło:</b></td><td><input style="background-repeat: no-repeat; background-color: #64a424; padding-left: 4px; background-image: url('templates/images/52.png'); border: 0px; width: 165px; height: 22px;" name="pw" type="password" /></td></tr>
<tr><td valign="top" align="right" style="padding-top: 5px;"><b>Powtórz hasło:</b></td><td><input style="background-repeat: no-repeat; background-color: #64a424; padding-left: 4px; background-image: url('templates/images/52.png'); border: 0px; width: 165px; height: 22px;" name="rpw" type="password" /></td></tr>
<tr><td valign="top" align="right" style="padding-top: 5px;"><b>Adres e-mail:</b></td><td><input style="background-repeat: no-repeat; background-color: #64a424; padding-left: 4px; background-image: url('templates/images/52.png'); border: 0px; width: 165px; height: 22px;" name="email" type="text" /></td></tr>
<tr><td valign="top" align="right" style="padding-top: 5px;"><b>Powtórz adres:</b></td><td><input style="background-repeat: no-repeat; background-color: #64a424; padding-left: 4px; background-image: url('templates/images/52.png'); border: 0px; width: 165px; height: 22px;" name="remail" type="text" /></td></tr>
<tr><td><input type='hidden' name='register' value='1' /></td><td style="padding-left: 100px;" align="left"> <input type="image" src="templates/images/53.png"  value="Załóż konto" /></td></tr>
</table>
</td>
</tr>
</table>
 
 
</div>
</form>   
 <script type="text/javascript">{literal}
 function error(tekst) {
	if (errfound) return;
		window.alert(tekst);
errfound = true;
}
function validate() {
	errfound = false;
	var bledy = "";
	with (document.forms[0]) {
		if (nick.value == "")   bledy += "Nie podałeś imienia. \n";
		if (pw.value == "")	    bledy += "Nie podałeś nazwiska.\n";		
    if (rpw.value == "")    bledy += "Nie napisałeś treści";
		if (email.value == "")  bledy += "Nie podałeś nazwiska.\n";		
    if (remail.value == "") bledy += "Nie napisałeś treści";
		if (bledy != "")  error(bledy); 
	}
return !errfound;
}
 {/literal}</script>
[HTML] pobierz, plaintext 

pomocy ?

help ?

hej hej help me :E

nikt nie wie ? dziwne :<