Changes:

[friendica.git] / src / Util / EMailer / SystemMailBuilder.php

diff --git a/src/Util/EMailer/SystemMailBuilder.php b/src/Util/EMailer/SystemMailBuilder.php
index 41d758ba0a3a173f9cf8d37e041609405ff1a8b5..702f4b14275a8484611b35216072eb52d7e67a84 100644 (file)
--- a/src/Util/EMailer/SystemMailBuilder.php
+++ b/src/Util/EMailer/SystemMailBuilder.php
@@ -1,15 +1,34 @@
 <?php
+/**
+ * @copyright Copyright (C) 2010-2024, the Friendica project
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
+ */
 
 namespace Friendica\Util\EMailer;
 
 use Exception;
-use Friendica\App;
 use Friendica\App\BaseURL;
 use Friendica\Content\Text\BBCode;
-use Friendica\Core\Config\IConfig;
+use Friendica\Core\Config\Capability\IManageConfigValues;
 use Friendica\Core\L10n;
 use Friendica\Core\Renderer;
 use Friendica\Network\HTTPException\InternalServerErrorException;
+use Psr\Log\LoggerInterface;
 
 /**
  * Builder for system-wide emails without any dependency to concrete entities (like items, activities, ..)
@@ -17,20 +36,19 @@ use Friendica\Network\HTTPException\InternalServerErrorException;
 class SystemMailBuilder extends MailBuilder
 {
        /** @var string */
-       protected $subject;
+       protected $subject = '';
        /** @var string */
-       protected $preamble;
+       protected $preamble = '';
        /** @var string */
-       protected $body;
+       protected $body = '';
 
        /** @var string */
        protected $siteAdmin;
 
-       public function __construct(App $a, L10n $l10n, BaseURL $baseUrl, IConfig $config)
+       public function __construct(L10n $l10n, BaseURL $baseUrl, IManageConfigValues $config, LoggerInterface $logger,
+                                   string $siteEmailAddress, string $siteName)
        {
-               parent::__construct($l10n, $baseUrl, $config);
-
-               $siteName = $this->config->get('config', 'sitename');
+               parent::__construct($l10n, $baseUrl, $config, $logger);
 
                if ($this->config->get('config', 'admin_name')) {
                        $this->siteAdmin = $l10n->t('%1$s, %2$s Administrator', $this->config->get('config', 'admin_name'), $siteName);
@@ -38,24 +56,21 @@ class SystemMailBuilder extends MailBuilder
                        $this->siteAdmin = $l10n->t('%s Administrator', $siteName);
                }
 
-               $this->senderAddress = $a->getSenderEmailAddress();
+               // Set the system wide site address/name as sender (default for system mails)
+               $this->withSender($siteName, $siteEmailAddress, $siteEmailAddress);
        }
 
        /**
         * Adds a message
         *
-        * @param string      $subject  The subject of the email
-        * @param string      $preamble The preamble of the email
-        * @param string|null $body     The body of the email (if not set, the preamble will get used as body)
+        * @param string $subject  The subject of the email
+        * @param string $preamble The preamble of the email
+        * @param string $body     The body of the email (optional)
         *
         * @return static
         */
-       public function withMessage(string $subject, string $preamble, string $body = null)
+       public function withMessage(string $subject, string $preamble, string $body = '')
        {
-               if (!isset($body)) {
-                       $body = $preamble;
-               }
-
                $this->subject  = $subject;
                $this->preamble = $preamble;
                $this->body     = $body;
@@ -79,15 +94,13 @@ class SystemMailBuilder extends MailBuilder
         */
        protected function getHtmlMessage()
        {
-               $htmlVersion = BBCode::convert($this->body);
-
                // load the template for private message notifications
                $tpl = Renderer::getMarkupTemplate('email/system/html.tpl');
                return Renderer::replaceMacros($tpl, [
                        '$preamble'    => str_replace("\n", "<br>\n", $this->preamble),
                        '$thanks'      => $this->l10n->t('thanks'),
                        '$site_admin'  => $this->siteAdmin,
-                       '$htmlversion' => $htmlVersion,
+                       '$htmlversion' => BBCode::convertForUriId(0, $this->body, BBCode::EXTERNAL),
                ]);
        }
 
@@ -98,26 +111,13 @@ class SystemMailBuilder extends MailBuilder
         */
        protected function getPlaintextMessage()
        {
-               $textVersion = BBCode::toPlaintext($this->body);
-
                // load the template for private message notifications
                $tpl = Renderer::getMarkupTemplate('email/system/text.tpl');
                return Renderer::replaceMacros($tpl, [
                        '$preamble'    => $this->preamble,
                        '$thanks'      => $this->l10n->t('thanks'),
                        '$site_admin'  => $this->siteAdmin,
-                       '$textversion' => $textVersion,
+                       '$textversion' => BBCode::toPlaintext($this->body),
                ]);
        }
-
-       /**
-        * {@inheritDoc}
-        */
-       public function build(bool $raw = false)
-       {
-               // for system emails, always use the sitename/site address as the sender
-               $this->withSender($this->config->get('config', 'sitename'), $this->senderAddress);
-
-               return parent::build($raw);
-       }
 }