forked from GNUsocial/gnu-social
[UI][FEEDS][ENTITY][Feed] Add way to customize the feeds that are displayed in the left panel. The user can add, delete, reoder and rename them
This commit is contained in:
parent
cdc8886bb8
commit
11d2cfb9ed
9
public/assets/icons/edit.svg.twig
Normal file
9
public/assets/icons/edit.svg.twig
Normal file
@ -0,0 +1,9 @@
|
||||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<!-- https://github.com/primer/octicons -->
|
||||
|
||||
<!-- MIT License -->
|
||||
|
||||
<svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" viewBox="0 0 16 16">
|
||||
<title>edit</title>
|
||||
<path fill="#fff" fill-rule="evenodd" d="M11.013 1.427a1.75 1.75 0 012.474 0l1.086 1.086a1.75 1.75 0 010 2.474l-8.61 8.61c-.21.21-.47.364-.756.445l-3.251.93a.75.75 0 01-.927-.928l.929-3.25a1.75 1.75 0 01.445-.758l8.61-8.61zm1.414 1.06a.25.25 0 00-.354 0L10.811 3.75l1.439 1.44 1.263-1.263a.25.25 0 000-.354l-1.086-1.086zM11.189 6.25L9.75 4.81l-6.286 6.287a.25.25 0 00-.064.108l-.558 1.953 1.953-.558a.249.249 0 00.108-.064l6.286-6.286z"/>
|
||||
</svg>
|
After Width: | Height: | Size: 676 B |
@ -35,17 +35,27 @@ declare(strict_types = 1);
|
||||
|
||||
namespace App\Controller;
|
||||
|
||||
use App\Core\Cache;
|
||||
use App\Core\Controller;
|
||||
use App\Core\DB\DB;
|
||||
use App\Core\Event;
|
||||
use App\Core\Form;
|
||||
use function App\Core\I18n\_m;
|
||||
use App\Core\Router\Router;
|
||||
use App\Core\VisibilityScope;
|
||||
use App\Entity\Feed;
|
||||
use App\Entity\Note;
|
||||
use App\Util\Common;
|
||||
use App\Util\Exception\ClientException;
|
||||
use App\Util\Exception\NotFoundException;
|
||||
use App\Util\Exception\NotImplementedException;
|
||||
use App\Util\Exception\RedirectException;
|
||||
use Functional as F;
|
||||
use Symfony\Component\Form\Extension\Core\Type\IntegerType;
|
||||
use Symfony\Component\Form\Extension\Core\Type\SubmitType;
|
||||
use Symfony\Component\Form\Extension\Core\Type\TextType;
|
||||
use Symfony\Component\HttpFoundation\Request;
|
||||
use Symfony\Component\Routing\Exception\ResourceNotFoundException;
|
||||
|
||||
class Feeds extends Controller
|
||||
{
|
||||
@ -128,6 +138,115 @@ class Feeds extends Controller
|
||||
];
|
||||
}
|
||||
|
||||
public function edit_feeds(Request $request)
|
||||
{
|
||||
$user = Common::ensureLoggedIn();
|
||||
$key = Feed::cacheKey($user);
|
||||
$feeds = Feed::getFeeds($user);
|
||||
|
||||
$form_definitions = [];
|
||||
foreach ($feeds as $feed) {
|
||||
$md5 = md5($feed->getUrl());
|
||||
$form_definitions[] = [$md5 . '-url', TextType::class, ['data' => $feed->getUrl(), 'label' => ' ']];
|
||||
$form_definitions[] = [$md5 . '-order', IntegerType::class, ['data' => $feed->getOrdering(), 'label' => ' ']];
|
||||
$form_definitions[] = [$md5 . '-title', TextType::class, ['data' => $feed->getTitle(), 'label' => ' ']];
|
||||
$form_definitions[] = [$md5 . '-remove', SubmitType::class, ['label' => _m('Remove')]];
|
||||
}
|
||||
|
||||
$form_definitions[] = ['url', TextType::class, ['label' => _m('New feed'), 'required' => false]];
|
||||
$form_definitions[] = ['order', IntegerType::class, ['label' => _m('Order'), 'data' => (\count($form_definitions) / 4) + 1]];
|
||||
$form_definitions[] = ['title', TextType::class, ['label' => _m('Title'), 'required' => false]];
|
||||
$form_definitions[] = ['add', SubmitType::class, ['label' => _m('Add')]];
|
||||
$form_definitions[] = ['update_exisiting', SubmitType::class, ['label' => _m('Update existing')]];
|
||||
$form_definitions[] = ['reset', SubmitType::class, ['label' => _m('Reset to default values')]];
|
||||
|
||||
$form = Form::create($form_definitions);
|
||||
|
||||
$form->handleRequest($request);
|
||||
if ($form->isSubmitted() && $form->isValid()) {
|
||||
array_pop($form_definitions);
|
||||
array_pop($form_definitions);
|
||||
array_pop($form_definitions);
|
||||
array_pop($form_definitions);
|
||||
array_pop($form_definitions);
|
||||
|
||||
$data = $form->getData();
|
||||
|
||||
if ($form->get('update_exisiting')->isClicked()) {
|
||||
// Each feed has a URL, an order and a title
|
||||
$feeds_data = array_chunk($data, 3, preserve_keys: true);
|
||||
// The last three would be the new one
|
||||
array_pop($feeds_data);
|
||||
// Sort by the order
|
||||
usort($feeds_data, fn ($fd_l, $fd_r) => next($fd_l) <=> next($fd_r));
|
||||
// Make the order sequential
|
||||
$order = 1;
|
||||
foreach ($feeds_data as $i => $fd) {
|
||||
next($fd);
|
||||
$feeds_data[$i][key($fd)] = $order++;
|
||||
}
|
||||
// Update the fields in the corresponding feed
|
||||
foreach ($feeds_data as $fd) {
|
||||
$md5 = str_replace('-url', '', array_key_first($fd));
|
||||
$feed = F\first($feeds, fn ($f) => md5($f->getUrl()) === $md5);
|
||||
$feed->setUrl($fd[$md5 . '-url']);
|
||||
$feed->setOrdering($fd[$md5 . '-order']);
|
||||
$feed->setTitle($fd[$md5 . '-title']);
|
||||
DB::merge($feed);
|
||||
}
|
||||
DB::flush();
|
||||
Cache::delete($key);
|
||||
throw new RedirectException();
|
||||
}
|
||||
|
||||
// Remove feed
|
||||
foreach ($form_definitions as [$field, $type, $opts]) {
|
||||
if (str_ends_with($field, '-url')) {
|
||||
$remove_id = str_replace('-url', '-remove', $field);
|
||||
if ($form->get($remove_id)->isClicked()) {
|
||||
DB::remove(DB::getReference('feed', ['actor_id' => $user->getId(), 'url' => $opts['data']]));
|
||||
DB::flush();
|
||||
Cache::delete($key);
|
||||
throw new RedirectException();
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if ($form->get('reset')->isClicked()) {
|
||||
F\map(DB::findBy('feed', ['actor_id' => $user->getId()]), fn ($f) => DB::remove($f));
|
||||
DB::flush();
|
||||
Cache::delete($key);
|
||||
Feed::createDefaultFeeds($user->getId(), $user);
|
||||
DB::flush();
|
||||
throw new RedirectException();
|
||||
}
|
||||
|
||||
// Add feed
|
||||
try {
|
||||
$match = Router::match($data['url']);
|
||||
$route = $match['_route'];
|
||||
DB::persist(Feed::create([
|
||||
'actor_id' => $user->getId(),
|
||||
'url' => $data['url'],
|
||||
'route' => $route,
|
||||
'title' => $data['title'],
|
||||
'ordering' => $data['order'],
|
||||
]));
|
||||
DB::flush();
|
||||
Cache::delete($key);
|
||||
throw new RedirectException();
|
||||
} catch (ResourceNotFoundException) {
|
||||
// throw new ClientException(_m('Invalid route'));
|
||||
// continue bellow
|
||||
}
|
||||
}
|
||||
|
||||
return [
|
||||
'_template' => 'feeds/edit_feeds.html.twig',
|
||||
'form' => $form->createView(),
|
||||
];
|
||||
}
|
||||
|
||||
public function replies(Request $request)
|
||||
{
|
||||
// TODO replies
|
||||
|
@ -8,8 +8,10 @@ use App\Core\Controller;
|
||||
use App\Core\DB\DB;
|
||||
use App\Core\Event;
|
||||
use App\Core\Form;
|
||||
use function App\Core\I18n\_m;
|
||||
use App\Core\Log;
|
||||
use App\Entity\Actor;
|
||||
use App\Entity\Feed;
|
||||
use App\Entity\LocalUser;
|
||||
use App\Entity\Subscription;
|
||||
use App\Security\Authenticator;
|
||||
@ -39,7 +41,6 @@ use Symfony\Component\Security\Guard\GuardAuthenticatorHandler;
|
||||
use Symfony\Component\Security\Http\Authentication\AuthenticationUtils;
|
||||
use Symfony\Component\Validator\Constraints\Length;
|
||||
use Symfony\Component\Validator\Constraints\NotBlank;
|
||||
use function App\Core\I18n\_m;
|
||||
|
||||
class Security extends Controller
|
||||
{
|
||||
@ -155,8 +156,11 @@ class Security extends Controller
|
||||
DB::persistWithSameId(
|
||||
$actor,
|
||||
$user,
|
||||
// Self subscription
|
||||
fn (int $id) => DB::persist(Subscription::create(['subscriber' => $id, 'subscribed' => $id])),
|
||||
function (int $id) use ($user) {
|
||||
// Self subscription
|
||||
DB::persist(Subscription::create(['subscriber' => $id, 'subscribed' => $id]));
|
||||
Feed::createDefaultFeeds($id, $user);
|
||||
},
|
||||
);
|
||||
|
||||
Event::handle('SuccessfulLocalUserRegistration', [$actor, $user]);
|
||||
|
179
src/Entity/Feed.php
Normal file
179
src/Entity/Feed.php
Normal file
@ -0,0 +1,179 @@
|
||||
<?php
|
||||
|
||||
declare(strict_types = 1);
|
||||
|
||||
// {{{ License
|
||||
// This file is part of GNU social - https://www.gnu.org/software/social
|
||||
//
|
||||
// GNU social is free software: you can redistribute it and/or modify
|
||||
// it under the terms of the GNU Affero General Public License as published by
|
||||
// the Free Software Foundation, either version 3 of the License, or
|
||||
// (at your option) any later version.
|
||||
//
|
||||
// GNU social is distributed in the hope that it will be useful,
|
||||
// but WITHOUT ANY WARRANTY; without even the implied warranty of
|
||||
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
||||
// GNU Affero General Public License for more details.
|
||||
//
|
||||
// You should have received a copy of the GNU Affero General Public License
|
||||
// along with GNU social. If not, see <http://www.gnu.org/licenses/>.
|
||||
// }}}
|
||||
|
||||
namespace App\Entity;
|
||||
|
||||
use App\Core\Cache;
|
||||
use App\Core\DB\DB;
|
||||
use App\Core\Entity;
|
||||
use function App\Core\I18n\_m;
|
||||
use DateTimeInterface;
|
||||
|
||||
/**
|
||||
* Entity for feeds a user follows
|
||||
*
|
||||
* @category DB
|
||||
* @package GNUsocial
|
||||
*
|
||||
* @author Hugo Sales <hugo@hsal.es>
|
||||
* @copyright 2021 Free Software Foundation, Inc http://www.fsf.org
|
||||
* @license https://www.gnu.org/licenses/agpl.html GNU AGPL v3 or later
|
||||
*/
|
||||
class Feed extends Entity
|
||||
{
|
||||
// {{{ Autocode
|
||||
// @codeCoverageIgnoreStart
|
||||
private int $actor_id;
|
||||
private string $url;
|
||||
private string $title;
|
||||
private string $route;
|
||||
private int $ordering;
|
||||
private DateTimeInterface $created;
|
||||
private DateTimeInterface $modified;
|
||||
|
||||
public function setActorId(int $actor_id): self
|
||||
{
|
||||
$this->actor_id = $actor_id;
|
||||
return $this;
|
||||
}
|
||||
|
||||
public function getActorId(): int
|
||||
{
|
||||
return $this->actor_id;
|
||||
}
|
||||
|
||||
public function setUrl(string $url): self
|
||||
{
|
||||
$this->url = $url;
|
||||
return $this;
|
||||
}
|
||||
|
||||
public function getUrl(): string
|
||||
{
|
||||
return $this->url;
|
||||
}
|
||||
|
||||
public function setTitle(string $title): self
|
||||
{
|
||||
$this->title = $title;
|
||||
return $this;
|
||||
}
|
||||
|
||||
public function getTitle(): string
|
||||
{
|
||||
return $this->title;
|
||||
}
|
||||
|
||||
public function setRoute(string $route): self
|
||||
{
|
||||
$this->route = $route;
|
||||
return $this;
|
||||
}
|
||||
|
||||
public function getRoute(): string
|
||||
{
|
||||
return $this->route;
|
||||
}
|
||||
|
||||
public function setOrdering(int $ordering): self
|
||||
{
|
||||
$this->ordering = $ordering;
|
||||
return $this;
|
||||
}
|
||||
|
||||
public function getOrdering(): int
|
||||
{
|
||||
return $this->ordering;
|
||||
}
|
||||
|
||||
public function setCreated(DateTimeInterface $created): self
|
||||
{
|
||||
$this->created = $created;
|
||||
return $this;
|
||||
}
|
||||
|
||||
public function getCreated(): DateTimeInterface
|
||||
{
|
||||
return $this->created;
|
||||
}
|
||||
|
||||
public function setModified(DateTimeInterface $modified): self
|
||||
{
|
||||
$this->modified = $modified;
|
||||
return $this;
|
||||
}
|
||||
|
||||
public function getModified(): DateTimeInterface
|
||||
{
|
||||
return $this->modified;
|
||||
}
|
||||
|
||||
// @codeCoverageIgnoreEnd
|
||||
// }}} Autocode
|
||||
|
||||
public static function cacheKey(LocalUser|Actor $user_actor): string
|
||||
{
|
||||
return 'feeds-' . $user_actor->getId();
|
||||
}
|
||||
|
||||
/**
|
||||
* @return self[]
|
||||
*/
|
||||
public static function getFeeds(LocalUser|Actor $user_actor): array
|
||||
{
|
||||
return Cache::getList(self::cacheKey($user_actor), fn () => DB::findBy('feed', ['actor_id' => $user_actor->getId()], order_by: ['ordering' => 'ASC']));
|
||||
}
|
||||
|
||||
/**
|
||||
* This is called in the register function, in `DB::persistWithSameId`,
|
||||
* so we don't have the $user with an id yet, hence the awkward
|
||||
* arguments
|
||||
*/
|
||||
public static function createDefaultFeeds(int $actor_id, LocalUser $user): void
|
||||
{
|
||||
DB::persist(self::create(['actor_id' => $actor_id, 'url' => '/main/public',
|
||||
'route' => 'main_public', 'title' => _m('Public'), 'ordering' => 1, ]));
|
||||
DB::persist(self::create(['actor_id' => $actor_id, 'url' => '/main/all',
|
||||
'route' => 'main_all', 'title' => _m('Network'), 'ordering' => 2, ]));
|
||||
DB::persist(self::create(['actor_id' => $actor_id, 'url' => '/@' . $user->getNickname() . '/all',
|
||||
'route' => 'home_all', 'title' => _m('Home'), 'ordering' => 3, ]));
|
||||
}
|
||||
|
||||
public static function schemaDef(): array
|
||||
{
|
||||
return [
|
||||
'name' => 'feed',
|
||||
'fields' => [
|
||||
'actor_id' => ['type' => 'int', 'foreign key' => true, 'target' => 'Actor.id', 'multiplicity' => 'many to one', 'not null' => true, 'description' => 'foreign key to actor table'],
|
||||
'url' => ['type' => 'text', 'not null' => true],
|
||||
'title' => ['type' => 'text', 'not null' => true],
|
||||
'route' => ['type' => 'text', 'not null' => true],
|
||||
'ordering' => ['type' => 'int', 'not null' => true],
|
||||
'created' => ['type' => 'datetime', 'not null' => true, 'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was created'],
|
||||
'modified' => ['type' => 'timestamp', 'not null' => true, 'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was modified'],
|
||||
],
|
||||
'primary key' => ['actor_id', 'url'],
|
||||
'indexes' => [
|
||||
'feed_actor_id_idx' => ['actor_id'],
|
||||
],
|
||||
];
|
||||
}
|
||||
}
|
@ -59,7 +59,7 @@ abstract class Main
|
||||
$r->connect('main_all', '/main/all', [C\Feeds::class, 'network']);
|
||||
$r->connect('home_all', '/@{nickname<' . Nickname::DISPLAY_FMT . '>}/all', [C\Feeds::class, 'home']);
|
||||
$r->connect('replies', '/@{nickname<' . Nickname::DISPLAY_FMT . '>}/replies', [C\Feeds::class, 'replies']);
|
||||
$r->connect('edit_feeds', '/edit-feeds', [C\Feeds::class, 'replies']);
|
||||
$r->connect('edit_feeds', '/edit-feeds', [C\Feeds::class, 'edit_feeds']);
|
||||
|
||||
$r->connect('panel', '/panel', [C\AdminPanel::class, 'site']);
|
||||
$r->connect('panel_site', '/panel/site', [C\AdminPanel::class, 'site']);
|
||||
|
@ -73,6 +73,7 @@ class Extension extends AbstractExtension
|
||||
new TwigFunction('handle_override_stylesheet', [Runtime::class, 'handleOverrideStylesheet']),
|
||||
|
||||
new TwigFunction('open_details', [Runtime::class, 'openDetails']),
|
||||
new TwigFunction('get_feeds', [Runtime::class, 'getFeeds']),
|
||||
];
|
||||
}
|
||||
}
|
||||
|
@ -33,6 +33,8 @@ declare(strict_types = 1);
|
||||
namespace App\Twig;
|
||||
|
||||
use App\Core\Event;
|
||||
use App\Entity\Actor;
|
||||
use App\Entity\Feed;
|
||||
use App\Entity\Note;
|
||||
use App\Util\Common;
|
||||
use App\Util\Formatting;
|
||||
@ -143,9 +145,14 @@ class Runtime implements RuntimeExtensionInterface, EventSubscriberInterface
|
||||
return $result;
|
||||
}
|
||||
|
||||
public function openDetails(?string $query, array $ids)
|
||||
public function openDetails(?string $query, array $ids): string
|
||||
{
|
||||
return in_array($query, $ids) ? 'open=""' : '';
|
||||
return \in_array($query, $ids) ? 'open=""' : '';
|
||||
}
|
||||
|
||||
public function getFeeds(Actor $actor): array
|
||||
{
|
||||
return Feed::getFeeds($actor);
|
||||
}
|
||||
|
||||
// ----------------------------------------------------------
|
||||
|
@ -16,13 +16,11 @@
|
||||
</li>
|
||||
</ul>
|
||||
{% else %} {# User custom feeds #}
|
||||
<a href="{{ path('edit_feeds') }}">{{ icon('edit', 'icon') }}</a>
|
||||
<a href="{{ path('edit_feeds') }}">{{ icon('edit', 'icon') | raw }}</a>
|
||||
<ul>
|
||||
{% for link in handle_event('GetLeftNavigationFeeds', {'actor': current_actor}) %}
|
||||
{% for link in get_feeds(current_actor) %}
|
||||
<li>
|
||||
<a href="{{ link['path'] }}" class='{{ active(link['path_id']) }}'>
|
||||
{{ link['title'] }}
|
||||
</a>
|
||||
<a href="{{ link.getUrl() }}" class='{{ active(link.getRoute()) }}'>{{ link.getTitle() }}</a>
|
||||
</li>
|
||||
{% endfor %}
|
||||
</ul>
|
||||
|
7
templates/feeds/edit_feeds.html.twig
Normal file
7
templates/feeds/edit_feeds.html.twig
Normal file
@ -0,0 +1,7 @@
|
||||
{% extends 'base.html.twig' %}
|
||||
|
||||
{% block body %}
|
||||
<div class="section-widget">
|
||||
{{ form(form) }}
|
||||
</div>
|
||||
{% endblock %}
|
Loading…
Reference in New Issue
Block a user