🚀 go-pugleaf

Maciek - Wally wrote on 2009-07-27 14:54:

> if (isset($_REQUEST['comment_id']) and $_REQUEST['comment_id'])

"and" moze i dziala ale lepiej uzywac && (po co sobie wyrobic
jakies niestandardowe nawyki)

>     $sql1 = "SELECT entry_id FROM comments WHERE id=" .
> $_REQUEST['comment_id'];
>     $result1 = mysql_query($sql1);

Nie sprawdzasz czy query sie wykonalo poprawnie, a powinienes.

>     $row1 = mysql_fetch_array($result1);
>     $link = "viewentry.php?entry_id=" . $row1['entry_id'] . "#comments";

Nie sprawdzasz czy mysql_fetch_array() sie wykonalo poprawnie, a powinines.

>     $sql = "DELETE FROM comments WHERE id=" . $_REQUEST['comment_id'];

Poczytaj o SQL Injection, bo aktualny kod u Ciebie to jedna wielka
dziura.

> Z góry dziękuję za pomoc w rozwiązaniu problemu i przepraszam, jeżeli
> popełniłem jakiś trywialny błąd...

ustaw sobie error_reporting(E_ALL) na poczatku skryptu.


Pozdrawiam,
-- 
Szyfrowanie skryptow PHP v4 oraz v5. Blokady: czasowe, IP, MAC
Loadery dla systemow: Windows, Linux, MacOS, Free/BSD, Solaris
Zabezpiecz swoje skrypty PHP!  Przetestuj http://phpencoder.pl