Move GET starred/{id} to POST item/{id}/star

src/Module/Item/Star.php

@@ -19,33 +19,38 @@
  *
  */
 
-namespace Friendica\Module;
+namespace Friendica\Module\Item;
 
 use Friendica\BaseModule;
+use Friendica\Core\Session;
+use Friendica\Core\System;
 use Friendica\DI;
 use Friendica\Model\Item;
 use Friendica\Model\Post;
+use Friendica\Network\HTTPException;
 
 /**
  * Toggle starred items
  */
-class Starred extends BaseModule
+class Star extends BaseModule
 {
 	public static function rawContent(array $parameters = [])
 	{
-		if (!local_user()) {
+		$l10n = DI::l10n();
-			throw new \Friendica\Network\HTTPException\ForbiddenException();
+
+		if (!Session::isAuthenticated()) {
+			throw new HttpException\ForbiddenException($l10n->t('Access denied.'));
 		}
 
-		if (empty($parameters['item'])) {
+		if (empty($parameters['id'])) {
-			throw new \Friendica\Network\HTTPException\BadRequestException();
+			throw new HTTPException\BadRequestException();
 		}
 
-		$itemId = intval($parameters['item']);
+		$itemId = intval($parameters['id']);
 
 		$item = Post::selectFirstForUser(local_user(), ['starred'], ['uid' => local_user(), 'id' => $itemId]);
 		if (empty($item)) {
-			throw new \Friendica\Network\HTTPException\NotFoundException();
+			throw new HTTPException\NotFoundException();
 		}
 
 		$starred = !(bool)$item['starred'];
@@ -53,14 +58,25 @@ class Starred extends BaseModule
 		Item::update(['starred' => $starred], ['id' => $itemId]);
 
 		// See if we've been passed a return path to redirect to
-		$returnPath = $_REQUEST['return'] ?? '';
+		$return_path = $_REQUEST['return'] ?? '';
-		if (!empty($returnPath)) {
+		if (!empty($return_path)) {
-			$rand = '_=' . time() . (strpos($returnPath, '?') ? '&' : '?') . 'rand';
+			$rand = '_=' . time();
-			DI::baseUrl()->redirect($returnPath . $rand);
+			if (strpos($return_path, '?')) {
+				$rand = "&$rand";
+			} else {
+				$rand = "?$rand";
 			}
 
-		// the json doesn't really matter, it will either be 0 or 1
+			DI::baseUrl()->redirect($return_path . $rand);
-		echo json_encode((int)$starred);
+		}
-		exit();
+
+		$return = [
+			'status'  => 'ok',
+			'item_id' => $itemId,
+			'verb'    => 'star',
+			'state'   => (int)$starred,
+		];
+
+		System::jsonExit($return);
 	}
 }

static/routes.config.php

@@ -294,6 +294,7 @@ return [
 		'/{id:\d+}/activity/{verb}' => [Module\Item\Activity::class,    [        R::POST]],
 		'/{id:\d+}/ignore'          => [Module\Item\Ignore::class,      [        R::POST]],
 		'/{id:\d+}/pin'             => [Module\Item\Pin::class,         [        R::POST]],
+		'/{id:\d+}/star'            => [Module\Item\Star::class,        [        R::POST]],
 	],
 
 	'/localtime'          => [Module\Debug\Localtime::class, [R::GET, R::POST]],
@@ -412,7 +413,6 @@ return [
 	'/rsd.xml'                       => [Module\ReallySimpleDiscovery::class, [R::GET]],
 	'/smilies[/json]'                => [Module\Smilies::class,               [R::GET]],
 	'/statistics.json'               => [Module\Statistics::class,            [R::GET]],
-	'/starred/{item:\d+}'            => [Module\Starred::class,               [R::GET]],
 	'/toggle_mobile'                 => [Module\ToggleMobile::class,          [R::GET]],
 	'/tos'                           => [Module\Tos::class,                   [R::GET]],
 

view/js/main.js

@@ -685,7 +685,7 @@ function dosubthread(ident) {
 function doStar(ident) {
 	ident = ident.toString();
 	$('#like-rotator-' + ident).show();
-	$.get('starred/' + ident)
+	$.post('item/' + ident + '/star')
 	.then(function(data) {
 		if (data.state === 1) {
 			$('#starred-' + ident)