*Update* VMS1.2 Tresor Addon !

[Koki12]

Hi.

Ich habe mir gerade nochmal das Tresor Addon angeschaut, und nen Schwerwiegenden Fehler gefunden !

Um diesen Fehler zu Beheben, öffne Folgende Datei:
content/konto/tresor.php

Suche:

if($userdaten['tresor'] >= $_POST['betrag']) { 

Füge darüber folgendes ein:

if (ereg("[^0-9]", $_POST['betrag'])) { 
            $msg .= "Userguthaben reicht nicht aus.";
        } else { 

Suche:

            } else {
                $msg .= "Userguthaben reicht nicht aus.";
            } 

Füge darunter folgendes ein:

} 

Suche:

            if($userdaten['tresor'] >= $_POST['betrag']) { 

Füge darüber folgendes ein:

if (ereg("[^0-9]", $_POST['betrag'])) { 
            $msg .= "Userguthaben reicht nicht aus.";
        } else { 

Süche:

        } else {
                $msg .= "Tresorguthaben reicht nicht aus."; 

Füge darunter folgendes ein:

} 

Baut dieses bitte umgehend ein, ansonsten könnte es sein, dass Verluste entstehen !

MfG;
Koki12

[jpwfour]

hm, geht immer noch, trotz des codes???

[Hardy]

Ich denke mal da einfach nur ne fehlermeldung ausgespuckt wird aber der vorgang nicht wirklich abgebrochen wird.da fehlte ne else schleife
Bsp.:

if (!ereg("[^0-9]", $_POST['betrag'])){
Hier wenn ok!
}else{
Hier wenn falsche eingabe!
} 

[jpwfour]

if($userdaten['kontostand'] >= $_POST['betrag']) {
if (ereg("[^0-9]", $_POST['betrag'])){
$msg .= 'Bei Betrag nur Zahlen!'; 
}else{
            kontobuchung_tresor('+',$_POST['betrag'],$_SESSION['uid']);
            buchungsliste_tresor($buchungsid,$_POST['betrag'],'Tresor Einzahlung',$_SESSION['uid']);
            kontobuchung('-',$_POST['betrag'],$_SESSION['uid']);            
            buchungsliste($buchungsid,$_POST['betrag'],'Tresor Einzahlung',$_SESSION['uid']);
            $msg .= "Betrag Eingezahlt.";
}
        } else {
            $msg .= "Userguthaben reicht nicht aus.";
        } 

so hab ich das jetzt bei mir gemacht.

[Hardy]

Würde das denn noch eher so machen:

if (!ereg("[^0-9]", $_POST['betrag'])){
if($userdaten['kontostand'] >= $_POST['betrag']) {
            kontobuchung_tresor('+',$_POST['betrag'],$_SESSION['uid']);
            buchungsliste_tresor($buchungsid,$_POST['betrag'],'Tresor Einzahlung',$_SESSION['uid']);
            kontobuchung('-',$_POST['betrag'],$_SESSION['uid']);            
            buchungsliste($buchungsid,$_POST['betrag'],'Tresor Einzahlung',$_SESSION['uid']);
            $msg .= "Betrag Eingezahlt.";
        } else {
            $msg .= "Userguthaben reicht nicht aus.";
        }
}else{
$msg .= 'Bei Betrag nur Zahlen!'; 
} 

Da dann erst gecheckt wird ob es zahlen sind und dann das konto geprüft wird.

if($userdaten['kontostand'] >= $_POST['betrag']) {
if (ereg("[^0-9]", $_POST['betrag'])){
$msg .= 'Bei Betrag nur Zahlen!'; 
}else{
            kontobuchung_tresor('+',$_POST['betrag'],$_SESSION['uid']);
            buchungsliste_tresor($buchungsid,$_POST['betrag'],'Tresor Einzahlung',$_SESSION['uid']);
            kontobuchung('-',$_POST['betrag'],$_SESSION['uid']);            
            buchungsliste($buchungsid,$_POST['betrag'],'Tresor Einzahlung',$_SESSION['uid']);
            $msg .= "Betrag Eingezahlt.";
}
        } else {
            $msg .= "Userguthaben reicht nicht aus.";
        } 

so hab ich das jetzt bei mir gemacht.

[jpwfour]

jo,d a muss ich dir recht geben, deine lösung ist schöner.

aber genial ist der effekt, wenn man diese überprüfung beim auszahlen eglässt, da dann, wenn ein user eine neagtive summe eingibt, es ja genau umgekehrt verbucht wird, also wie wenn er die summe einzahlen würde