[PHP] Problem z prywatnymi wiadomosciami Opcje

[stiw]

Witam, na swojej stronie uzylem skryptu Z TEJ STRONY do stworzenia prywatnych wiadomosci.

No i mam pewien problem :/ Wysylanie wiadomosci dziala ok, wiadomosci odczytac moge tylko te ktore wysle sam do siebie, kasowac tez moge tylko swoje wiadomosci (nadawczej tylko te ktore do siebie wyslalem, te ktore wyslalem do innego uzytkownika juz nie moge usunac)

Jak wysle jakas wiadomosc do drugiego uzytkownika to nie moge jej podgladnac w skrzynce nadawczej, a ten drugi nie moze jej odczytac w odbiorczej chociaz widnieje u niego na liscie , nie moze zobaczyc jej tresci .

Nie wiem za bardzo o co w tym chodzi bo zielony w tym jestem, takze prosilbym o wszelkie wskazowki, pozdrawiam.

Ten post edytował stiw 17.01.2011, 21:47:52

[Kshyhoo]

Testowałem kiedyś (jakieś 3 miesiące temu) ten skrypt i u mnie działał bez zarzutu... nawet dorobiłem sobie nieco funkcji - odpowiadanie ze skrzynki nadawczej i odbiorczej, wiadomości grupowe.
Kopiowałeś kod ze strony, czy użyłeś tego archiwum?

[stiw]

Skrypt jest pobrany z archiwum, nie jest kopiowany ze strony.

[Kshyhoo]

No to problem leży u Ciebie. Jedynym mankamentem rozwiązania w tym skrypcie jest znikanie wiadomości. Jak odpowiesz na wiadomość, zapisywana jest w skrzynce z wysłanymi a znika z odebranych. Można to jednak "naprawić", dodając odpowiednie warunki albo zmieniając nieco (dodając pole) w bazie danych.

[stiw]

Kurcze no nie wiem co jest nie tak . Jedyne co zmieniam w kodzie to polaczenie z baza danych a to chyba nie powinno generowac tego typu problemow ?

Jutro postaram sie przedstawic moze troche dokladniej ten problem, teraz jeszcze sam troche powalcze :/

[Kshyhoo]

Spróbuj zmienić w skrypcie odbiorczej z wiad_czyj=0 na wiad_czyj=1 i analogicznie w nadawczej.

[stiw]

Zmienilem pierwotne wiad_czyj=1 w nadawczej na 0, a wiad_czyj=0 w odbiorczej na 1 i nic :/ Ciagle to samo, z odbiorczej moge usunac tylko wiadomosci otrzymane od siebie samego, natomiast odczytac nie moge zadnych. W nadawczej natomiast nie moge odczytac i skasowac wiadomosc od siebie , natomiast nie moge nic zrobic jak ktos mi cos wysle :|

Moze to kwestia bazy danych ? Mam ja zrobiona na cba.pl . Typ tych 2 tabeli to innoDB . Nie mam pojecia gdzie moze lezec blad :/ W kodzie modyfikowalem tylko polaczenie z baza. W sesje :

mysql_connect("localhost", "root", "");
mysql_select_db("test");

na

$host = "mysql.cba.pl";
$user = "***";
$pass = "***";
$connect = mysql_connect($host,$user,$pass) or die("Sprawdz polaczenie z serwerem");
mysql_set_charset('utf8',$connect);
mysql_select_db('***_cba_pl',$connect) or die('nie wybrano bazy');



Oraz rejestruj

mysql_connect("localhost", "root", "")or die("Nie mo¿na nawi¹zaæ po³¹czenia z baz¹"); //po³¹czenie z baz¹ danych
mysql_select_db("test")or die("Wyst¹pi³ b³¹d podczas wybierania bazy danych");

na to samo co wyzej :

$host = "mysql.cba.pl";
$user = "***";
$pass = "***";
$connect = mysql_connect($host,$user,$pass) or die("Sprawdz polaczenie z serwerem");
mysql_set_charset('utf8',$connect);
mysql_select_db('***_cba_pl',$connect) or die('nie wybrano bazy');

No ale to chyba nie powinno miec zadnego wplywu na dzialanie skryptu ?

[Kshyhoo]

Coś mi świta, że na początku miałem jakiś problem, ale szybko się z nim uporałem. Wszystko jest sprawą odpowiednich warunków w zapytaniu i wyświetleniu danych. Przemyśl je na spokojnie - na logikę...
EDIT:
Ty masz jakichś userów w bazie danych? Z tego, co widzę, jestem tylko ja...

Ten post edytował Kshyhoo 18.01.2011, 09:36:03

[stiw]

