Onderwerp: Wachtwoord wijzigen werkt niet

Hallo

ik ben bezig met een scriptje zodat leden hun wachtwoord kunnen wijzigen, nu is het zo dat je 3 velden hebt met het:
1. Oude wachtwoord
2. Nieuwe wachtwoord
3. Herhaling nieuwe wachtwoord

Ik wil dus dat het script het oude wachtwoord checkt en ook de 2 nieuwe of die gelijk zijn.

dit script heb ik nu maar het werkt niet en volgens mij heeft het wat aanpassingen nodig?

het script:

Code:

<?
require("verbinden.php"); 

session_start(); 

if(IsSet($_SESSION['reg_sessie'])) 
{ 
if($_SESSION['reg_sessie']->login==1 AND 
IsSet($_SESSION['reg_sessie']->memberid) AND 
$_SESSION['reg_sessie']->ipadres==$_SERVER['REMOTE_ADDR']) 
 { 


    $memberid=$_SESSION['reg_sessie']->memberid; 
    $wachtwoord=addslashes($_POST['wachtwoord2']); 
    $wachtwoord2=addslashes($_POST['wachtwoord3']); 

	
	require("wijzigww.inc");
 
   if($_SERVER['REQUEST_METHOD'] == "POST") 
   { 
     if(isset($_POST['wijzig'])) 
     { 
        if(isset($_POST['wachtwoord2']) AND isset($_POST['wachtwoord3'])) 
        { 
            if(trim($_POST['wachtwoord2']) != "" and trim($_POST['wachtwoord3']) != ""  AND  $_POST['wachtwoord2'] == $_POST['wachtwoord3'])   
			{
			echo "Oude wachtwoord klopt niet!"; 
			}               
            if($wachtwoord) 
                { 
                // controleren of beide wachtwoorden gelijk zijn 				
                if($wachtwoord!=$wachtwoord2) 
                    { 
                    echo "Wachtwoorden zijn niet gelijk aan elkaar..."; 
                    } 
                    
                else 
                    { 
                    // versleutelen van het wachtwoord 
			

                     mysql_query(" 
                                UPDATE perso  
                                SET wachtwoord='$md5wachtwoord' 
                                WHERE id='$memberid' 
                                ")  
                    or die(mysql_error()); 					
                         
                    echo "Je wachtwoord is succesvol gewijzigd, 
                    log nu opnieuw in met je nieuwe wachtwoord...<br> 
                    <br><a href=\"login.php\">Log in</a>";  

                    }
				}	
             } 
        } 
		
     } 

   } 

 

else 
{ 
// er is niet juist ingelogd, of de sessie is geopend vanaf een ander ip adres 
session_destroy(); 
require("loginscherm.inc"); 
  } 
} 

else 
{ 
// Object $reg_sessie is niet geregistreerd in de sessie 
session_destroy(); 
require("loginscherm.inc"); 
} 
?>

Voor de handigheid plaats ik het form er ook maar even bij

Code:

<form method="post" action = "<?php echo $_SERVER['PHP_SELF']; ?>"> 
  <table width="353" border="0">
    <tr>
      <td width="177">Oude wachtwoord:</td>
      <td width="160"><input type="password" name="wachtwoord"></td>
    </tr>
    <tr>
      <td>Nieuwe wachtwoord:</td>
      <td><input type="password" name="wachtwoord2"></td>
    </tr>
    <tr>
      <td>Herhaal nieuwe wachtwoord: </td>
      <td><input type="password" name="wachtwoord3"></td>
    </tr>
    <tr>
      <td><input type="submit" name="wijzig" value="wijzig"></td>
      <td>&nbsp;</td>
    </tr>
  </table>
</form>

Je weet zeker dat je sessies actief zijn?
Ik weet niet veel van PHP, maar moet je session_start() niet voor andere dingen zetten?

als jee session_start() ergens in het midden plaatst krijg je een "headers already sent" error.

Dus die staat daar wel goed.

verder vind ik het iets te vroeg om heel de code door te spitten

okey dan wacht ik nog even:P

ik kom er namelijk ff niet meer uit:S

Welke melding krijg je? Dan kan er gekeken worden WAAR het fout gaat. Krijg je niks, php errors, foutmelding die je in je script heb gezet, etc.




if(trim($_POST['wachtwoord2']) != "" and trim($_POST['wachtwoord3']) != "" AND $_POST['wachtwoord2'] == $_POST['wachtwoord3'])
{
echo "Oude wachtwoord klopt niet!";
}

Je zegt dus: als ww2 iet anders dan niks is, ww3 iets anders dan niks is en ww2 is gelijk aan ww3 dan kloppen de oude wachtwoorden niet ?

Waar contoleer jij het odue wachtwoord?




Denk meer aan:

if ($_POST["wachtwoord"] != "" && $_POST["wachtwoord2"] != "" AND $_POST["wachtwoord3"] != "") {

if (mysql_numrows(mysql_query("SEELCT id FROM perso WHERE wachtwoord = '".$_POST["wachtwoord"]."' AND id = '".$memberid."'")) == '1') [

if ($_POST["wachtwoord2"] == $_POST["wachtwoord3"]) {

mysql_query("UPDATE perso SET wachtwoord='".md5($_POST["wachtwoord2"])."' WHERE id='".$memberid."'");
echo "Wachtwoord gewijzigd";

}
else "nieuwe wachtwoorden komen hier overeen";

}
else echo "oud wachtwoord klopt niet";

}
else echo "Wel alles invullen he";


Uiteraard wel beveiligen, optimaliseren etc maar zo krijg je wel een idee


edit 2: hmm, tabs doen het dus niet

Dat is juist het probleem ik ben eigenlijk nog een beginner

ik weet daarom ook niet precies wat dat stukje script doet

Heb jij toevallig niet zoiets wat ik zoek?

Zie mijn edit

ik zal er even na gaan kijken
dit ziet er ook wat beter uit

1 foutje zie ik wel je hebt SEELCT inplaats van SELECT
ik denk ik zeg het maar even

Ik krijg nu nog een foutje

dat heb met het laatste gedeelte te maken

hij geeft dit aan: Parse error: syntax error, unexpected T_ELSE in C:\apachefriends\xampp\htdocs\apriliafreaks\wijzig ww.php on line 45

dit is nu het script:

Code:

<? 

error_reporting(E_ALL);

session_start(); 

require("verbinden.php"); 

if(IsSet($_SESSION['reg_sessie'])) 
    { 
    if($_SESSION['reg_sessie']->login==1 AND  
       IsSet($_SESSION['reg_sessie']->memberid) AND     
          $_SESSION['reg_sessie']->ipadres==$_SERVER['REMOTE_ADDR']) 
        { 


    $memberid=$_SESSION['reg_sessie']->memberid; 
    $wachtwoord=addslashes($_POST['wachtwoord2']); 
    $wachtwoord2=addslashes($_POST['wachtwoord3']); 

	
	require("wijzigww.inc");
 
   if($_SERVER['REQUEST_METHOD'] == "POST") 
   { 
     if ($_POST["wachtwoord"] != "" && $_POST["wachtwoord2"] != "" AND $_POST["wachtwoord3"] != "") {

     if (mysql_numrows(mysql_query("SELECT id FROM perso WHERE wachtwoord = '".$_POST["wachtwoord"]."' AND id = '".$memberid."'")) == '1') {

     if ($_POST["wachtwoord2"] == $_POST["wachtwoord3"]) {

mysql_query("UPDATE perso SET wachtwoord='".md5($_POST["wachtwoord2"])."' WHERE id='".$memberid."'"); 
echo "Wachtwoord gewijzigd";

}
else "nieuwe wachtwoorden komen hier overeen";

}
else echo "oud wachtwoord klopt niet";

}
else echo "Wel alles invullen he";


    else 
        { 
        // er is niet juist ingelogd, of de sessie is geopend vanaf een ander ip adres 
            session_destroy(); 
            require("loginscherm.inc"); 
        } 
    } 

else 
    { 
    // Object $reg_sessie is niet geregistreerd in de sessie     
    session_destroy(); 
    require("loginscherm.inc"); 
    } 
?>

Hoe zorg ik er voor dat die Else goed staat
volgens mij moet ik ergens een } zetten ofzo maar waar?

Ik heb je script iets veranderd, omdat het niet echt fijn was om te 'lezen'.
Wat verstandig is, is om eerst de IF en ELSE functies te maken en daarna pas de scripts daar tussen te plaatsen/schrijven.

Ik heb even geprobeerd het werkend te krijgen, zie hiervoor de bijlage. Ik heb het script niet getest.

Veel Succes!

/edit: Attachment gewist, omdat er een fout is zat

Zenda iig hardstikke bedankt het ziet er zo wel wat overzichtelijker uit
en ik weet gelijk wat elke regel doet

alleen krijg ik deze fout Parse error: syntax error, unexpected T_ELSE in C:\apachefriends\xampp\htdocs\apriliafreaks\wachtw oordwijzigen.php on line 42

ook een onverwachte ELSE zoals ik die al had
zie je soms dit foutje

Volgens mij zag ik je bedoeling in het script over het hoofd. Deze zal waarschijnlijk wel werken.

Succes!

/edit: Script verwijderd

Nu is het zo dat hij aangeeft dat ik niet ben ingelogt terwijl ik dat wel ben!

en als ik dan toch het formulier invul zegt hij dat het oude wachtwoord niet klopt, wat ik dus wel goed invul.

Sorry, ik heb even geen tijd meer.

Zorg er voor dat de IF en ELSE goed worden gesloten, want in jouw geval was dat niet (er moesten totaal nog iets van 5 keer een } in voorkomen).

okey ik zal er voortaan zeker op gaan letten omdat anders het script niet werkt, en ik ga nu zelf ook nog wel ff gaan kijken heel mischien dat ik toch de fout zie zitten

iig erg bedankt voor de tijd die je hebt genomen