Skip to content

Commit 2810566

Browse files
committed
change callable to Closure for php5 support. See #1 issue
1 parent b1af05c commit 2810566

File tree

7 files changed

+17
-8
lines changed

7 files changed

+17
-8
lines changed

src/Transport/FakeTransport.php

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
namespace Ddrv\Mailer\Transport;
44

5+
use Closure;
56
use Ddrv\Mailer\Exception\RecipientsListEmptyException;
67
use Ddrv\Mailer\Message;
78
use Ddrv\Mailer\TransportInterface;
@@ -27,7 +28,7 @@ public function send(Message $message)
2728
return true;
2829
}
2930

30-
public function setLogger(callable $logger)
31+
public function setLogger(Closure $logger)
3132
{
3233
$this->logger = $logger;
3334
}

src/Transport/FileTransport.php

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
namespace Ddrv\Mailer\Transport;
44

5+
use Closure;
56
use Ddrv\Mailer\Exception\RecipientsListEmptyException;
67
use Ddrv\Mailer\Message;
78
use Ddrv\Mailer\TransportInterface;
@@ -52,7 +53,7 @@ public function send(Message $message)
5253
return true;
5354
}
5455

55-
public function setLogger(callable $logger)
56+
public function setLogger(Closure $logger)
5657
{
5758
$this->logger = $logger;
5859
}

src/Transport/SendmailTransport.php

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
namespace Ddrv\Mailer\Transport;
44

5+
use Closure;
56
use Ddrv\Mailer\Exception\RecipientsListEmptyException;
67
use Ddrv\Mailer\Message;
78
use Ddrv\Mailer\TransportInterface;
@@ -47,7 +48,7 @@ public function send(Message $message)
4748
return mail($to, $subject, $body, $headers, $this->options);
4849
}
4950

50-
public function setLogger(callable $logger)
51+
public function setLogger(Closure $logger)
5152
{
5253
$this->logger = $logger;
5354
}

src/Transport/SmtpTransport.php

+3-2
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
namespace Ddrv\Mailer\Transport;
44

5+
use Closure;
56
use Ddrv\Mailer\Exception\RecipientsListEmptyException;
67
use Ddrv\Mailer\Message;
78
use Ddrv\Mailer\TransportInterface;
@@ -81,9 +82,9 @@ public function send(Message $message)
8182
}
8283

8384
/**
84-
* @param callable $logger
85+
* @param Closure $logger
8586
*/
86-
public function setLogger(callable $logger)
87+
public function setLogger(Closure $logger)
8788
{
8889
$this->logger = $logger;
8990
}

src/TransportInterface.php

+3-2
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
namespace Ddrv\Mailer;
44

5+
use Closure;
56
use Ddrv\Mailer\Exception\RecipientsListEmptyException;
67

78
interface TransportInterface
@@ -17,9 +18,9 @@ interface TransportInterface
1718
public function send(Message $message);
1819

1920
/**
20-
* @param callable $logger
21+
* @param Closure $logger
2122
* @return void
2223
*/
23-
public function setLogger(callable $logger);
24+
public function setLogger(Closure $logger);
2425
}
2526

tests/Support/Mock/Transport/MockTransport.php

+2-1
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
namespace Tests\Ddrv\Mailer\Support\Mock\Transport;
44

5+
use Closure;
56
use Ddrv\Mailer\Exception\RecipientsListEmptyException;
67
use Ddrv\Mailer\Message;
78
use Ddrv\Mailer\TransportInterface;
@@ -32,7 +33,7 @@ public function send(Message $message)
3233
return true;
3334
}
3435

35-
public function setLogger(callable $logger)
36+
public function setLogger(Closure $logger)
3637
{
3738
$this->logger = $logger;
3839
}

tests/Unit/MailerTest.php

+3
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,9 @@ public function setUp()
3232
parent::setUp();
3333
$this->factory = new MessageFactory();
3434
$this->transport = new MockTransport();
35+
$this->transport->setLogger(function($log) {
36+
echo $log.PHP_EOL.PHP_EOL.PHP_EOL.PHP_EOL;
37+
});
3538
$this->mailer = new Mailer(new MemorySpool($this->transport));
3639
}
3740

0 commit comments

Comments
 (0)