]> git.mxchange.org Git - friendica.git/commitdiff
Refactor API notification usage
authornupplaPhil <admin@philipp.info>
Tue, 28 Jan 2020 20:28:57 +0000 (21:28 +0100)
committernupplaPhil <admin@philipp.info>
Tue, 28 Jan 2020 20:28:57 +0000 (21:28 +0100)
- Remove "mapFields()" from BaseModel
- Add new Notification API entity (including collection)
- Add new NotificationFactory method "getApiList()"

include/api.php
src/BaseCollection.php
src/BaseEntity.php
src/BaseModel.php
src/Collection/Api/Notifications.php [new file with mode: 0644]
src/DI.php
src/Factory/Notification/Notification.php
src/Model/Notify.php
src/Object/Api/Friendica/Notification.php [new file with mode: 0644]

index a69bd01b4971edfcea66cb166a3542111c3cd931..3f4d5625b0f8b44f1d4931e141de6cdf33982edf 100644 (file)
@@ -5892,10 +5892,11 @@ api_register_func('api/friendica/activity/unattendmaybe', 'api_friendica_activit
  * Returns notifications
  *
  * @param string $type Known types are 'atom', 'rss', 'xml' and 'json'
+ *
  * @return string|array
- * @throws BadRequestException
  * @throws ForbiddenException
- * @throws InternalServerErrorException
+ * @throws BadRequestException
+ * @throws Exception
  */
 function api_friendica_notification($type)
 {
@@ -5908,7 +5909,7 @@ function api_friendica_notification($type)
                throw new BadRequestException("Invalid argument count");
        }
 
-       $notifications = DI::notify()->select(['uid' => api_user()], ['order' => ['seen' => 'ASC', 'date' => 'DESC'], 'limit' => 50]);
+       $notifications = DI::notification()->getApiList(local_user());
 
        if ($type == "xml") {
                $xmlnotes = false;
index 5a20acee7f4f2a0f1ccf6e34fd6d2a6e03330d64..9a9efdb0681a8e125bd438d98dc8ca97255942e8 100644 (file)
@@ -17,14 +17,14 @@ abstract class BaseCollection extends \ArrayIterator
        protected $totalCount = 0;
 
        /**
-        * @param BaseModel[] $models
-        * @param int|null    $totalCount
+        * @param BaseEntity[] $entities
+        * @param int|null     $totalCount
         */
-       public function __construct(array $models = [], int $totalCount = null)
+       public function __construct(array $entities = [], int $totalCount = null)
        {
-               parent::__construct($models);
+               parent::__construct($entities);
 
-               $this->totalCount = $totalCount ?? count($models);
+               $this->totalCount = $totalCount ?? count($entities);
        }
 
        /**
index 9f0cb31f8ea5c8297b34c3112dc6c9f59fe24f53..14f95c197ecef43caa27d34e32e0c97e55ee376a 100644 (file)
@@ -11,7 +11,22 @@ namespace Friendica;
  */
 abstract class BaseEntity implements \JsonSerializable
 {
+       /**
+        * Returns the current entity as an json array
+        *
+        * @return array
+        */
        public function jsonSerialize()
+       {
+               return $this->toArray();
+       }
+
+       /**
+        * Returns the current entity as an array
+        *
+        * @return array
+        */
+       public function toArray()
        {
                return get_object_vars($this);
        }
index decc627521e50309a69dfe9195a1705ceab92b7d..2c952888b2e54edb766856756e7124252d70ced7 100644 (file)
@@ -12,7 +12,7 @@ use Psr\Log\LoggerInterface;
  *
  * @property int id
  */
-abstract class BaseModel
+abstract class BaseModel extends BaseEntity
 {
        /** @var Database */
        protected $dba;
@@ -48,23 +48,9 @@ abstract class BaseModel
                $this->originalData = $data;
        }
 
-       /**
-        * Maps a data array (original/current) to a known field list of the chosen model
-        *
-        * This is useful to filter out additional attributes, which aren't part of the db-table (like readonly cached fields)
-        *
-        * @param array $data The data array to map to db-fields
-        *
-        * @return array the mapped data array
-        */
-       protected function mapFields(array $data)
-       {
-               return $data;
-       }
-
        public function getOriginalData()
        {
-               return $this->mapFields($this->originalData);
+               return $this->originalData;
        }
 
        public function resetOriginalData()
@@ -129,16 +115,9 @@ abstract class BaseModel
                $this->data[$name] = $value;
        }
 
-       /**
-        * Returns the values of the current model as an array
-        *
-        * @param bool $dbOnly True, if just the db-relevant fields should be returned
-        *
-        * @return array The values of the current model
-        */
-       public function toArray(bool $dbOnly = false)
+       public function toArray()
        {
-               return $dbOnly ? $this->mapFields($this->data) : $this->data;
+               return $this->data;
        }
 
        protected function checkValid()
diff --git a/src/Collection/Api/Notifications.php b/src/Collection/Api/Notifications.php
new file mode 100644 (file)
index 0000000..5bd8983
--- /dev/null
@@ -0,0 +1,17 @@
+<?php
+
+namespace Friendica\Collection\Api;
+
+use Friendica\BaseCollection;
+use Friendica\Object\Api\Friendica\Notification;
+
+class Notifications extends BaseCollection
+{
+       /**
+        * @return Notification
+        */
+       public function current()
+       {
+               return parent::current();
+       }
+}
index fc18b30b8ea04f3cc9a436973d6baa4e9bf5ca8f..65c82225d54c1f3757283e6781094701cd1b2724 100644 (file)
@@ -280,14 +280,6 @@ abstract class DI
                return self::$dice->create(Model\User\Cookie::class);
        }
 
-       /**
-        * @return Repository\Notify
-        */
-       public static function notify()
-       {
-               return self::$dice->create(Repository\Notify::class);
-       }
-
        /**
         * @return Model\Storage\IStorage
         */
@@ -324,6 +316,14 @@ abstract class DI
                return self::$dice->create(Repository\ProfileField::class);
        }
 
