[MySQL][PHP] Problem z banem, Opcje

[Samael_Pl]

Witam!
Posiadam swój czat gg i mam pewien problem z banowaniem.

Orginalna funkcja wygląda tak:

[PHP] pobierz, plaintext 
function cmd_ban()
{
eval(buildFunc());
   if($user['rank'] < 3)
     $api->reply($text->dialog('no_rights'), 0, 1, 1);
    $select = $db->query("SELECT id, nick, rank, banned, online FROM users WHERE nick LIKE '".$parts[1]."' LIMIT 1");
@   $secuser = $select->fetch_assoc();
    if($select->num_rows == 0 || $secuser['rank'] >= $user['rank'] || $secuser['online'] == 0 || $secuser['banned'] == 1)
      $api->reply($text->dialog('no_rights'), 0, 1, 1);
   $ban = $db->query("UPDATE users SET online=0, banned=1 WHERE id=".$secuser['id']." LIMIT 1");
    $api->send($main->nick($secuser['nick'], $secuser['rank']).' '.$text->dialog('banned_by').' '.$main->nick($user['nick'], $user['rank']));
[PHP] pobierz, plaintext 

Chciałem zrobić coś, aby wyświetlało takze powód zbanowania, więc dodałem dodatkową kolumnę w MySQL "reason" i funkcja wygląda tak:

[PHP] pobierz, plaintext 
function cmd_ban()
{
 eval(buildFunc());
    if($user['rank'] < 3)
      $api->reply($text->dialog('no_rights'), 0, 1, 1);
    $select = $db->query("SELECT id, nick, rank, banned, online, reason FROM users WHERE nick LIKE '".$parts[1]."'LIMIT 10");
@   $secuser = $select->fetch_assoc();
    if($select->num_rows == 0 || $secuser['rank'] >= $user['rank'] || $secuser['online']=0|| $secuser['banned'] == 1)
      $api->reply($text->dialog('no_rights'), 0, 1, 1);
   $txt=$parts[2];
 $ban = $db->query("UPDATE users SET online=0, banned=1, reason=$txt WHERE id='".$secuser['id']."'LIMIT 10");
    $api->send($main->nick($secuser['nick'], $secuser['rank']).' '.$text->dialog('banned_by').' '.$main->nick($user['nick'], $user['rank']).' '.$text->dialog('banned_for').' '.$txt);
[PHP] pobierz, plaintext 

Problem jest w tym, że po dodaniu "reason=$txt" w 11 linijce funkcja przestaje działać.
Po wpisaniu .ban nick powód wyświetla "<NICK> Zostal(a) zbanowany przez <##NICK> Powod: XXX" , ale nie banuje..

[erix]

[PHP] pobierz, plaintext 
eval(buildFunc());
[PHP] pobierz, plaintext 

CO TO JEST?!

[Samael_Pl]

To jest w każdej funkcji i nie tutaj problem.
Chodzi mi o to, że po dodaniu "reason=$txt" funkcja nawala.

[erix]

Ciekawe, skąd Ci się bierze zawartość do tej zmiennej.

Wybacz, ale jak widzę kwiatka w stylu eval = aplikacja źle zaprojektowana. Widocznie się nie pomyliłem, skoro używasz zmiennej, która bierze się niewiadomo skąd.

[Samael_Pl]

Zrozum, że

[PHP] pobierz, plaintext 
    function cmd_ban()
    {
    eval(buildFunc());
    if($user['rank'] < 3)
    $api->reply($text->dialog('no_rights'), 0, 1, 1);
    $select = $db->query("SELECT id, nick, rank, banned, online FROM users WHERE nick LIKE '".$parts[1]."' LIMIT 1");
    @ $secuser = $select->fetch_assoc();
    if($select->num_rows == 0 || $secuser['rank'] >= $user['rank'] || $secuser['online'] == 0 || $secuser['banned'] == 1)
    $api->reply($text->dialog('no_rights'), 0, 1, 1);
    $ban = $db->query("UPDATE users SET online=0, banned=1 WHERE id=".$secuser['id']." LIMIT 1");
    $api->send($main->nick($secuser['nick'], $secuser['rank']).' '.$text->dialog('banned_by').' '.$main->nick($user['nick'], $user['rank']));
[PHP] pobierz, plaintext 

działa poprawnie.

[erix]

Czytaj uważnie:

skoro używasz zmiennej, która bierze się niewiadomo skąd

I z łaski swojej, przejrzyj kod, który Ci nie działa mając na uwadze to, co pogrubiłem.

Albo użyj IDE zamiast edytora, bo takie błędy wykrywają w trakcie pisania.