Forum PHP.pl

Witam,
Mam taki skrypt do logowania, który sprawdza czy minął okres ważności hasła, jeśli tak, to wywołana zostaje zmiana tego hasła, a następnie ma zostać odświeżona strona z dostępnym dla danego konta menu.
Na lokalnym serwerze mi to działa. Niestety, gdy wrzucam na serwer produkcyjny to już nie.
Wyświetla mi się informacja o poprawnej zmianie hasła, ale strona nie zostaje odświeżona.
Czy jest tu jakiś problem?

[PHP] pobierz, plaintext 
 
<?php
if (isset($_SESSION['login']))  {
 $prac = mysql_query("SELECT `id`, `pesel`, `herb`,  TO_DAYS(`data_konta`) - TO_DAYS(NOW()) AS `zmiana` FROM `dawca_rej` WHERE `herb` = '{$_SESSION['login']}'");
					while ($row = mysql_fetch_assoc ($prac)){
					$osoba = $row['osoba'];
					$zmiana = $row['zmiana'];
 
					if ( $row['zmiana'] <= 0 ){
 
           ?>
<h3>Zmień hasło</h3>					 
<table>
 
<form action="<?php $_SERVER['PHP_SELF']; ?>" method="POST">
    <tr><td>Login:</td><td><input type="text" name="herb" value="<?php echo $row['herb']; ?>" readonly="readonly"/></td></tr>
    <tr><td>Nowe hasło:</td><td><input type="text" name="haslo" size="40"/></td></tr>	
    <tr><td></td><td><input type="hidden" name="data_konta" value="<?php echo date("Y-m-d",strtotime("+30 days")); ?>" readonly="readonly" /></td></tr>
    <tr><td></td><td><input type="submit"  style="background-color:yellow" name="edit" value="Zmień hasło"/></tr>
</form>
</table>
<?php  
 
		if( !empty( $_POST['herb']) && !empty( $_POST['haslo']) && !empty( $_POST['data_konta'])){
	       $herb = mysql_real_escape_string( $_POST['herb']);				 
               $haslo = hash('sha512', $_POST['haslo']);
               $data_konta =  mysql_real_escape_string ($_POST['data_konta']);
 
		$info5 = 'Hasło zostało zmienione.';
		$color5='green';
$query = mysql_query("UPDATE `dawca_rej` SET  `haslo` = '$haslo', `data_konta` = '$data_konta' WHERE `herb` = '{$_SESSION['login']}'") or die( mysql_error());
 
		header('refresh:2;url= ../admin_sz/panel.php');
	//ten zapis powyżej nie działa
	}
 
		if( empty( $_POST['haslo'])){
		$info5 = 'Musisz wpisać nowe hasło.';
		$color5= 'red';
		}
 
 echo '<font color="'.$color5.'" size="3"><b>'.$info5.'</b></font>'; ?>
<br>
<?php	}
 
 
              elseif ( $row['zmiana'] > 0 ){	
?>
<h3>Panel Administr</h3> 
<?php
require_once('../admin_sz/uprawnienia.php');
?>
<hr>
<?php 
}}}
 
echo  'Witaj <font color="green" size="3"><b>'.$osoba.'</b></font><br>';	
if (isset($_SESSION['login']))  {
echo  '<font color="black" size="3">Jesteś zalogowany jako: </font>
<font color="red" size="3"><b>'.$_SESSION['login'].'</b></font><br>';
 
unset($_SESSION['kto']);
 echo 'Dziś jest: '; echo date('<b>Y-m-d H:i:s</b>');
}
?>
<br>
<?php
if ( $zmiana <= 0 ){
echo  'Hasło wygasło <font color="red" size="3"><b>'.$zmiana.' </b></font>dni temu';}
if ( $zmiana > 0 ){
echo  'Do zmiany hasła zostało <font color="green" size="3"><b>'.$zmiana.' </b></font>dni';}
?>
[PHP] pobierz, plaintext 

Przed wyslaniem jakiegokolwiek naglowka nie mozna pojsc do przegladarki zadne dodatkowe info. A u cibeie leci cala masa kodu html. Takie skutki niestosowania sie do zasady:
najpierw logika aplikacji a potem dopiero wyswietlanie

Lokalnie ci dziala bo masz ustawione buforowanie automatyczne. Albo zmien ustawienia serwera na to buforowanie, albo sam wlaczaj buforowanie w kodzie php (ob_start()) albo zacznij pisac poprawnie czyli:
najpierw logika aplikacji a potem dopiero wyswietlanie

Ostatnia opcja jest najlepsza.

