tleilax / exportforce

Wrapper for klamm.de's API "ExportForce" in PHP
MIT License
2 stars 1 forks source link

Bug in confirm() #3

Closed tleilax closed 12 years ago

tleilax commented 12 years ago

== Von Mone ==

Hm, das sieht mir wie ein echter Bug aus .

{{{ if ($kres[0]!=1001) return false; $timestamp = $kret[2]; $tmpinfo = explode('|', $kret[1]); return $tmpinfo[0]; }}}

Du hast da folgendes nicht bedacht: wenn ein Transcode bei EF nicht existiert, kommt direkt 1089.

{{{ Returncodes: 1001 // Alles OK 1002 // EF Account existiert nicht 1003 // EF Passwort falsch 1089 // Transaktions-Code nicht vorhanden 1098 // EF Account ist gesperrt 1099 // Unbekannter Fehler }}}

Da fehlt also

{{{ if ($kres[0]==1089) return 1089; }}}

Mit deiner Version bekomme ich false, wenn EF immer noch nicht erreichbar ist und auch false, wenn der Code nicht existiert. 1089 bekomme ich in keinem Fall.