From b3b2bc96bab2ed578ae1d1264cd0927a907c68bf Mon Sep 17 00:00:00 2001 From: Hypolite Petovan Date: Sat, 30 Jan 2021 18:05:40 -0500 Subject: [PATCH 1/2] Rename dopin() to doPIn --- view/js/main.js | 17 +++++++++++------ view/templates/wall_thread.tpl | 2 +- view/theme/frio/templates/search_item.tpl | 4 ++-- view/theme/frio/templates/wall_thread.tpl | 8 ++++---- view/theme/quattro/templates/wall_thread.tpl | 4 ++-- view/theme/smoothly/templates/wall_thread.tpl | 2 +- view/theme/vier/templates/wall_thread.tpl | 4 ++-- 7 files changed, 23 insertions(+), 18 deletions(-) diff --git a/view/js/main.js b/view/js/main.js index a0ffea371..089f78010 100644 --- a/view/js/main.js +++ b/view/js/main.js @@ -701,21 +701,26 @@ function dostar(ident) { }); } -function dopin(ident) { +function doPin(ident) { ident = ident.toString(); $('#like-rotator-' + ident).show(); - $.get('pinned/' + ident, function(data) { + $.get('pinned/' + ident) + .then(function(data) { if (data.match(/1/)) { - $('#pinned-' + ident).addClass('pinned'); - $('#pinned-' + ident).removeClass('unpinned'); + $('#pinned-' + ident) + .addClass('pinned') + .removeClass('unpinned'); $('#pin-' + ident).addClass('hidden'); $('#unpin-' + ident).removeClass('hidden'); } else { - $('#pinned-' + ident).addClass('unpinned'); - $('#pinned-' + ident).removeClass('pinned'); + $('#pinned-' + ident) + .addClass('unpinned') + .removeClass('pinned'); $('#pin-' + ident).removeClass('hidden'); $('#unpin-' + ident).addClass('hidden'); } + }) + .always(function () { $('#like-rotator-' + ident).hide(); }); } diff --git a/view/templates/wall_thread.tpl b/view/templates/wall_thread.tpl index 893bd7291..4b1a42785 100644 --- a/view/templates/wall_thread.tpl +++ b/view/templates/wall_thread.tpl @@ -102,7 +102,7 @@ {{/if}} {{if $item.pin}} - + {{/if}} {{if $item.star}} diff --git a/view/theme/frio/templates/search_item.tpl b/view/theme/frio/templates/search_item.tpl index 717340b8b..835b3d667 100644 --- a/view/theme/frio/templates/search_item.tpl +++ b/view/theme/frio/templates/search_item.tpl @@ -199,8 +199,8 @@ {{if $item.pin}}
  • -  {{$item.pin.do}} -  {{$item.pin.undo}} +  {{$item.pin.do}} +  {{$item.pin.undo}}
    • {{/if}} diff --git a/view/theme/frio/templates/wall_thread.tpl b/view/theme/frio/templates/wall_thread.tpl index 25d6885fc..b98cefb9c 100644 --- a/view/theme/frio/templates/wall_thread.tpl +++ b/view/theme/frio/templates/wall_thread.tpl @@ -351,8 +351,8 @@ as the value of $top_child_total (this is done at the end of this file) {{if $item.pin}}
  • -  {{$item.pin.do}} -  {{$item.pin.undo}} +  {{$item.pin.do}} +  {{$item.pin.undo}}
    • {{/if}} @@ -516,8 +516,8 @@ as the value of $top_child_total (this is done at the end of this file) {{if $item.pin}}
  • -  {{$item.pin.do}} -  {{$item.pin.undo}} +  {{$item.pin.do}} +  {{$item.pin.undo}}
    • {{/if}} diff --git a/view/theme/quattro/templates/wall_thread.tpl b/view/theme/quattro/templates/wall_thread.tpl index f7d4b0f8c..5f682dd3e 100644 --- a/view/theme/quattro/templates/wall_thread.tpl +++ b/view/theme/quattro/templates/wall_thread.tpl @@ -98,8 +98,8 @@
    {{if $item.pin}} - {{$item.pin.do}} - {{$item.pin.undo}} + {{$item.pin.do}} + {{$item.pin.undo}} {{/if}} {{if $item.star}} {{$item.star.do}} diff --git a/view/theme/smoothly/templates/wall_thread.tpl b/view/theme/smoothly/templates/wall_thread.tpl index 6fd787afe..817d4d5ed 100644 --- a/view/theme/smoothly/templates/wall_thread.tpl +++ b/view/theme/smoothly/templates/wall_thread.tpl @@ -114,7 +114,7 @@ {{/if}} {{if $item.pin}} - + {{/if}} {{if $item.star}} diff --git a/view/theme/vier/templates/wall_thread.tpl b/view/theme/vier/templates/wall_thread.tpl index 7a04b5efe..6963fd411 100644 --- a/view/theme/vier/templates/wall_thread.tpl +++ b/view/theme/vier/templates/wall_thread.tpl @@ -136,8 +136,8 @@ {{/if}} {{if $item.pin}} - {{$item.pin.do}} - {{$item.pin.undo}} + {{$item.pin.do}} + {{$item.pin.undo}} {{/if}} {{if $item.star}} {{$item.star.do}} From 188f7b0eb07a37afa6ff276e9b479560d0123c4d Mon Sep 17 00:00:00 2001 From: Hypolite Petovan Date: Sat, 30 Jan 2021 18:42:23 -0500 Subject: [PATCH 2/2] Move GET pinned/{id} to POST item/{id}/pin --- src/Module/{Pinned.php => Item/Pin.php} | 44 +++++++++++++++++-------- static/routes.config.php | 2 +- view/js/main.js | 4 +-- 3 files changed, 33 insertions(+), 17 deletions(-) rename src/Module/{Pinned.php => Item/Pin.php} (58%) diff --git a/src/Module/Pinned.php b/src/Module/Item/Pin.php similarity index 58% rename from src/Module/Pinned.php rename to src/Module/Item/Pin.php index 97364ceff..d99b1a345 100644 --- a/src/Module/Pinned.php +++ b/src/Module/Item/Pin.php @@ -19,42 +19,58 @@ * */ -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\Network\HTTPException; /** * Toggle pinned items */ -class Pinned extends BaseModule +class Pin extends BaseModule { public static function rawContent(array $parameters = []) { - if (!local_user()) { - throw new \Friendica\Network\HTTPException\ForbiddenException(); + $l10n = DI::l10n(); + + if (!Session::isAuthenticated()) { + throw new HttpException\ForbiddenException($l10n->t('Access denied.')); } - if (empty($parameters['item'])) { - throw new \Friendica\Network\HTTPException\BadRequestException(); + if (empty($parameters['id'])) { + throw new HTTPException\BadRequestException(); } - $itemId = intval($parameters['item']); + $itemId = intval($parameters['id']); $pinned = !Item::getPinned($itemId, local_user()); Item::setPinned($itemId, local_user(), $pinned); // See if we've been passed a return path to redirect to - $returnPath = $_REQUEST['return'] ?? ''; - if (!empty($returnPath)) { - $rand = '_=' . time() . (strpos($returnPath, '?') ? '&' : '?') . 'rand'; - DI::baseUrl()->redirect($returnPath . $rand); + $return_path = $_REQUEST['return'] ?? ''; + if (!empty($return_path)) { + $rand = '_=' . time(); + if (strpos($return_path, '?')) { + $rand = "&$rand"; + } else { + $rand = "?$rand"; + } + + DI::baseUrl()->redirect($return_path . $rand); } - // the json doesn't really matter, it will either be 0 or 1 - echo json_encode((int)$pinned); - exit(); + $return = [ + 'status' => 'ok', + 'item_id' => $itemId, + 'verb' => 'pin', + 'state' => (int)$pinned, + ]; + + System::jsonExit($return); } } diff --git a/static/routes.config.php b/static/routes.config.php index 8108c24cd..5a2ee2ffd 100644 --- a/static/routes.config.php +++ b/static/routes.config.php @@ -292,6 +292,7 @@ return [ '/item' => [ '/ignore/{id}' => [Module\Item\Ignore::class, [R::GET]], + '/{id:\d+}/pin' => [Module\Item\Pin::class, [ R::POST]], ], '/like/{item:\d+}' => [Module\Like::class, [R::GET]], @@ -351,7 +352,6 @@ return [ '/{type}/{customize}/{name}' => [Module\Photo::class, [R::GET]], ], - '/pinned/{item:\d+}' => [Module\Pinned::class, [R::GET]], '/pretheme' => [Module\ThemeDetails::class, [R::GET]], '/probe' => [Module\Debug\Probe::class, [R::GET]], diff --git a/view/js/main.js b/view/js/main.js index 089f78010..1503529d1 100644 --- a/view/js/main.js +++ b/view/js/main.js @@ -704,9 +704,9 @@ function dostar(ident) { function doPin(ident) { ident = ident.toString(); $('#like-rotator-' + ident).show(); - $.get('pinned/' + ident) + $.post('item/' + ident + '/pin') .then(function(data) { - if (data.match(/1/)) { + if (data.state === 1) { $('#pinned-' + ident) .addClass('pinned') .removeClass('unpinned');