youngguns-nl/moneybird_php_api

"Undefined index" bij versturen met de hand

Closed this issue · 2 comments

Probleem: indien een factuur met de hand verstuurd wordt, krijg je een foutmelding te zien.

Foutmelding:
Notice: Undefined index: email in Invoice.php on line 332
Notice: Undefined index: message in Invoice.php on line 336

Oorzaak: de twee variabelen worden niet opgeslagen indien de factuur met de hand gestuurd wordt, best logisch. Echter wordt er op de verkeerde manier gechecked waardoor er een foutmelding volgt.

Relevante code: Invoice.php, regel 332 t/m 339
properties['email'] != null) { $xml .= ' '.htmlspecialchars($this->properties['email']).''.PHP_EOL; } if ($this->properties['message'] != null) { $xml .= ' '.htmlspecialchars($this->properties['message']).''.PHP_EOL; } ?>

Verbeterde code: Invoice.php, regel 332 t/m 339
properties['email'])) { $xml .= ' '.htmlspecialchars($this->properties['email']).''.PHP_EOL; } if (!empty($this->properties['message'])) { $xml .= ' '.htmlspecialchars($this->properties['message']).''.PHP_EOL; } ?>

bedankt voor het melden en de code, ik neem het mee met een volgende aanpassing

opgelost