UKMNorge / UKMapi

0 stars 0 forks source link

Ny epost-klasse bør kaste Exception dersom Svar_Til mangler #84

Closed AsgeirSH closed 4 years ago

AsgeirSH commented 4 years ago

Evt ikke kreve at den er satt og heller defaulte til avsender - innspill @mariusmandal ?

Får pdd. den mer kryptiske

FatalThrowableError in Epost.php line 198:
Call to a member function getEpost() on null
mariusmandal commented 4 years ago

Hva tror du om denne @AsgeirSH ? if ($mottaker->getEpost() == UKM_MAIL_REPLY && ($this->svar_til !== null && $this->svar_til->getEpost() == UKM_MAIL_REPLY)) {

https://github.com/UKMNorge/UKMapi/blob/master/Kommunikasjon/Epost.php#L197

AsgeirSH commented 4 years ago

Liker den om det kun er der vi bruker svar_til? @mariusmandal

mariusmandal commented 4 years ago

Skal gjøre et søk i klassen i morra, og pushe den ut @AsgeirSH. Takk!