+       /**
+        * @return Repository\Notify
+        */
+       public static function notify()
+       {
+               return self::$dice->create(Repository\Notify::class);
+       }
+
        //
        // "Protocol" namespace instances
        //
index 5f2c2231b89cdd79df3b469b6d9d290baa82f186..2b179cca53845c4ee4a3523aa3c9f1bf43a807ce 100644 (file)
@@ -6,6 +6,7 @@ use Exception;
 use Friendica\App;
 use Friendica\App\BaseURL;
 use Friendica\BaseFactory;
+use Friendica\Collection\Api\Notifications as ApiNotifications;
 use Friendica\Content\Text\BBCode;
 use Friendica\Core\L10n;
 use Friendica\Core\PConfig\IPConfig;
@@ -15,6 +16,7 @@ use Friendica\Database\Database;
 use Friendica\Model\Item;
 use Friendica\Module\BaseNotifications;
 use Friendica\Network\HTTPException\InternalServerErrorException;
+use Friendica\Object\Api\Friendica\Notification as ApiNotification;
 use Friendica\Protocol\Activity;
 use Friendica\Repository;
 use Friendica\Util\DateTimeFormat;
@@ -352,4 +354,26 @@ class Notification extends BaseFactory
 
                return $formattedNotifications;
        }
+
+       /**
+        * @param int   $uid    The user id of the API call
+        * @param array $params Additional parameters
+        *
+        * @return ApiNotifications
+        *
+        * @throws Exception
+        */
+       public function getApiList(int $uid, array $params = ['order' => ['seen' => 'ASC', 'date' => 'DESC'], 'limit' => 50])
+       {
+               $notifies = $this->notification->select(['uid' => $uid], $params);
+
+               /** @var ApiNotification[] $notifications */
+               $notifications = [];
+
+               foreach ($notifies as $notify) {
+                       $notifications[] = new ApiNotification($notify);
+               }
+
+               return new ApiNotifications($notifications);
+       }
 }
