literat / srazvs

app/services/Emailer.php

<?php

namespace App\Services;

use App\Models\SettingsModel;
use Nette\Mail\IMailer;
use Nette\Mail\Message;
use Tracy\Debugger;

/**
 * Emailer
 *
 * Class for hadling and sending e-mails
 *
 * @created 2011-09-16
 * @author Tomas Litera <tomaslitera@hotmail.com>
 */
class Emailer
{
	/** @var SmtpMailer */
	private $mailer;

	/** @var SettingsModel */
	private $settings;

	/* Constructor */
	public function __construct(SettingsModel $settings, IMailer $mailer)
	{
		$this->mailer = $mailer;

It seems like $mailer of type object<Nette\Mail\IMailer> is incompatible with the declared type object<App\Services\SmtpMailer> of property $mailer.

		$this->settings = $settings;
	}

	/**
	 * Send an e-mail to recipient
	 *
	 * @param	array	recipient e-mail and name
	 * @param	string	subject
	 * @param	string	message
	 * @param	array	bcc
	 * @return	bool	true | false (log the exception)
	 */
	public function sendMail($recipients, $subject, $body, array $bccMail = NULL)
	{
		$message = new Message;
		$message->setFrom('srazyvs@hkvs.cz', 'Srazy VS');

		foreach($recipients as $recipient) {
			$message->addTo(
			    $recipient->email,
                trim($recipient->name . ' ' . $recipient->surname)
            );
		}
		// add bcc
		if(!empty($bccMail)) {
			foreach ($bccMail as $bccMail => $bccName) {

The expression $bccMail of type integer|string|array is not guaranteed to be traversable. How about adding an additional type check?

				$message->addBcc($bccMail, $bccName);
			}
		}
		// create subject
		$message->subject = $subject;
		// create HTML body
		$message->htmlBody = $body;
		// create alternative message without HTML tags
		$message->body = strip_tags($body);
		// sending e-mail or error status
		try {
			$this->mailer->send($message);
			return true;
		} catch(Exception $e) {

The class App\Services\Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

			Debugger::log($e, 'error');
			return false;
		}
	}

	/**
	 * Get e-mail template from settings
	 *
	 * @param	string	type of template
	 * @return	array	subject and message
	 */
	public function getTemplate($type)
	{
		$json = $this->settings->getMailJSON($type);

		$subject = html_entity_decode($json->subject);
		$message = html_entity_decode($json->message);

		return array(
			'subject' => $subject,
			'message' => $message,
		);
	}

	/**
	 * Sends an e-mail to lecture master
	 *
	 * @param	array	recipient mail and name
	 * @param	string	guid
	 * @param	string	program | block
	 * @return	mixed	true | error information
	 */
	public function tutor($recipients, $guid, $type)
	{
		$lang['block']['cs'] = "bloku";

$lang was never initialized. Although not strictly required by PHP, it is generally a good practice to add $lang = array(); before regardless.

		$lang['program']['cs'] = "programu";

		$tutorFormUrl = PRJ_DIR . "annotation/edit/{$type}/{$guid}";

		// e-mail templates
		$template = $this->getTemplate('tutor');
		$subject = $template['subject'];
		$message = $template['message'];

		// replacing text variables
		$subject = preg_replace('/%%\[typ-anotace\]%%/', $lang[$type]['cs'], $subject);
		$message = preg_replace('/%%\[typ-anotace\]%%/', $lang[$type]['cs'], $message);
		$message = preg_replace('/%%\[url-formulare\]%%/', $tutorFormUrl, $message);

		// send it
		return $this->sendMail($recipients, $subject, $message);
	}

	/**
	 * Sends an after registration summary e-mail to visitor
	 *
	 * @param	array	recipient mail
	 * @param	int		check hash code
	 * @param	string	code for recognition of bank transaction
	 * @return	mixed	true | error information
	 */
	public function sendRegistrationSummary(array $recipientMail, $hash, $code4bank)
	{
		// e-mail templates
		$template = $this->getTemplate('post_reg');
		$subject = $template['subject'];
		$message = $template['message'];

		// replacing text variables
		$message = preg_replace('/%%\[kontrolni-hash\]%%/', $hash, $message);
		$message = preg_replace('/%%\[variabilni-symbol\]%%/', $code4bank, $message);

		// send it
		return $this->sendMail($recipientMail, $subject, $message);
	}

	/**
	 * Get e-mail templates from settings
	 *
	 * @param	mixed	id numbers in row
	 * @param	string	type of template
	 * @return	array	subject and message
	 */
	public function sendPaymentInfo($recipients, $type)
	{
		// e-mail templates
		$template = $this->getTemplate($type);
		$subject = $template['subject'];
		$message = $template['message'];

		return $this->sendMail($recipients, $subject, $message);
	}
}