Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: [HTML][MySQL][PHP] Zmiana hasła - brak informacji o błędach
Forum PHP.pl > Forum > Przedszkole
Laxus
10.05.2017, 19:33:20
Mam problem ze skryptem odpowiedzialnym za zmianę hasła użytkownika.

W poniższym kodzie linie 7-40 odpowiadają za walidację formularza do zmiany hasła.
Formularz to linie 84-108.
Zaś w liniach 118-133 umieściłem dwa spany, w których ma wyświetlać się informacja o zmianie hasła, bądź błędzie.

Kod w zasadzie działa poprawnie, bo hasło się zmienia, jeśli spełnione są wszystkie warunki. Problem w tym, że nie wyświetlają mi się w spanach żadne informacje.

Gapię się w kod już z godzinę i nie mogę dojść co jest nie tak sad.gif

[PHP] pobierz, plaintext 
  1. <?php
  2. 	session_start();
  3. 	if( !isset( $_SESSION[ 'logged' ] ) ) {
  4. 		header( 'Location: ../index.php' );
  5. 		exit();
  6. 	}
  7. 	if( isset( $_POST[ 'change-password-submit' ] ) ) {
  8. 		$validate = true;
  9. 		if( $_POST[ 'old-password' ] != password_verify($_POST[ 'old-password' ], $_SESSION[ 'logged' ] [ 'password' ] ) ) {
  10. 			$_SESSION[ 'change_password_error' ] = 'Stare hasło jest niepoprawne';
  11. 			$validate = false;
  12. 		}
  13. 		if( strlen( $_POST[ 'new-password' ] ) < 8 || strlen( $_POST[ 'new-password' ] ) > 12 ) {
  14. 			$_SESSION[ 'change_password_error' ] = 'Nowe hasło musi mieć 8-12 znaków';
  15. 			$validate = false;
  16. 		}
  17. 		if( $_POST[ 'new-password' ] != $_POST[ 'new-password2' ] ) {
  18. 			$_SESSION[ 'change_password_error' ] = 'Nowe hasła nie są identyczne';
  19. 			$validate = false;
  20. 		}
  21. 		if( empty( $_POST[ 'old-password' ] ) || empty( $_POST[ 'new-password' ] ) || empty( $_POST[ 'new-password2' ] ) ) {
  22. 			$_SESSION[ 'change_password_error' ] = 'Musisz wypełnić wszystkie pola;';
  23. 			$validate = false;
  24. 		}
  25. 		if( $validate == false ) {
  26. 			header( 'Location: profile.php' );
  27. 		} else {
  28. 			$password = password_hash( $_POST[ 'new-password' ], PASSWORD_DEFAULT );
  29. 			try {
  30. 				require_once( 'connect.php' );
  31. 				$change_password = $pdo -> prepare( "UPDATE users SET password = :password" );
  32. 				$change_password -> bindParam( ':password', $password );
  33. 				$change_password -> execute();
  34. 			} catch(PDOException $e) {
  35. 				echo 'ERROR: ' . $e -> getMessage();
  36. 			}
  37. 			$_SESSION[ 'change_password_success' ] = 'Twoje hasło zostało zmienione';
  38. 			header( 'Location: profile.php' );
  39. 		}
  40. 	}
  41. ?>
  42.  
  43. <!DOCTYPE html>
  44. <html>
  45. <head>
  46. 	<meta charset="utf-8">
  47. 	<title>
  48. 		Profil użytkownika
  49. 		<?php
  50. 			echo $_SESSION[ 'logged' ] [ 'login' ];
  51. 		?>
  52. 	</title>
  53. 	<link rel="stylesheet" type="text/css" href="../css/screen.css" media="screen">
  54. </head>
  55. <body>
  56. 	<?php
  57. 		include_once( 'top.php' );
  58. 	?>
  59.  
  60. 	<aside id="profile">
  61. 		<h1>
  62. 			Profil użytkownika
  63. 			<?php
  64. 				echo $_SESSION[ 'logged' ] [ 'login' ];
  65. 			?>
  66. 		</h1>
  67. 		<img src="../img/users/<?php echo $_SESSION[ 'logged' ] [ 'avatar' ]; ?>" alt="<?php echo $_SESSION[ 'logged' ] [ 'avatar' ]; ?>">
  68. 		<table border="1">
  69. 			<tr>
  70. 				<td>Nazwa użytkownika: </td>
  71. 				<td><?php echo $_SESSION[ 'logged' ] [ 'login' ]; ?></td>
  72. 			</tr>
  73. 			<tr>
  74. 				<td>Adres e-mail: </td>
  75. 				<td><?php echo $_SESSION[ 'logged' ] [ 'email' ]; ?></td>
  76. 			</tr>
  77. 			<tr>
  78. 				<td>Data założenia konta: </td>
  79. 				<td><?php echo $_SESSION[ 'logged' ] [ 'date' ]; ?></td>
  80. 			</tr>
  81. 			<tr>
  82. 				<td colspan="2"><a href="#" id="change_password" onclick="toggle('change_password_div');">Zmień hasło</a></td>
  83. 				<div id="change_password_div" style="display: none;">
  84. 					<form action="profile.php" method="post">
  85. 						<ul>
  86. 							<li>
  87. 								<label for="old-password">Stare hasło</label>
  88. 							</li>
  89. 							<li>
  90. 								<input type="password" name="old-password" id="old-password">
  91. 							</li>
  92. 							<li>
  93. 								<label for="new-password">Nowe hasło</label>
  94. 							</li>
  95. 							<li>
  96. 								<input type="password" name="new-password" id="new-password">
  97. 							</li>
  98. 							<li>
  99. 								<label for="new-password2">Powtórz hasło</label>
  100. 							</li>
  101. 							<li>
  102. 								<input type="password" name="new-password2" id="new-password2">
  103. 							</li>
  104. 							<li>
  105. 								<input type="submit" name="change-password-submit" id="change-password-submit" value="Zmień hasło">
  106. 							</li>
  107. 						</ul>
  108. 					</form>
  109. 				</div>
  110. 			</tr>
  111. 			<tr>
  112. 				<td colspan="2"><a href="#" id="change_avatar">Zmień obraz konta</a></td>
  113. 			</tr>
  114. 			<tr>
  115. 				<td colspan="2"><a href="logout.php">Wyloguj się</a></td>
  116. 			</tr>
  117. 		</table>
  118. 		<span class="error change-password-error">
  119. 			<?php
  120. 				if( isset( $_SESSION[ 'change_password_error' ] ) ) {
  121. 					echo $_SESSION[ 'change_password_error' ];
  122. 					unset( $_SESSION[ 'change_password_error' ] );
  123. 				}
  124. 			?>
  125. 		</span>
  126. 		<span class="success change-password-success">
  127. 			<?php
  128. 				if( isset( $_SESSION[ 'change_password_success' ] ) ) {
  129. 					echo $_SESSION[ 'change_password_success' ];
  130. 					unset( $_SESSION[ 'change_password_success' ] );
  131. 				}
  132. 			?>
  133. 		</span>
  134. 	</aside>
  135. 	<script type="text/javascript" src="../js/script.js"></script>
  136. </body>
  137. </html>
