Witam. O to mój kod:

[PHP] pobierz, plaintext 
<?php
$id=intval($_GET["usun"]); 
$odbiorca = $user->id;  
 
if($_GET["usun"]){
 
if (usun_list($id, $odbiorca)) {
          echo 'Usunięto.<br />';
        }else{
          echo 'Nie udało się.<br />';
      }  
}
?>
[PHP] pobierz, plaintext 

I funkcja usuń_list();

[PHP] pobierz, plaintext 
<?php
function usun_list($id, $odbiorca)
{
    $lacz = lacz_bd();
  if (!$lacz->query("delete from wiadomosci 
                     where wiad_id='$id' and wiad_do='$odbiorca' and wiad_czyj=0"))
    throw new Exception('Usunięcie listu nie powiodło się.');
 return true;  
}
?>
[PHP] pobierz, plaintext 

Wiadomość usuwa się dobrze jeśli jest to odbiorca listu. Jednak kiedy w linku wpisze np. ?usun=1 (nie mój list) to listu nie usuwa ale pisze "usunięto wiadomość".

Jeżeli rzucasz wyjątkami to musisz je łapać.

Poczytaj o blokach try {}catch() {}

A no tak.. zapomniałem Dałem teraz tak:

[PHP] pobierz, plaintext 
<?php
try
   {
if($_GET["usun"]){
 
 
if (usun_list($id, $odbiorca)) {
          echo 'Usunięto.<br />';
        }else{
          echo 'Nie udało się.<br />';
      }  
}
}
catch (Exception $e)
   {
     echo $e->getMessage();
     exit;
   }
?>
[PHP] pobierz, plaintext 

i nadal piszę "usunięto"

usun_list zwraca true lub rzuca wyjątek...

Więc spróbuj tu if (usun_list($id, $odbiorca)) dać == true

[PHP] pobierz, plaintext 
<?php
if (usun_list($id, $odbiorca) == TRUE) {
         echo 'Usunięto.<br />';
       }else{
         echo 'Nie udało się.<br />';
     }
?>
[PHP] pobierz, plaintext 

Niestety nadal pisze usunięto.

Spróbuj zamiast rzucania wyjątku zwrócić po prostu false...

Ale jeśli np. dam:

[PHP] pobierz, plaintext 
<?php
if (strlen($id)<4)
     {
        throw new Exception('Podaj 4 znaki!');
     }
?>
[PHP] pobierz, plaintext 

To wszytko ładnie się wyświetla i jak dam co najmniej 4 znaki to wyjątku nie ma ;P Popróbuje jeszcze z tym... może wyjdzie.

Przecież w usuń_list() podajesz odbiorcę, który przy usuwaniu jest sprawdzany. Więc jeśli usuwasz nie swój list to zamiast $odbiorca = $user->id; z początku kodów musisz wpisać coś innego. (zakładam, że $user to przeglądający witrynę).

$user->id; To id zalogowanego usera. I jeśli nie jestem odbiorcą czyli np. moje id: 1 to nie mogę usunąć i tak się dzieje, problem w tym, ze i tak wyswietla 'usunięto' zamiast 'nie udało sie'

Połączenie z bazą jest w porządku, zapytanie jest poprawne i się wykonuje prawidłowo, więc query zwraca true (czy coś w tym stylu) - bo nic innego nie może. Musiałbyś sprawdzić liczbę przetworzonych wierszy, żeby wykryć "błąd".