index c48fa0f679051af1506e25bc90ddadc3a05ead02..b12eb341b0d188d4adbcdf4f8308e6301925584f 100644 (file)
@@ -5,19 +5,12 @@ namespace Friendica\Model;
 use Exception;
 use Friendica\BaseModel;
 use Friendica\Content\Text\BBCode;
-use Friendica\Content\Text\HTML;
 use Friendica\Database\Database;
 use Friendica\Network\HTTPException\InternalServerErrorException;
-use Friendica\Util\DateTimeFormat;
-use Friendica\Util\Temporal;
 use Psr\Log\LoggerInterface;
 
 /**
  * Model for an entry in the notify table
- * - Including additional, calculated properties
- *
- * Is used either for frontend interactions or for API-based interaction
- * @see https://github.com/friendica/friendica/blob/develop/doc/API-Entities.md#notification
  *
  * @property string  hash
  * @property integer type
@@ -36,11 +29,6 @@ use Psr\Log\LoggerInterface;
  *
  * @property-read string name_cache Full name of the contact subject
  * @property-read string msg_cache  Plaintext version of the notification text with a placeholder (`{0}`) for the subject contact's name.
- *
- * @property-read integer timestamp  Unix timestamp
- * @property-read string  dateRel       Time since the note was posted, eg "1 hour ago"
- * @property-read string  $msg_html
- * @property-read string  $msg_plain
  */
 class Notify extends BaseModel
 {
@@ -59,8 +47,7 @@ class Notify extends BaseModel
                $this->repo = $repo;
 
                $this->setNameCache();
-               $this->setTimestamp();
-               $this->setMsg();
+               $this->setMsgCache();
        }
 
        /**
@@ -81,41 +68,23 @@ class Notify extends BaseModel
                }
        }
 
-       /**
-        * Set some extra properties to the notification from db:
-        *  - timestamp as int in default TZ
-        *  - date_rel : relative date string
-        */
-       private function setTimestamp()
-       {
-               try {
-                       $this->timestamp = strtotime(DateTimeFormat::local($this->date));
-               } catch (Exception $e) {
-               }
-               $this->dateRel = Temporal::getRelativeDate($this->date);
-       }
-
        /**
         * Sets the pre-formatted name (caching)
-        *
-        * @throws InternalServerErrorException
         */
        private function setNameCache()
        {
-               $this->name_cache = strip_tags(BBCode::convert($this->source_name ?? ''));
+               try {
+                       $this->name_cache = strip_tags(BBCode::convert($this->source_name ?? ''));
+               } catch (InternalServerErrorException $e) {
+               }
        }
 
        /**
-        * Set some extra properties to the notification from db:
-        *  - msg_html: message as html string
-        *  - msg_plain: message as plain text string
-        *  - msg_cache: The pre-formatted message (caching)
+        * Sets the pre-formatted msg (caching)
         */