uzytkownicy to login : 1 haslo : 1 , oraz taki uzytkownik 2, haslo 2

Ten post edytował stiw 18.01.2011, 16:20:01

[Kshyhoo]

Założyłem sobie konto
Widzę tylko siebie, mogę wysyłać tylko do siebie i czytać tylko swoje. Coś pokićkałeś w kodzie.

[stiw]

Hm, jak widzisz tylko siebie ? Jak sie chce wyslac wiadomosc masz do wyboru uzytkownikow, 1, 2, oraz siebie. No a w kodzie nic nie pokickalem bo jest zywcem wyjety z archiwum :/

[Kshyhoo]

Teraz widzę teraz usera 1 i 2, widzę wiadomość od usera 1 ale nie mogę jej przeczytać, więc warunki leżą...

[stiw]

O rety to ja juz nie wiem co jest zle :/ Czy to moze byc kwestia serwera cba albo bazy ? :/

Nadawcza.php

[PHP] pobierz, plaintext 
<?
require "sesje.php";
require "naglowek.php";
 
if($_GET["id"]){
$id=intval($_GET["id"]);//zamieniamy zmiennš na liczbe, zapobiega to atakom typu sql injection
if(mysql_num_rows(mysql_query("select wiad_od from wiadomosci where wiad_id=$id and wiad_czyj=0 and wiad_do=".$_SESSION["zalogowany"]))){
mysql_query("delete from wiadomosci where wiad_id=$id");   //sprawdzamy czy wiadomoœć którš użytkownik chce usunšć faktycznie ależy do niego
echo "Usunięto wiadomoœć!<br>";
}
}
 
else if($_GET["co"]){
$co=intval($_GET["co"]);//zamieniamy zmiennš na liczbe, zapobiega to atakom typu sql injection
if(mysql_num_rows(mysql_query("select wiad_od from wiadomosci where wiad_id=$co and wiad_czyj=0 and wiad_do=".$_SESSION["zalogowany"]))){
$wynik=mysql_query("select * from wiadomosci where wiad_id=$co and wiad_czyj=0");  //po raz kolejny sprawdzamy czy wiadomoœć należy do użytkownika
$rekord=mysql_fetch_array($wynik);
$odbiorca=mysql_fetch_array(mysql_query("select user_login from users where user_id=".$rekord["wiad_do"]));
echo "<br><br><table border=1><tr><td>Odbiorca: ".$odbiorca["user_login"]."</td><td>Data: ".date("d/m/Y H:i", strtotime($rekord["wiad_data"]))."</td><td><a href='odbiorcza.php?id=".$rekord["wiad_id"]."'>usuń</a></td></tr>";
echo "<tr><td colspan=3>Temat: ".$rekord["wiad_temat"]."</td></tr>";
echo "<tr><td colspan=3>".$rekord["wiad_tresc"]."</td></tr>";
echo "</table>";
}
}
 
else{
$wynik=mysql_query("select * from wiadomosci where wiad_od=".$_SESSION["zalogowany"]." and wiad_czyj=0 order by wiad_data ");
echo "<table><tr><td>Odbiorca</td><td>Temat</td><td>Data</td><td>&nbsp;</td></tr>";
if(!mysql_num_rows($wynik))echo "<tr><td colspan=4 style='text-align:center'>Nie masz żadnych wiadomoœci!</td></tr>";
else while($rekord=mysql_fetch_array($wynik)){
$odbiorca=mysql_fetch_array(mysql_query("select user_login from users where user_id=".$rekord["wiad_do"]));
echo "<tr><td>".$odbiorca["user_login"]."</td><td><a href='nadawcza.php?co=".$rekord["wiad_id"]."'>".$rekord["wiad_temat"]."</td><td>".date("d/m/Y H:i", strtotime($rekord["wiad_data"]))."</td><td><a href='nadawcza.php?id=".$rekord["wiad_id"]."'>usuń</a></td></tr>";
}
echo "</table>";
}
require "stopka.php";
?>
[PHP] pobierz, plaintext 

Odbiorcza.php

[PHP] pobierz, plaintext 
<?
require "sesje.php";
require "naglowek.php";
 
if($_GET["id"]){
$id=intval($_GET["id"]); //zamieniamy zmiennš na liczbe, zapobiega to atakom typu sql injection
if(mysql_num_rows(mysql_query("select wiad_od from wiadomosci where wiad_id=$id and wiad_czyj=1 and wiad_od=".$_SESSION["zalogowany"]))){
mysql_query("delete from wiadomosci where wiad_id=$id");   //sprawdzamy czy wiadomoœć którš użytkownik chce usunšć faktycznie ależy do niego
echo "Usunięto wiadomoœć!<br>";
}
}
 
