Beiträge von cottton

<?php
    public function login($email, $password)
    {
    	if(!strlen($email)) { 
    		// error ausgeben: feld kann nicht leer sein 
    		return false;
    	} 
    	if(!strlen($password)) { 
    		// error ausgeben: feld kann nicht leer sein 
    		return false;
    	}
    	$user = $this->getUserByEmail($email);
		if($user and is_array($user) and !empty($user)) { 
			// if würde hier noch die felder prüfen wie zB if($this->isValidUser($user)) ... nur so am rande
            if(password_verify($password, $user['password'])) {
            	// würde das session setzen auslagern like: $this->setSession($user); ... auch nur so am rande
              	$_SESSION['email'] = $userRow['email'];
              	$_SESSION['vorname'] = $userRow['vorname'];            	
              	//$_SESSION['password'] = $userRow['password']; das passwort sollte nicht mehr gebraucht werden
                return true;
            } 
		} 
		// error ausgeben: email oder passwort falsch
		return false;
    } 
    public function getUserByEmail($email) 
    {
    	/* INFO
    	function gibt einfach nur den gefundenen nutzer zurück, oder false (wenn nicht email existiert)
    	*/ 
        try  {
        	$sql = "SELECT `email`, `password`, `vorname` FROM `user_daten` WHERE `email` = :email LIMIT 1;";
        	/* INFO
        	- LIMIT 1, weil wir ja nur einen nutzer erwarten. (gäbe es 2 mit der gleichen email, dann wäre etwas von grund auf falsch)
			- abschließendes semikolon ; immer. warum sollte man die query "offen" lassen (es sei denn du hängst per PHP noch was an ...).
        	*/
        	// prepare
        	$stmt = $this->db->prepare($sql);
        	if(!$stmt) {
        	    // fehler auslesen mit: $this->db->errorInfo() 
        	    // könnte hier $this->errorLast = $this->db->errorInfo(); dir überlassen
        	    return false;
        	} 




        	// execute
        	$para = array(':email'=>$email);
        	if(!$stmt->execute($para)) {
        	    // fehler auslesen mit: $this->db->errorInfo() 
        	    // könnte hier $this->errorLast = $this->db->errorInfo(); dir überlassen
        	    return false;
        	}




        	// fetch
	        /* INFO
			bei PDO gibt es kein ->fetchFirst(). du könntest fetchAll() nutzen, oder eben so:
	        */
	        $row = $stmt->fetch(PDO::FETCH_ASSOC); // eine row lesen (erwarten ja nur eine)
	        /* INFO
	        $row sollte in etwa so aussehen:
	        $row = array(
				'email' => '...daten...',
				'password' => '...daten...',
				'vorname' => '...daten...',
	        )
	        */
	        return $row;  
        } catch(PDOException $e) {
            //echo $e->getMessage(); // exception ausgeben kann fatal sein, wenn benutzer daten in der exception enthalten sind (pw!)
            return false; // NEU -- hier sollte auch false zurückgegeben werden
        }
    }

public function login($email,$password, $vorname) // , (komma) vergessen
    {
        try
        {
            $stmt = $this->db->prepare("SELECT email,password,vorname FROM user_daten WHERE email = :email OR :vorname");
            /* fatal error: die query hier ergibt folgendes:
            	SELECT email,password,vorname FROM user_daten 
            	WHERE email = 'bsp@bsp.de' OR 'benutzername'




            	diese query gibt immer den nutzer zurück, der die email besitzt
            	ODER den ersten in der reihe!




            	warum: OR 'ein nicht leerer string' 
            		wird als WAHR (es ist, also es ist nicht "nicht") gedeutet.
            		Ähnliches bsp: OR 1




            	richtig wäre: 
	            	SELECT email,password,vorname FROM user_daten 
            		WHERE email = :email OR vorname = :vorname -- NEU: vorname = 
            */
            $stmt->execute(array(':email'=>$email, ':vorname'=>$vorname));
            #$userRow=$stmt->fetch(PDO::FETCH_ASSOC); unnötig
            $count = $stmt->columnCount()// NEU
            #if($stmt->rowCount() > 0)
            if($count > 0) // NEU
            {
                if(password_verify($password, $userRow['password']))
                {
                    $_SESSION['email'] = $userRow['email'];
                    $_SESSION['vorname'] = $userRow['vorname'];
                    return true;
                }
                else
                {
                    return false;
                }
            }
        }
        catch(PDOException $e)
        {
            echo $e->getMessage();
            return false; // NEU -- hier sollte auch false zurückgegeben werden
        }
    }

$email = $_POST['email'];
$password = $_POST['password'];
$vorname = $_POST['vorname'];
$nachname = $_POST['nachname'];




$error = '';




include 'sec/verbindung.inc.php'; // ich denke mal $DB_con wird hier definiert
// evtl hast du die verbindung schon vorher aufgebaut -
// und $DB_con ist schon definiert -- dann brauchst du das include gar nicht
function check($DB_con, $email) { // $DB_con mit reinschicken -- die 
	$stmt = $DB_con->prepare("SELECT email FROM user_daten WHERE email = ?"); 
	$stmt->bindParam(1, $email);
	$stmt->execute();




	if($stmt->rowCount() > 0) {
    	return true;
	} else {
    	return false;
	}
}




