Remove redundant body creation in case of only a preamble is set

This commit is contained in:
nupplaPhil 2020-03-05 00:40:42 +01:00
parent 44eebde9cb
commit 6730ddfd99
No known key found for this signature in database
GPG key ID: D8365C3D36B77D90

View file

@ -36,11 +36,11 @@ use Psr\Log\LoggerInterface;
class SystemMailBuilder extends MailBuilder
{
/** @var string */
protected $subject;
protected $subject = '';
/** @var string */
protected $preamble;
protected $preamble = '';
/** @var string */
protected $body;
protected $body = null;
/** @var string */
protected $siteAdmin;
@ -71,10 +71,6 @@ class SystemMailBuilder extends MailBuilder
*/
public function withMessage(string $subject, string $preamble, string $body = null)
{
if (!isset($body)) {
$body = $preamble;
}
$this->subject = $subject;
$this->preamble = $preamble;
$this->body = $body;
@ -98,7 +94,7 @@ class SystemMailBuilder extends MailBuilder
*/
protected function getHtmlMessage()
{
$htmlVersion = BBCode::convert($this->body);
$htmlVersion = !empty($this->body) ? BBCode::convert($this->body) : '';
// load the template for private message notifications
$tpl = Renderer::getMarkupTemplate('email/system/html.tpl');
@ -117,7 +113,7 @@ class SystemMailBuilder extends MailBuilder
*/
protected function getPlaintextMessage()
{
$textVersion = BBCode::toPlaintext($this->body);
$textVersion = !empty($this->body) ? BBCode::toPlaintext($this->body) : '';
// load the template for private message notifications
$tpl = Renderer::getMarkupTemplate('email/system/text.tpl');