frage zum schönes UserManagement

[frank7l7]

sag mal wie geht das den du hast schneller gepostet als ich? wenn man die sekunden an den postzeit sehen könnte -> im millisekunden bereich

also du meinst lieber neues passwort generieren als mit dem schon verschlüsselten auch eine unverschlüsselte kopie zu speichern -> richtig

[Abraxax]

richtig

[Fry2k]

wie kann ich am besten in der IF bedingung den captcha mit einbauen ?

http://www.php-resource.de/forum/sho...threadid=39848

[TobiaZ]

Heute schon der zeweite, der zwar ehrenwerter Weise gesucht hat und sich auf einen vorhandenen Thread bezieht, aber sein Posting so unverständlich und bezugslos verfasst, dass man nicht erahnen kann, was die wirkliche Absicht ist.

[Fry2k]

meine absicht ist es das UserManagement script hier(http://www.php-resource.de/forum/sho...threadid=15290) mit dem Captcha script zu verbinden. Das heißt er soll prüfen ob username, passwort + captcha code richtig ist.Wenn ja soll er sich einloggen. Und wenn es falsch ist soll er sich nicht einloggen.

[TobiaZ]

Gut. wo sind deine Ansätze? Was klappt nicht? Fehlermeldungen? Das übliche halt.

[Fry2k]

also hab mir gedacht das ich die captcha abfrage auch in einer Funktion mache wie die function CheckUser das sollte denn so aussehen:

<?php
$errorC = TRUE;
$error = TRUE;

function CheckCaptcha ($session) {
if(!empty($_POST))
{
    if(!isset($_SESSION['secCheck']))
        $errorC = TRUE;

    if($_SESSION['secCheck'] != $_POST['secCheck'])
        $errorC = TRUE;
    else
        $errorC = FALSE;
$error = FALSE;
}
if(!$errorC && !$error)
{
....... ?!?!?

function CheckUser

function CheckUserLogin ($un, $pw, $session) {
    $result = mysql_query("SELECT UserId,Name,Vorname,UserName,UserPasswd FROM AdminUser
                    WHERE UserName='$un' AND UserPasswd=MD5('$pw')") or mysql_error();
    $zeile = mysql_fetch_array($result);

    if ( $zeile["UserId"] == "" )
    {
    }
    else
    {
      $result = mysql_query("UPDATE AdminUser SET UserSession = '".$session."',UserLogin = NOW()
                    WHERE UserId = '".$zeile["UserId"]."'") or mysql_error();
      ........??!?!
    }
  } 

mit der index.php soll jetzt geprüft werden ob function CheckUser und function CheckCaptcha zutreffen wenn ja soll er auf admin.php gehen sonst zurück auf index.php


index.php

<?php

    session_start();

    include("connect2db.inc.php");
    include("usermanagment.inc.php");

    if (isset($_POST['action']) && $_POST['action']=='login')
    {
        $un = (isset($_POST['un'])) ? $_POST['un'] : '';
        $pw = (isset($_POST['pw'])) ? $_POST['pw'] : '';
        If ((CheckUserLogin ($un, $pw, session_id());) & (function CheckCaptcha))
        {        
        header ("Location: admin.php");
        else
        header ("Location: index.php");
    }
    
    if (isset($_GET['action']) && $_GET['action']=='logout')
    {
        UserLogout (session_id());
    }

?>

mein Problem ist es das ich nicht weiß was am Ende von CheckUser und CheckCaptcha hinmuss

[TobiaZ]

Du bekommst doch bestimmt nen parseerror, oder?

[Fry2k]

jo

Parse error: parse error, unexpected T_STRING in /var/www/script/index.php on line 13

ich schätze mal das er damit nix in der index.php anfangen kann sondern brauch die im UserManagement.php wo sie auch waren (also wo auch die ganzen function CheckUser ... u.s.w. drin sind). Aber wenn ich die jetzt da wieder einbaue kriege ich wieder nicht meine 2 bedingungen zustande

<?php

    session_start();

    include("connect2db.inc.php");
    include("usermanagment.inc.php");

    if (isset($_POST['action']) && $_POST['action']=='login')
    {
        $un = (isset($_POST['un'])) ? $_POST['un'] : '';
        $pw = (isset($_POST['pw'])) ? $_POST['pw'] : '';
        if((CheckUserLogin ($un, $pw, session_id()) & (CheckCaptcha))
        header ("Location: admin.php");
        else
        header ("Location: index.php");
    }

    if (isset($_GET['action']) && $_GET['action']=='logout')
    {
        UserLogout (session_id());
    }

?>

[TobiaZ]

Und was passt jetzt an dem neuen Code nicht?

[Fry2k]

wie ich die beiden sachen verknüpfen kann interessiert mich

hier sieht man das wenn das erste zutrifft soll er index.php springen sonst auf admin.php

function CheckUserLogin ($un, $pw, $session) {
    $result = mysql_query("SELECT UserId,Name,Vorname,UserName,UserPasswd FROM AdminUser
                    WHERE UserName='$un' AND UserPasswd=MD5('$pw')") or mysql_error();
    $zeile = mysql_fetch_array($result);

    if ( $zeile["UserId"] == "" )
    {
      header ("Location: index.php");
    }
    else
    {
      $result = mysql_query("UPDATE AdminUser SET UserSession = '".$session."',UserLogin = NOW()
                    WHERE UserId = '".$zeile["UserId"]."'") or mysql_error();
      header ("Location: admin.php");
    }
  } 

und hier ist es so das wenn $errorC und $error true sind gewährt er den zugriff und sonst nicht denke ich.

if(!empty($_POST))
{
    if(!isset($_SESSION['secCheck']))
        $errorC = TRUE;

    if($_SESSION['secCheck'] != $_POST['secCheck'])
        $errorC = TRUE;
    else
        $errorC = FALSE;

    $error = FALSE;
}
if(!$errorC && !$error)
{
?>
     <b>Ich bin drin!</b>
<?php
}
else
{

dieses ganze true und false irretiert mich ein wenig. und das ausrufezeichen von "!empty" bedeutet meiner Meinung nach das Gegenteil wenn ich das richtig verstehe.Die Frage ist nur wie verknüpfe ich das jetzt am besten so das der Username und Passwort vom ersten teil zutreffen muss und der zweite teil auch (dann auf admin.php) und wenn eines von den 3 sachen falsch ist dann auf index.php)