Forum PHP.pl

Czy takie coś zadziałać powinno? Bo u mnie jakoś nie chce.

[PHP] pobierz, plaintext 
<?
if ($_SESSION["zalogowany"] == "no")
{
?>
<table border="0" width="540">
	<tr>
	<td class="menu" colspan="2"> &nbsp; </td>
	</tr>
</table>
<?
}
elseif ($_SESSION["zalogowany"] == "yes")  // Funkcje dostepne po zalogowaniu
{
?>
<form action="index.php?ids=popraw" method="post">
	<table border="0" width="540">
	  <tr>
		<td>&nbsp;</td>
	</tr>
	  <tr>
		<td class="tekst_nagl" colspan="2"> Zmiana hasła użytkownika: </td>
	  </tr>
	  <tr>
		<td>&nbsp;</td>
	</tr>
	<tr>
		<td class="tekst"> Poprzednie hasło: </td>
		<td><input type="text" name="stare_haslo" size="20" /></td>
	</tr>
	<tr>
		<td class="tekst"> Nowe hasło: </td>
		<td><input type="password" name="newpass1" size="20" maxlength="20"/></td>
	</tr>
	<tr>
		<td class="tekst"> Powtórzenie nowego hasła: </td>
		<td><input type="password" name="newpass2" size="20" maxlength="20"/></td>
	</tr>
	  <tr>
		<td>&nbsp;</td>
	</tr>
	 <tr>
		<td align="center"><input type="submit" value="Zmiana hasła" /></td>
	</tr>
	<tr>
		<td>&nbsp;</td>
	</tr>
	</table>
	</form>
<?
if ($_GET["ids"] == "popraw")
{
?>
<form action="index.php?ids1=popraw" method="post">
<table border="0" width="540">
<?
$newpass1 = $_POST['newpass1'];
$newpass2 = $_POST['newpass2'];
$login = $_SESSION['login'];
$pytanie = "select * from user where login ='$login'";
$wyslij = mysql_query($pytanie);
$tablica = mysql_fetch_array($wyslij);
$md5newpass = md5($newpass1);
$oldpass = $tablica['haslo'];
if (!$newpass1 || !$newpass2)
{
$_SESSION['komunikat2'] = "<br>Wpisz hasło<br>";
if ($newpass1 != $newpass2)
{
$_SESSION['komunikat2'] = "<br>Hasła powinny być jednakowe.<br>";
if (strlen($newpass1) <4 || strlen ($newpass1) > 10)
{
$_SESSION['komunikat2'] = "<br>Hasło powinno mieć od 4 do 10 znaków.<br>";
}
if ($oldpass == $md5newpass)
{
$_SESSION['komunikat2'] = "<br>Po co zmieniać hasło na takie samo?!<br>";
}
else
{
}
}
$pytanie2 = "update users set haslo='$md5newpass' where login='$login'";
  $sending = mysql_query($pytanie2);
  $_SESSION['komunikat2'] = "<br>Hasło zostało pomyślnie zmienione.<br>";
}
?>
</table>
</form>
<?
}
}
[PHP] pobierz, plaintext 

Przyznaję się że znalazłem jakiegoś gotowca i chciałem go dostosować do siebie, ale... za cienki Bolek jestem.

Włącz sobie error_reporting" title="Zobacz w manualu php" target="_manual na 2047 ( nie pamiętam stałych )
Dodaj na początku session_start" title="Zobacz w manualu php" target="_manual

Z tym error to nie wiem co do mnie mówisz, bo jestem ciemno zielony A co do session_start... to tu mam główny plik

