Fehler in Funktion

[Trexx]

Hallo,
habe ein login script was nach usernamen und userssesions überprüft nun wollte ich es aber auch für mehrere USer zur verfügung stellen jedoch in einer und der selben Tabelle nur bin ich ein ziemliicher grünschnabel in sachen php und mysql.
Bitte schaut euch mal die folgende funktion an und nennt mir bitte meine fehler wäre für eure hilfe recht dankbar

function logged_in()
{
    $sql="SELECT UserIdFROM users WHERE "Domain ='".Domain."' AND "UserSession='".session_id()."'LIMIT 1";
    $result= mysql_query($sql);
      return ( mysql_num_rows($result)==1);
} 

Und hier einmal das Formular worüber sich der user einloggen muss.

<?php


session_start();
include 'inc/connect.inc.php';
include 'Config/Datenbank.php';
if (isset($_POST['login']))
{
    $userid=check_user($_POST['domain'],$_POST['username'], $_POST['userpass']);
    if ($userid!=false)
        login($userid);
    else
        echo 'Ihre Anmeldedaten waren nicht korrekt!';
}
if (!logged_in()){
    echo '<form method="post" action="login.php">
        <label>Domain:</label><input name="domain" type="text"><br>
        <label>Benutzername:</label><input name="username" type="text"><br>
        <label>Passwort: </label><input name="userpass" type="password" id="userpass"><br>
        <input name="login" type="submit" id="login" value="Einloggen">
    </form>';
    }
else {



$connection = mysql_connect($host, $user, $Passwort)or die("Keine Verbindung möglich: " . mysql_error());
mysql_select_db($Datenbank) or die ("Die Datenbank konnte nicht ausgewählt werden!");
$anfrage = mysql_query("SELECT * FROM users WHERE  Domain = '$domain' AND UserName = '$username'");
//echo mysql_errno().": ".mysql_error()."<BR>";


?>

Thanks für eure hilfe

[Abraxax]

Re: Fehler in Funktion

Original geschrieben von Trexx

function logged_in()
{
    $sql="SELECT UserIdFROM users WHERE "Domain ='".Domain."' AND "UserSession='".session_id()."'LIMIT 1";
    $result= mysql_query($sql);
      return ( mysql_num_rows($result)==1);
} 

schaue dir doch mal das syntaxhighlighting genau an. da siehst du auch, was du schon alles falsch gemacht hast.

desweiteren scheinen dir die grundlagen in mysql zu fehlen. möglicherweise fehlen dir auch grundlagen des copy&paste, denn das sieht als basis sehr nach meinem 'schönen usermanagement' aus.

[Trexx]

was denn?

[Skaschy]

les dir 3 mal den query durch ... dann weisstes

[jahlives]

$sql="SELECT UserIdFROM users WHERE "Domain ='".Domain."' AND "UserSession='".session_id()."'LIMIT 1"; 

ändere das mal in

$sql="SELECT UserId FROM users WHERE Domain ='$Domain' AND UserSession='session_id()' LIMIT 1"; 

Wenn du doppelte Anführungszeichen für den String verwendest, dann darfst du innerhalb des Strings keine doppelten Anführungszeichen haben (ausser du escapest sie). Bei deinem Beispiel sucht er nach einer Funktion domain. Ausserdem solltest du noch eine Var $domain haben, denn sonst suchst du nach dem String domain in deiner DB.

Gruss

tobi

[schlimmerfinger]

Original geschrieben von jahlives

$sql="SELECT UserId FROM users WHERE Domain ='$Domain' AND UserSession='session_id()' LIMIT 1"; 

Du hast zwar den Fehler korrigiert dafür aber ein neues "eingabaut".

cheers

[Abraxax]

damit ist der nun auch weg ....

$sql = "SELECT UserId FROM users
        WHERE
           Domain ='".mysql_escape_string($Domain)."' AND
           UserSession='".session_id()."' LIMIT 1";