Skip to content

Commit 7378b84

Browse files
fabpotfabpot
committed
Merge branch '4.4'
* 4.4: rename SmtpEnvelope to Envelope
2 parents fe48fac + c23d641 commit 7378b84

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

43 files changed

+126
-124
lines changed

‎AbstractApiTransport.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@
1111

1212
namespace Symfony\Component\Mailer\Transport;
1313

14+
use Symfony\Component\Mailer\Envelope;
1415
use Symfony\Component\Mailer\Exception\RuntimeException;
1516
use Symfony\Component\Mailer\SentMessage;
16-
use Symfony\Component\Mailer\SmtpEnvelope;
1717
use Symfony\Component\Mime\Address;
1818
use Symfony\Component\Mime\Email;
1919
use Symfony\Component\Mime\MessageConverter;
@@ -24,7 +24,7 @@
2424
*/
2525
abstract class AbstractApiTransport extends AbstractHttpTransport
2626
{
27-
abstract protected function doSendApi(Email $email, SmtpEnvelope $envelope): ResponseInterface;
27+
abstract protected function doSendApi(Email $email, Envelope $envelope): ResponseInterface;
2828

2929
protected function doSendHttp(SentMessage $message): ResponseInterface
3030
{
@@ -37,7 +37,7 @@ protected function doSendHttp(SentMessage $message): ResponseInterface
3737
return $this->doSendApi($email, $message->getEnvelope());
3838
}
3939

40-
protected function getRecipients(Email $email, SmtpEnvelope $envelope): array
40+
protected function getRecipients(Email $email, Envelope $envelope): array
4141
{
4242
return array_filter($envelope->getRecipients(), function (Address $address) use ($email) {
4343
return false === \in_array($address, array_merge($email->getCc(), $email->getBcc()), true);

‎AbstractTransport.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,9 @@
1313

1414
use Psr\Log\LoggerInterface;
1515
use Psr\Log\NullLogger;
16+
use Symfony\Component\Mailer\Envelope;
1617
use Symfony\Component\Mailer\Event\MessageEvent;
1718
use Symfony\Component\Mailer\SentMessage;
18-
use Symfony\Component\Mailer\SmtpEnvelope;
1919
use Symfony\Component\Mime\Address;
2020
use Symfony\Component\Mime\RawMessage;
2121
use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;
@@ -51,10 +51,10 @@ public function setMaxPerSecond(float $rate): self
5151
return $this;
5252
}
5353

54-
public function send(RawMessage $message, SmtpEnvelope $envelope = null): ?SentMessage
54+
public function send(RawMessage $message, Envelope $envelope = null): ?SentMessage
5555
{
5656
$message = clone $message;
57-
$envelope = null !== $envelope ? clone $envelope : SmtpEnvelope::create($message);
57+
$envelope = null !== $envelope ? clone $envelope : Envelope::create($message);
5858

5959
if (null !== $this->dispatcher) {
6060
$event = new MessageEvent($message, $envelope, (string) $this);

‎AbstractTransportTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@
1212
namespace Symfony\Component\Mailer\Tests\Transport;
1313

1414
use PHPUnit\Framework\TestCase;
15+
use Symfony\Component\Mailer\Envelope;
1516
use Symfony\Component\Mailer\Exception\LogicException;
16-
use Symfony\Component\Mailer\SmtpEnvelope;
1717
use Symfony\Component\Mailer\Transport\NullTransport;
1818
use Symfony\Component\Mime\Address;
1919
use Symfony\Component\Mime\RawMessage;
@@ -28,7 +28,7 @@ public function testThrottling()
2828
$transport = new NullTransport();
2929
$transport->setMaxPerSecond(2 / 10);
3030
$message = new RawMessage('');
31-
$envelope = new SmtpEnvelope(new Address('fabien@example.com'), [new Address('helene@example.com')]);
31+
$envelope = new Envelope(new Address('fabien@example.com'), [new Address('helene@example.com')]);
3232

3333
$start = time();
3434
$transport->send($message, $envelope);

‎Amazon/Transport/SesApiTransport.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@
1212
namespace Symfony\Component\Mailer\Bridge\Amazon\Transport;
1313

1414
use Psr\Log\LoggerInterface;
15+
use Symfony\Component\Mailer\Envelope;
1516
use Symfony\Component\Mailer\Exception\HttpTransportException;
16-
use Symfony\Component\Mailer\SmtpEnvelope;
1717
use Symfony\Component\Mailer\Transport\AbstractApiTransport;
1818
use Symfony\Component\Mime\Email;
1919
use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;
@@ -48,7 +48,7 @@ public function __toString(): string
4848
return sprintf('ses+api://%s@%s', $this->accessKey, $this->getEndpoint());
4949
}
5050

51-
protected function doSendApi(Email $email, SmtpEnvelope $envelope): ResponseInterface
51+
protected function doSendApi(Email $email, Envelope $envelope): ResponseInterface
5252
{
5353
$date = gmdate('D, d M Y H:i:s e');
5454
$auth = sprintf('AWS3-HTTPS AWSAccessKeyId=%s,Algorithm=HmacSHA256,Signature=%s', $this->accessKey, $this->getSignature($date));
@@ -81,7 +81,7 @@ private function getSignature(string $string): string
8181
return base64_encode(hash_hmac('sha256', $string, $this->secretKey, true));
8282
}
8383

84-
private function getPayload(Email $email, SmtpEnvelope $envelope): array
84+
private function getPayload(Email $email, Envelope $envelope): array
8585
{
8686
if ($email->getAttachments()) {
8787
return [

‎CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,8 @@ CHANGELOG
44
4.4.0
55
-----
66

7+
* [BC BREAK] renamed `SmtpEnvelope` to `Envelope`, renamed `DelayedSmtpEnvelope` to
8+
`DelayedEnvelope`
79
* [BC BREAK] changed the syntax for failover and roundrobin DSNs
810

911
Before:

‎DelayedSmtpEnvelope.php renamed to ‎DelayedEnvelope.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
*
2222
* @internal
2323
*/
24-
final class DelayedSmtpEnvelope extends SmtpEnvelope
24+
final class DelayedEnvelope extends Envelope
2525
{
2626
private $senderSet = false;
2727
private $recipientsSet = false;

‎SmtpEnvelope.php renamed to ‎Envelope.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
/**
2020
* @author Fabien Potencier <fabien@symfony.com>
2121
*/
22-
class SmtpEnvelope
22+
class Envelope
2323
{
2424
private $sender;
2525
private $recipients = [];
@@ -39,7 +39,7 @@ public static function create(RawMessage $message): self
3939
throw new LogicException('Cannot send a RawMessage instance without an explicit Envelope.');
4040
}
4141

42-
return new DelayedSmtpEnvelope($message);
42+
return new DelayedEnvelope($message);
4343
}
4444

4545
public function setSender(Address $sender): void

‎SmtpEnvelopeTest.php renamed to ‎EnvelopeTest.php

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -12,71 +12,71 @@
1212
namespace Symfony\Component\Mailer\Tests;
1313

1414
use PHPUnit\Framework\TestCase;
15+
use Symfony\Component\Mailer\Envelope;
1516
use Symfony\Component\Mailer\Exception\LogicException;
16-
use Symfony\Component\Mailer\SmtpEnvelope;
1717
use Symfony\Component\Mime\Address;
1818
use Symfony\Component\Mime\Header\Headers;
1919
use Symfony\Component\Mime\Message;
2020
use Symfony\Component\Mime\RawMessage;
2121

22-
class SmtpEnvelopeTest extends TestCase
22+
class EnvelopeTest extends TestCase
2323
{
2424
public function testConstructorWithAddressSender()
2525
{
26-
$e = new SmtpEnvelope(new Address('fabien@symfony.com'), [new Address('thomas@symfony.com')]);
26+
$e = new Envelope(new Address('fabien@symfony.com'), [new Address('thomas@symfony.com')]);
2727
$this->assertEquals(new Address('fabien@symfony.com'), $e->getSender());
2828
}
2929

3030
public function testConstructorWithNamedAddressSender()
3131
{
32-
$e = new SmtpEnvelope(new Address('fabien@symfony.com', 'Fabien'), [new Address('thomas@symfony.com')]);
32+
$e = new Envelope(new Address('fabien@symfony.com', 'Fabien'), [new Address('thomas@symfony.com')]);
3333
$this->assertEquals(new Address('fabien@symfony.com'), $e->getSender());
3434
}
3535

3636
public function testConstructorWithAddressRecipients()
3737
{
38-
$e = new SmtpEnvelope(new Address('fabien@symfony.com'), [new Address('thomas@symfony.com'), new Address('lucas@symfony.com', 'Lucas')]);
38+
$e = new Envelope(new Address('fabien@symfony.com'), [new Address('thomas@symfony.com'), new Address('lucas@symfony.com', 'Lucas')]);
3939
$this->assertEquals([new Address('thomas@symfony.com'), new Address('lucas@symfony.com')], $e->getRecipients());
4040
}
4141

4242
public function testConstructorWithNoRecipients()
4343
{
4444
$this->expectException(\InvalidArgumentException::class);
45-
$e = new SmtpEnvelope(new Address('fabien@symfony.com'), []);
45+
$e = new Envelope(new Address('fabien@symfony.com'), []);
4646
}
4747

4848
public function testConstructorWithWrongRecipients()
4949
{
5050
$this->expectException(\InvalidArgumentException::class);
51-
$e = new SmtpEnvelope(new Address('fabien@symfony.com'), ['lucas@symfony.com']);
51+
$e = new Envelope(new Address('fabien@symfony.com'), ['lucas@symfony.com']);
5252
}
5353

5454
public function testSenderFromHeaders()
5555
{
5656
$headers = new Headers();
5757
$headers->addPathHeader('Return-Path', new Address('return@symfony.com', 'return'));
5858
$headers->addMailboxListHeader('To', ['from@symfony.com']);
59-
$e = SmtpEnvelope::create(new Message($headers));
59+
$e = Envelope::create(new Message($headers));
6060
$this->assertEquals(new Address('return@symfony.com', 'return'), $e->getSender());
6161

6262
$headers = new Headers();
6363
$headers->addMailboxHeader('Sender', new Address('sender@symfony.com', 'sender'));
6464
$headers->addMailboxListHeader('To', ['from@symfony.com']);
65-
$e = SmtpEnvelope::create(new Message($headers));
65+
$e = Envelope::create(new Message($headers));
6666
$this->assertEquals(new Address('sender@symfony.com', 'sender'), $e->getSender());
6767

6868
$headers = new Headers();
6969
$headers->addMailboxListHeader('From', [new Address('from@symfony.com', 'from'), 'some@symfony.com']);
7070
$headers->addMailboxListHeader('To', ['from@symfony.com']);
71-
$e = SmtpEnvelope::create(new Message($headers));
71+
$e = Envelope::create(new Message($headers));
7272
$this->assertEquals(new Address('from@symfony.com', 'from'), $e->getSender());
7373
}
7474

7575
public function testSenderFromHeadersWithoutFrom()
7676
{
7777
$headers = new Headers();
7878
$headers->addMailboxListHeader('To', ['from@symfony.com']);
79-
$e = SmtpEnvelope::create($message = new Message($headers));
79+
$e = Envelope::create($message = new Message($headers));
8080
$message->getHeaders()->addMailboxListHeader('From', [new Address('from@symfony.com', 'from')]);
8181
$this->assertEquals(new Address('from@symfony.com', 'from'), $e->getSender());
8282
}
@@ -88,7 +88,7 @@ public function testRecipientsFromHeaders()
8888
$headers->addMailboxListHeader('To', [new Address('to@symfony.com')]);
8989
$headers->addMailboxListHeader('Cc', [new Address('cc@symfony.com')]);
9090
$headers->addMailboxListHeader('Bcc', [new Address('bcc@symfony.com')]);
91-
$e = SmtpEnvelope::create(new Message($headers));
91+
$e = Envelope::create(new Message($headers));
9292
$this->assertEquals([new Address('to@symfony.com'), new Address('cc@symfony.com'), new Address('bcc@symfony.com')], $e->getRecipients());
9393
}
9494

@@ -99,14 +99,14 @@ public function testRecipientsFromHeadersWithNames()
9999
$headers->addMailboxListHeader('To', [new Address('to@symfony.com', 'to')]);
100100
$headers->addMailboxListHeader('Cc', [new Address('cc@symfony.com', 'cc')]);
101101
$headers->addMailboxListHeader('Bcc', [new Address('bcc@symfony.com', 'bcc')]);
102-
$e = SmtpEnvelope::create(new Message($headers));
102+
$e = Envelope::create(new Message($headers));
103103
$this->assertEquals([new Address('to@symfony.com', 'to'), new Address('cc@symfony.com', 'cc'), new Address('bcc@symfony.com', 'bcc')], $e->getRecipients());
104104
}
105105

106106
public function testFromRawMessages()
107107
{
108108
$this->expectException(LogicException::class);
109109

110-
SmtpEnvelope::create(new RawMessage('Some raw email message'));
110+
Envelope::create(new RawMessage('Some raw email message'));
111111
}
112112
}

‎Event/MessageEvent.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111

1212
namespace Symfony\Component\Mailer\Event;
1313

14-
use Symfony\Component\Mailer\SmtpEnvelope;
14+
use Symfony\Component\Mailer\Envelope;
1515
use Symfony\Component\Mime\RawMessage;
1616
use Symfony\Contracts\EventDispatcher\Event;
1717

@@ -27,7 +27,7 @@ final class MessageEvent extends Event
2727
private $transport;
2828
private $queued;
2929

30-
public function __construct(RawMessage $message, SmtpEnvelope $envelope, string $transport, bool $queued = false)
30+
public function __construct(RawMessage $message, Envelope $envelope, string $transport, bool $queued = false)
3131
{
3232
$this->message = $message;
3333
$this->envelope = $envelope;

‎Mailchimp/Transport/MandrillApiTransport.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,8 @@
1212
namespace Symfony\Component\Mailer\Bridge\Mailchimp\Transport;
1313

1414
use Psr\Log\LoggerInterface;
15+
use Symfony\Component\Mailer\Envelope;
1516
use Symfony\Component\Mailer\Exception\HttpTransportException;
16-
use Symfony\Component\Mailer\SmtpEnvelope;
1717
use Symfony\Component\Mailer\Transport\AbstractApiTransport;
1818
use Symfony\Component\Mime\Email;
1919
use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;
@@ -41,7 +41,7 @@ public function __toString(): string
4141
return sprintf('mandrill+api://%s', $this->getEndpoint());
4242
}
4343

44-
protected function doSendApi(Email $email, SmtpEnvelope $envelope): ResponseInterface
44+
protected function doSendApi(Email $email, Envelope $envelope): ResponseInterface
4545
{
4646
$response = $this->client->request('POST', 'https://'.$this->getEndpoint().'/api/1.0/messages/send.json', [
4747
'json' => $this->getPayload($email, $envelope),
@@ -64,7 +64,7 @@ private function getEndpoint(): ?string
6464
return ($this->host ?: self::HOST).($this->port ? ':'.$this->port : '');
6565
}
6666

67-
private function getPayload(Email $email, SmtpEnvelope $envelope): array
67+
private function getPayload(Email $email, Envelope $envelope): array
6868
{
6969
$payload = [
7070
'key' => $this->key,
@@ -105,7 +105,7 @@ private function getPayload(Email $email, SmtpEnvelope $envelope): array
105105
return $payload;
106106
}
107107

108-
protected function getRecipients(Email $email, SmtpEnvelope $envelope): array
108+
protected function getRecipients(Email $email, Envelope $envelope): array
109109
{
110110
$recipients = [];
111111
foreach ($envelope->getRecipients() as $recipient) {

0 commit comments

Comments
 (0)