[PHP] pobierz, plaintext 
<?
// Plik startowy strony
session_start();
session_register('zalogowany'); // Status zalogowania
if (!IsSet($_SESSION["zalogowany"]))
{
	$_SESSION["zalogowany"] = "no";
}
if (!IsSet($_SESSION["rola"])) // Rola uzytkownika
{
	$_SESSION["rola"] = "no";
}
include ("./nagl.php");  // Wstawianie naglowka
?>
	<table border="1" width="800" cellspacing="0" cellpadding="0" align="center">
		<tr>
		<? // Wstawianie menu gornego menu_g.php
			 // Wstawianie menu lewego menu_l.php
		// Wstawianie menu prawego menu_p.php
		?>
		<td class="uwaga" colspan="3"><? include ("./menu_g.php"); ?></td>
		</tr>
	<tr>
		<?
		if (($_SESSION["zalogowany"] == "no") and ($_GET["ids"] == "logowanie"))
		{
		$tab = mysql_query("SELECT * from users where login='$_POST[login]'");
		$tab_wyn= mysql_fetch_array($tab);
		if ($tab_wyn[haslo] == md5($_POST["haslo"]))
		{
		$_SESSION["rola"] = $tab_wyn[rola];
		$_SESSION["zalogowany"] = "yes";
		$tab1 = "insert into logowania (id_log,id_logina,data_log) values ('','$tab_wyn[id_users]','')";
		$tab_wyn1 = mysql_query($tab1,$connection);
		?>
		<td width="125" valign="top"><? include ("./menu_l.php"); ?> </td>
		<td width="550" class="tekst_nagl" align="center"> Zostałeś zalogowany !!!
			<br>
			<br>
			<br>
			<font class="uwaga" size=4>
			<?
			$dzisiaj = getdate();		 //Przypomnienie-komunikat pojawiajacy sie po zalogowaniu
			$dzien = $dzisiaj['mday'];
			$miesiac = $dzisiaj['mon'];
			$pytanie = mysql_query("select * from przypom where dzien_prz='$dzien' and miesi_prz='$miesiac'");
			$odpowiedz = mysql_fetch_array($pytanie);
			{
			echo $odpowiedz[tekst];
			}
			?>
			</td>
			<?
		}
		else
		{
		?>
		<td width="125" valign="top"><? include ("./menu_l.php"); ?> </td>
		<td width="550" align="center"><a href="./index.php?idl=login" class="uwaga"> ZŁY LOGIN LUB HASŁO !!! WPROWADŹ PONOWNIE !!! </a></td>
		<?
		}
		}
		  elseif (($_SESSION["zalogowany"] == "no") and ($_GET["idl"] <> "login"))
		{
		?>
		<td width="125" valign="top"><? include ("./menu_l.php"); ?> </td>
		<td width="550"> &nbsp; </td>
		<?
		}
		  elseif (($_SESSION["zalogowany"] == "no") and ($_GET["idl"] == "login"))
		{
		?>
		<td width="125" valign="top"><? include ("./menu_l.php"); ?> </td>
		<td width="550"> <? include ("./login.php"); ?> </td>
		<?
		}
		  elseif (($_SESSION["zalogowany"] == "yes") and ($_GET["idl"] == "login"))
		{
		?>
		<td width="125" valign="top"><? include ("./menu_l.php"); ?> </td>
		<td width="550" class="uwaga" align="center"> JESTEŚ JUŻ ZALOGOWANY !!! </td>
		<?
		}
		  elseif (($_SESSION["zalogowany"] == "yes") and ($_GET["idl"] == "popraw")) //Poprawianie hasla uzytkownika
		{
		?>
		<td width="125" valign="top"><? include ("./menu_l.php"); ?> </td>
		  <td width="550"> <? include ("./popraw.php"); ?> </td>
		<?
		}
		  elseif (($_SESSION["zalogowany"] == "yes") and ($_GET["idl"] == "wylogowanie"))
		{
		?>
		<td width="125" valign="top"><? include ("./menu_l.php"); ?> </td>
		<td width="550" class="tekst_nagl" align="center"> Zostałeś wylogowany !!! </td>
		<?
		$_SESSION["zalogowany"] = "no";
		}		
		elseif ($_SESSION["zalogowany"] == "yes")
		{
		?>
		<td width="125" valign="top"><? include ("./menu_l.php"); ?> </td>
		<td width="550" class="uwaga" align="center"> JESTEŚ JUŻ ZALOGOWANY !!! </td>
		<?
		}
		?>
		<td width="125" valign="top"><? include ("./menu_p.php"); ?> </td>
		</tr>
	</table>
	<table border="0" width="800" cellspacing="0" cellpadding="0" align="center">
		<tr>
			<td align="center" colspan="3"><a href="./index.php"><img src="./obrazy/powrot.gif" border="0" alt="powrót" /></a></td>
		</tr>
		<tr>
			<td colspan="3" align="center">
		<p>
			<a href="http://validator.w3.org/check?uri=referer"><img src="http://www.w3.org/Icons/valid-xhtml10" alt="Valid XHTML 1.0!" height="31" width="89" border="0" /></a>
		</p>
		</td>
		</tr>
	</table>
</body>
</html>
[PHP] pobierz, plaintext 

