Skip to content

Commit 8cdd59f

Browse files
committed
fix: subsequent email actions can't access attachments (closes #137)
1 parent ced2719 commit 8cdd59f

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

classes/Actions/EmailAction.php

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -298,15 +298,14 @@ protected function attachments(): array
298298
} else { // is PHP file object
299299
$files = array_values(A::filter($value->value(), fn ($file) => $file['error'] === UPLOAD_ERR_OK));
300300
foreach ($files as $file) {
301-
$name = $file['tmp_name'];
302-
$tmpName = pathinfo($name);
301+
$tmpName = pathinfo($file['tmp_name']);
303302
$filename = $tmpName['dirname'] . '/' . F::safeName($file['name']);
304303

305-
if (rename($file['tmp_name'], $filename)) {
306-
$name = $filename;
304+
if (!F::exists($filename)) {
305+
rename($file['tmp_name'], $filename);
307306
}
308307

309-
$attachments[] = $name;
308+
$attachments[] = $filename;
310309
}
311310
}
312311
}

0 commit comments

Comments
 (0)