else if($_GET["co"]){
$co=intval($_GET["co"]); //zamieniamy zmiennš na liczbe, zapobiega to atakom typu sql injection
if(mysql_num_rows(mysql_query("select wiad_od from wiadomosci where wiad_id=$id and wiad_czyj=1 and wiad_od=".$_SESSION["zalogowany"]))){
mysql_query("update wiadomosci set wiad_przeczytane=1 where wiad_id=$co");          //po raz kolejny zostaje sprawdzony warunek, 
$wynik=mysql_query("select * from wiadomosci where wiad_id=$co and wiad_czyj=1");   //który również sprawdza właœciciela wiadomoœci
$rekord=mysql_fetch_array($wynik);
$nadawca=mysql_fetch_array(mysql_query("select user_login from users where user_id=".$rekord["wiad_od"]));
echo "<br><br><table><tr><td>Nadawca: ".$nadawca["user_login"]."</td><td>Data: ".date("d/m/Y H:i", strtotime($rekord["wiad_data"]))."</td><td><a href='odbiorcza.php?id=".$rekord["wiad_id"]."'>usuń</a></td></tr>";
echo "<tr><td colspan=3>".$rekord["wiad_temat"]."</td></tr>";
echo "<tr><td colspan=3>".$rekord["wiad_tresc"]."</td></tr>";
echo "</table>";
}
}
 
else{
$wynik=mysql_query("select * from wiadomosci where wiad_do=".$_SESSION["zalogowany"]." and wiad_czyj=1 order by wiad_data");
 
echo "<table><tr><td>Nadawca</td><td>Temat</td><td>Data</td><td>&nbsp;</td></tr>";
if(!mysql_num_rows($wynik))echo "<tr><td colspan=4 style='text-align:center'>Nie masz żadnych wiadomoœci!</td></tr>";
else while($rekord=mysql_fetch_array($wynik)){
$nadawca=mysql_fetch_array(mysql_query("select user_login from users where user_id=".$rekord["wiad_od"]));
$kw1="";$kw2="";
if(!$rekord["wiad_przeczytane"]){$kw1="<b>";$kw2="</b>";}
echo "<tr><td>".$nadawca["user_login"]."</td><td><a href='odbiorcza.php?co=".$rekord["wiad_id"]."'>$kw1".$rekord["wiad_temat"]."$kw2</td><td>".date("d/m/Y H:i", strtotime($rekord["wiad_data"]))."</td><td><a href='odbiorcza.php?id=".$rekord["wiad_id"]."'>usuń</a></td></tr>";
}
echo "</table>";
}
require "stopka.php";
?>
[PHP] pobierz, plaintext 

newmsg.php

[PHP] pobierz, plaintext 
<?
require "sesje.php";
require "naglowek.php";
 
if($_POST["tresc"] && $_POST["do"] && $_POST["temat"]){
mysql_query("insert into wiadomosci values(NULL, '".htmlspecialchars($_POST["tresc"])."', ".$_SESSION["zalogowany"].", ".intval($_POST["do"]).", 0, NOW(), '".htmlspecialchars($_POST["temat"])."', 0)");
mysql_query("insert into wiadomosci values(NULL, '".htmlspecialchars($_POST["tresc"])."', ".$_SESSION["zalogowany"].", ".intval($_POST["do"]).", 0, NOW(), '".htmlspecialchars($_POST["temat"])."', 1)");
echo "<br><br>Wysłano wiadomoœć!<br>";
}
else if($_POST["submit"]){
echo "<br><br>Nie uzupełniono wszystkich pól!<br>";
}
echo "<form action='newmsg.php' method=post>";
echo "<br>Temat: <input name=temat size=30>";
echo "<br>Do kogo: <select name=do>";
$wynik=mysql_query("select user_login, user_id from users order by user_login");
while($rekord=mysql_fetch_array($wynik)){
echo "<option value=".$rekord["user_id"].">".$rekord["user_login"];
}
echo "</select><br>";
echo "Treœć: <br><textarea name='tresc' rows=8 cols=50></textarea>";
echo "<br><input type=submit value='wyœlij wiadomoœć' name=submit>";
require "stopka.php";
?>
[PHP] pobierz, plaintext 

Ten post edytował stiw 18.01.2011, 09:50:08

[Kshyhoo]

Raczej nie, to sprawa warunków, tak jak napisałem wcześniej. Mam ten skrypt w wersji podstawowej. Daj mi trochę czasu, to może do czegoś dojdę.

[stiw]