Po pierwsze wywal:

[PHP] pobierz, plaintext 
<?php
session_register('zalogowany');
?>
[PHP] pobierz, plaintext 

Powyższe zawsze ustawia zmienną zalogowany, bez nadawania wartości YES/NO

Potem powiedz co ci nie działa? Jak to się objawia, itp.?

Dobra, wywaliłem co miałem wywalić i jest jak było poprzednio... czyli po zmianie hasła mówi... jesteś już zalogowany, a hasła nie zmienia


Czyli to sprawa beznadziejna?

Czyżbym tak namieszał że nie da się już nic z tym zrobić?

Ten post edytował BIG_MICHU 26.11.2006, 20:04:26

Sprawdz wogóle czy ten rekord jest rzeczywiście aktualizowany -> mysql_affected_rows" title="Zobacz w manualu php" target="_manual bo póki co to tylko wyświetlasz komunikat że jest

Po pierwsze nie używaj '<?' tylko '<?php'

Po drugie dlaczego nie spr. czy jest przekazane: $_GET["ids"]

Po trzecie gdzie się łączysz z bazą? bo chyba to robisz

4) $_POST[login] chyba $_POST['login']

i dalsze $tab_wyn[haslo] ... oraz spr. czy to co użytkownik miał przesłać naprawdę przesłał.

[PHP] pobierz, plaintext 
<?php
$tab1 = "insert into logowania (id_log,id_logina,data_log) values ('','$tab_wyn[id_users]','')";
?>
[PHP] pobierz, plaintext 

id_log zapisujesz puste (i do tego string)?
A i te $tablica[cos] to to cos jak to jest string to w ciapkach

Ten post edytował Sedziwoj 27.11.2006, 19:40:47

Chyba sobie to jednak odpuszcze, bo to dla mnie chinszczyzna.

A tak się spytam, co w tym takiego dziwnego??
Jak ma w bazie auto_incremment to jest wszystko ok.


Jak robisz wywołanie funkji mysql_query to zrób tak

[PHP] pobierz, plaintext 
<?php
mysql_query($zapytanie) or die mysql_error();
?>
[PHP] pobierz, plaintext 

i napisz jaki błąd masz

Ten post edytował artur81 28.11.2006, 07:12:26

tylko chodzi że automatyczna inkrementacja działa jak nie masz podanej wartości a tu podajesz pusty string, chyba że zrobili idioto odporne przyjmowanie... w co wątpię.

ehkm, żarty sobie robisz?

Masz prostą tabelę o strukturze:

[SQL] pobierz, plaintext 
CREATE TABLE `test` (
  `id` tinyint(4) NOT NULL AUTO_INCREMENT,
  `wpis` varchar(30) NOT NULL DEFAULT '',
  PRIMARY KEY  (`id`)
) TYPE=MyISAM AUTO_INCREMENT=1 ;
[SQL] pobierz, plaintext 

Wykonaj do niej kilka takich zapytań:

[SQL] pobierz, plaintext 
INSERT INTO `test` VALUES ('', 'blalalalal');
[SQL] pobierz, plaintext 

i zobacz co się dzieje, bo albo się nie rozumiemy albo coś źle doczytałeś...

[SQL] pobierz, plaintext 
INSERT INTO aktorzy VALUES ('', 'Testowy aktor');
[SQL] pobierz, plaintext 

i wyrzuca:

[SQL] pobierz, plaintext 
ERROR:  invalid input syntax FOR integer: ""
[SQL] pobierz, plaintext 

Przy:

[SQL] pobierz, plaintext 
CREATE TABLE aktorzy
(
  id_a serial NOT NULL,
  name varchar(40) NOT NULL,
  CONSTRAINT aktorzy_pkey PRIMARY KEY (id_a),
  CONSTRAINT aktorzy_name_key UNIQUE (name)
) 
 
CREATE SEQUENCE aktorzy_id_a_seq
  INCREMENT 1
  MINVALUE 1
  MAXVALUE 9223372036854775807
  START 8
  CACHE 1;
[SQL] pobierz, plaintext 

Może po prostu różnica między MySQL i PostgreSQL?
Szczerze to mocny w SQL'u się nie czuję więc różnice implementacji mogą mi umknąć.
Co nie zmienia faktu, że tak:

[SQL] pobierz, plaintext 
INSERT INTO aktorzy (name) VALUES ('Testowy aktor');
[SQL] pobierz, plaintext 

