Problem mit mysql_num_rows

[PHPbeginner]

hi

ich wollte für mein rechtesystem eine funktion schreiben und ein teil davon gibt nur eine error meldung aus. die variablen $rightgroup und $option sind beide gesetzt.

weiss jemand was da falsch ist?

danke

$sql2="SELECT id
    FROM rights
    WHERE groupid='".$rightgroup."' AND option='".$option."' OR option ='ALL' 
    LIMIT 1";
    $result2 = mysql_query($sql2);
if ( mysql_num_rows($result2)==1) 
    {
        return true;
    }
  else {
      return false;
      } 

Fehlermeldung:

Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in 
/home/.../functions.php on line 62

[Deathrow]

Ich würd dir mal empfehlen, mal mysql_error(); zu benutzen

[PHPbeginner]

Hier der MySQL Error:

You have an error in your SQL syntax. 
Check the manual that corresponds to your MySQL server version for the right syntax to use near
 'option='ADMCP' OR option ='ALL' LIMIT 1' at line 3

[peetjoff]

oder so ?

hi.

ich hatte mal mit dem selben problem zu tun.
da gibts die wohl möglichkeit diesen befehl zu verwenden:

mysql_numrows()

sorry, ich weis nicht genau warum, aber das ist wohl ne kompatibilitätsfrage.

keine garantie, aber probiers einfach mal.

ciao


nachtrag:
hab eben die error-rport meldung gelesen.
jetzt: fraglich, ob mein tip was bringt.

[penizillin]

brich den code um!
und guck dir die query an, nämlich zwischen der zusammenstellung und der ausführung, mit die() oder so.

[derHund]

http://dev.mysql.com/doc/mysql/en/reserved-words.html

[PHPbeginner]

Original geschrieben von derHund
http://dev.mysql.com/doc/mysql/en/reserved-words.html

Dann liegt es wohl daran das eine Spalte "option" heisst? Wenn ja, gibt es dann noch eine Möglichekeit das zu umgehen oder muss ich die Spalte umbenennen?

[penizillin]

nachdem du deinen beitrag editiert hast und den code lesbarkeitshalber umgebrochen hast, setzst du das wort option in backticks.

[PHPbeginner]

Okay es funktioniert nun.

Ich hab aber noch ne andere Frage: mein query ist ja im moment so aufgebaut:

[grouip=?, und option=???] ODER [option=ALL]

ich will aber das die bedingungen so aufgebaut sind:

[grouip=?, und option=???] ODER [grouip=?, und option=ALL]

[TobiaZ]

grouip ausklammern, der rest dann in die klammer. logisch drüber nachdenken

[penizillin]

die klammern sind deine freunde.

[TobiaZ]

die klammern sind deine freunde.

OffTopic:

---

klar, wenn man sonst keine hat...

---

[penizillin]

OffTopic:

---

"kennen sie das - sie haben mehr erdnußflips, als freunde?" © schlechte comedy-serie auf einem schlechten sender.

---