Ok dzieki Mam nadzieje ze Ci sie uda dojsc co i jak . Wyzej wstawilem 3 kody, nadawcza odbiorcza i newmsg , bo tam pewnie gdzies tkwi problem.

Ja tez przepatrze co i jak, a nuz cos wynajde :|

Pozdrawiam.

[Kshyhoo]

U mnie:
1. Skrzynka odbiorcza działa dobrze:
- pokazuje listę wiadomości
- mogę odczytać wszystkie przychodzące wiadomości
2. Skrzynka nadawcza działa źle:
- wyświetla listę wiadomości
- nie mogę odczytać wysłanych wiadomości
Wnioski:
Zapytanie w nadawcza.php:

[PHP] pobierz, plaintext 
if(mysql_num_rows(mysql_query("select wiad_od from wiadomosci where wiad_id=$co and wiad_czyj=1 and wiad_do=".$_SESSION["zalogowany"]))) {
[PHP] pobierz, plaintext 

powinno wyglądać:

[PHP] pobierz, plaintext 
if(mysql_num_rows(mysql_query("select wiad_od from wiadomosci where wiad_id=$co and wiad_czyj=1 and wiad_od=".$_SESSION["zalogowany"]))) {
[PHP] pobierz, plaintext 

Czyli dwa razy trzeba zmienić wiad_do na wiad_od
To tyle, mam nadzieję, że już wszystko będzie ok (chyba, że coś skopałeś).

[stiw]

Ok, teraz nadawcza juz dziala tzn moze zobaczyc wiadomosc ktora wyslalem do kogos innego, nie tylko do siebie, natomiast dalej nie dziala w nadawczej kasowanie

Odbiorcza dalej ten sam problem co byl.

[Kshyhoo]

Zobaczę, co się dzieje. U mnie odbiorcza działa, więc potem podeślę kod.

To kody, które działają u mnie poprawnie:
odbiorcza.php

[PHP] pobierz, plaintext 
<?
require "sesje.php";
require "naglowek.php";
 
if($_GET["id"]) {
	$id=intval($_GET["id"]); //zamieniamy zmienną na liczbe, zapobiega to atakom typu sql injection
	if(mysql_num_rows(mysql_query("select wiad_od from wiadomosci where wiad_id=$id and wiad_czyj=0 and wiad_do=".$_SESSION["zalogowany"]))) {
		mysql_query("delete from wiadomosci where wiad_id=$id");   //sprawdzamy czy wiadomość którą użytkownik chce usunąć faktycznie ależy do niego
		echo "Usunięto wiadomość!<br>";
	}
} else if($_GET["co"]) {
	$co=intval($_GET["co"]); //zamieniamy zmienną na liczbe, zapobiega to atakom typu sql injection
	if(mysql_num_rows(mysql_query("select wiad_od from wiadomosci where wiad_id=$co and wiad_czyj=0 and wiad_do=".$_SESSION["zalogowany"]))) {
		mysql_query("update wiadomosci set wiad_przeczytane=1 where wiad_id=$co");  //po raz kolejny zostaje sprawdzony warunek, 
		$wynik=mysql_query("select * from wiadomosci where wiad_id=$co and wiad_czyj=0");   //który również sprawdza właściciela wiadomości
		$rekord=mysql_fetch_array($wynik);
		$nadawca =mysql_fetch_array(mysql_query("select user_login from users where user_id=".$rekord["wiad_od"]));
		echo "<br><br><table><tr><td>Nadawca: ".$nadawca["user_login"]."</td><td>Data: ".date("d/m/Y H:i", strtotime($rekord["wiad_data"]))."</td><td><a href='odbiorcza.php?id=".$rekord["wiad_id"]."'>usuń</a></td></tr>";
		echo "<tr><td colspan=3>".$rekord["wiad_temat"]."</td></tr>";
		echo "<tr><td colspan=3>".$rekord["wiad_tresc"]."</td></tr>";
		echo "</table>";
	}
} else {
	$wynik=mysql_query("select * from wiadomosci where wiad_do=".$_SESSION["zalogowany"]." and wiad_czyj=0 order by wiad_data");
	echo "<table><tr><td>Nadawca</td><td>Temat</td><td>Data</td><td>&nbsp;</td></tr>";
	if(!mysql_num_rows($wynik))echo "<tr><td colspan=4 style='text-align:center'>Nie masz żadnych wiadomości!</td></tr>";
	else while($rekord=mysql_fetch_array($wynik)) {
		$nadawca=mysql_fetch_array(mysql_query("select user_login from users where user_id=".$rekord["wiad_od"]));
		$kw1="";
		$kw2="";
		if(!$rekord["wiad_przeczytane"]) {
			$kw1="<b>";
			$kw2="</b>";
		}
		echo "<tr><td>".$nadawca["user_login"]."</td><td><a href='odbiorcza.php?co=".$rekord["wiad_id"]."'>$kw1".$rekord["wiad_temat"]."$kw2</td><td>".date("d/m/Y H:i", strtotime($rekord["wiad_data"]))."</td><td><a href='odbiorcza.php?id=".$rekord["wiad_id"]."'>usuń</a></td></tr>";
	}
	echo "</table>";
}
require "stopka.php";
?>
[PHP] pobierz, plaintext 

nadawcza.php

[PHP] pobierz, plaintext 
<?
require "sesje.php";
require "naglowek.php";
 
if($_GET["id"]) {
	$id=intval($_GET["id"]);//zamieniamy zmienną na liczbe, zapobiega to atakom typu sql injection
	if(mysql_num_rows(mysql_query("select wiad_do from wiadomosci where wiad_id=$id and wiad_czyj=1 and wiad_od=".$_SESSION["zalogowany"]))) {
		mysql_query("delete from wiadomosci where wiad_id=$id");   //sprawdzamy czy wiadomość którą użytkownik chce usunąć faktycznie ależy do niego
		echo "Usunięto wiadomość!<br>";
	}
} else if($_GET["co"]) {
	$co=intval($_GET["co"]); //zamieniamy zmienną na liczbe, zapobiega to atakom typu sql injection
	if(mysql_num_rows(mysql_query("select wiad_do from wiadomosci where wiad_id=$co and wiad_czyj=1 and wiad_od=".$_SESSION["zalogowany"]))) {
		$wynik=mysql_query("select * from wiadomosci where wiad_id=$co and wiad_czyj=1"); //po raz kolejny sprawdzamy czy wiadomość należy do użytkownika
		$rekord=mysql_fetch_array($wynik);
		$odbiorca=mysql_fetch_array(mysql_query("select user_login from users where user_id=".$rekord["wiad_do"]));
		echo "<br><br><table border=1><tr><td>Odbiorca: ".$odbiorca["user_login"]."</td><td>Data: ".date("d/m/Y H:i", strtotime($rekord["wiad_data"]))."</td><td><a href='odbiorcza.php?id=".$rekord["wiad_id"]."'>usuń</a></td></tr>";
		echo "<tr><td colspan=3>Temat: ".$rekord["wiad_temat"]."</td></tr>";
		echo "<tr><td colspan=3>".$rekord["wiad_tresc"]."</td></tr>";
		echo "</table>";
	}
} else {
	$wynik=mysql_query("select * from wiadomosci where wiad_od=".$_SESSION["zalogowany"]." and wiad_czyj=1 order by wiad_data ");
	echo "<table><tr><td>Odbiorca</td><td>Temat</td><td>Data</td><td>&nbsp;</td></tr>";
	if(!mysql_num_rows($wynik))echo "<tr><td colspan=4 style='text-align:center'>Nie masz żadnych wiadomości!</td></tr>";
	else while($rekord=mysql_fetch_array($wynik)) {
		$odbiorca=mysql_fetch_array(mysql_query("select user_login from users where user_id=".$rekord["wiad_do"]));
		echo "<tr><td>".$odbiorca["user_login"]."</td><td><a href='nadawcza.php?co=".$rekord["wiad_id"]."'>".$rekord["wiad_temat"]."</td><td>".date("d/m/Y H:i", strtotime($rekord["wiad_data"]))."</td><td><a href='nadawcza.php?id=".$rekord["wiad_id"]."'>usuń</a></td></tr>";
	}
	echo "</table>";
}
require "stopka.php";
 
?>
[PHP] pobierz, plaintext 

[stiw]

Ok, nadawcza dziala juz calkowicie poprawnie, moge odczytywac i kasowac wszystkie wiadomosci . Problem polegal na tym ze tam byly 2 zapytania takie same, te ktore wymieniles wyzej by zmienic, ja zmienilem tylko jedno .

Natomiast teraz w odbiorczej nic nie dziala Poza kasowaniem tylko swoich wiadomosci. Tez cos pewnie nie tak z tymi wiad_do, wiad_od :/



EDIT : dopiero teraz zauwazylem twoj wpis. Juz odbiorcza dziala, wszystko elegancko Jeszcze potestuje, ale wielkie dzieki Ksyhoo, nie wiem co sie za blad tam wkradl :|:|

Ten post edytował stiw 18.01.2011, 13:24:25

[Kshyhoo]

Nie wiedzieć czemu, widzę tylko kilka znaków w tytule i w treści... Nie grzebałeś coś w bazie danych?