Wydawało mi się, że to jest logiczne. Przesyłam najpierw login, sprawdzam w bazie czy jest spełniony warunek. Jeśli jest spełniony to wywołuję formularz do zmiany hasła, wyświetlam komunikat o poprawnej zmianie i odświeżam stronę.
Po odświeżeniu warunek nie jest spełniony wiec przechodzi dalej i wyświetla pozostałe informacje. Ale widać byłem w błędzie. Jaka więc kolejność powinna być poprawna?

No juz ci napisalem: najpierw logika aplikacji, potem wyswietlanie.
U ciebie jest, ze najpierw wyswietlasz - np.formularz , a potem dopiero robisz header.

Wysweitlanie ma sie pojawic dopiero gdy cala logika aplikacji sie zakonczy. Nie wczesniej.

Jakoś to mi nie wychodzi. Może jakaś podpowiedz, ale bardziej szczegółowa.

No instrukcje sa jasne: header to logika aplikacji. Robiac header nie mozesz nic wczesniej wyswietlac.
Przeciez w request, ktore wywali header nie potrzebujesz wyswietlac formularza. To go nie wysietlaj. Przeciez wiesz ktory request wymaga formularza a ktory nie.

Takie coś też nie działa Formularz jest na końcu.

[PHP] pobierz, plaintext 
<?php
 
 
if (isset($_SESSION['login']))  {
 $prac = mysql_query("SELECT `id`, `pesel`, `herb`,  TO_DAYS(`data_konta`) - TO_DAYS(NOW()) AS `zmiana` FROM `dawca_rej` WHERE `herb` = '{$_SESSION['login']}'");
 
					while ($row = mysql_fetch_assoc ($prac)){
 
					$osoba = $row['osoba'];
					$zmiana = $row['zmiana'];
 
 
					if ( $row['zmiana'] <= 0 ){
 
 
 
				if( !empty( $_POST['herb']) && !empty( $_POST['haslo']) && !empty( $_POST['data_konta'])){
 
							 $herb = mysql_real_escape_string( $_POST['herb']);
 
               $haslo = hash('sha512', $_POST['haslo']);
 
 
                $data_konta =  mysql_real_escape_string ($_POST['data_konta']);
 
 
		$info5 = 'Hasło zostało zmienione.';
		$color5='green';
 
 
$query = mysql_query("UPDATE `dawca_rej` SET  `haslo` = '$haslo', `data_konta` = '$data_konta' WHERE `herb` = '{$_SESSION['login']}'") or die( mysql_error());
 
 
 
	}			
 
 
 
		if( empty( $_POST['haslo'])){
 
 
 
		$info5 = 'Musisz wpisać nowe hasło.';
		$color5= 'red';
 
		}		
 
 
?>
 
 
<br>
 
<?php	}
			elseif ( $row['zmiana'] > 0 ){	
 
 
 
?>
 
 
 
<h3>Panel Administr</h3> 
 
<?php
 
require_once('../admin_sz/uprawnienia.php');
 
 
?>
<hr>
 
 
<?php 
}}
 
 
 
 
header('refresh:2;url= ../admin_sz/panel.php');	
}
echo  'Witaj <font color="green" size="3"><b>'.$osoba.'</b></font><br>';	
 
 
if (isset($_SESSION['login']))  {
 
echo  '<font color="black" size="3">Jesteś zalogowany jako: </font>
<font color="red" size="3"><b>'.$_SESSION['login'].'</b></font><br>
';
 
unset($_SESSION['kto']);
 
echo 'Dziś jest: '; echo date('<b>Y-m-d H:i:s</b>');
 
}
?>
<br>
<?php
if ( $zmiana <= 0 ){
 
echo '<font color="'.$color5.'" size="3"><b>'.$info5.'</b></font>'; ?>
 
<h3>Zmień hasło</h3>					 
<table>
 
<form action="<?php $_SERVER['PHP_SELF']; ?>" method="POST">
 
 
 
    <tr><td>Login:</td><td><input type="text" name="herb" value="<?php echo $_SESSION['login']; ?>" readonly="readonly"/></td></tr>
 
    <tr><td>Nowe hasło:</td><td><input type="text" name="haslo" size="40"/></td></tr>	
 
		<tr><td></td><td><input type="hidden" name="data_konta" value="<?php echo date("Y-m-d",strtotime("+30 days")); ?>" readonly="readonly" /></td></tr>
 
    <tr><td></td><td><input type="submit"  style="background-color:yellow" name="edit" value="Zmień hasło"/></tr>
 
</form>
 
</table>
 
<?php 
 
 
 
 
echo  'Hasło wygasło <font color="red" size="3"><b>'.$zmiana.' </b></font>dni temu';}
if ( $zmiana > 0 ){
 
 
echo '<font color="'.$color5.'" size="3"><b>'.$info5.'</b></font>'; 
echo  'Do zmiany hasła zostało <font color="green" size="3"><b>'.$zmiana.' </b></font>dni'
;}
 
 
?>
 