if(check($email) === false) {
    $error = 'Email existiert';
}

<?php
require_once 'dbconfig.php'; 




if(isset($_POST['senden']) && strtolower($_POST['senden'])) {




    if(strtoupper($_SERVER['REQUEST_METHOD']) == 'POST') { // du guckst in $_POST nach daten, dann is klar, dass es eine POST request is =)

        $email = htmlentities($_POST['email'], ENT_QUOTES, "UTF-8"); // NEIN ... warum denn?
        $password = $_POST['password'];
        $vorname = htmlentities($_POST['vorname'], ENT_QUOTES, "UTF-8");// NEIN ... warum denn?
        $nachname = htmlentities($_POST['nachname'], ENT_QUOTES, "UTF-8");// NEIN ... warum denn?




        $error = '';




        // die function sollte nicht hier definiert werden. die kann doch am anfang oder sogar am ende des scripts stehen
        function check($email) {
	        include 'sec/verbindung.inc.php'; // sicherlich deine db verbindung. das wird bei JEDEM funktions aufruf erneut geladen
	        									// bei jedem aufruf wird (evtl) erneut eine verbindung aufgebaut, anstatt die vorhandene zu nutzen
	        #$sql = 'SELECT email FROM user_daten WHERE email = $email'; // NEIN!!!! "SQL-Injection" -- forum/google suchen und lesen!

	        // hier liegt btw das eigl problem
	        // 	in '' einfachen anführungszeichen wird eine $var nicht platziert. zu hast den >text< $email da rein geschrieben
			// 	du hast also immer abgefragt: gibt mir email von user_daten wo email $email ist =)
			// 	ergo: immer 0 (keine) einträge gefunden




	        $sql = 'SELECT email FROM user_daten WHERE email = ?';
	        $insert->BindValue(1, $email); // NEU 
	        $stmt = $DB_con->prepare($sql);
	        // hier fehlt doch ->execute ? 
	        $stmt->execute() // NEU 
	        $check_row = $stmt->fetch(PDO::FETCH_ASSOC); // weiss grad nicht ob du unbeding fetch´en musst, wenn du nur den rowcount brauchst




	        if($stmt->rowCount() > 0) {
	            return true;
	        } else {
	            return false;
	        }
	    }

    	// schreib doch das, was in der check funktion steht hier direkt hin =)
        if(check($email) === false) {
            $error = 'Email existiert';
        }




        if(!$error) { 
	        $insert = $DB_con->prepare("INSERT INTO user_daten (email, password, vorname, nachname) VALUES (?, ?, ?, ?"); 
	        $insert->BindValue(1, $email);
	        $insert->BindValue(2, password_hash($password, PASSWORD_BCRYPT, $option)); 
	        $insert->BindValue(3, $vorname);
	        $insert->BindValue(4, $nachname);
	        if($insert->execute()) {
	        	exit("<article id='erfolg_insert'><section id='content'><p class='susc_reg'>Sie haben erfolgreich einen Account erstellt.</p> <br> <p class='after_reg'><a href='login.php' class='now_log_after_reg'>Jetzt Einloggen</a></p> </section></article><?php");
	        	// was macht das <?php über mir am ende des strings? ...section></article><?php	     
	        }
   		}




	}
}
?>
    <article id="a1-reg">
        <section id="content">
        <h2 class="h2-reg"> Registrieren </h2>
        <h3 class="h3-reg"> Kostenlos und schnell Registrieren </h3>
        <p class="genericon genericon-user"></p>
        </section>
    </article>

    <article id="a2-reg">
        <section id="content">
        <form action="<?php $_SERVER['REQUEST_METHOD']?>" method="POST">


         <?php if(isset($error)) {   ?>
         <article id='error_info'>
         <section id='content'>
         <?php echo $error; ?>
         </section>
         </article>
         <?php  } ?>

<p><input type="email" name="email" placeholder="E-Mail Adresse"> <input type="email" name="atmail-g" placeholder="E-Mail Adresse Wiederholen"></p>
        <p><input type="password" name="password" placeholder="Passwort"> <input type="password" name="psw-g" placeholder="Passwort Wiederholen"></p>
        <p><input name="vorname" placeholder="Ihr Vorname"> <input name="nachname" placeholder="Ihr Nachname"></p>
        <p class="info-agb"> Beim Senden der Registration stimmen Sie automatisch der <a href="#" class="agb"> AGB</a> zu.</p>
        <p><input type="submit" class="send" value="Senden" name="senden"> <input type="reset" class="reset" value="Zur&uuml;cksetzen"></p>
        </section>
    </article>




</main>
</body>
</html>

-- tabelle kunden
id  | kdnr  | name | ...
------------------------
1   |123    | hans
2   |124    | bernd
3   |125    | wurst




-- tabelle kundenerinnerung
id  | msg           | something         | ...
-------------------------------------------------
3   |'text 3.1'     | 'something 3.1'
3   |'text 3.2'     | 'something 3.2' 
3   |'text 3.3'     | 'something 3.3' 
2   |'text 2.1'     | 'something 2.1'