]> git.mxchange.org Git - friendica.git/commitdiff
Function renamed, documentation added
authorMichael <heluecht@pirati.ca>
Fri, 30 Jul 2021 10:24:08 +0000 (10:24 +0000)
committerMichael <heluecht@pirati.ca>
Fri, 30 Jul 2021 10:24:08 +0000 (10:24 +0000)
src/Factory/Api/Mastodon/ScheduledStatus.php
src/Module/Api/Mastodon/ScheduledStatuses.php
src/Module/Api/Mastodon/Statuses.php
src/Object/Api/Mastodon/ScheduledStatus.php

index f0d846fc00326a0aa64cc73da9fe36ff53182fba..ce79cc80221626097c394cec68794d3e40ea0f98 100644 (file)
@@ -45,7 +45,7 @@ class ScheduledStatus extends BaseFactory
         * @return \Friendica\Object\Api\Mastodon\ScheduledStatus
         * @throws HTTPException\InternalServerErrorException
         */
-       public function createFromId(int $id, int $uid): \Friendica\Object\Api\Mastodon\ScheduledStatus
+       public function createFromDelayedPostId(int $id, int $uid): \Friendica\Object\Api\Mastodon\ScheduledStatus
        {
                $delayed_post = $this->dba->selectFirst('delayed-post', [], ['id' => $id, 'uid' => $uid]);
                if (empty($delayed_post)) {
@@ -53,6 +53,9 @@ class ScheduledStatus extends BaseFactory
                }
 
                $parameters = Post\Delayed::getParametersForid($delayed_post['id']);
+               if (empty($parameters)) {
+                       throw new HTTPException\NotFoundException('Scheduled status with ID ' . $id . ' not found for user ' . $uid . '.');
+               }
 
                return new \Friendica\Object\Api\Mastodon\ScheduledStatus($delayed_post, $parameters);
        }
index 5e7f4b9547b0703c154429d0c1ed106377c333df..fab70738f0fb0709d22a6a01647466893eefd0ed 100644 (file)
@@ -72,7 +72,7 @@ class ScheduledStatuses extends BaseApi
                $uid = self::getCurrentUserID();
 
                if (isset($parameters['id'])) {
-                               System::jsonExit(DI::mstdnScheduledStatus()->createFromId($parameters['id'], $uid)->toArray());
+                               System::jsonExit(DI::mstdnScheduledStatus()->createFromDelayedPostId($parameters['id'], $uid)->toArray());
                }
 
                $request = self::getRequest([
@@ -104,7 +104,7 @@ class ScheduledStatuses extends BaseApi
                $statuses = [];
                while ($post = DBA::fetch($posts)) {
                        self::setBoundaries($post['id']);
-                       $statuses[] = DI::mstdnScheduledStatus()->createFromId($post['id'], $uid);
+                       $statuses[] = DI::mstdnScheduledStatus()->createFromDelayedPostId($post['id'], $uid);
                }
                DBA::close($posts);
 
index 40bbca0cf067ec6a8d90d7d00ec8747cc7295c48..cd6bbda3f890e6c2b68b7475ca768dd58b5912dd 100644 (file)
@@ -190,7 +190,7 @@ class Statuses extends BaseApi
                        if (empty($id)) {
                                DI::mstdnError()->InternalError();
                        }
-                       System::jsonExit(DI::mstdnScheduledStatus()->createFromId($id, $uid)->toArray());
+                       System::jsonExit(DI::mstdnScheduledStatus()->createFromDelayedPostId($id, $uid)->toArray());
                }
 
                $id = Item::insert($item, true);
index f63ba8db51da733cf425a750c24f8c82acc90400..473cd99a5436cb2cc9ab0fa92105f272579ab16c 100644 (file)
@@ -55,7 +55,8 @@ class ScheduledStatus extends BaseDataTransferObject
        /**
         * Creates a status record from a delayed-post record.
         *
-        * @param array   $delayed_post
+        * @param array $delayed_post Record with the delayed post
+        * @param array $parameters   Parameters for the workerqueue entry for the delayed post
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        public function __construct(array $delayed_post, array $parameters)