[PHP] pobierz, plaintext 

No ale przed header masz teraz jeszcze to:

<h3>Panel Administr</h3>

Masz rowniez cala mase pustych linii (ENTEROW). Zrozum. Nic nie moze pojsc. Zaden tekst, zadna spacja, nic. nie mozesz pojsc tez zaden znaczik html ala: <html><body> jesli takowe masz ghdzies tam a nie pokazales tu w kodzie

W kodzie pokazałem wszystko.
Zmieniłem to tak. Nic już nie powinno się wyświetlać przed header, ale to i tak nie działa. Gdzie tu jeszcze robie błąd.

[PHP] pobierz, plaintext 
 
<?php
if (isset($_SESSION['login']))  {
 $prac = mysql_query("SELECT `id`, `pesel`, `herb`,  TO_DAYS(`data_konta`) - TO_DAYS(NOW()) AS `zmiana` FROM `dawca_rej` WHERE `herb` = '{$_SESSION['login']}'");
 
					while ($row = mysql_fetch_assoc ($prac)){
 
					$osoba = $row['osoba'];
					$zmiana = $row['zmiana'];
 
					if ( $row['zmiana'] <= 0 ){
 
 
				if( !empty( $_POST['herb']) && !empty( $_POST['haslo']) && !empty( $_POST['data_konta'])){
							 $herb = mysql_real_escape_string( $_POST['herb']);
 
               $haslo = hash('sha512', $_POST['haslo']);
 
                $data_konta =  mysql_real_escape_string ($_POST['data_konta']);
 
		$info5 = 'Hasło zostało zmienione.';
		$color5='green';
$query = mysql_query("UPDATE `dawca_rej` SET  `haslo` = '$haslo', `data_konta` = '$data_konta' WHERE `herb` = '{$_SESSION['login']}'") or die( mysql_error());
	}			
 
 
 
		if( empty( $_POST['haslo'])){
 
		$info5 = 'Musisz wpisać nowe hasło.';
		$color5= 'red';
				}		
?>
<br>
<?php	
header('refresh:2;url= ../admin_sz/panel.php');
}
			elseif ( $row['zmiana'] > 0 ){	
require_once('../admin_sz/uprawnienia.php');
?>
<hr>
<?php 
}
}
}
echo  'Witaj <font color="green" size="3"><b>'.$osoba.'</b></font><br>';	
 
if (isset($_SESSION['login']))  {
echo  '<font color="black" size="3">Jesteś zalogowany jako: </font>
<font color="red" size="3"><b>'.$_SESSION['login'].'</b></font><br>
';
 
unset($_SESSION['kto']);
 
echo 'Dziś jest: '; echo date('<b>Y-m-d H:i:s</b>');
 
}
?>
<br>
<?php
if ( $zmiana <= 0 ){
 
echo '<font color="'.$color5.'" size="3"><b>'.$info5.'</b></font>'; ?>
 
<h3>Zmień hasło</h3>					 
<table>
 
<form action="<?php $_SERVER['PHP_SELF']; ?>" method="POST">
 
    <tr><td>Login:</td><td><input type="text" name="herb" value="<?php echo $_SESSION['login']; ?>" readonly="readonly"/></td></tr>
 
    <tr><td>Nowe hasło:</td><td><input type="text" name="haslo" size="40"/></td></tr>	
 
		<tr><td></td><td><input type="hidden" name="data_konta" value="<?php echo date("Y-m-d",strtotime("+30 days")); ?>" readonly="readonly" /></td></tr>
 
    <tr><td></td><td><input type="submit"  style="background-color:yellow" name="edit" value="Zmień hasło"/></tr>
 
</form>
 
</table>
 
<?php 
echo  'Hasło wygasło <font color="red" size="3"><b>'.$zmiana.' </b></font>dni temu';}
if ( $zmiana > 0 ){
echo '<font color="'.$color5.'" size="3"><b>'.$info5.'</b></font>'; 
echo  'Do zmiany hasła zostało <font color="green" size="3"><b>'.$zmiana.' </b></font>dni'
;}
?>
 
 
[PHP] pobierz, plaintext 

Dobra, ja sie poddaje. Wyraznie ci napisalem NIC a ty dalej walisz COS. Nie bede ci palcem kazdej linijki pokazywal bo to juz lekka przesada

