Tresor Addon Problem

[NeoGriever]

Ähm.

function kontobuchung($art, $menge, $uid, $loseart = null) {
    if($loseart === true) {
        // nur normal-lose
    }elseif($loseart === false) {
        // nur bonus-lose
    }else{
        // beide (standard)
    }
}

Und was heißt hier "unsauber"? Das ist grundlegendes php-standard. O_o

Mit $loseart = null leg ich erstmal fest, dass der standard-wert, wenn der letzte parameter nicht angegeben wird, "Null" ist. Es handelt sich hierbei um KEIN boolean-wert.

Mit $loseart === true (dem 3fach-istgleich) verhinder ich, dass $loseart als boolean angesehen wird, wenn man 1 z. b. angibt.
Ebenso bei === false, wenn 0 angegeben wird. Es wird nur True oder False erlaubt.

Wird keines der beiden erkannt, standard.

Somit können alte addons die übliche Funktionsweise verwenen und neue addons den parameter hinzufügen. Darüber hinaus lassen sich so alte addons schnell und einfach damit erweitern.

Von wegen unsauber ...

[Alpha10]

Hallo!
Ich stelle gerade fest das man beim Tresor Addon auch ohne Passwort auszahlen kann.
Gibt man das PW falsch ein steht richtig da das Passwort ist falsch.
Wenn man aber gar kein Passwort eingibt und auf auszahlen geht dann zahlt der Tresor aus und das it ein BUG!!

Hat da jemand einen Ansatz ich kann nix finden!

[Alpha10]

Problem gefunden es erfolgt KEINE 0 Eingabe Prüfung!!!
Datei ab Zeile 68 anpassen:

} else {
if ( empty ($_POST['tpw']) == TRUE ){
$msg .= "Keine Eingabe.";
} else {
if($userdaten['tresor_pw'] != $_POST['tpw']) {
$msg .= "Tresor Passwort Falsch.";
} else {
if (ereg("[^0-9]", $_POST['betrag'])) {
$msg .= "Bei Betrag nur Zahlen.";
} else {
if($userdaten['tresor'] >= $_POST['betrag']) {
$buchungsid = create_code(14);
kontobuchung_tresor('-',$_POST['betrag'],$_SESSION['uid']);
buchungsliste_tresor($buchungsid,'-'.$_POST['betrag'],'Tresor Auszahlung',$_SESSION['uid']);
kontobuchung('+',$_POST['betrag'],$_SESSION['uid']);
buchungsliste($buchungsid,'+'.$_POST['betrag'],'Tresor Auszahlung',$_SESSION['uid']);
$msg .= "Betrag Ausgezahlt.";
} else {
$msg .= "Tresorguthaben reicht nicht aus.";
}
}
}
}
}
}

bei keiner Eingabe erfolgt jetzt die Ausgabe "Keine Eingabe"! Hat diesen kritischen Bug den 10 Jahre niemand endeckt???

[Alpha10]

Peinlich Peinlich: Ohne den Eintrag Tresor_PW in den _kontodaten, ja da kann mann dann auch ohne PW auszahlen... Leider!



Schaden kann die Ergänzung des Codes OBEN aber trotzdem nicht.