search

dotkernel / api

DotKernel's PSR-7 REST style API built around the Mezzio API skeleton.
https://docs.dotkernel.org/api-documentation/
MIT License
35 stars 5 forks source link

AccountActivateHandler.php POST #51

Closed bidi47 closed 3 years ago

bidi47 commented 3 years ago

the last line has a call for the old location for email. this needs investigating as i can't be sure what the intended result should be

return $this->infoResponse(sprintf(Message::MAIL_SENT_USER_ACTIVATION, $user->getEmail()));

the 2nd parameter in sprintf should be either $user->getIdentity() OR $user->getDetail()->getEmail()

alexmerlin commented 3 years ago

@bidi47 As long as user accounts need to (and currently can only) be activated via email, the info message should use $user->getDetail()->getEmail(). See this PR for the fix.