Nie oczekuje pokazywania każdej linijki, wystarczy napisać np. tak:
najpierw zrób sprwdzenie loginu w bazie;
jesli warunek spełniony to zrob to;
jeśli nie to zrób tamto;
tutaj ma być header;
tu ma być wyświetlanie.
Wszystkie puste linie wykasowane

[PHP] pobierz, plaintext 
<?php
if (isset($_SESSION['login']))  {
 $prac = mysql_query("SELECT `id`, `pesel`, `herb`,  TO_DAYS(`data_konta`) - TO_DAYS(NOW()) AS `zmiana` FROM `dawca_rej` WHERE `herb` = '{$_SESSION['login']}'");
					while ($row = mysql_fetch_assoc ($prac)){
					$osoba = $row['osoba'];
					$zmiana = $row['zmiana'];
					if ( $row['zmiana'] <= 0 ){
				if( !empty( $_POST['herb']) && !empty( $_POST['haslo']) && !empty( $_POST['data_konta'])){
							 $herb = mysql_real_escape_string( $_POST['herb']);
               $haslo = hash('sha512', $_POST['haslo']);
                $data_konta =  mysql_real_escape_string ($_POST['data_konta']);
		$info5 = 'Hasło zostało zmienione.';
		$color5='green';
$query = mysql_query("UPDATE `dawca_rej` SET  `haslo` = '$haslo', `data_konta` = '$data_konta' WHERE `herb` = '{$_SESSION['login']}'") or die( mysql_error());
	}			
		if( empty( $_POST['haslo'])){
		$info5 = 'Musisz wpisać nowe hasło.';
		$color5= 'red';
				}		
?>
<br>
<?php	
header('refresh:2;url= ../admin_sz/panel.php');
}
			elseif ( $row['zmiana'] > 0 ){	
require_once('../admin_sz/uprawnienia.php');
?>
<hr>
<?php 
}
}
}
echo  'Witaj <font color="green" size="3"><b>'.$osoba.'</b></font><br>';	
if (isset($_SESSION['login']))  {
echo  '<font color="black" size="3">Jesteś zalogowany jako: </font>
<font color="red" size="3"><b>'.$_SESSION['login'].'</b></font><br>';
unset($_SESSION['kto']);
 echo 'Dziś jest: '; echo date('<b>Y-m-d H:i:s</b>');
}
?>
<br>
<?php
if ( $zmiana <= 0 ){
echo '<font color="'.$color5.'" size="3"><b>'.$info5.'</b></font>'; ?>
<h3>Zmień hasło</h3>					 
<table>
<form action="<?php $_SERVER['PHP_SELF']; ?>" method="POST">
    <tr><td>Login:</td><td><input type="text" name="herb" value="<?php echo $_SESSION['login']; ?>" readonly="readonly"/></td></tr> 
    <tr><td>Nowe hasło:</td><td><input type="text" name="haslo" size="40"/></td></tr>	
		<tr><td></td><td><input type="hidden" name="data_konta" value="<?php echo date("Y-m-d",strtotime("+30 days")); ?>" readonly="readonly" /></td></tr>
    <tr><td></td><td><input type="submit"  style="background-color:yellow" name="edit" value="Zmień hasło"/></tr>
</form>
</table>
<?php 
echo  'Hasło wygasło <font color="red" size="3"><b>'.$zmiana.' </b></font>dni temu';}
if ( $zmiana > 0 ){
echo '<font color="'.$color5.'" size="3"><b>'.$info5.'</b></font>'; 
echo  'Do zmiany hasła zostało <font color="green" size="3"><b>'.$zmiana.' </b></font>dni'
;}
?>
[PHP] pobierz, plaintext 

Ten post edytował dopal 25.06.2018, 13:09:37

Napisalem ci wyraznie co masz zrobic. Instrukcje byly naprawde banalne. Banalne.

Napislaem ci wyraznide co ma byc skasowane. Puste linie byly tylko jedne z wielu.

No nic ja też się poddaje i zrobię to zupełnie inaczej, bez sprawdzania ważności hasła ( choć szkoda), bo jednak trochę bardziej liczyłem na pomoc a nie na powtarzanie ciągle tego samego przez Ciebie. Skoro to jest miejsce dla początkujących to oczekiwałbym trochę więcej zrozumienia, nie wszyscy to eksperci. A nie oczekuję podania rozwiązania na tacy, a jedynie konkretnych wskazówek. Gdybym wiedział co masz na mysli, to bym się nie dopytywał. Mimo to dzięki.