jest zawsze dobrze, niezależnie od bazy.

EDIT lit.

Ten post edytował Sedziwoj 28.11.2006, 12:49:49

Oho, auto_increment jak się patrzy

Nic nie rozumiem co piszecie, ale mam tak :
include ("./nagl.php") a w nagl.php include ("./stale/dblib.inc.php") i tu funkcje łączenia się z bazą, rozumiem można to uznać, że z bazą się łączę?

Jedyne string(i) jakie znam, to mojej dziewczyny i one na bank puste nie są , ale raczej nie o to chodziło?

Co do bazy to:
CREATE TABLE `logowania` (
`id_log` int(5) NOT NULL auto_increment,
`id_logina` int(3) NOT NULL default '0',
`data_log` int(10) NOT NULL default '0',
PRIMARY KEY (`id_log`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=2 ;

Nie wiem czy w dobrym miejscu to wstawiłem, ale

[PHP] pobierz, plaintext 
<?
$newpass1 = $_POST['newpass1'];
$newpass2 = $_POST['newpass2'];
$login = $_SESSION['login'];
$pytanie = "select * from user where login ='$login'";
$wyslij = mysql_query($zapytanie) or die mysql_error();
$tablica = mysql_fetch_array($wyslij);
$md5newpass = md5($newpass1);
$oldpass = $tablica['haslo'];
if (!$newpass1 || !$newpass2)
?>
[PHP] pobierz, plaintext 

i wyszedł komunikat

Parse error: syntax error, unexpected T_STRING in /home3/dealer/main/pkg/bleble/web/html/popraw.php on line 60

Ten post edytował BIG_MICHU 28.11.2006, 17:32:38

to jest różnica między MySQL a PostgreSQL, tu jest pseudo typ serial, lub można stosować 'dziwną' konstrukcje pobierającą następną wartość z SEQUENCE aktorzy_id_a_seq.
Takie pomieszane, ale logiczne


Aby łączyła, musi być wywołana.

[PHP] pobierz, plaintext 
<?
//może sprawdź najpierw czy są if (isset($_POST['newpass1']))...
$newpass1 = $_POST['newpass1'];
$newpass2 = $_POST['newpass2'];
//to samo czy podane
$login = $_SESSION['login'];
//tu masz zmienną $pytanie
$pytanie = "select * from user where login ='$login'";
//a tu $zapytanie, może prościej :
//$wyslij = mysql_query("select * from user where login ='$login'") or die mysql_error();
$wyslij = mysql_query($zapytanie) or die mysql_error();
$tablica = mysql_fetch_array($wyslij);
$md5newpass = md5($newpass1);
$oldpass = $tablica['haslo'];
//a to co ma robić?
if (!$newpass1 || !$newpass2)
?>
[PHP] pobierz, plaintext 

ale my nie mamy całego pliku więc zaznacz która to jest tak 60 linia.

EDIT ach te polskie znaczki

Ten post edytował Sedziwoj 28.11.2006, 18:42:31

to jest wspominana linia 60 $wyslij = mysql_query($zapytanie) or die mysql_error();
ale postaram się to teraz jakoś logicznie popoprawiać i zobaczymy co wyjdzie- może mi się uda

if (!$newpass1 || !$newpass2) a to ma sprawdzać czy pola są wypełnione

No i nigdzie nie mam czegoś takiego if (isset($_POST['newpass1']))...

Po przerobieniu (takim jak umiałem)

[PHP] pobierz, plaintext 
<form action="index.php?ids1=popraw" method="post">
<table border="0" width="540">
<?
$newpass1 = $_POST['newpass1'];
$newpass2 = $_POST['newpass2'];
$login = $_SESSION['login'];
$pytanie = "select * from user where login ='$login'";
$wyslij = mysql_query("select * from user where login ='$login'") or die mysql_error();
$tablica = mysql_fetch_array($wyslij);
$md5newpass = md5($newpass1);
$oldpass = $tablica['haslo'];
if (!$newpass1 || !$newpass2)
{
$_SESSION['komunikat2'] = "<br>Wpisz hasło<br>";
if ($newpass1 != $newpass2)
?>
[PHP] pobierz, plaintext 

wywala błąd tu $wyslij = mysql_query("select * from user where login ='$login'") or die mysql_error();

Ten post edytował BIG_MICHU 28.11.2006, 20:13:14