PDA

Archiv verlassen und diese Seite im Standarddesign anzeigen : *Update* VMS1.2 Tresor Addon !



Koki12
07.03.2008, 17:54
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
07.03.2008, 20:56
hm, geht immer noch, trotz des codes???

Hardy
07.03.2008, 21:09
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
07.03.2008, 21:19
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
07.03.2008, 21:30
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
07.03.2008, 21:35
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 :hand:

Koki12
07.03.2008, 22:22
Oh. Sry, sehs grad Oo.

In der Ruhe liegt die Kraft ...

*Mal wieder nicht aufgepasst*

SilentRunner
08.03.2008, 12:00
Oh. Sry, sehs grad Oo.

In der Ruhe liegt die Kraft ...

*Mal wieder nicht aufgepasst*

ändere bitte mal oben deinen Post, was man nun genau machen soll, danke.

Koki12
08.03.2008, 18:07
So.
Geupdatet !

Für weitere Fragen, bitte per P.N Melden, oder gleich hier fragen.

MfG;
Koki12