Skip to content

Commit 4e9311d

Browse files
authored
Merge pull request #170 from michielkeijts/master
Fix to follow email transport defined in app.php
2 parents 6c17304 + 8d468e2 commit 4e9311d

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

src/Shell/Task/QueueEmailTask.php

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,12 +73,16 @@ public function run(array $data, $jobId) {
7373
return false;
7474
}
7575

76+
if (!isset($data['transport'])) {
77+
$this->err('Queue Email task needs a transport to be set to actually send the email.');
78+
return false;
79+
}
80+
7681
/** @var \Cake\Mailer\Email $email */
7782
$email = $data['settings'];
7883
if (is_object($email) && $email instanceof Email) {
7984
try {
80-
$transportClassNames = $email->configuredTransport();
81-
$result = $email->transport($transportClassNames[0])->send();
85+
$result = $email->setTransport($data['transport'])->send();
8286

8387
if (!isset($config['log']) || !empty($config['logTrace']) && $config['logTrace'] === true) {
8488
$config['log'] = 'email_trace';

0 commit comments

Comments
 (0)