From: Michael Vogel Date: Sun, 31 Jan 2021 10:35:40 +0000 (+0100) Subject: Merge pull request #9892 from MrPetovan/task/9872-item-pin X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=2d687b1052e980b8119525e3d298ce1b0b9e52fe;hp=9885c1f128313514026d0214a06034d818e1c32a;p=friendica.git Merge pull request #9892 from MrPetovan/task/9872-item-pin Move GET pinned/{id} to POST item/{id}/pin --- diff --git a/src/Module/Item/Pin.php b/src/Module/Item/Pin.php new file mode 100644 index 0000000000..d99b1a3452 --- /dev/null +++ b/src/Module/Item/Pin.php @@ -0,0 +1,76 @@ +. + * + */ + +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 Pin extends BaseModule +{ + public static function rawContent(array $parameters = []) + { + $l10n = DI::l10n(); + + if (!Session::isAuthenticated()) { + throw new HttpException\ForbiddenException($l10n->t('Access denied.')); + } + + if (empty($parameters['id'])) { + throw new HTTPException\BadRequestException(); + } + + $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 + $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); + } + + $return = [ + 'status' => 'ok', + 'item_id' => $itemId, + 'verb' => 'pin', + 'state' => (int)$pinned, + ]; + + System::jsonExit($return); + } +} diff --git a/src/Module/Pinned.php b/src/Module/Pinned.php deleted file mode 100644 index 97364ceffb..0000000000 --- a/src/Module/Pinned.php +++ /dev/null @@ -1,60 +0,0 @@ -. - * - */ - -namespace Friendica\Module; - -use Friendica\BaseModule; -use Friendica\DI; -use Friendica\Model\Item; - -/** - * Toggle pinned items - */ -class Pinned extends BaseModule -{ - public static function rawContent(array $parameters = []) - { - if (!local_user()) { - throw new \Friendica\Network\HTTPException\ForbiddenException(); - } - - if (empty($parameters['item'])) { - throw new \Friendica\Network\HTTPException\BadRequestException(); - } - - $itemId = intval($parameters['item']); - - $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); - } - - // the json doesn't really matter, it will either be 0 or 1 - echo json_encode((int)$pinned); - exit(); - } -} diff --git a/static/routes.config.php b/static/routes.config.php index 8108c24cdc..5a2ee2ffd1 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 a0ffea3718..1503529d1f 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) { - if (data.match(/1/)) { - $('#pinned-' + ident).addClass('pinned'); - $('#pinned-' + ident).removeClass('unpinned'); + $.post('item/' + ident + '/pin') + .then(function(data) { + if (data.state === 1) { + $('#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 893bd7291a..4b1a427859 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 717340b8b3..835b3d6677 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 25d6885fc8..b98cefb9c9 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 f7d4b0f8c8..5f682dd3e2 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 6fd787afee..817d4d5ed0 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 7a04b5efed..6963fd411a 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}}