Passwort-Ändern-Seite

[marcel580]

Guten Morgen PHP.de-Forum,
ich habe ein kleines Problem mir meiner Passwort Ändern-Website.
Diese soll den User der grade per Session eingeloggt ist die möglichkeit geben, sein
Passwort zu ändern.
Allerdings bekomm ich grade den Fehler:


" Parse error: syntax error, unexpected '{' in /var/www/html/marcel/changepass.php on line 53"


Vielen Dank schonmal im vorraus

<?php
$_SESSION["username"]= $username;
    $oldpw = md5 ($_POST["pw"]); 
    $changepw1 = md5 ($_POST["pw"]); 
    $changepw2 = md5 ($_POST["pw2"]); 
    $errors = array();

    if(!isset ($oldpw) OR (!isset ($changepw1)  OR (!isset ($changepw2)
    {
    echo "Bitte alle Felder ausfüllen";
    }
    else
    {
    if ($changepw1 !== $changepw2) 
    {
        echo "Deine Passwörter stimmen nicht überein.";
    }
    else
    {
    
$ueberpruefen = "SELECT
                                password
                        FROM
                                USER                        
                        WHERE
                            username = ".($_SESSION)."
                            AND 
                                password = ".($oldpw)."
                        ";
                        echo $ueberpruefen;
                        $result = mysql_query($ueberpruefen) OR die("<pre>\n".$ueberpruefen."</pre>\n" .mysql_error());
                $row = mysql_fetch_assoc($result);
                var_dump($_POST);
                if(!$row)
                        $errors[] = "Ihr Passwort konnte leider nicht geändert werden.\n".
                    
    }
    if (count($errors))
    {
        echo "Fehler beim ändern des Passwortes.<br>\n".
                "<br>\n";
        foreach($errors as $error)
                echo $error.  "<br>\n";
    echo  "<br>\n";
                            echo  "<a href=\"changepw.php\"Wiederhole deine Eingabe Bitte, ansonsten wende dich Bitte an den Webmaster.</a>\n";
    }
    else
    {
    
    
    $ueberpruefen = "UPDATE
                                    user
                            SET
                                    password = ".($changepw1)."
                            WHERE
                                    username = ".($_SESSION)."
                            ";
    
    }
    }
                        ?>

[VPh]

Betroffene Zeile hervorheben wäre gut bei solchen Meldungen.
Da fehlt wahrscheinlich irgendwo ein ";" bevor die Klammer zum Einsatz kommt.

[lstegelitz]

 if(!isset ($oldpw) OR (!isset ($changepw1)  OR (!isset ($changepw2) 
    { 
    echo "Bitte alle Felder ausfüllen"; 
    } 

Zähle die Klammern... wieviele geöffnete, wieviele geschlossene

[marcel580]

Die Klammern Stimmen soweit.
Ausjeden Fall wird jede Klammer wieder geschloßen.
Bei:
ini_set('display_errors',1);
error_reporting(-1);

wird mir leider gar nichts angezeigt.

Habe bei
$oldpw = md5 ($_POST['pw']);
die ' durch " erstezt, es hat sich allerdings nichts geändert

[Moewe]

zudem solltest du auch für dich besser einrücken.
Eine verschachtelte If-Anweisung in einer Linie nach unten fortzuführen ist sehr unübersichtlich

    if(!isset ($oldpw) OR (!isset ($changepw1)  OR (!isset ($changepw2)
    {
    echo "Bitte alle Felder ausfüllen";
    }
    else
    {
    if ($changepw1 !== $changepw2) 
    {
        echo "Deine Passwörter stimmen nicht überein.";
    }
    else
    {
         ...
     } 

besser so

if(!isset ($oldpw) OR (!isset ($changepw1)  OR (!isset ($changepw2)
{
    echo "Bitte alle Felder ausfüllen";
}
else
{
    if ($changepw1 !== $changepw2) 
    {
        echo "Deine Passwörter stimmen nicht überein.";
    }
    else
    {
        ....
    }
} 

Zudem kannst du auch gerne ein else if verwenden

[Moewe]

if(!isset ($oldpw) OR (!isset ($changepw1)  OR (!isset ($changepw2) 

BTW:
Da fehlen => )

[MuellerLukas]

Nein, wird sie nicht.
Wir bei sowas immer mal einen Blick auf die Zeile vor dem Fehler.

[Gast]

$oldpw = md5 ($_POST["pw"]);

JUST SAY NO!
http://php.net/manual/de/faq.passwords.php

[marcel580]

if(!isset ($oldpw) OR (!isset ($changepw1)  OR (!isset ($changepw2) 

BTW:
Da fehlen => )

Wen ich bei "
if(!isset ($oldpw)) OR (!isset ($changepw1)) OR (!isset ($changepw2))

" Klammern setzte kommt die Meldung:
Parse error: syntax error, unexpected 'OR' (T_LOGICAL_OR) in /var/www/html/marcel/changepass.php on line 52
.

Ich habe auch Probiert jeweils "if"
zu schreiben:
if (!isset ($oldpw)) OR if(!isset ($changepw1)) OR if(isset ($changepw2))

allerding kommt die Meldung:
Parse error: syntax error, unexpected 'OR' (T_LOGICAL_OR) in /var/www/html/marcel/changepass.php on line 52

[Moewe]

Wen ich bei "
if(!isset ($oldpw)) OR (!isset ($changepw1)) OR (!isset ($changepw2))

" Klammern setzte kommt die Meldung:
Parse error: syntax error, unexpected 'OR' (T_LOGICAL_OR) in /var/www/html/marcel/changepass.php on line 52
.

Ich habe auch Probiert jeweils "if"
zu schreiben:
if (!isset ($oldpw)) OR if(!isset ($changepw1)) OR if(isset ($changepw2))

allerding kommt die Meldung:
Parse error: syntax error, unexpected 'OR' (T_LOGICAL_OR) in /var/www/html/marcel/changepass.php on line 52

MOment, hab nicht deinen nicht gehighlitententen Text gesehen
Jetzt ist es min. eine zu viel

Für den Anfang

if (
        (isset($dings))
        OR
        (!isset($bums))
        OR
        (isset($hans))
   (
   {
    } 

so gingen mir früher weniger durch die Lappen

[marcel580]

JUST SAY NO!
http://php.net/manual/de/faq.passwords.php

Die feinheiten kann ich hinerher noch ändern, darum geht es in meiner Frage aber auch nicht.
Trotzdem Danke

[marcel580]

MOment, hab nicht deinen nicht gehighlitententen Text gesehen
Jetzt ist es min. eine zu viel

Sorry, aber welche Klammern meinst du den?
Habe es jetzt so:

if (!isset ($oldpw)) OR if(!isset ($changepw1)) OR if(isset ($changepw2))

[lstegelitz]

Die Klammern Stimmen soweit.

Eeehm... nein, die stimmen nicht. Ganz sicher...

 if(!isset ($oldpw) OR (!isset ($changepw1)  OR (!isset ($changepw2) 

Wen ich bei "
if(!isset ($oldpw)) OR (!isset ($changepw1)) OR (!isset ($changepw2))

" Klammern setzte kommt die Meldung:
Parse error: syntax error, unexpected 'OR' (T_LOGICAL_OR) in /var/www/html/marcel/changepass.php on line 52

Ja, wenn du Klammern einfach frei Schnauze verteilst, kein Wunder.
Deine IF-Anweisung endet bereits nach "if(!isset ($oldpw))"

Lerne Grundlagen!

[marcel580]

MOment, hab nicht deinen nicht gehighlitententen Text gesehen
Jetzt ist es min. eine zu viel

Für den Anfang

if (
        (isset($dings))
        OR
        (!isset($bums))
        OR
        (isset($hans))
   (
   {
    } 

so gingen mir früher weniger durch die Lappen

Oke, übersichtlicher ist es jetzt auf jeden Fall schonmal habe es jetzt so:

if (!isset ($oldpw))
OR
if(!isset ($changepw1))
OR
if(isset ($changepw2))

[VPh]

$errors[] = "Ihr Passwort konnte leider nicht geändert werden.\n". 

"." auch mal durch ";" ersetzen.