search

billbeeio / billbee-php-sdk

🔌 The official Billbee API SDK for PHP 💻
MIT License
23 stars 25 forks source link

Null-Pruefung im Konstruktor bei Model Stock #4

Closed jkahnt closed 6 years ago

jkahnt commented 6 years ago

Hallo,

zuallerst großes Lob für den umfangreichen API Client. Bis jetzt konnte ich damit wunderbar arbeiten allerdings viel mir eine Kleinigkeit auf. Wenn ich ein Objekt vom Typ Stock erstelle und im Konstruktor für NewQuantity den Wert 0 übergebe wird dieser nicht akzeptiert bzw. mit dem Wert von OldQuantity überschrieben. Mein Workaround aktuell ist nach dem Konstruktor-Aufruf nochmal die SetNewQuantity-Methode verwende um den Wert 0 übertragen zu können(da wird nicht auf null geprüft). Wäre es dann nicht besser die Prüfung in Zeile 62 von (newQuantity == null) auf (newQuantity === null) anzupassen. Dann könnte man sich den Extra-Aufruf sparen.

devtronic commented 6 years ago

Danke für das Lob. Du hast natürlich recht, ich passe das entsprechend an.

devtronic commented 6 years ago

Ist mit 26b435f508c / v1.1.2 gefixt, einfach mit composer update die aktuelle Version ziehen.