From 82e271589f9e6332635147d6a32ed1fce1443893 Mon Sep 17 00:00:00 2001
From: Philipp Holzer <admin+github@philipp.info>
Date: Sat, 18 May 2019 21:38:02 +0200
Subject: [PATCH] Move mod/starred to src/Module/Starred

---
 mod/starred.php        | 51 -----------------------------------
 src/App/Router.php     |  1 +
 src/Module/Starred.php | 60 ++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 61 insertions(+), 51 deletions(-)
 delete mode 100644 mod/starred.php
 create mode 100644 src/Module/Starred.php

diff --git a/mod/starred.php b/mod/starred.php
deleted file mode 100644
index 0705d5436a..0000000000
--- a/mod/starred.php
+++ /dev/null
@@ -1,51 +0,0 @@
-<?php
-/**
- * @file mod/starred.php
- */
-use Friendica\App;
-use Friendica\Database\DBA;
-use Friendica\Model\Item;
-
-function starred_init(App $a) {
-	$starred = 0;
-	$message_id = null;
-
-	if (!local_user()) {
-		exit();
-	}
-	if ($a->argc > 1) {
-		$message_id = intval($a->argv[1]);
-	}
-	if (!$message_id) {
-		exit();
-	}
-
-	$item = Item::selectFirstForUser(local_user(), ['starred'], ['uid' => local_user(), 'id' => $message_id]);
-	if (!DBA::isResult($item)) {
-		exit();
-	}
-
-	if (!intval($item['starred'])) {
-		$starred = 1;
-	}
-
-	Item::update(['starred' => $starred], ['id' => $message_id]);
-
-	// See if we've been passed a return path to redirect to
-	$return_path = defaults($_REQUEST, 'return', '');
-	if ($return_path) {
-		$rand = '_=' . time();
-		if (strpos($return_path, '?')) {
-			$rand = "&$rand";
-		} else {
-			$rand = "?$rand";
-		}
-
-		$a->internalRedirect($return_path . $rand);
-	}
-
-	// the json doesn't really matter, it will either be 0 or 1
-
-	echo json_encode($starred);
-	exit();
-}
diff --git a/src/App/Router.php b/src/App/Router.php
index 133544c819..e9eeebf86a 100644
--- a/src/App/Router.php
+++ b/src/App/Router.php
@@ -207,6 +207,7 @@ class Router
 		$this->routeCollector->addRoute(['GET'],         '/rsd.xml',             Module\ReallySimpleDiscovery::class);
 		$this->routeCollector->addRoute(['GET'],         '/smilies[/json]',      Module\Smilies::class);
 		$this->routeCollector->addRoute(['GET'],         '/statistics.json',     Module\Statistics::class);
+		$this->routeCollector->addRoute(['GET'],         '/starred/{item:\d+}',  Module\Starred::class);
 		$this->routeCollector->addRoute(['GET'],         '/toggle_mobile',       Module\ToggleMobile::class);
 		$this->routeCollector->addRoute(['GET'],         '/tos',                 Module\Tos::class);
 		$this->routeCollector->addRoute(['GET'],         '/viewsrc/{item:\d+}',  Module\ItemBody::class);
diff --git a/src/Module/Starred.php b/src/Module/Starred.php
new file mode 100644
index 0000000000..df7da40992
--- /dev/null
+++ b/src/Module/Starred.php
@@ -0,0 +1,60 @@
+<?php
+
+namespace Friendica\Module;
+
+use Friendica\BaseModule;
+use Friendica\Model\Item;
+
+/**
+ * Toggle starred items
+ */
+class Starred extends BaseModule
+{
+	public static function rawContent()
+	{
+		$a = self::getApp();
+		$starred = 0;
+		$itemId = null;
+
+		if (!local_user()) {
+			exit();
+		}
+
+		// @TODO: Replace with parameter from router
+		if ($a->argc > 1) {
+			$itemId = intval($a->argv[1]);
+		}
+
+		if (!$itemId) {
+			exit();
+		}
+
+		$item = Item::selectFirstForUser(local_user(), ['starred'], ['uid' => local_user(), 'id' => $itemId]);
+		if (empty($item)) {
+			exit();
+		}
+
+		if (!intval($item['starred'])) {
+			$starred = 1;
+		}
+
+		Item::update(['starred' => $starred], ['id' => $itemId]);
+
+		// See if we've been passed a return path to redirect to
+		$returnPath = defaults($_REQUEST, 'return', '');
+		if ($returnPath) {
+			$rand = '_=' . time();
+			if (strpos($returnPath, '?')) {
+				$rand = "&$rand";
+			} else {
+				$rand = "?$rand";
+			}
+
+			$a->internalRedirect($returnPath . $rand);
+		}
+
+		// the json doesn't really matter, it will either be 0 or 1
+		echo json_encode($starred);
+		exit();
+	}
+}