[PHP] pobierz, plaintext
bostaf
11.05.2017, 08:53:23
A w profile.php masz session_start na początku?

(ps. bardzo ładnie sformatowany kod. Porządek to podstawa, tak trzymać)
Laxus
11.05.2017, 09:09:00
To jest plik profile.php

Chciałem, żeby cały kod wykonywał się w tym samym pliku. Dzisiaj trochę pozmieniałem, bo myślałem, że może sesje powodują problem i napisałem to za pomocą zmiennych i tablic, lecz nic się nie zmieniło.
Nie wiem co robię źle.

[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. 	session_start();
  4. 	$messages = array(
  5. 		'change_password_error' => array(
  6. 			'Stare hasło jest niepoprawne',
  7. 			'Nowe hasło musi mieć 8-12 znaków',
  8. 			'Nowe hasła nie są identyczne',
  9. 			'Musisz wypełnić wszystkie pola'
  10. 		),
  11. 		'change_password_success' => array(
  12. 			'Twoje hasło zostało zmienione'
  13. 		)
  14. 	);
  15. 	if( !isset( $_SESSION[ 'logged' ] ) ) {
  16. 		header( 'Location: ../index.php' );
  17. 		exit();
  18. 	}
  19. 	if( isset( $_POST[ 'change-password-submit' ] ) ) {
  20. 		$validate = true;
  21. 		if( $_POST[ 'old-password' ] != password_verify($_POST[ 'old-password' ], $_SESSION[ 'logged' ] [ 'password' ] ) ) {
  22. 			$change_password_message = $messages[ 'change_password_error' ] [0];
  23. 			$validate = false;
  24. 		}
  25. 		if( strlen( $_POST[ 'new-password' ] ) < 8 || strlen( $_POST[ 'new-password' ] ) > 12 ) {
  26. 			$change_password_message = $messages[ 'change_password_error' ] [1];
  27. 			$validate = false;
  28. 		}
  29. 		if( $_POST[ 'new-password' ] != $_POST[ 'new-password2' ] ) {
  30. 			$change_password_message = $messages[ 'change_password_error' ] [2];
  31. 			$validate = false;
  32. 		}
  33. 		if( empty( $_POST[ 'old-password' ] ) || empty( $_POST[ 'new-password' ] ) || empty( $_POST[ 'new-password2' ] ) ) {
  34. 			$change_password_message = $messages[ 'change_password_error' ] [3];
  35. 			$validate = false;
  36. 		}
  37. 		if( $validate == false ) {
  38. 			header( 'Location: profile.php' );
  39. 		} else {
  40. 			$password = password_hash( $_POST[ 'new-password' ], PASSWORD_DEFAULT );
  41. 			try {
  42. 				require_once( 'connect.php' );
  43. 				$change_password = $pdo -> prepare( "UPDATE users SET password = :password" );
  44. 				$change_password -> bindParam( ':password', $password );
  45. 				$change_password -> execute();
  46. 			} catch(PDOException $e) {
  47. 				echo 'ERROR: ' . $e -> getMessage();
  48. 			}
  49. 			$change_password_message = $messages[ 'change_password_success' ] [0];
  50. 				header( 'Location: profile.php' );
  51. 		}
  52. 	}
  53. ?>
  54.  
  55. <!DOCTYPE html>
  56. <html>
  57. <head>
  58. 	<meta charset="utf-8">
  59. 	<title>
  60. 		Profil użytkownika
  61. 		<?php
  62. 			echo $_SESSION[ 'logged' ] [ 'login' ];
  63. 		?>
  64. 	</title>
  65. 	<link rel="stylesheet" type="text/css" href="../css/screen.css" media="screen">
  66. </head>
  67. <body>
  68. 	<?php
  69. 		include_once( 'top.php' );
  70. 	?>
  71. 	<aside id="profile">
  72. 		<h1>
  73. 			Profil użytkownika
  74. 			<?php
  75. 				echo $_SESSION[ 'logged' ] [ 'login' ];
  76. 			?>
  77. 		</h1>
  78. 		<img src="../img/users/<?php echo $_SESSION[ 'logged' ] [ 'avatar' ]; ?>" alt="<?php echo $_SESSION[ 'logged' ] [ 'avatar' ]; ?>">
  79. 		<table border="1">
  80. 			<tr>
  81. 				<td>Nazwa użytkownika: </td>
  82. 				<td><?php echo $_SESSION[ 'logged' ] [ 'login' ]; ?></td>
  83. 			</tr>
  84. 			<tr>
  85. 				<td>Adres e-mail: </td>
  86. 				<td><?php echo $_SESSION[ 'logged' ] [ 'email' ]; ?></td>
  87. 			</tr>
  88. 			<tr>
  89. 				<td>Data założenia konta: </td>
  90. 				<td><?php echo $_SESSION[ 'logged' ] [ 'date' ]; ?></td>
  91. 			</tr>
  92. 			<tr>
  93. 				<td colspan="2"><a href="#" id="change_password" onclick="toggle('change_password_div');">Zmień hasło</a></td>
  94. 				<div id="change_password_div" style="display: none;">
  95. 					<form action="profile.php" method="post">
  96. 						<ul>
  97. 							<li>
  98. 								<label for="old-password">Stare hasło</label>
  99. 							</li>
  100. 							<li>
  101. 								<input type="password" name="old-password" id="old-password">
  102. 							</li>
  103. 							<li>
  104. 								<label for="new-password">Nowe hasło</label>
  105. 							</li>
  106. 							<li>
  107. 								<input type="password" name="new-password" id="new-password">
  108. 							</li>
  109. 							<li>
  110. 								<label for="new-password2">Powtórz hasło</label>
  111. 							</li>
  112. 							<li>
  113. 								<input type="password" name="new-password2" id="new-password2">
  114. 							</li>
  115. 							<li>
  116. 								<input type="submit" name="change-password-submit" id="change-password-submit" value="Zmień hasło">
  117. 							</li>
  118. 						</ul>
  119. 					</form>
  120. 				</div>
  121. 			</tr>
  122. 			<tr>
  123. 				<td colspan="2"><a href="#" id="change_avatar">Zmień obraz konta</a></td>
  124. 			</tr>
  125. 			<tr>
  126. 				<td colspan="2"><a href="logout.php">Wyloguj się</a></td>
  127. 			</tr>
  128. 		</table>
  129. 		<span class="error change-password-error">
  130. 			<?php
  131. 				if( isset( $change_password_message ) ) {
  132. 					echo $change_password_message;
  133. 					unset( $change_password_message );
  134. 				}
  135. 			?>
  136. 		</span>
  137. 		<span class="success change-password-success">
  138. 			<?php
  139. 				if( isset( $change_password_message ) ) {
  140. 					echo $change_password_message;
  141. 					unset( $change_password_message );
  142. 				}
  143. 			?>
  144. 		</span>
  145. 	</aside>
  146. 	<script type="text/javascript" src="../js/script.js"></script>
  147. </body>
  148. </html>
[PHP] pobierz, plaintext
trueblue
11.05.2017, 09:46:26
Przypisujesz komunikaty do zmiennej, która nie przetrzymuje wartości po przekierowaniu.
goartur
11.05.2017, 09:58:03
Cytat(trueblue @ 11.05.2017, 09:46:26 ) *
Przypisujesz komunikaty do zmiennej, która nie przetrzymuje wartości po przekierowaniu.



Aby to rozwiazac zapisz komunikaty w sesji smile.gif I pozniej je usun.
bostaf
11.05.2017, 10:08:01
Cytat(goartur @ 11.05.2017, 10:58:03 ) *
Aby to rozwiazac zapisz komunikaty w sesji smile.gif I pozniej je usun.

Niom, w pierwszej wersji tak miał. Teraz zmienił. W pierwszej wersji teoretycznie wszystko wyglądało OK, ale nie działało tak jak powinno.
goartur
11.05.2017, 10:43:35
To najpiert w tym IF ktory sprawdza czy jest jakies blad zrob echo 'test'; naprzyklad, jak dziala to zrob var_dump($_SESSION['change_password_message'])
W innym wypadku poza if zrob var_dump($_SESSION) i zobacz co zwraca
bostaf
11.05.2017, 11:07:39
Znalazłem. Ta pierwsza wersja była oczywiście OK, ale po przekierowaniu w linii 38 brakowało instrukcji exit albo die.
Laxus
11.05.2017, 12:40:50
Dzięki wielkie. Faktycznie to był problem. W pierwszej wersji kodu było trzeba dopisać exit() po liniach 26 i 38, czyli kiedy wykonuje przekierowanie. Teraz wszystko śmiga jak należy smile.gif
To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.
Invision Power Board © 2001-2025 Invision Power Services, Inc.