h4kuna\MailManager\Mailer\FileMailer

Complexity

Total Complexity

8

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	setLive()	0	3	1
A	getLastFile()	0	3	1
A	autoremove()	0	14	4
A	__construct()	0	4	1
A	send()	0	6	1

<?php declare(strict_types=1);

namespace h4kuna\MailManager\Mailer;

use Nette\Mail;
use Nette\Utils;

/**
 * File Mailer - store mail to server uploads (file)
 * @todo https://github.com/romanmatyus/FileMailer
 * @author David Grudl
 */
class FileMailer implements Mail\IMailer

The interface Nette\Mail\IMailer has been deprecated: use Nette\Mail\Mailer

{

	/** @var string */
	private $path;

	/** @var string|null */
	private $live;

	/** @var string */
	private $lastFile;


	public function __construct(string $path)
	{
		Utils\FileSystem::createDir($path);
		$this->path = realpath($path) . DIRECTORY_SEPARATOR;
	}


	public function setLive(string $live): void
	{
		$this->live = $live;
	}


	public function getLastFile(): string
	{
		return $this->lastFile;
	}


	public function send(Mail\Message $mail): void
	{
		$this->autoremove();
		list($sec) = explode(' ', substr(microtime(), 2));
		$this->lastFile = $this->path . date('Y-m-d_H-i-s-') . $sec . '.eml';
		file_put_contents($this->lastFile, $mail->generateMessage());
	}


	private function autoremove(): void
	{
		if (!$this->live) {
			return;
		}
		$finder = Utils\Finder::findFiles('*.eml');
		if (is_string($this->live)) {

The condition is_string($this->live) is always true.

			$finder->date('<=', "- {$this->live}");
		}

		foreach ($finder->in($this->path) as $file) {
			@unlink($file->getPathname());

It seems like you do not handle an error condition for unlink(). This can introduce security issues, and is generally not recommended.

		}
		$this->live = null; // clear temporary onetime per execution.
	}

}