Merge remote-tracking branch 'upstream/develop' into api4

[friendica.git] / src / Module / FriendSuggest.php

diff --git a/src/Module/FriendSuggest.php b/src/Module/FriendSuggest.php
index 6bbfd71166a5eaf534201fa044cefadc61a857d4..b0456377f1c53d8ed76f29996d3b9f81300ec12d 100644 (file)
--- a/src/Module/FriendSuggest.php
+++ b/src/Module/FriendSuggest.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2021, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -38,16 +38,16 @@ use Friendica\Worker\Delivery;
  */
 class FriendSuggest extends BaseModule
 {
-       public static function init(array $parameters = [])
+       public function init()
        {
                if (!local_user()) {
                        throw new ForbiddenException(DI::l10n()->t('Permission denied.'));
                }
        }
 
-       public static function post(array $parameters = [])
+       public function post()
        {
-               $cid = intval($parameters['contact']);
+               $cid = intval($this->parameters['contact']);
 
                // We do query the "uid" as well to ensure that it is our contact
                if (!DI::dba()->exists('contact', ['id' => $cid, 'uid' => local_user()])) {
@@ -68,25 +68,24 @@ class FriendSuggest extends BaseModule
 
                $note = Strings::escapeHtml(trim($_POST['note'] ?? ''));
 
-               $suggest = DI::fsuggest()->insert([
-                       'uid'     => local_user(),
-                       'cid'     => $cid,
-                       'name'    => $contact['name'],
-                       'url'     => $contact['url'],
-                       'request' => $contact['request'],
-                       'photo'   => $contact['avatar'],
-                       'note'    => $note,
-                       'created' => DateTimeFormat::utcNow()
-               ]);
+               $suggest = DI::fsuggest()->save(DI::fsuggestFactory()->createNew(
+                       local_user(),
+                       $cid,
+                       $contact['name'],
+                       $contact['url'],
+                       $contact['request'],
+                       $contact['avatar'],
+                       $note
+               ));
 
                Worker::add(PRIORITY_HIGH, 'Notifier', Delivery::SUGGESTION, $suggest->id);
 
                info(DI::l10n()->t('Friend suggestion sent.'));
        }
 
-       public static function content(array $parameters = [])
+       public function content(): string
        {
-               $cid = intval($parameters['contact']);
+               $cid = intval($this->parameters['contact']);
 
                $contact = DI::dba()->selectFirst('contact', [], ['id' => $cid, 'uid' => local_user()]);
                if (empty($contact)) {
@@ -94,7 +93,7 @@ class FriendSuggest extends BaseModule
                        DI::baseUrl()->redirect();
                }
 
-               $contacts = ContactModel::selectToArray(['id', 'name'], [
+               $suggestableContacts = ContactModel::selectToArray(['id', 'name'], [
                        '`uid` = ? 
                        AND `id` != ? 
                        AND `network` = ? 
@@ -111,8 +110,8 @@ class FriendSuggest extends BaseModule
 
                $formattedContacts = [];
 
-               foreach ($contacts as $contact) {
-                       $formattedContacts[$contact['id']] = $contact['name'];
+               foreach ($suggestableContacts as $suggestableContact) {
+                       $formattedContacts[$suggestableContact['id']] = $suggestableContact['name'];
                }
 
                $tpl = Renderer::getMarkupTemplate('fsuggest.tpl');