Skip to content

Commit

Permalink
Merge pull request #10 from iqnection-programming/master
Browse files Browse the repository at this point in the history
fixed bug with class declaration
  • Loading branch information
lekoala authored Apr 22, 2022
2 parents 4716989 + 7f4f68d commit b6577bd
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/MailgunSwiftTransport.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
use \Swift_MimePart;
use \Swift_Transport;
use \Swift_Attachment;
use \Swift_Mime_SimpleMessage;
use \Swift_Mime_Message;
use \Swift_Events_SendEvent;
use \Swift_Events_EventListener;
use Psr\Log\LoggerInterface;
Expand Down Expand Up @@ -93,12 +93,12 @@ public function ping()
}

/**
* @param Swift_Mime_SimpleMessage $message
* @param Swift_Mime_Message $message
* @param null $failedRecipients
* @return int Number of messages sent
*/
public function send(
Swift_Mime_SimpleMessage $message,
Swift_Mime_Message $message,
&$failedRecipients = null
) {
$this->resultApi = null;
Expand Down

2 comments on commit b6577bd

@k-coidan
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @lekoala. Just installed your package and an error is being thrown as the method signature no longer matches the one from the Swift_Transport interface. Changing the type back to Swift_Mime_SimpleMessage on the $message param fixes the issue.

@lekoala
Copy link
Owner Author

@lekoala lekoala commented on b6577bd Aug 1, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kris-coidan you can try 1.1.1 or 1.1.2, one of them should be good it depends on which version you are using

Please sign in to comment.