motze92 / office365-mail

Office365 transport for Laravel
MIT License
62 stars 21 forks source link

Transport/Office365MailTransport.php:273 Syntax error, unexpected ')' #16

Closed EndOfSource closed 3 years ago

EndOfSource commented 3 years ago

Hey,

I just installed the latest version and it seems like there is a little copy and past issue which came with the last merge.

{
  "message": "syntax error, unexpected ')'",
  "exception": "ParseError",
  "file": "/var/www/app/vendor/motze92/office365-mail/src/Transport/Office365MailTransport.php",
  "line": 273
}

This is the code:

return array_merge(
    (array) $message->getTo(),
    (array) $message->getCc(),
    (array) $message->getBcc(),
    (array) $message->getReplyTo(),
);

After changing the file inside the vendor directory, it works. I had to remove the last ","

return array_merge(
    (array) $message->getTo(),
    (array) $message->getCc(),
    (array) $message->getBcc(),
    (array) $message->getReplyTo()
);
motze92 commented 3 years ago

17 fixed