Merge pull request #10565 from annando/schedule2

Scheduled posts are now listed and can be deleted

src/Model/Post/Delayed.php

@@ -98,6 +98,23 @@ class Delayed
 		return DBA::delete('delayed-post', ['uri' => $uri, 'uid' => $uid]);
 	}
 
+	/**
+	 * Delete scheduled posts and the associated workerqueue entry
+	 *
+	 * @param integer $id
+	 * @return void
+	 */
+	public static function deleteById(int $id)
+	{
+		$post = DBA::selectFirst('delayed-post', ['wid'], ['id' => $id]);
+		if (empty($post['wid'])) {
+			return;
+		}
+
+		DBA::delete('delayed-post', ['id' => $id]);
+		DBA::delete('workerqueue', ['id' => $post['wid']]);
+	}
+
 	/**
 	 * Check if an entry exists
 	 *

src/Module/Api/Mastodon/ScheduledStatuses.php

@@ -25,6 +25,7 @@ use Friendica\App\Router;
 use Friendica\Core\System;
 use Friendica\Database\DBA;
 use Friendica\DI;
+use Friendica\Model\Post;
 use Friendica\Module\BaseApi;
 
 /**
@@ -49,15 +50,11 @@ class ScheduledStatuses extends BaseApi
 			DI::mstdnError()->UnprocessableEntity();
 		}
 
-		$condtion = ['id' => $parameters['id'], 'uid' => $uid];
-		$post = DBA::selectFirst('delayed-post', ['id'], $condtion);
-		if (empty($post['id'])) {
+		if (!DBA::exists('delayed-post', ['id' => $parameters['id'], 'uid' => $uid])) {
 			DI::mstdnError()->RecordNotFound();
 		}
 
-		if (!DBA::delete('delayed-post', $condtion)) {
-			DI::mstdnError()->RecordNotFound();
-		}
+		Post\Delayed::deleteById($parameters['id']);
 
 		System::jsonExit([]);
 	}

src/Module/Profile/Schedule.php

@@ -21,12 +21,34 @@
 
 namespace Friendica\Module\Profile;
 
+use Friendica\BaseModule;
+use Friendica\Content\Text\BBCode;
+use Friendica\Core\Renderer;
+use Friendica\Database\DBA;
 use Friendica\DI;
+use Friendica\Model\Post;
 use Friendica\Module\BaseProfile;
 use Friendica\Network\HTTPException;
 
 class Schedule extends BaseProfile
 {
+	public static function post(array $parameters = [])
+	{
+		if (!local_user()) {
+			throw new HTTPException\ForbiddenException(DI::l10n()->t('Permission denied.'));
+		}
+
+		if (empty($_REQUEST['delete'])) {
+			throw new HTTPException\BadRequestException();
+		}
+
+		if (!DBA::exists('delayed-post', ['id' => $_REQUEST['delete'], 'uid' => local_user()])) {
+			throw new HTTPException\NotFoundException();
+		}
+
+		Post\Delayed::deleteById($_REQUEST['delete']);
+	}
+
 	public static function content(array $parameters = [])
 	{
 		if (!local_user()) {
@@ -37,7 +59,33 @@ class Schedule extends BaseProfile
 
 		$o = self::getTabsHTML($a, 'schedule', true, $a->user);
 
-		$o .= DI::l10n()->t('Currently here is no functionality here. Please use an app to have a look at your scheduled posts.');
+		$schedule = [];
+		$delayed = DBA::select('delayed-post', [], ['uid' => local_user()]);
+		while ($row = DBA::fetch($delayed)) {
+			$parameter = Post\Delayed::getParametersForid($row['id']);
+			if (empty($parameter)) {
+				continue;
+			}
+			$schedule[] = [
+				'id'           => $row['id'],
+				'scheduled_at' => $row['delayed'],
+				'content'      => BBCode::toPlaintext($parameter['item']['body'], false)
+			];
+		}
+		DBA::close($delayed);
+
+		$tpl = Renderer::getMarkupTemplate('profile/schedule.tpl');
+		$o .= Renderer::replaceMacros($tpl, [
+			'$form_security_token' => BaseModule::getFormSecurityToken("profile_schedule"),
+			'$baseurl'             => DI::baseUrl()->get(true),
+			'$title'               => DI::l10n()->t('Scheduled Posts'),
+			'$nickname'            => $parameters['nickname'] ?? '',
+			'$scheduled_at'        => DI::l10n()->t('Scheduled'),
+			'$content'             => DI::l10n()->t('Content'),
+			'$delete'              => DI::l10n()->t('Remove post'),
+			'$schedule'            => $schedule,
+		]);
+
 		return $o;
 	}
 }

static/routes.config.php

@@ -33,7 +33,7 @@ use Friendica\Module;
 $profileRoutes = [
 	''                                         => [Module\Profile\Index::class,    [R::GET]],
 	'/profile'                                 => [Module\Profile\Profile::class,  [R::GET]],
-	'/schedule'                                => [Module\Profile\Schedule::class, [R::GET]],
+	'/schedule'                                => [Module\Profile\Schedule::class, [R::GET, R::POST]],
 	'/contacts/common'                         => [Module\Profile\Common::class,   [R::GET]],
 	'/contacts[/{type}]'                       => [Module\Profile\Contacts::class, [R::GET]],
 	'/status[/{category}[/{date1}[/{date2}]]]' => [Module\Profile\Status::class,   [R::GET]],

view/lang/C/messages.po

@@ -8,7 +8,7 @@ msgid ""
 msgstr ""
 "Project-Id-Version: 2021.09-dev\n"
 "Report-Msgid-Bugs-To: \n"
-"POT-Creation-Date: 2021-08-02 12:15+0000\n"
+"POT-Creation-Date: 2021-08-02 17:00+0000\n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
 "Last-Translator: FULL NAME <EMAIL@ADDRESS>\n"
 "Language-Team: LANGUAGE <LL@li.org>\n"
@@ -845,7 +845,7 @@ msgstr ""
 #: src/Module/Invite.php:127 src/Module/Notifications/Notification.php:47
 #: src/Module/Notifications/Notification.php:76
 #: src/Module/Profile/Common.php:56 src/Module/Profile/Contacts.php:57
-#: src/Module/Profile/Schedule.php:33 src/Module/Register.php:62
+#: src/Module/Profile/Schedule.php:38 src/Module/Register.php:62
 #: src/Module/Register.php:75 src/Module/Register.php:193
 #: src/Module/Register.php:232 src/Module/Search/Directory.php:38
 #: src/Module/Settings/Delegation.php:42 src/Module/Settings/Delegation.php:70
@@ -7096,7 +7096,7 @@ msgstr ""
 msgid "Only You Can See This"
 msgstr ""
 
-#: src/Module/BaseProfile.php:117
+#: src/Module/BaseProfile.php:117 src/Module/Profile/Schedule.php:68
 msgid "Scheduled Posts"
 msgstr ""
 
@@ -8779,10 +8779,16 @@ msgstr ""
 msgid "%s's comments"
 msgstr ""
 
-#: src/Module/Profile/Schedule.php:40
-msgid ""
-"Currently here is no functionality here. Please use an app to have a look at "
-"your scheduled posts."
+#: src/Module/Profile/Schedule.php:70
+msgid "Scheduled"
+msgstr ""
+
+#: src/Module/Profile/Schedule.php:71
+msgid "Content"
+msgstr ""
+
+#: src/Module/Profile/Schedule.php:72
+msgid "Remove post"
 msgstr ""
 
 #: src/Module/Register.php:69

view/templates/profile/schedule.tpl

@@ -0,0 +1,25 @@
+<div class="generic-page-wrapper">
+	<h1>{{$title}}</h1>
+	<table id="application-block" class="table table-condensed table-striped">
+		<thead>
+			<tr>
+				<th>{{$scheduled_at}}</th>
+				<th>{{$content}}</th>
+			</tr>
+		</thead>
+		<tbody>
+			{{foreach $schedule as $entry}}
+			<tr>
+				<td>{{$entry.scheduled_at}}</td>
+				<td>{{$entry.content}}</td>
+				<td>
+					<form action="{{$baseurl}}/profile/{{$nickname}}/schedule" method="post">
+						<input type="hidden" name="form_security_token" value="{{$form_security_token}}">
+						<button type="submit" name="delete" value="{{$row.id}}" title="{{$delete}}" class="icon s22 delete"></button>
+					</form>
+				</td>
+			</tr>
+			{{/foreach}}
+		</tbody>
+	</table>
+</div>

view/theme/frio/templates/profile/schedule.tpl

@@ -0,0 +1,28 @@
+<div class="generic-page-wrapper">
+	{{* include the title template for the settings title *}}
+	{{include file="section_title.tpl" title=$title}}
+	<table id="application-block" class="table table-condensed table-striped">
+		<thead>
+			<tr>
+				<th>{{$scheduled_at}}</th>
+				<th>{{$content}}</th>
+			</tr>
+		</thead>
+		<tbody>
+			{{foreach $schedule as $row}}
+			<tr>
+				<td>{{$row.scheduled_at}}</td>
+				<td>{{$row.content}}</td>
+				<td>
+					<form action="{{$baseurl}}/profile/{{$nickname}}/schedule" method="post">
+						<input type="hidden" name="form_security_token" value="{{$form_security_token}}">
+						<button class="btn btn-default" type="submit" name="delete" value="{{$row.id}}" title="{{$delete}}">
+							<i class="fa fa-trash" aria-hidden="true"></i>
+						</button>
+					</form>
+				</td>
+			</tr>
+			{{/foreach}}
+		</tbody>
+	</table>
+</div>