Merge pull request '[bluesky] Fix double dollar sign in bluesky_fetch_notifications...

[friendica-addons.git] / notifyall / notifyall.php

diff --git a/notifyall/notifyall.php b/notifyall/notifyall.php
index 824e306b92cd90b56776b4406a6fd69be00d49dc..966a91843890aae4757f0eacd98f65d0cf0757e6 100644 (file)
--- a/notifyall/notifyall.php
+++ b/notifyall/notifyall.php
@@ -8,33 +8,29 @@
  * Author: Rabuzarus <https://friendica.kommune4.de/profile/rabuzarus> (Port to Friendica)
  */
 
+use Friendica\Addon\notifyall\NotifyAllEmail;
 use Friendica\App;
-use Friendica\Content\Text\BBCode;
-use Friendica\Core\Config;
-use Friendica\Core\L10n;
-use Friendica\Util\Emailer;
-
-function notifyall_install()
-{
-       logger("installed notifyall");
-}
-
-function notifyall_uninstall()
-{
-       logger("removed notifyall");
-}
+use Friendica\Database\DBA;
+use Friendica\Core\Logger;
+use Friendica\Core\Renderer;
+use Friendica\DI;
 
+/**
+ * This is a statement rather than an actual function definition. The simple
+ * existence of this method is checked to figure out if the addon offers a
+ * module.
+ */
 function notifyall_module() {}
 
-function notifyall_addon_admin(App $a, &$o)
+function notifyall_addon_admin(string &$o)
 {
-       $o = '<div></div>&nbsp;&nbsp;&nbsp;&nbsp;<a href="' . $a->getBaseURL() . '/notifyall">' . L10n::t('Send email to all members') . '</a></br/>';
+       $o = '<div></div>&nbsp;&nbsp;&nbsp;&nbsp;<a href="' . DI::baseUrl() . '/notifyall">' . DI::l10n()->t('Send email to all members') . '</a></br/>';
 }
 
 
-function notifyall_post(App $a)
+function notifyall_post()
 {
-       if(!is_site_admin()) {
+       if (!DI::userSession()->isSiteAdmin()) {
                return;
        }
 
@@ -44,72 +40,47 @@ function notifyall_post(App $a)
                return;
        }
 
-       $sitename = Config::get('config', 'sitename');
-
-       if (empty(Config::get('config', 'admin_name'))) {
-               $sender_name = '"' . L10n::t('%s Administrator', $sitename) . '"';
-       } else {
-               $sender_name = '"' . L10n::t('%1$s, %2$s Administrator', Config::get('config', 'admin_name'), $sitename) . '"';
-       }
-
-       if (! x(Config::get('config', 'sender_email'))) {
-               $sender_email = 'noreply@' . $a->getHostName();
-       } else {
-               $sender_email = Config::get('config', 'sender_email');
-       }
-
-       $subject = $_REQUEST['subject'];
-
-
-       $textversion = strip_tags(html_entity_decode(BBCode::convert(stripslashes(str_replace(["\\r", "\\n"], ["", "\n"], $text))), ENT_QUOTES, 'UTF-8'));
-
-       $htmlversion = BBCode::convert(stripslashes(str_replace(["\\r", "\\n"], ["", "<br />\n"], $text)));
+       $condition = ['account_removed' => false, 'account_expired' => false];
 
        // if this is a test, send it only to the admin(s)
        // admin_email might be a comma separated list, but we need "a@b','c@d','e@f
        if (intval($_REQUEST['test'])) {
-               $email = Config::get('config', 'admin_email');
-               $email = "'" . str_replace([" ",","], ["","','"], $email) . "'";
+               $adminEmails = \Friendica\Model\User::getAdminListForEmailing(['email']);
+
+               $condition['email'] = array_column($adminEmails, 'email');
        }
-       $sql_extra = ((intval($_REQUEST['test'])) ? sprintf(" AND `email` in ( %s )", $email) : '');
 
-       $recips = q("SELECT DISTINCT `email` FROM `user` WHERE `verified` AND NOT `account_removed` AND NOT `account_expired` $sql_extra");
+       $recipients = DBA::p("SELECT DISTINCT `email` FROM `user`" . DBA::buildCondition($condition), $condition);
 
-       if (! $recips) {
-               notice(L10n::t('No recipients found.') . EOL);
+       if (! $recipients) {
+               DI::sysmsg()->addNotice(DI::l10n()->t('No recipients found.'));
                return;
        }
 
-       foreach ($recips as $recip) {
-               Emailer::send([
-                       'fromName'             => $sender_name,
-                       'fromEmail'            => $sender_email,
-                       'replyTo'              => $sender_email,
-                       'toEmail'              => $recip['email'],
-                       'messageSubject'       => $subject,
-                       'htmlVersion'          => $htmlversion,
-                       'textVersion'          => $textversion
-               ]);
+       $notifyEmail = new NotifyAllEmail(DI::l10n(), DI::config(), DI::baseUrl(), $text);
+
+       foreach ($recipients as $recipient) {
+               DI::emailer()->send($notifyEmail->withRecipient($recipient['email']));
        }
 
-       notice(L10n::t('Emails sent'));
-       goaway('admin');
+       DI::sysmsg()->addInfo(DI::l10n()->t('Emails sent'));
+       DI::baseUrl()->redirect('admin');
 }
 
-function notifyall_content(&$a)
+function notifyall_content()
 {
-       if (! is_site_admin()) {
-               return;
+       if (!DI::userSession()->isSiteAdmin()) {
+               return '';
        }
 
-       $title = L10n::t('Send email to all members of this Friendica instance.');
+       $title = DI::l10n()->t('Send email to all members of this Friendica instance.');
 
-       $o = replace_macros(get_markup_template('notifyall_form.tpl', 'addon/notifyall/'), [
+       $o = Renderer::replaceMacros(Renderer::getMarkupTemplate('notifyall_form.tpl', 'addon/notifyall/'), [
                '$title' => $title,
-               '$text' => htmlspecialchars(defaults($_REQUEST, 'text', '')),
-               '$subject' => ['subject', L10n::t('Message subject'), defaults($_REQUEST, 'subject', ''),''],
-               '$test' => ['test',L10n::t('Test mode (only send to administrator)'), 0,''],
-               '$submit' => L10n::t('Submit')
+               '$text' => htmlspecialchars($_REQUEST['text'] ?? ''),
+               '$subject' => ['subject', DI::l10n()->t('Message subject'), $_REQUEST['subject'] ?? '',''],
+               '$test' => ['test',DI::l10n()->t('Test mode (only send to administrator)'), 0,''],
+               '$submit' => DI::l10n()->t('Submit')
        ]);
 
        return $o;