-       private function setMsg()
+       private function setMsgCache()
        {
                try {
-                       $this->msg_html  = BBCode::convert($this->msg, false);
-                       $this->msg_plain = explode("\n", trim(HTML::toPlaintext($this->msg_html, 0)))[0];
                        $this->msg_cache = self::formatMessage($this->name_cache, strip_tags(BBCode::convert($this->msg)));
                } catch (InternalServerErrorException $e) {
                }
@@ -125,12 +94,8 @@ class Notify extends BaseModel
        {
                parent::__set($name, $value);
 
-               if ($name == 'date') {
-                       $this->setTimestamp();
-               }
-
                if ($name == 'msg') {
-                       $this->setMsg();
+                       $this->setMsgCache();
                }
 
                if ($name == 'source_name') {
@@ -163,29 +128,4 @@ class Notify extends BaseModel
 
                return $message;
        }
-
-       /**
-        * {@inheritDoc}
-        */
-       protected function mapFields(array $data)
-       {
-               return [
-                       'hash'       => $data['hash'] ?? '',
-                       'type'       => $data['type'] ?? 0,
-                       'name'       => $data['name'] ?? '',
-                       'url'        => $data['url'] ?? '',
-                       'photo'      => $data['photo'] ?? '',
-                       'date'       => $data['date'] ?? DateTimeFormat::utcNow(),
-                       'msg'        => $data['msg'] ?? '',
-                       'uid'        => $data['uid'] ?? 0,
-                       'link'       => $data['link'] ?? '',
-                       'iid'        => $data['iid'] ?? 0,
-                       'parent'     => $data['parent'] ?? 0,
-                       'seen'       => $data['seen'] ?? false,
-                       'verb'       => $data['verb'] ?? '',
-                       'otype'      => $data['otype'] ?? '',
-                       'name_cache' => $data['name_cache'] ?? null,
-                       'msg_cache'  => $data['msg_cache'] ?? null,
-               ];
-       }
 }
diff --git a/src/Object/Api/Friendica/Notification.php b/src/Object/Api/Friendica/Notification.php
new file mode 100644 (file)
index 0000000..910d5ec
--- /dev/null
@@ -0,0 +1,79 @@
+<?php
+
+namespace Friendica\Object\Api\Friendica;
+
+use Friendica\BaseEntity;
+use Friendica\Content\Text\BBCode;
+use Friendica\Content\Text\HTML;
+use Friendica\Model\Notify;
+use Friendica\Util\DateTimeFormat;
+use Friendica\Util\Temporal;
+
+/**
+ * Friendica Notification
+ *
+ * @see https://github.com/friendica/friendica/blob/develop/doc/API-Entities.md#notification
+ */
+class Notification extends BaseEntity
+{
+       /** @var integer */
+       protected $id;
+       /** @var string */
+       protected $hash;
+       /** @var integer */
+       protected $type;
+       /** @var string Full name of the contact subject */
+       protected $name;
+       /** @var string Profile page URL of the contact subject */
+       protected $url;
+       /** @var string Profile photo URL of the contact subject */
+       protected $photo;
+       /** @var string YYYY-MM-DD hh:mm:ss local server time */
+       protected $date;
+       /** @var string The message (BBCode) */
+       protected $msg;
+       /** @var integer Owner User Id */
+       protected $uid;
+       /** @var string Notification URL */
+       protected $link;
+       /** @var integer Item Id */
+       protected $iid;
+       /** @var integer Parent Item Id */
+       protected $parent;
+       /** @var boolean  Whether the notification was read or not. */
+       protected $seen;
+       /** @var string Verb URL @see http://activitystrea.ms */
+       protected $verb;
+       /** @var string Subject type (`item`, `intro` or `mail`) */
+       protected $otype;
+       /** @var string Full name of the contact subject (HTML) */
+       protected $name_cache;
+       /** @var string Plaintext version of the notification text with a placeholder (`{0}`) for the subject contact's name. (Plaintext) */
+       protected $msg_cache;
+       /** @var integer  Unix timestamp */
+       protected $timestamp;
+       /** @var string Time since the note was posted, eg "1 hour ago" */
+       protected $date_rel;
+       /** @var string Message (HTML) */
+       protected $msg_html;
+       /** @var string Message (Plaintext) */
+       protected $msg_plain;
+
+       public function __construct(Notify $notify)
+       {
+               // map each notify attribute to the entity
+               foreach ($notify->toArray() as $key => $value) {
+                       $this->{$key} = $value;
+               }
+
+               // add additional attributes for the API
+               try {
+                       $this->timestamp = strtotime(DateTimeFormat::local($this->date));
+                       $this->msg_html  = BBCode::convert($this->msg, false);
+                       $this->msg_plain = explode("\n", trim(HTML::toPlaintext($this->msg_html, 0)))[0];
+               } catch (\Exception $e) {
+               }
+
+               $this->date_rel = Temporal::getRelativeDate($this->date);
+       }
+}