[COMPONENT][Notification] Feed: Fix typo in query

Fixed a couple of bugs

.env.test

@@ -3,4 +3,4 @@ KERNEL_CLASS='App\Kernel'
 APP_SECRET='$ecretf0rt3st'
 SYMFONY_DEPRECATIONS_HELPER=999999
 PANTHER_APP_ENV=panther
-DATABASE_URL=postgresql://postgres:password@db:5432/social
+DATABASE_URL=postgresql://postgres:password@db:5432/test

Makefile

@@ -69,4 +69,54 @@ remove-file:
 flush-redis-cache:
 	docker exec -it $(call translate-container-name,$(strip $(DIR))_redis_1) sh -c 'redis-cli flushall'
 
-force-nuke-everything: down up flush-redis-cache database-force-nuke remove-var remove-file
+install-plugins:
+	docker exec -it $(call translate-container-name,$(strip $(DIR))_php_1) /var/www/social/bin/install_plugins.sh
+
+update-dependencies:
+	docker exec -it $(call translate-container-name,$(strip $(DIR))_php_1) sh -c 'cd /var/www/social && composer update'
+
+update-autocode:
+	docker exec -it $(call translate-container-name,$(strip $(DIR))_php_1) sh -c 'cd /var/www/social && bin/update_autocode'
+
+backup-actors:
+	docker exec -it $(call translate-container-name,$(strip $(DIR))_db_1) \
+	sh -c 'su postgres -c "mkdir -p /tmp/backup"' && \
+	docker exec -it $(call translate-container-name,$(strip $(DIR))_php_1) \
+	sh -c "cd /var/www/social && bin/console doctrine:query:sql \"\
+	copy actor to '/tmp/backup/actor.csv';\
+	copy local_user to '/tmp/backup/local_user.csv';\
+	copy local_group to '/tmp/backup/local_group.csv';\
+	\
+	copy activitypub_actor to '/tmp/backup/ap_actor.csv';\
+	copy activitypub_rsa to '/tmp/backup/ap_rsa.csv';\
+	\
+	copy actor_subscription to '/tmp/backup/actor_subscription.csv';\
+	copy group_member to '/tmp/backup/group_member.csv';\
+	\
+	copy feed to '/tmp/backup/feed.csv';\
+	copy (SELECT 'ALTER SEQUENCE ' || c.relname || ' RESTART WITH ' || nextval(c.relname::regclass) || ';'\
+	FROM pg_class c WHERE c.relkind = 'S') to '/tmp/backup/sequences';\"" && \
+	mkdir -p /tmp/social-sql-backup && \
+	docker cp $(call translate-container-name,$(strip $(DIR))_db_1):/tmp/backup/. /tmp/social-sql-backup
+
+restore-actors:
+	docker cp /tmp/social-sql-backup/. $(call translate-container-name,$(strip $(DIR))_db_1):/tmp/backup
+	docker exec -it $(call translate-container-name,$(strip $(DIR))_db_1) sh -c 'chown postgres /tmp/backup' && \
+	docker exec -it $(call translate-container-name,$(strip $(DIR))_php_1) \
+	sh -c "cd /var/www/social && bin/console doctrine:query:sql \"\
+	copy actor from '/tmp/backup/actor.csv';\
+	copy local_user from '/tmp/backup/local_user.csv';\
+	copy local_group from '/tmp/backup/local_group.csv';\
+	\
+	copy activitypub_actor from '/tmp/backup/ap_actor.csv';\
+	copy activitypub_rsa from '/tmp/backup/ap_rsa.csv';\
+	\
+	copy actor_subscription from '/tmp/backup/actor_subscription.csv';\
+	copy group_member from '/tmp/backup/group_member.csv';\
+	\
+	copy feed from '/tmp/backup/feed.csv';\
+	`cat /tmp/social-sql-backup/sequences`\""
+
+force-nuke-everything: down remove-var remove-file up flush-redis-cache database-force-nuke install-plugins
+
+force-delete-content: backup-actors force-nuke-everything restore-actors

bin/configure

@@ -352,8 +352,8 @@ SOCIAL_DBMS=${DBMS}
 SOCIAL_DB=${DB_NAME}
 SOCIAL_USER=${DB_USER}
 SOCIAL_PASSWORD=${DB_PASSWORD}
-SOCIAL_DOMAIN=${DOMAIN}
+CONFIG_DOMAIN=${DOMAIN}
-SOCIAL_NODE_NAME=${NODE_NAME}
+CONFIG_NODE_NAME=${NODE_NAME}
 SOCIAL_ADMIN_EMAIL=${EMAIL}
 SOCIAL_SITE_PROFILE=${PROFILE}
 MAILER_DSN=${MAILER_DSN}

bin/install_plugins.sh

@@ -0,0 +1,11 @@
+#!/bin/sh
+
+for plugin in plugins/*; do
+    install="${plugin}/bin/install.sh"
+    if [ -x "${install}" ]; then
+        ( # subshell, to clear options/environment
+            set -x
+            "${install}"
+        )
+    fi
+done

bin/update_autocode

@@ -20,26 +20,23 @@ const types = [
     'text'         => 'string',
     'varchar'      => 'string',
     'phone_number' => 'PhoneNumber',
+    'float'        => 'float', // TODO REMOVE THIS
 ];
 
 $files = array_merge(glob(ROOT . '/src/Entity/*.php'),
                      array_merge(glob(ROOT . '/components/*/Entity/*.php'),
                                  glob(ROOT . '/plugins/*/Entity/*.php')));
 
-$classes = [];
 $nullable_no_defaults_warning = [];
 
 foreach ($files as $file) {
 
     require_once $file;
 
-    $declared = get_declared_classes();
+    $class = str_replace(['/', 'src', 'components', 'plugins'], ['\\', 'App', 'Component', 'Plugin'], substr($file, strlen(ROOT) + 1, -4));
-    foreach ($declared as $dc) {
+
-        if (preg_match('/(App|(Component|Plugin)\\\\[^\\\\]+)\\\\Entity/', $dc) && !in_array($dc, $classes)) {
+    if (!method_exists($class, 'schemaDef')) {
-            $class = $dc;
+        continue;
-            $classes[] = $class;
-            break;
-        }
     }
 
     $no_ns_class  = preg_replace('/.*?\\\\/', '', $class);

codeception.yml

@@ -1,10 +0,0 @@
-paths:
-    tests: tests/CodeCeption
-    output: tests/CodeCeption/_output
-    data: tests/CodeCeption/_data
-    support: tests/CodeCeption/_support
-    envs: tests/CodeCeption/_envs
-actor_suffix: Tester
-extensions:
-    enabled:
-        - Codeception\Extension\RunFailed

components/Attachment/Attachment.php

@@ -70,7 +70,14 @@ class Attachment extends Component
 
     public function onCollectionQueryAddJoins(QueryBuilder &$note_qb, QueryBuilder &$actor_qb): bool
     {
-        $note_qb->leftJoin(E\AttachmentToNote::class, 'attachment_to_note', Expr\Join::WITH, 'note.id = attachment_to_note.note_id');
+        if (!\in_array('attachment_to_note', $note_qb->getAllAliases())) {
+            $note_qb->leftJoin(
+                join: E\AttachmentToNote::class,
+                alias: 'attachment_to_note',
+                conditionType: Expr\Join::WITH,
+                condition: 'note.id = attachment_to_note.note_id',
+            );
+        }
         return Event::next;
     }
 

components/Attachment/Controller/Attachment.php

@@ -35,6 +35,7 @@ use App\Util\Exception\NoSuchFileException;
 use App\Util\Exception\NotFoundException;
 use App\Util\Exception\ServerException;
 use Component\Attachment\Entity\AttachmentThumbnail;
+use Component\Attachment\Entity\AttachmentToNote;
 use Symfony\Component\HttpFoundation\HeaderUtils;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
@@ -50,7 +51,12 @@ class Attachment extends Controller
         $attachment = DB::findOneBy('attachment', ['id' => $attachment_id]);
         $note       = \is_int($note) ? Note::getById($note) : $note;
 
-        // Before anything, ensure proper scope
+        // Before anything, two very important things!
+        // first: ensure this attachment is associated with this note
+        if (DB::count(AttachmentToNote::class, ['attachment_id' => $attachment->getId(), 'note_id' => $note->getId()]) <= 0) {
+            throw new ClientException(_m('No such attachment.'), 404);
+        }
+        // second: ensure proper scope
         if (!$note->isVisibleTo(Common::actor())) {
             throw new ClientException(_m('You don\'t have permissions to view this attachment.'), 401);
         }
@@ -89,7 +95,7 @@ class Attachment extends Controller
         try {
             return $this->attachment($attachment_id, $note_id, function ($res) use ($note_id, $attachment_id) {
                 return [
-                    '_template'        => '/cards/attachments/show.html.twig',
+                    '_template'        => 'attachment/view.html.twig',
                     'download'         => $res['attachment']->getDownloadUrl(note: $note_id),
                     'title'            => $res['title'],
                     'attachment'       => $res['attachment'],
@@ -145,12 +151,18 @@ class Attachment extends Controller
      */
     public function attachmentThumbnailWithNote(Request $request, int $note_id, int $attachment_id, string $size = 'small'): Response
     {
-        // Before anything, ensure proper scope
-        if (!Note::getById($note_id)->isVisibleTo(Common::actor())) {
-            throw new ClientException(_m('You don\'t have permissions to view this thumbnail.'), 401);
-        }
-
         $attachment = DB::findOneBy('attachment', ['id' => $attachment_id]);
+        $note       = Note::getById($note_id);
+
+        // Before anything, two very important things!
+        // first: ensure this attachment is associated with this note
+        if (DB::count(AttachmentToNote::class, ['attachment_id' => $attachment->getId(), 'note_id' => $note->getId()]) <= 0) {
+            throw new ClientException(_m('No such attachment.'), 404);
+        }
+        // second: ensure proper scope
+        if (!$note->isVisibleTo(Common::actor())) {
+            throw new ClientException(_m('You don\'t have permissions to view this attachment.'), 401);
+        }
 
         $crop = Common::config('thumbnail', 'smart_crop');
 

components/Attachment/Entity/Attachment.php

@@ -35,6 +35,7 @@ use App\Entity\Note;
 use App\Util\Common;
 use App\Util\Exception\ClientException;
 use App\Util\Exception\DuplicateFoundException;
+use App\Util\Exception\NoSuchFileException;
 use App\Util\Exception\NotFoundException;
 use App\Util\Exception\ServerException;
 use DateTimeInterface;
@@ -223,8 +224,7 @@ class Attachment extends Entity
                     $this->setFilename(null);
                     $this->setSize(null);
                     // Important not to null neither width nor height
-                    DB::persist($this);
+                    DB::wrapInTransaction(fn () => DB::persist($this));
-                    DB::flush();
                 }
             } else {
                 // @codeCoverageIgnoreStart
@@ -339,7 +339,11 @@ class Attachment extends Entity
      */
     public function getThumbnails()
     {
-        return DB::findBy('attachment_thumbnail', ['attachment_id' => $this->id]);
+        return DB::findBy(
+            AttachmentThumbnail::class,
+            ['attachment_id' => $this->id],
+            order_by: ['size' => 'ASC', 'mimetype' => 'ASC'],
+        );
     }
 
     public function getPath()
@@ -367,15 +371,17 @@ class Attachment extends Entity
      * @throws ClientException
      * @throws NotFoundException
      * @throws ServerException
-     *
-     * @return AttachmentThumbnail
      */
     public function getThumbnail(?string $size = null, bool $crop = false): ?AttachmentThumbnail
     {
-        return AttachmentThumbnail::getOrCreate(attachment: $this, size: $size, crop: $crop);
+        try {
+            return AttachmentThumbnail::getOrCreate(attachment: $this, size: $size, crop: $crop);
+        } catch (NoSuchFileException) {
+            return null;
+        }
     }
 
-    public function getThumbnailUrl(Note|int $note, ?string $size = null)
+    public function getThumbnailUrl(Note|int $note, ?string $size = null): string
     {
         return Router::url('note_attachment_thumbnail', ['note_id' => \is_int($note) ? $note : $note->getId(), 'attachment_id' => $this->getId(), 'size' => $size ?? Common::config('thumbnail', 'default_size')]);
     }

components/Attachment/Entity/AttachmentThumbnail.php

@@ -30,6 +30,7 @@ use App\Core\Event;
 use App\Core\GSFile;
 use App\Core\Log;
 use App\Core\Router\Router;
+use App\Entity\Note;
 use App\Util\Common;
 use App\Util\Exception\ClientException;
 use App\Util\Exception\NotFoundException;
@@ -179,7 +180,7 @@ class AttachmentThumbnail extends Entity
         if (isset($this->attachment) && !\is_null($this->attachment)) {
             return $this->attachment;
         } else {
-            return $this->attachment = DB::findOneBy('attachment', ['id' => $this->attachment_id]);
+            return $this->attachment = DB::findOneBy(Attachment::class, ['id' => $this->attachment_id]);
         }
     }
 
@@ -204,7 +205,7 @@ class AttachmentThumbnail extends Entity
         try {
             return Cache::get(
                 self::getCacheKey($attachment->getId(), $size_int),
-                fn () => DB::findOneBy('attachment_thumbnail', ['attachment_id' => $attachment->getId(), 'size' => $size_int]),
+                fn () => DB::findOneBy(self::class, ['attachment_id' => $attachment->getId(), 'size' => $size_int]),
             );
         } catch (NotFoundException) {
             if (\is_null($attachment->getWidth()) || \is_null($attachment->getHeight())) {
@@ -213,7 +214,7 @@ class AttachmentThumbnail extends Entity
             [$predicted_width, $predicted_height] = self::predictScalingValues($attachment->getWidth(), $attachment->getHeight(), $size, $crop);
             if (\is_null($attachment->getPath()) || !file_exists($attachment->getPath())) {
                 // Before we quit, check if there's any other thumb
-                $alternative_thumbs = DB::findBy('attachment_thumbnail', ['attachment_id' => $attachment->getId()]);
+                $alternative_thumbs = DB::findBy(self::class, ['attachment_id' => $attachment->getId()]);
                 usort($alternative_thumbs, fn ($l, $r) => $r->getSize() <=> $l->getSize());
                 if (empty($alternative_thumbs)) {
                     throw new NotStoredLocallyException();
@@ -253,14 +254,14 @@ class AttachmentThumbnail extends Entity
         }
     }
 
-    public function getPath()
+    public function getPath(): string
     {
         return Common::config('thumbnail', 'dir') . \DIRECTORY_SEPARATOR . $this->getFilename();
     }
 
-    public function getUrl()
+    public function getUrl(Note|int $note): string
     {
-        return Router::url('attachment_thumbnail', ['id' => $this->getAttachmentId(), 'size' => self::sizeIntToStr($this->getSize())]);
+        return Router::url('note_attachment_thumbnail', ['note_id' => \is_int($note) ? $note : $note->getId(), 'attachment_id' => $this->getAttachmentId(), 'size' => self::sizeIntToStr($this->getSize())]);
     }
 
     /**
@@ -277,9 +278,10 @@ class AttachmentThumbnail extends Entity
             }
         }
         Cache::delete(self::getCacheKey($this->getAttachmentId(), $this->getSize()));
-        DB::remove($this);
         if ($flush) {
-            DB::flush();
+            DB::wrapInTransaction(fn () => DB::remove($this));
+        } else {
+            DB::remove($this);
         }
     }
 

components/Attachment/tests/Controller/AttachmentTest.php

@@ -21,10 +21,12 @@ declare(strict_types = 1);
 
 // }}}
 
-namespace App\Tests\Controller;
+namespace Component\Attachment\tests\Controller;
 
 use App\Core\DB\DB;
 use App\Util\GNUsocialTestCase;
+use Component\Attachment\Entity\Attachment;
+use Component\Attachment\Entity\AttachmentToNote;
 
 class AttachmentTest extends GNUsocialTestCase
 {
@@ -32,31 +34,33 @@ class AttachmentTest extends GNUsocialTestCase
     {
         // This calls static::bootKernel(), and creates a "client" that is acting as the browser
         $client = static::createClient();
-        $client->request('GET', '/attachment');
+        //$client->request('GET', '/attachment');
+        //$this->assertResponseStatusCodeSame(404);
+        $client->request('GET', '/object/note/1/attachment/-1');
         $this->assertResponseStatusCodeSame(404);
-        $client->request('GET', '/attachment/-1');
+        $client->request('GET', '/object/note/1/attachment/asd');
         $this->assertResponseStatusCodeSame(404);
-        $client->request('GET', '/attachment/asd');
+        $client->request('GET', '/object/note/1/attachment/0');
-        $this->assertResponseStatusCodeSame(404);
-        $client->request('GET', '/attachment/0');
         // In the meantime, throwing ClientException doesn't actually result in the reaching the UI, as it's intercepted
         // by the helpful framework that displays the stack traces and such. This should be easily fixable when we have
         // our own error pages
-        $this->assertSelectorTextContains('.stacktrace', 'ClientException');
+        $this->assertResponseStatusCodeSame(500); // TODO (exception page) 404
+        $this->assertSelectorTextContains('.stacktrace', 'No such attachment.');
     }
 
     private function testAttachment(string $suffix = '')
     {
-        $client  = static::createClient();
+        $client        = static::createClient();
-        $id      = DB::findOneBy('attachment', ['filehash' => '5d8ee7ead51a28803b4ee5cb2306a0b90b6ba570f1e5bcc2209926f6ab08e7ea'])->getId();
+        $attachment_id = DB::findOneBy(Attachment::class, ['filehash' => '5d8ee7ead51a28803b4ee5cb2306a0b90b6ba570f1e5bcc2209926f6ab08e7ea'])->getId();
-        $crawler = $client->request('GET', "/attachment/{$id}{$suffix}");
+        $note_id       = DB::findOneBy(AttachmentToNote::class, ['attachment_id' => $attachment_id])->getNoteId();
+        $crawler       = $client->request('GET', "/object/note/{$note_id}/attachment/{$attachment_id}{$suffix}");
     }
 
     public function testAttachmentShow()
     {
         $this->testAttachment();
         $this->assertResponseIsSuccessful();
-        $this->assertSelectorTextContains('figure figcaption', '5d8ee7ead51a28803b4ee5cb2306a0b90b6ba570f1e5bcc2209926f6ab08e7ea');
+        $this->assertSelectorTextContains('figure figcaption', 'image.jpg');
     }
 
     public function testAttachmentView()
@@ -65,16 +69,6 @@ class AttachmentTest extends GNUsocialTestCase
         $this->assertResponseIsSuccessful();
     }
 
-    public function testAttachmentViewNotStored()
-    {
-        $client          = static::createClient();
-        $last_attachment = DB::findBy('attachment', [], order_by: ['id' => 'DESC'], limit: 1)[0];
-        $id              = $last_attachment->getId() + 1;
-        $crawler         = $client->request('GET', "/attachment/{$id}/view");
-        $this->assertResponseStatusCodeSame(500); // TODO (exception page) 404
-        $this->assertSelectorTextContains('.stacktrace', 'ClientException');
-    }
-
     public function testAttachmentDownload()
     {
         $this->testAttachment('/download');

components/Attachment/tests/Entity/AttachmentTest.php

@@ -19,15 +19,17 @@ declare(strict_types = 1);
 // along with GNU social.  If not, see <http://www.gnu.org/licenses/>.
 // }}}
 
-namespace App\Tests\Entity;
+namespace Component\Attachment\tests\Entity;
 
 use App\Core\DB\DB;
 use App\Core\Event;
 use App\Core\GSFile;
+use App\Core\Router\Router;
 use App\Entity\Note;
 use App\Util\GNUsocialTestCase;
 use App\Util\TemporaryFile;
 use Component\Attachment\Entity\AttachmentToNote;
+use Component\Conversation\Conversation;
 use Jchook\AssertThrows\AssertThrows;
 use SplFileInfo;
 use Symfony\Component\HttpFoundation\File\File;
@@ -107,6 +109,7 @@ class AttachmentTest extends GNUsocialTestCase
 
         $actor = DB::findOneBy('actor', ['nickname' => 'taken_user']);
         DB::persist($note = Note::create(['actor_id' => $actor->getId(), 'content' => 'attachment: some content', 'content_type' => 'text/plain', 'is_local' => true]));
+        Conversation::assignLocalConversation($note, null);
         DB::persist(AttachmentToNote::create(['attachment_id' => $attachment->getId(), 'note_id' => $note->getId(), 'title' => 'A title']));
         DB::flush();
 
@@ -118,7 +121,7 @@ class AttachmentTest extends GNUsocialTestCase
         static::bootKernel();
         $attachment = DB::findBy('attachment', ['mimetype' => 'image/png'], limit: 1)[0];
         $id         = $attachment->getId();
-        static::assertSame("/attachment/{$id}/view", $attachment->getUrl());
+        static::assertSame("/object/note/42/attachment/{$id}/view", $attachment->getUrl(note: 42, type: Router::ABSOLUTE_PATH));
     }
 
     public function testMimetype()

components/Attachment/tests/Entity/AttachmentThumbnailTest.php

@@ -19,12 +19,13 @@ declare(strict_types = 1);
 // along with GNU social.  If not, see <http://www.gnu.org/licenses/>.
 // }}}
 
-namespace App\Tests\Entity;
+namespace Component\Attachment\tests\Entity;
 
 use App\Core\DB\DB;
 use App\Core\Event;
 use App\Util\Exception\NotStoredLocallyException;
 use App\Util\GNUsocialTestCase;
+use Component\Attachment\Entity\Attachment;
 use Component\Attachment\Entity\AttachmentThumbnail;
 use Functional as F;
 use Jchook\AssertThrows\AssertThrows;
@@ -42,9 +43,9 @@ class AttachmentThumbnailTest extends GNUsocialTestCase
         $file = new SplFileInfo(INSTALLDIR . '/tests/sample-uploads/attachment-lifecycle-target.jpg');
         $hash = null;
         Event::handle('HashFile', [$file->getPathname(), &$hash]);
-        $attachment = DB::findOneBy('attachment', ['filehash' => $hash]);
+        $attachment = DB::findOneBy(Attachment::class, ['filehash' => $hash]);
 
-        $thumbs = [
+        $expected = [
             AttachmentThumbnail::getOrCreate($attachment, 'small', crop: false),
             AttachmentThumbnail::getOrCreate($attachment, 'medium', crop: false),
             $thumb = AttachmentThumbnail::getOrCreate($attachment, 'big', crop: false),
@@ -54,21 +55,29 @@ class AttachmentThumbnailTest extends GNUsocialTestCase
         $thumb->setAttachment(null);
         static::assertSame($attachment, $thumb->getAttachment());
 
-        $sort      = fn ($l, $r) => [$l->getWidth(), $l->getHeight()] <=> [$r->getWidth(), $r->getHeight()];
+        $actual = $attachment->getThumbnails();
-        $at_thumbs = F\sort($attachment->getThumbnails(), $sort);
+        static::assertSame(\count($expected), \count($actual));
-        static::assertSame($thumbs, $at_thumbs);
+        foreach ($expected as $e) {
-        array_pop($thumbs);
+            $a = array_shift($actual);
-        $thumb->delete(flush: true);
+            static::assertObjectEquals($e, $a);
-        $at_thumbs = F\sort($attachment->getThumbnails(), $sort);
+        }
-        static::assertSame($thumbs, $at_thumbs);
+
+        array_pop($expected);
+        $thumb->delete();
+        $actual = $attachment->getThumbnails();
+        static::assertSame(\count($expected), \count($actual));
+        foreach ($expected as $e) {
+            $a = array_shift($actual);
+            static::assertObjectEquals($e, $a);
+        }
 
         $attachment->deleteStorage();
 
-        foreach (array_reverse($thumbs) as $t) {
+        foreach (array_reverse($attachment->getThumbnails()) as $t) {
             // Since we still have thumbnails, those will be used as the new thumbnail, even though we don't have the original
             $new = AttachmentThumbnail::getOrCreate($attachment, 'big', crop: false);
             static::assertSame([$t->getFilename(), $t->getSize()], [$new->getFilename(), $new->getSize()]);
-            $t->delete(flush: true);
+            $t->delete();
         }
 
         // Since the backed storage was deleted and we don't have any more previous thumnbs, we can't generate another thumbnail
@@ -159,10 +168,10 @@ class AttachmentThumbnailTest extends GNUsocialTestCase
     public function testGetUrl()
     {
         parent::bootKernel();
-        $attachment = DB::findBy('attachment', ['mimetype' => 'image/png'], limit: 1)[0];
+        $attachment = DB::findBy(Attachment::class, ['mimetype' => 'image/png'], limit: 1)[0];
         $thumb      = AttachmentThumbnail::getOrCreate($attachment, 'big', crop: false);
         $id         = $attachment->getId();
-        $url        = "/attachment/{$id}/thumbnail/big";
+        $expected   = "/object/note/42/attachment/{$id}/thumbnail/big";
-        static::assertSame($url, $thumb->getUrl());
+        static::assertSame($expected, $thumb->getUrl(note: 42));
     }
 }

components/Blog/Blog.php

@@ -1,8 +1,8 @@
 <?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
@@ -17,21 +17,21 @@ declare(strict_types = 1);
 //
 // 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\Tests\Util\Notification;
+namespace Component\Blog;
 
-use App\Entity\Actor;
+use App\Core\Event;
-use App\Util\Notification\Notification;
+use App\Core\Modules\Plugin;
-use Jchook\AssertThrows\AssertThrows;
+use App\Core\Router\RouteLoader;
-use PHPUnit\Framework\TestCase;
+use Component\Blog\Controller as C;
 
-class NotificationTest extends TestCase
+class Blog extends Plugin
 {
-    use AssertThrows;
+    public function onAddRoute(RouteLoader $r): bool
-
-    public function testNotificationBitmap()
     {
-        static::assertTrue((new Notification(Notification::DM, new Actor())) instanceof Notification);
+        $r->connect(id: 'blog_post', uri_path: '/blog/post', target: [C\Post::class, 'makePost']);
+        return Event::next;
     }
 }

components/Blog/Controller/Post.php

@@ -0,0 +1,165 @@
+<?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 Component\Blog\Controller;
+
+use App\Core\ActorLocalRoles;
+use App\Core\Controller;
+use App\Core\Event;
+use App\Core\Form;
+use function App\Core\I18n\_m;
+use App\Core\VisibilityScope;
+use App\Entity\Actor;
+use App\Util\Common;
+use App\Util\Exception\ClientException;
+use App\Util\Exception\RedirectException;
+use App\Util\Form\FormFields;
+use Component\Posting\Posting;
+use InvalidArgumentException;
+use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
+use Symfony\Component\Form\Extension\Core\Type\FileType;
+use Symfony\Component\Form\Extension\Core\Type\SubmitType;
+use Symfony\Component\Form\Extension\Core\Type\TextareaType;
+use Symfony\Component\Form\Extension\Core\Type\TextType;
+use Symfony\Component\HttpFoundation\File\Exception\FormSizeFileException;
+use Symfony\Component\HttpFoundation\RedirectResponse;
+use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\Validator\Constraints\Length;
+
+class Post extends Controller
+{
+    /**
+     * Creates and handles Blog post creation form
+     *
+     * @throws \App\Util\Exception\DuplicateFoundException
+     * @throws \App\Util\Exception\NoLoggedInUser
+     * @throws \App\Util\Exception\ServerException
+     * @throws ClientException
+     * @throws RedirectException
+     */
+    public function makePost(Request $request)
+    {
+        $actor = Common::ensureLoggedIn()->getActor();
+
+        $placeholder_strings = ['How are you feeling?', 'Have something to share?', 'How was your day?'];
+        Event::handle('PostingPlaceHolderString', [&$placeholder_strings]);
+        $placeholder = $placeholder_strings[array_rand($placeholder_strings)];
+
+        $initial_content = '';
+        Event::handle('PostingInitialContent', [&$initial_content]);
+
+        $available_content_types = [
+            _m('Plain Text') => 'text/plain',
+        ];
+        Event::handle('PostingAvailableContentTypes', [&$available_content_types]);
+
+        if (!\is_int($this->int('in'))) {
+            throw new InvalidArgumentException('You must specify an In group/org.');
+        }
+        $context_actor = Actor::getById($this->int('in'));
+        if (!$context_actor->isGroup()) {
+            throw new InvalidArgumentException('Only group blog posts are supported for now.');
+        }
+        $in_targets    = ["!{$context_actor->getNickname()}" => $context_actor->getId()];
+        $form_params[] = ['in', ChoiceType::class, ['label' => _m('In:'), 'multiple' => false, 'expanded' => false, 'choices' => $in_targets]];
+
+        $visibility_options = [
+            _m('Public')    => VisibilityScope::EVERYWHERE->value,
+            _m('Local')     => VisibilityScope::LOCAL->value,
+            _m('Addressee') => VisibilityScope::ADDRESSEE->value,
+        ];
+        if (!\is_null($context_actor) && $context_actor->isGroup()) { // @phpstan-ignore-line currently an open bug. See https://web.archive.org/web/20220226131651/https://github.com/phpstan/phpstan/issues/6234
+            if ($actor->canModerate($context_actor)) {
+                if ($context_actor->getRoles() & ActorLocalRoles::PRIVATE_GROUP) {
+                    $visibility_options = array_merge([_m('Group') => VisibilityScope::GROUP->value], $visibility_options);
+                } else {
+                    $visibility_options[_m('Group')] = VisibilityScope::GROUP->value;
+                }
+            }
+        }
+        $form_params[] = ['visibility', ChoiceType::class, ['label' => _m('Visibility:'), 'multiple' => false, 'expanded' => false, 'choices' => $visibility_options]];
+
+        $form_params[] = ['title', TextType::class, ['label' => _m('Title:'), 'constraints' => [new Length(['max' => 129])], 'required' => true]];
+        $form_params[] = ['content', TextareaType::class, ['label' => _m('Content:'), 'data' => $initial_content, 'attr' => ['placeholder' => _m($placeholder)], 'constraints' => [new Length(['max' => Common::config('site', 'text_limit')])]]];
+        $form_params[] = ['attachments', FileType::class, ['label' => _m('Attachments:'), 'multiple' => true, 'required' => false, 'invalid_message' => _m('Attachment not valid.')]];
+        $form_params[] = FormFields::language($actor, $context_actor, label: _m('Note language'), help: _m('The selected language will be federated and added as a lang attribute, preferred language can be set up in settings'));
+
+        if (\count($available_content_types) > 1) {
+            $form_params[] = ['content_type', ChoiceType::class,
+                [
+                    'label'   => _m('Text format:'), 'multiple' => false, 'expanded' => false,
+                    'data'    => $available_content_types[array_key_first($available_content_types)],
+                    'choices' => $available_content_types,
+                ],
+            ];
+        }
+
+        Event::handle('PostingAddFormEntries', [$request, $actor, &$form_params]);
+
+        $form_params[] = ['post_note', SubmitType::class, ['label' => _m('Post')]];
+        $form          = Form::create($form_params);
+
+        $form->handleRequest($request);
+        if ($form->isSubmitted()) {
+            try {
+                if ($form->isValid()) {
+                    $data = $form->getData();
+                    Event::handle('PostingModifyData', [$request, $actor, &$data, $form_params, $form]);
+
+                    if (empty($data['content']) && empty($data['attachments'])) {
+                        // TODO Display error: At least one of `content` and `attachments` must be provided
+                        throw new ClientException(_m('You must enter content or provide at least one attachment to post a note.'));
+                    }
+
+                    if (\is_null(VisibilityScope::tryFrom($data['visibility']))) {
+                        throw new ClientException(_m('You have selected an impossible visibility.'));
+                    }
+
+                    $content_type = $data['content_type'] ?? $available_content_types[array_key_first($available_content_types)];
+                    $extra_args   = [];
+                    Event::handle('AddExtraArgsToNoteContent', [$request, $actor, $data, &$extra_args, $form_params, $form]);
+
+                    [,$note,] = Posting::storeLocalPage(
+                        actor: $actor,
+                        content: $data['content'],
+                        content_type: $content_type,
+                        locale: $data['language'],
+                        scope: VisibilityScope::from($data['visibility']),
+                        attentions: [(int) $data['in']],
+                        attachments: $data['attachments'],
+                        process_note_content_extra_args: $extra_args,
+                        title: $data['title'],
+                    );
+
+                    return new RedirectResponse($note->getConversationUrl());
+                }
+            } catch (FormSizeFileException $e) {
+                throw new ClientException(_m('Invalid file size given'), previous: $e);
+            }
+        }
+        return [
+            '_template'       => 'blog/make_post.html.twig',
+            'blog_entry_form' => $form->createView(),
+        ];
+    }
+}

components/Blog/templates/blog/make_post.html.twig

@@ -0,0 +1,10 @@
+{% extends 'stdgrid.html.twig' %}
+{% block title %}{% trans %}Create a blog post{% endtrans %}{% endblock %}
+
+{% block body %}
+    {{ parent() }}
+    <section class="frame-section frame-section-padding">
+        <h1>{% trans %}Create a blog post{% endtrans %}</h1>
+        {{ form(blog_entry_form) }}
+    </section>
+{% endblock body %}

components/Circle/Circle.php

@@ -57,8 +57,8 @@ class Circle extends Component
 {
     use MetaCollectionTrait;
     public const TAG_CIRCLE_REGEX = '/' . Nickname::BEFORE_MENTIONS . '@#([\pL\pN_\-\.]{1,64})/';
-    protected string $slug        = 'circle';
+    protected const SLUG          = 'circle';
-    protected string $plural_slug = 'circles';
+    protected const PLURAL_SLUG   = 'circles';
 
     public function onAddRoute(RouteLoader $r): bool
     {
@@ -97,7 +97,7 @@ class Circle extends Component
 
     public function onPopulateSettingsTabs(Request $request, string $section, array &$tabs): bool
     {
-        if ($section === 'profile' && $request->get('_route') === 'settings') {
+        if ($section === 'profile' && \in_array($request->get('_route'), ['person_actor_settings', 'group_actor_settings'])) {
             $tabs[] = [
                 'title'      => 'Self tags',
                 'desc'       => 'Add or remove tags on yourself',
@@ -195,6 +195,17 @@ class Circle extends Component
         return \in_array($vars['path'], ['actor_view_nickname', 'actor_view_id']);
     }
 
+    /**
+     * Retrieves an array of Collections owned by an Actor.
+     * In this case, Collections of those within Actor's own circle of Actors, aka ActorCircle.
+     *
+     * Differs from the overwritten method in MetaCollectionsTrait, since retrieved Collections come from the $owner
+     * itself, and from every Actor that is a part of its ActorCircle.
+     *
+     * @param Actor      $owner    the Actor, and by extension its own circle of Actors
+     * @param null|array $vars     Page vars sent by AppendRightPanelBlock event
+     * @param bool       $ids_only true if only the Collections ids are to be returned
+     */
     protected function getCollectionsBy(Actor $owner, ?array $vars = null, bool $ids_only = false): array
     {
         $tagged_id = !\is_null($vars) ? $this->getActorIdFromVars($vars) : null;

components/Circle/Controller/Circle.php

@@ -31,6 +31,14 @@ use Component\Collection\Util\Controller\CircleController;
 
 class Circle extends CircleController
 {
+    /**
+     * Render an existing ActorCircle with the given id as a Collection of Actors
+     *
+     * @param ActorCircle|int $circle_id the desired ActorCircle id
+     *
+     * @throws \App\Util\Exception\ServerException
+     * @throws ClientException
+     */
     public function circleById(int|ActorCircle $circle_id): array
     {
         $circle = \is_int($circle_id) ? ActorCircle::getByPK(['id' => $circle_id]) : $circle_id;

components/Circle/Controller/Circles.php

@@ -33,9 +33,9 @@ use Component\Collection\Util\Controller\MetaCollectionController;
 
 class Circles extends MetaCollectionController
 {
-    protected string $slug        = 'circle';
+    protected const SLUG         = 'circle';
-    protected string $plural_slug = 'circles';
+    protected const PLURAL_SLUG  = 'circles';
-    protected string $page_title  = 'Actor circles';
+    protected string $page_title = 'Actor circles';
 
     public function createCollection(int $owner_id, string $name)
     {

components/Circle/Controller/SelfTagsSettings.php

@@ -23,11 +23,12 @@ class SelfTagsSettings extends Controller
 {
     /**
      * Generic settings page for an Actor's self tags
+     * TODO: We should have $actor->setSelfTags(), $actor->addSelfTags(), $actor->removeSelfTags()
      */
     public static function settingsSelfTags(Request $request, E\Actor $target, string $details_id)
     {
         $actor = Common::actor();
-        if (!$actor->canAdmin($target)) {
+        if (!$actor->canModerate($target)) {
             throw new ClientException(_m('You don\'t have enough permissions to edit {nickname}\'s settings', ['{nickname}' => $target->getNickname()]));
         }
 

components/Circle/Entity/ActorCircle.php

@@ -49,7 +49,7 @@ class ActorCircle extends Entity
     // {{{ Autocode
     // @codeCoverageIgnoreStart
     private int $id;
-    private ?int $tagger = null; // If null, is the special global self-tag circle
+    private ?int $tagger = null;
     private string $tag;
     private ?string $description = null;
     private ?bool $private       = false;
@@ -202,7 +202,7 @@ class ActorCircle extends Entity
             'description' => 'An actor can have lists of actors, to separate their feed or quickly mention his friend',
             'fields'      => [
                 'id'          => ['type' => 'serial',    'not null' => true, 'description' => 'unique identifier'], // An actor can be tagged by many actors
-                'tagger'      => ['type' => 'int',       'default' => null, 'foreign key' => true, 'target' => 'Actor.id', 'multiplicity' => 'many to one', 'name' => 'actor_list_tagger_fkey', 'description' => 'user making the tag, null if self-tag'],
+                'tagger'      => ['type' => 'int',       'default' => null, 'foreign key' => true, 'target' => 'Actor.id', 'multiplicity' => 'many to one', 'name' => 'actor_list_tagger_fkey', 'description' => 'user making the tag, null if self-tag. If null, is the special global self-tag circle'],
                 'tag'         => ['type' => 'varchar',   'length' => 64, 'foreign key' => true, 'target' => 'ActorTag.tag', 'multiplicity' => 'many to one', 'not null' => true, 'description' => 'actor tag'], // Join with ActorTag // // so, Doctrine doesn't like that the target is not unique, even though the pair is  // Many Actor Circles can reference (and probably will) an Actor Tag
                 'description' => ['type' => 'text',      'description' => 'description of the people tag'],
                 'private'     => ['type' => 'bool',      'default' => false, 'description' => 'is this tag private'],

components/Collection/Collection.php

@@ -23,20 +23,31 @@ class Collection extends Component
      * Supports a variety of query terms and is used both in feeds and
      * in search. Uses query builders to allow for extension
      */
-    public static function query(string $query, int $page, ?string $locale = null, ?Actor $actor = null): array
+    public static function query(string $query, int $page, ?string $locale = null, ?Actor $actor = null, array $note_order_by = [], array $actor_order_by = []): array
     {
         $note_criteria  = null;
         $actor_criteria = null;
         if (!empty($query = trim($query))) {
             [$note_criteria, $actor_criteria] = Parser::parse($query, $locale, $actor);
         }
+
         $note_qb  = DB::createQueryBuilder();
         $actor_qb = DB::createQueryBuilder();
         // TODO consider selecting note related stuff, to avoid separate queries (though they're cached, so maybe it's okay)
-        $note_qb->select('note')->from('App\Entity\Note', 'note')->orderBy('note.created', 'DESC')->addOrderBy('note.id', 'DESC');
+        $note_qb->select('note')->from('App\Entity\Note', 'note');
-        $actor_qb->select('actor')->from('App\Entity\Actor', 'actor')->orderBy('actor.created', 'DESC')->addOrderBy('actor.id', 'DESC');
+        $actor_qb->select('actor')->from('App\Entity\Actor', 'actor');
         Event::handle('CollectionQueryAddJoins', [&$note_qb, &$actor_qb, $note_criteria, $actor_criteria]);
 
+        // Handle ordering
+        $note_order_by  = !empty($note_order_by) ? $note_order_by : ['note.created' => 'DESC', 'note.id' => 'DESC'];
+        $actor_order_by = !empty($actor_order_by) ? $actor_order_by : ['actor.created' => 'DESC', 'actor.id' => 'DESC'];
+        foreach ($note_order_by as $field => $order) {
+            $note_qb->addOrderBy($field, $order);
+        }
+        foreach ($actor_order_by as $field => $order) {
+            $actor_qb->addOrderBy($field, $order);
+        }
+
         $notes  = [];
         $actors = [];
         if (!\is_null($note_criteria)) {
@@ -55,8 +66,13 @@ class Collection extends Component
 
     public function onCollectionQueryAddJoins(QueryBuilder &$note_qb, QueryBuilder &$actor_qb): bool
     {
-        $note_qb->leftJoin(ActorSubscription::class, 'subscription', Expr\Join::WITH, 'note.actor_id = subscription.subscribed_id')
+        $note_aliases = $note_qb->getAllAliases();
-            ->leftJoin(Actor::class, 'note_actor', Expr\Join::WITH, 'note.actor_id = note_actor.id');
+        if (!\in_array('subscription', $note_aliases)) {
+            $note_qb->leftJoin(ActorSubscription::class, 'subscription', Expr\Join::WITH, 'note.actor_id = subscription.subscribed_id');
+        }
+        if (!\in_array('note_actor', $note_aliases)) {
+            $note_qb->leftJoin(Actor::class, 'note_actor', Expr\Join::WITH, 'note.actor_id = note_actor.id');
+        }
         return Event::next;
     }
 
@@ -104,7 +120,7 @@ class Collection extends Component
                             if (\in_array($type, ['actor', 'actors'])) {
                                 $type_consts = null;
                             } else {
-                                $type_consts[] = \constant(Actor::class . '::' . mb_strtoupper($type));
+                                $type_consts[] = \constant(Actor::class . '::' . mb_strtoupper($type === 'organisation' ? 'group' : $type));
                             }
                         }
                     }
@@ -127,8 +143,7 @@ class Collection extends Component
                         foreach (
                             [
                                 Actor::PERSON => ['person', 'people'],
-                                Actor::GROUP => ['group', 'groups'],
+                                Actor::GROUP => ['group', 'groups', 'org', 'orgs', 'organisation', 'organisations', 'organization', 'organizations'],
-                                Actor::ORGANISATION => ['org', 'orgs', 'organization', 'organizations', 'organisation', 'organisations'],
                                 Actor::BOT => ['bot', 'bots'],
                             ] as $type => $match) {
                             if (array_intersect(explode(',', $term[1]), $match) !== []) {

components/Collection/Util/ActorControllerTrait.php

@@ -1,71 +0,0 @@
-<?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/>.
-// }}}
-
-/**
- * Base class for feed controllers
- *
- * @package  GNUsocial
- * @category Controller
- *
- * @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
- */
-
-namespace Component\Collection\Util;
-
-use App\Core\DB\DB;
-use function App\Core\I18n\_m;
-use App\Core\Router\Router;
-use App\Util\Exception\ClientException;
-
-trait ActorControllerTrait
-{
-    /**
-     * Generic function that handles getting a representation for an actor from id
-     */
-    protected function handleActorById(int $id, callable $handle)
-    {
-        $actor = DB::findOneBy('actor', ['id' => $id]);
-        if ($actor->getIsLocal()) {
-            return ['_redirect' => $actor->getUrl(Router::ABSOLUTE_PATH), 'actor' => $actor];
-        }
-        if (empty($actor)) {
-            throw new ClientException(_m('No such actor.'), 404);
-        } else {
-            return $handle($actor);
-        }
-    }
-
-    /**
-     * Generic function that handles getting a representation for an actor from nickname
-     */
-    protected function handleActorByNickname(string $nickname, callable $handle)
-    {
-        $user  = DB::findOneBy('local_user', ['nickname' => $nickname]);
-        $actor = DB::findOneBy('actor', ['id' => $user->getId()]);
-        if (empty($actor)) {
-            throw new ClientException(_m('No such actor.'), 404);
-        } else {
-            return $handle($actor);
-        }
-    }
-}

components/Collection/Util/Controller/Collection.php

@@ -11,10 +11,10 @@ use Component\Collection\Collection as CollectionModule;
 
 class Collection extends Controller
 {
-    public function query(string $query, ?string $locale = null, ?Actor $actor = null): array
+    public function query(string $query, ?string $locale = null, ?Actor $actor = null, array $note_order_by = [], array $actor_order_by = []): array
     {
         $actor  ??= Common::actor();
         $locale ??= Common::currentLanguage()->getLocale();
-        return CollectionModule::query($query, $this->int('page') ?? 1, $locale, $actor);
+        return CollectionModule::query($query, $this->int('page') ?? 1, $locale, $actor, $note_order_by, $actor_order_by);
     }
 }

components/Collection/Util/Controller/FeedController.php

@@ -50,7 +50,7 @@ abstract class FeedController extends OrderedCollection
         $actor = Common::actor();
         if (\array_key_exists('notes', $result)) {
             $notes = $result['notes'];
-            self::enforceScope($notes, $actor);
+            self::enforceScope($notes, $actor, $result['actor'] ?? null);
             Event::handle('FilterNoteList', [$actor, &$notes, $result['request']]);
             Event::handle('FormatNoteList', [$notes, &$result['notes'], &$result['request']]);
         }
@@ -58,8 +58,8 @@ abstract class FeedController extends OrderedCollection
         return $result;
     }
 
-    private static function enforceScope(array &$notes, ?Actor $actor): void
+    private static function enforceScope(array &$notes, ?Actor $actor, ?Actor $in = null): void
     {
-        $notes = F\select($notes, fn (Note $n) => $n->isVisibleTo($actor));
+        $notes = F\select($notes, fn (Note $n) => $n->isVisibleTo($actor, $in));
     }
 }

components/Collection/Util/Controller/MetaCollectionController.php

@@ -43,9 +43,9 @@ use Symfony\Component\HttpFoundation\Request;
 
 abstract class MetaCollectionController extends FeedController
 {
-    protected string $slug        = 'collectionsEntry';
+    protected const SLUG         = 'collectionsEntry';
-    protected string $plural_slug = 'collectionsList';
+    protected const PLURAL_SLUG  = 'collectionsList';
-    protected string $page_title  = 'Collections';
+    protected string $page_title = 'Collections';
 
     abstract public function getCollectionUrl(int $owner_id, string $owner_nickname, int $collection_id): string;
     abstract public function getCollectionItems(int $owner_id, $collection_id): array;
@@ -76,8 +76,8 @@ abstract class MetaCollectionController extends FeedController
     {
         $collections = $this->getCollectionsByActorId($id);
 
-        $create_title      = _m('Create a ' . mb_strtolower(preg_replace('/([a-z0-9])([A-Z])/', '$1 $2', $this->slug)));
+        $create_title      = _m('Create a ' . mb_strtolower(preg_replace('/([a-z0-9])([A-Z])/', '$1 $2', static::SLUG)));
-        $collections_title = _m('The ' . mb_strtolower(preg_replace('/([a-z0-9])([A-Z])/', '$1 $2', $this->plural_slug)));
+        $collections_title = _m('The ' . mb_strtolower(preg_replace('/([a-z0-9])([A-Z])/', '$1 $2', static::PLURAL_SLUG)));
         // create collection form
         $create = null;
         if (Common::user()?->getId() === $id) {
@@ -111,8 +111,8 @@ abstract class MetaCollectionController extends FeedController
         //
         // Instead, I'm using an anonymous class to encapsulate
         // the functions and passing that class to the template.
-        // This is suggested at https://stackoverflow.com/a/50364502.
+        // This is suggested at https://web.archive.org/web/20220226132328/https://stackoverflow.com/questions/3595727/twig-pass-function-into-template/50364502
-        $fn = new class($id, $nickname, $request, $this, $this->slug) {
+        $fn = new class($id, $nickname, $request, $this, static::SLUG) {
             private $id;
             private $nick;
             private $request;

components/Collection/Util/MetaCollectionTrait.php

@@ -46,8 +46,8 @@ use Symfony\Component\HttpFoundation\Request;
 
 trait MetaCollectionTrait
 {
-    //protected string $slug        = 'collection';
+    //protected const SLUG        = 'collection';
-    //protected string $plural_slug = 'collections';
+    //protected const PLURAL_SLUG = 'collections';
 
     /**
      * create a collection owned by Actor $owner.
@@ -127,9 +127,9 @@ trait MetaCollectionTrait
                 },
             ]],
             ['add', SubmitType::class, [
-                'label' => _m('Add to ' . $this->plural_slug),
+                'label' => _m('Add to ' . static::PLURAL_SLUG),
                 'attr'  => [
-                    'title' => _m('Add to ' . $this->plural_slug),
+                    'title' => _m('Add to ' . static::PLURAL_SLUG),
                 ],
             ]],
         ]);
@@ -151,17 +151,17 @@ trait MetaCollectionTrait
         // form: add to new collection
         $create_form = Form::create([
             ['name', TextType::class, [
-                'label' => _m('Add to a new ' . $this->slug),
+                'label' => _m('Add to a new ' . static::SLUG),
                 'attr'  => [
-                    'placeholder' => _m('New ' . $this->slug . ' name'),
+                    'placeholder' => _m('New ' . static::SLUG . ' name'),
                     'required'    => 'required',
                 ],
                 'data' => '',
             ]],
             ['create', SubmitType::class, [
-                'label' => _m('Create a new ' . $this->slug),
+                'label' => _m('Create a new ' . static::SLUG),
                 'attr'  => [
-                    'title' => _m('Create a new ' . $this->slug),
+                    'title' => _m('Create a new ' . static::SLUG),
                 ],
             ]],
         ]);
@@ -176,7 +176,7 @@ trait MetaCollectionTrait
         $res[] = Formatting::twigRenderFile(
             'collection/widget_add_to.html.twig',
             [
-                'ctitle'          => _m('Add to ' . $this->plural_slug),
+                'ctitle'          => _m('Add to ' . static::PLURAL_SLUG),
                 'user'            => $user,
                 'has_collections' => \count($collections) > 0,
                 'add_form'        => $add_form->createView(),

components/Collection/Util/Parser.php

@@ -50,8 +50,9 @@ abstract class Parser
      * recognises either spaces (currently `or`, should be fuzzy match), `OR` or `|` (`or`) and `AND` or `&` (`and`)
      *
      * TODO: Better fuzzy match, implement exact match with quotes and nesting with parens
+     * TODO: Proper parser, tokenize better. Mostly a rewrite
      *
-     * @return Criteria[]
+     * @return array{?Criteria, ?Criteria} [?$note_criteria, ?$actor_criteria]
      */
     public static function parse(string $input, ?string $locale = null, ?Actor $actor = null, int $level = 0): array
     {
@@ -80,15 +81,16 @@ abstract class Parser
                     $actor_res = null;
                     Event::handle('CollectionQueryCreateExpression', [$eb, $term, $locale, $actor, &$note_res, &$actor_res]);
                     if (\is_null($note_res) && \is_null($actor_res)) { // @phpstan-ignore-line
-                        throw new ServerException("No one claimed responsibility for a match term: {$term}");
+                        //throw new ServerException("No one claimed responsibility for a match term: {$term}");
+                        // It's okay if the term doesn't exist, just perform a regular search
                     }
-                    if (!empty($note_res)) { // @phpstan-ignore-line
+                    if (!empty($note_res)) { // @phpstan-ignore-line currently an open bug. See https://web.archive.org/web/20220226131651/https://github.com/phpstan/phpstan/issues/6234
                         if (\is_array($note_res)) {
                             $note_res = $eb->orX(...$note_res);
                         }
                         $note_parts[] = $note_res;
                     }
-                    if (!empty($actor_res)) {
+                    if (!empty($actor_res)) { // @phpstan-ignore-line currently an open bug. See https://web.archive.org/web/20220226131651/https://github.com/phpstan/phpstan/issues/6234
                         if (\is_array($actor_res)) {
                             $actor_res = $eb->orX(...$actor_res);
                         }
@@ -107,18 +109,18 @@ abstract class Parser
                 }
             }
             // TODO
-            if (!$match) { // @phpstan-ignore-line
+            if (!$match) {
                 ++$right;
             }
         }
 
         $note_criteria  = null;
         $actor_criteria = null;
-        if (!empty($note_parts)) { // @phpstan-ignore-line
+        if (!empty($note_parts)) {
             self::connectParts($note_parts, $note_criteria_arr, $last_op, $eb, force: true);
             $note_criteria = new Criteria($eb->orX(...$note_criteria_arr));
         }
-        if (!empty($actor_parts)) { // @phpstan-ignore-line
+        if (!empty($actor_parts)) { // @phpstan-ignore-line weird, but this whole thing needs a rewrite
             self::connectParts($actor_parts, $actor_criteria_arr, $last_op, $eb, force: true);
             $actor_criteria = new Criteria($eb->orX(...$actor_criteria_arr));
         }

components/Collection/templates/collection/actors.html.twig

@@ -3,8 +3,17 @@
 {% block title %}{{ title }}{% endblock %}
 
 {% block body %}
-<div class="frame-section frame-section-padding">
+<section class="frame-section frame-section-padding">
-    <h1 class="frame-section-title">{{ title }}</h1>
+    <header class="feed-header">
+        {% if actors_feed_title is defined %}
+            {{ actors_feed_title.getHtml() }}
+        {% endif %}
+    </header>
+
+    {% set prepend_actors_collection = handle_event('PrependActorsCollection', request) %}
+    {% for widget in prepend_actors_collection %}
+        {{ widget | raw }}
+    {% endfor %}
 
     <details class="frame-section section-details-title">
         <summary class="details-summary-title">
@@ -45,16 +54,17 @@
         </form>
     </details>
 
-    <section class="frame-section-padding">
+    <section class="frame-section frame-section-padding">
+        <h2>{% trans %}Results{% endtrans %}</h2>
         {% if actors is defined and actors is not empty %}
             {% for actor in actors %}
-                {% block profile_view %}{% include 'cards/profile/view.html.twig' %}{% endblock profile_view %}
+                {% block profile_view %}{% include 'cards/blocks/profile.html.twig' %}{% endblock profile_view %}
                 <hr>
             {% endfor %}
-            <p>{% trans %}Page: %page%{% endtrans %}</p>
+            <span class="frame-section-button-like">{% trans %}Page: %page%{% endtrans %}</span>
         {% else %}
-            <h2>{{ empty_message }}</h2>
+            <span>{{ empty_message }}</span>
         {% endif %}
     </section>
-</div>
+</section>
 {% endblock body %}

components/Collection/templates/collection/collection_entry_view.html.twig

@@ -1,10 +1,10 @@
 {% extends '/collection/notes.html.twig' %}
 
-{% block title %}{{ page_title | trans }}{% endblock %}
+{% block title %}{% trans %}%page_title%{% endtrans %}{% endblock %}
 
 {% block body %}
     <div class="frame-section frame-section-padding">
-        <h2 class="frame-section-title">{{ page_title | trans }}</h2>
+        <h2 class="frame-section-title">{% trans %}%page_title%{% endtrans %}</h2>
         {% block collection_items %}
         {% endblock collection_items %}
     </div>

components/Collection/templates/collection/meta_collections.html.twig

@@ -1,17 +1,17 @@
 {% extends 'stdgrid.html.twig' %}
 
-{% block title %}{{ page_title | trans }}{% endblock %}
+{% block title %}{% trans %}%page_title%{% endtrans %}{% endblock %}
 
 {% block body %}
     <div class="frame-section frame-section-padding">
-        <h2 class="frame-section-title">{{ page_title | trans }}</h2>
+        <h2 class="frame-section-title">{% trans %}%page_title%{% endtrans %}</h2>
         {% if add_collection %}
             <div class="frame-section section-form">
                 {{ form(add_collection) }}
             </div>
         {% endif %}
         <div class="frame-section collections-list">
-            <h3>{{ list_title | trans }}</h3>
+            <h3>{% trans %}%list_title%{% endtrans %}</h3>
             {% for col in collections %}
                 <div class="collection-item">
                     <a class="name" href="{{ fn.getUrl(col.id) }}">{{ col.name }}</a>

components/Collection/templates/collection/notes.html.twig

@@ -1,11 +1,11 @@
 {% extends 'stdgrid.html.twig' %}
-{% import '/cards/note/view.html.twig' as noteView %}
+{% import '/cards/macros/note/factory.html.twig' as NoteFactory %}
 
-{% block title %}{% if page_title is defined %}{{ page_title | trans }}{% endif %}{% endblock %}
+{% block title %}{% if page_title is defined %}{% trans %}%page_title%{% endtrans %}{% endif %}{% endblock %}
 
 {% block stylesheets %}
     {{ parent() }}
-    <link rel="stylesheet" href="{{ asset('assets/default_theme/css/pages/feeds.css') }}" type="text/css">
+    <link rel="stylesheet" href="{{ asset('assets/default_theme/feeds.css') }}" type="text/css">
 {% endblock stylesheets %}
 
 {% block body %}
@@ -15,41 +15,44 @@
 
     {% if notes is defined %}
         <header class="feed-header">
-            {% if page_title is defined %}
+            {% set current_path = app.request.get('_route') %}
-                <h1 class="heading-no-margin">{{ page_title | trans }}</h1>
+            {% if notes_feed_title is defined %}
-            {% else %}
+                {{ notes_feed_title.getHtml() }}
-                <h3 class="heading-no-margin">{{ 'Notes' | trans }}</h3>
             {% endif %}
-            <nav class="feed-actions">
+            <nav class="feed-actions" title="{% trans %}Actions that change how the feed behaves{% endtrans %}">
-                <details class="feed-actions-details">
+                <details class="feed-actions-details" role="group">
                     <summary>
                         {{ icon('filter', 'icon icon-feed-actions') | raw }} {# button-container #}
                     </summary>
-                    <div class="feed-actions-details-dropdown">
+                    <menu class="feed-actions-details-dropdown" role="toolbar">
-                        <menu>
+                        {% for block in handle_event('AddFeedActions', app.request, notes is defined and notes is not empty) %}
-                            {% for block in handle_event('AddFeedActions', app.request, notes is defined and notes is not empty) %}
+                            {{ block | raw }}
-                                {{ block | raw }}
+                        {% endfor %}
-                            {% endfor %}
+                    </menu>
-                        </menu>
-                    </div>
                 </details>
             </nav>
         </header>
 
         {% if notes is not empty %}
             {# Backwards compatibility with hAtom 0.1 #}
-            <section class="feed h-feed hfeed notes" tabindex="0" role="feed">
+            <section class="feed h-feed hfeed notes" role="feed" aria-busy="false" title="{% trans %}Feed content{% endtrans %}">
                 {% for conversation in notes %}
                     {% block current_note %}
                         {% if conversation is instanceof('array') %}
-                            {{ noteView.macro_note(conversation['note'], conversation['replies']) }}
+                            {% set args = conversation | merge({'type': 'vanilla_full'}) %}
-                        {% else %}
+                            {{ NoteFactory.constructor(args) }}
-                            {{ noteView.macro_note(conversation) }}
+{#                        {% else %}
+                            {% set args = { 'type': 'vanilla_full', 'note': conversation, 'extra': { 'depth': 0 } } %}
+                            {{ NoteFactory.constructor(args) }}#}
                         {% endif %}
-                        <hr tabindex="0" title="{{ 'End of note and replies.' | trans }}">
+                        <hr class="hr-replies-end" role="separator" aria-label="{% trans %}Marks the end of previous conversation's initial note{% endtrans %}">
                     {% endblock current_note %}
                 {% endfor %}
             </section>
+        {% else %}
+            <section class="feed h-feed hfeed notes" tabindex="0" role="feed">
+                <span>{% trans %}No notes here...{% endtrans %}</span>
+            </section>
         {% endif %}
     {% endif %}
 {% endblock body %}

components/Collection/templates/collection/widget_add_to.html.twig

@@ -1,7 +1,7 @@
 <section class="frame-section collections">
     <details class="section-details-title" title="Expand if you want to access more options.">
         <summary class="details-summary-title">
-            <h2>{{ctitle}}</h2>
+            <span>{{ctitle}}</span>
         </summary>
         {% if has_collections %}
             <section class="section-form">

components/Conversation/Controller/Conversation.php

@@ -40,6 +40,7 @@ use App\Util\Exception\NoLoggedInUser;
 use App\Util\Exception\NoSuchNoteException;
 use App\Util\Exception\RedirectException;
 use App\Util\Exception\ServerException;
+use App\Util\HTML\Heading;
 use Component\Collection\Util\Controller\FeedController;
 use Component\Conversation\Entity\ConversationMute;
 use Symfony\Component\Form\Extension\Core\Type\SubmitType;
@@ -58,11 +59,17 @@ class Conversation extends FeedController
      */
     public function showConversation(Request $request, int $conversation_id): array
     {
+        $page_title = _m('Conversation');
+
         return [
-            '_template'     => 'collection/notes.html.twig',
+            '_template' => 'collection/notes.html.twig',
-            'notes'         => $this->query(query: "note-conversation:{$conversation_id}")['notes'] ?? [],
+            'notes'     => $this->query(
-            'should_format' => false,
+                query: "note-conversation:{$conversation_id}",
-            'page_title'    => _m('Conversation'),
+                note_order_by: ['note.created' => 'ASC', 'note.id' => 'ASC'],
+            )['notes'] ?? [],
+            'should_format'    => false,
+            'page_title'       => $page_title,
+            'notes_feed_title' => (new Heading(1, [], $page_title)),
         ];
     }
 
@@ -134,9 +141,12 @@ class Conversation extends FeedController
 
         return [
             '_template' => 'conversation/mute.html.twig',
-            'notes'     => $this->query(query: "note-conversation:{$conversation_id}")['notes'] ?? [],
+            'notes'     => $this->query(
-            'is_muted'  => $is_muted,
+                query: "note-conversation:{$conversation_id}",
-            'form'      => $form->createView(),
+                note_order_by: ['note.created' => 'ASC', 'note.id' => 'ASC'],
+            )['notes'] ?? [],
+            'is_muted' => $is_muted,
+            'form'     => $form->createView(),
         ];
     }
 }

components/Conversation/Conversation.php

@@ -1,9 +1,7 @@
 <?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
@@ -18,9 +16,15 @@ declare(strict_types = 1);
 //
 // 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/>.
-
 // }}}
 
+/**
+ * @author    Hugo Sales <hugo@hsal.es>
+ * @author    Eliseu Amaro <mail@eliseuama.ro>
+ * @copyright 2021-2022 Free Software Foundation, Inc http://www.fsf.org
+ * @license   https://www.gnu.org/licenses/agpl.html GNU AGPL v3 or later
+ */
+
 namespace Component\Conversation;
 
 use App\Core\Cache;
@@ -34,6 +38,7 @@ use App\Entity\Activity;
 use App\Entity\Actor;
 use App\Entity\Note;
 use App\Util\Common;
+use App\Util\Formatting;
 use Component\Conversation\Entity\Conversation as ConversationEntity;
 use Component\Conversation\Entity\ConversationMute;
 use Functional as F;
@@ -78,14 +83,12 @@ class Conversation extends Component
         } else {
             // It's a reply for sure
             // Set reply_to property in newly created Note to parent's id
-            $current_note->setReplyTo($parent_id);
-
             // Parent will have a conversation of its own, the reply should have the same one
             $parent_note = Note::getById($parent_id);
             $current_note->setConversationId($parent_note->getConversationId());
         }
 
-        DB::merge($current_note);
+        DB::persist($current_note);
     }
 
     /**
@@ -99,6 +102,8 @@ class Conversation extends Component
      *                                  'id' (HTML markup id used to redirect user to this anchor upon performing the action)
      *
      * @throws \App\Util\Exception\ServerException
+     *
+     * @return bool EventHook
      */
     public function onAddNoteActions(Request $request, Note $note, array &$actions): bool
     {
@@ -114,7 +119,8 @@ class Conversation extends Component
             'conversation_reply_to',
             [
                 'reply_to_id' => $note->getId(),
-                'from'        => $from . '#note-anchor-' . $note->getId(),
+                'from'        => $from,
+                '_fragment'   => 'note-anchor-' . $note->getId(),
             ],
             Router::ABSOLUTE_PATH,
         );
@@ -131,29 +137,20 @@ class Conversation extends Component
         return Event::next;
     }
 
-    /**
-     * Posting event to add extra info to a note
-     */
-    public function onPostingModifyData(Request $request, Actor $actor, array &$data): bool
-    {
-        $data['reply_to_id'] = $request->get('_route') === 'conversation_reply_to' && $request->query->has('reply_to_id')
-                             ? $request->query->getInt('reply_to_id')
-                             : null;
-
-        if (!\is_null($data['reply_to_id'])) {
-            Note::ensureCanInteract(Note::getById($data['reply_to_id']), $actor);
-        }
-        return Event::next;
-    }
-
     /**
      * Append on note information about user actions.
      *
      * @param array $vars   Contains information related to Note currently being rendered
      * @param array $result Contains keys 'actors', and 'action'. Needed to construct a string, stating who ($result['actors']), has already performed a reply ($result['action']), in the given Note (vars['note'])
+     *
+     * @return bool EventHook
      */
     public function onAppendCardNote(array $vars, array &$result): bool
     {
+        if (str_contains($vars['request']->getPathInfo(), 'conversation')) {
+            return Event::next;
+        }
+
         // The current Note being rendered
         $note = $vars['note'];
 
@@ -175,6 +172,22 @@ class Conversation extends Component
         return Event::next;
     }
 
+    private function getReplyToIdFromRequest(Request $request): ?int
+    {
+        if (!\is_array($request->get('post_note')) || !\array_key_exists('_next', $request->get('post_note'))) {
+            return null;
+        }
+        $next = parse_url($request->get('post_note')['_next']);
+        if (!\array_key_exists('query', $next)) {
+            return null;
+        }
+        parse_str($next['query'], $query);
+        if (!\array_key_exists('reply_to_id', $query)) {
+            return null;
+        }
+        return (int) $query['reply_to_id'];
+    }
+
     /**
      * Informs **\App\Component\Posting::onAppendRightPostingBlock**, of the **current page context** in which the given
      * Actor is in. This is valuable when posting within a group route, allowing \App\Component\Posting to create a
@@ -182,10 +195,12 @@ class Conversation extends Component
      *
      * @param \App\Entity\Actor      $actor         The Actor currently attempting to post a Note
      * @param null|\App\Entity\Actor $context_actor The 'owner' of the current route (e.g. Group or Actor), used to target it
+     *
+     * @return bool EventHook
      */
-    public function onPostingGetContextActor(Request $request, Actor $actor, ?Actor &$context_actor)
+    public function onPostingGetContextActor(Request $request, Actor $actor, ?Actor &$context_actor): bool
     {
-        $to_note_id = $request->query->get('reply_to_id');
+        $to_note_id = $this->getReplyToIdFromRequest($request);
         if (!\is_null($to_note_id)) {
             // Getting the actor itself
             $context_actor = Actor::getById(Note::getById((int) $to_note_id)->getActorId());
@@ -194,12 +209,44 @@ class Conversation extends Component
         return Event::next;
     }
 
+    /**
+     * Posting event to add extra information to Component\Posting form data
+     *
+     * @param array $data Transport data to be filled with reply_to_id
+     *
+     * @throws \App\Util\Exception\ClientException
+     * @throws \App\Util\Exception\NoSuchNoteException
+     *
+     * @return bool EventHook
+     */
+    public function onPostingModifyData(Request $request, Actor $actor, array &$data): bool
+    {
+        $to_note_id = $this->getReplyToIdFromRequest($request);
+        if (!\is_null($to_note_id)) {
+            Note::ensureCanInteract(Note::getById($to_note_id), $actor);
+            $data['reply_to_id'] = $to_note_id;
+        }
+
+        return Event::next;
+    }
+
+    /**
+     * Add minimal Note card to RightPanel template
+     */
+    public function onPrependPostingForm(Request $request, array &$elements): bool
+    {
+        $elements[] = Formatting::twigRenderFile('cards/blocks/note_compact_wrapper.html.twig', ['note' => Note::getById((int) $request->query->get('reply_to_id'))]);
+        return Event::next;
+    }
+
     /**
      * Event launched when deleting given Note, it's deletion implies further changes to object related to this Note.
      * Please note, **replies are NOT deleted**, their reply_to is only set to null since this Note no longer exists.
      *
      * @param \App\Entity\Note  $note  Note being deleted
      * @param \App\Entity\Actor $actor Actor that performed the delete action
+     *
+     * @return bool EventHook
      */
     public function onNoteDeleteRelated(Note &$note, Actor $actor): bool
     {
@@ -220,7 +267,7 @@ class Conversation extends Component
      *
      * @return bool EventHook
      */
-    public function onAddExtraNoteActions(Request $request, Note $note, array &$actions)
+    public function onAddExtraNoteActions(Request $request, Note $note, array &$actions): bool
     {
         if (\is_null($user = Common::user())) {
             return Event::next;
@@ -234,7 +281,8 @@ class Conversation extends Component
             'conversation_mute',
             [
                 'conversation_id' => $note->getConversationId(),
-                'from'            => $from . '#note-anchor-' . $note->getId(),
+                'from'            => $from,
+                '_fragment'       => 'note-anchor-' . $note->getId(),
             ],
             Router::ABSOLUTE_PATH,
         );
@@ -248,7 +296,14 @@ class Conversation extends Component
         return Event::next;
     }
 
-    public function onNewNotificationShould(Activity $activity, Actor $actor)
+    /**
+     * Prevents new Notifications to appear for muted conversations
+     *
+     * @param Activity $activity Notification Activity
+     *
+     * @return bool EventHook
+     */
+    public function onNewNotificationShould(Activity $activity, Actor $actor): bool
     {
         if ($activity->getObjectType() === 'note' && ConversationMute::isMuted($activity, $actor)) {
             return Event::stop;

components/Conversation/Entity/Conversation.php

@@ -26,7 +26,6 @@ namespace Component\Conversation\Entity;
 use App\Core\DB\DB;
 use App\Core\Entity;
 use App\Core\Router\Router;
-use App\Entity\Note;
 
 /**
  * Entity class for Conversations

components/Feed/Controller/Feeds.php

@@ -37,6 +37,7 @@ namespace Component\Feed\Controller;
 
 use function App\Core\I18n\_m;
 use App\Util\Common;
+use App\Util\HTML\Heading;
 use Component\Collection\Util\Controller\FeedController;
 use Symfony\Component\HttpFoundation\Request;
 
@@ -47,11 +48,13 @@ class Feeds extends FeedController
      */
     public function public(Request $request): array
     {
-        $data = $this->query('note-local:true');
+        $data       = $this->query('note-local:true');
+        $page_title = _m(\is_null(Common::user()) ? 'Feed' : 'Planet');
         return [
-            '_template'  => 'collection/notes.html.twig',
+            '_template'        => 'collection/notes.html.twig',
-            'page_title' => _m(\is_null(Common::user()) ? 'Feed' : 'Planet'),
+            'page_title'       => $page_title,
-            'notes'      => $data['notes'],
+            'notes_feed_title' => (new Heading(level: 1, classes: ['feed-title'], text: $page_title)),
+            'notes'            => $data['notes'],
         ];
     }
 
@@ -63,9 +66,10 @@ class Feeds extends FeedController
         Common::ensureLoggedIn();
         $data = $this->query('note-from:subscribed-person,subscribed-group,subscribed-organisation');
         return [
-            '_template'  => 'collection/notes.html.twig',
+            '_template'        => 'collection/notes.html.twig',
-            'page_title' => _m('Home'),
+            'page_title'       => _m('Home'),
-            'notes'      => $data['notes'],
+            'notes_feed_title' => (new Heading(level: 1, classes: ['feed-title'], text: 'Home')),
+            'notes'            => $data['notes'],
         ];
     }
 }

components/Feed/tests/Controller/FeedsTest.php

@@ -0,0 +1,54 @@
+<?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 Component\Feed\tests\Controller;
+
+use App\Core\Router\Router;
+use App\Util\GNUsocialTestCase;
+use Component\Feed\Controller\Feeds;
+use Jchook\AssertThrows\AssertThrows;
+
+class FeedsTest extends GNUsocialTestCase
+{
+    use AssertThrows;
+
+    public function testPublic()
+    {
+        // This calls static::bootKernel(), and creates a "client" that is acting as the browser
+        $client  = static::createClient();
+        $crawler = $client->request('GET', Router::url('feed_public'));
+        $this->assertResponseIsSuccessful();
+    }
+
+    public function testHome()
+    {
+        // This calls static::bootKernel(), and creates a "client" that is acting as the browser
+        $client  = static::createClient();
+        $crawler = $client->request('GET', Router::url('feed_home'));
+        $this->assertResponseStatusCodeSame(302);
+    }
+
+    // TODO: It would be nice to actually test whether the feeds are respecting scopes and spitting
+    // out the expected notes... The ActivityPub plugin have a somewhat obvious way of testing it so,
+    // for now, having that, might fill that need, let's see
+}

components/FreeNetwork/Controller/HostMeta.php

@@ -20,7 +20,7 @@ class HostMeta extends XrdController
 
     public function setXRD()
     {
-        if (Event::handle('StartHostMetaLinks', [&$this->xrd->links]) !== Event::stop) {
+        if (Event::handle('StartHostMetaLinks', [&$this->xrd->links])) {
             Event::handle('EndHostMetaLinks', [&$this->xrd->links]);
         }
     }

components/FreeNetwork/Entity/FreeNetworkActorProtocol.php

@@ -125,7 +125,7 @@ class FreeNetworkActorProtocol extends Entity
         } else {
             $attributed_protocol->setProtocol($protocol);
         }
-        DB::wrapInTransaction(fn () => DB::persist($attributed_protocol));
+        DB::persist($attributed_protocol);
     }
 
     public static function canIActor(string $protocol, int|Actor $actor_id): bool

components/FreeNetwork/FreeNetwork.php

@@ -44,6 +44,7 @@ use App\Util\Exception\NicknameTakenException;
 use App\Util\Exception\NicknameTooLongException;
 use App\Util\Exception\NoSuchActorException;
 use App\Util\Exception\ServerException;
+use App\Util\Formatting;
 use App\Util\Nickname;
 use Component\FreeNetwork\Controller\Feeds;
 use Component\FreeNetwork\Controller\HostMeta;
@@ -53,6 +54,7 @@ use Component\FreeNetwork\Util\Discovery;
 use Component\FreeNetwork\Util\WebfingerResource;
 use Component\FreeNetwork\Util\WebfingerResource\WebfingerResourceActor;
 use Component\FreeNetwork\Util\WebfingerResource\WebfingerResourceNote;
+use Doctrine\Common\Collections\ExpressionBuilder;
 use Exception;
 use const PREG_SET_ORDER;
 use Symfony\Component\HttpFoundation\JsonResponse;
@@ -76,6 +78,13 @@ class FreeNetwork extends Component
     public const OAUTH_ACCESS_TOKEN_REL  = 'http://apinamespace.org/oauth/access_token';
     public const OAUTH_REQUEST_TOKEN_REL = 'http://apinamespace.org/oauth/request_token';
     public const OAUTH_AUTHORIZE_REL     = 'http://apinamespace.org/oauth/authorize';
+    private static array $protocols      = [];
+
+    public function onInitializeComponent(): bool
+    {
+        Event::handle('AddFreeNetworkProtocol', [&self::$protocols]);
+        return Event::next;
+    }
 
     public function onAddRoute(RouteLoader $m): bool
     {
@@ -152,7 +161,7 @@ class FreeNetwork extends Component
             $parts = explode('@', mb_substr(urldecode($resource), 5)); // 5 is strlen of 'acct:'
             if (\count($parts) === 2) {
                 [$nick, $domain] = $parts;
-                if ($domain !== $_ENV['SOCIAL_DOMAIN']) {
+                if ($domain !== Common::config('site', 'server')) {
                     throw new ServerException(_m('Remote profiles not supported via WebFinger yet.'));
                 }
 
@@ -169,7 +178,7 @@ class FreeNetwork extends Component
                     // This means $resource is a valid url
                     $resource_parts = parse_url($resource);
                     // TODO: Use URLMatcher
-                    if ($resource_parts['host'] === $_ENV['SOCIAL_DOMAIN']) { // XXX: Common::config('site', 'server')) {
+                    if ($resource_parts['host'] === Common::config('site', 'server')) {
                         $str = $resource_parts['path'];
                         // actor_view_nickname
                         $renick = '/\/@(' . Nickname::DISPLAY_FMT . ')\/?/m';
@@ -379,7 +388,7 @@ class FreeNetwork extends Component
             $actor = null;
 
             $resource_parts = explode($preMention, $target);
-            if ($resource_parts[1] === $_ENV['SOCIAL_DOMAIN']) { // XXX: Common::config('site', 'server')) {
+            if ($resource_parts[1] === Common::config('site', 'server')) {
                 $actor = LocalUser::getByPK(['nickname' => $resource_parts[0]])->getActor();
             } else {
                 Event::handle('FreeNetworkFindMentions', [$target, &$actor]);
@@ -408,7 +417,7 @@ class FreeNetwork extends Component
                 // This means $resource is a valid url
                 $resource_parts = parse_url($url);
                 // TODO: Use URLMatcher
-                if ($resource_parts['host'] === $_ENV['SOCIAL_DOMAIN']) { // XXX: Common::config('site', 'server')) {
+                if ($resource_parts['host'] === Common::config('site', 'server')) {
                     $str = $resource_parts['path'];
                     // actor_view_nickname
                     $renick = '/\/@(' . Nickname::DISPLAY_FMT . ')\/?/m';
@@ -489,22 +498,39 @@ class FreeNetwork extends Component
 
     public static function notify(Actor $sender, Activity $activity, array $targets, ?string $reason = null): bool
     {
-        $protocols = [];
+        foreach (self::$protocols as $protocol) {
-        Event::handle('AddFreeNetworkProtocol', [&$protocols]);
+            $protocol::freeNetworkDistribute($sender, $activity, $targets, $reason);
-        $delivered = [];
-        foreach ($protocols as $protocol) {
-            $protocol::freeNetworkDistribute($sender, $activity, $targets, $reason, $delivered);
         }
-        $failed_targets = array_udiff($targets, $delivered, fn (Actor $a, Actor $b): int => $a->getId() <=> $b->getId());
-        // TODO: Implement failed queues
         return false;
     }
 
-    public static function mentionToName(string $nickname, string $uri): string
+    public static function mentionTagToName(string $nickname, string $uri): string
     {
         return '@' . $nickname . '@' . parse_url($uri, \PHP_URL_HOST);
     }
 
+    public static function groupTagToName(string $nickname, string $uri): string
+    {
+        return '!' . $nickname . '@' . parse_url($uri, \PHP_URL_HOST);
+    }
+
+    /**
+     * Add fediverse: query expression
+     * // TODO: adding WebFinger would probably be nice
+     */
+    public function onCollectionQueryCreateExpression(ExpressionBuilder $eb, string $term, ?string $locale, ?Actor $actor, &$note_expr, &$actor_expr): bool
+    {
+        if (Formatting::startsWith($term, ['fediverse:'])) {
+            foreach (self::$protocols as $protocol) {
+                // 10 is strlen of `fediverse:`
+                if ($protocol::freeNetworkGrabRemote(mb_substr($term, 10))) {
+                    break;
+                }
+            }
+        }
+        return Event::next;
+    }
+
     public function onPluginVersion(array &$versions): bool
     {
         $versions[] = [

components/Group/Controller/Group.php

@@ -23,155 +23,52 @@ declare(strict_types = 1);
 
 namespace Component\Group\Controller;
 
+use App\Core\ActorLocalRoles;
 use App\Core\Cache;
+use App\Core\Controller;
 use App\Core\DB\DB;
 use App\Core\Form;
 use function App\Core\I18n\_m;
 use App\Core\Log;
-use App\Core\UserRoles;
 use App\Entity as E;
 use App\Util\Common;
 use App\Util\Exception\ClientException;
+use App\Util\Exception\DuplicateFoundException;
 use App\Util\Exception\NicknameEmptyException;
+use App\Util\Exception\NicknameException;
 use App\Util\Exception\NicknameInvalidException;
 use App\Util\Exception\NicknameNotAllowedException;
 use App\Util\Exception\NicknameTakenException;
 use App\Util\Exception\NicknameTooLongException;
-use App\Util\Exception\NoLoggedInUser;
+use App\Util\Exception\NotFoundException;
 use App\Util\Exception\RedirectException;
 use App\Util\Exception\ServerException;
 use App\Util\Form\ActorForms;
 use App\Util\Nickname;
-use Component\Collection\Util\Controller\FeedController;
 use Component\Group\Entity\GroupMember;
 use Component\Group\Entity\LocalGroup;
 use Component\Subscription\Entity\ActorSubscription;
+use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
 use Symfony\Component\Form\Extension\Core\Type\SubmitType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
+use Symfony\Component\Form\FormInterface;
 use Symfony\Component\HttpFoundation\Request;
 
-class Group extends FeedController
+class Group extends Controller
 {
-    /**
-     * View a group feed by its nickname
-     *
-     * @param string $nickname The group's nickname to be shown
-     *
-     * @throws NicknameEmptyException
-     * @throws NicknameNotAllowedException
-     * @throws NicknameTakenException
-     * @throws NicknameTooLongException
-     * @throws ServerException
-     *
-     * @return array
-     */
-    public function groupViewNickname(Request $request, string $nickname)
-    {
-        Nickname::validate($nickname, which: Nickname::CHECK_LOCAL_GROUP); // throws
-        $group          = LocalGroup::getActorByNickname($nickname);
-        $actor          = Common::actor();
-        $subscribe_form = null;
-
-        if (!\is_null($group)
-            && !\is_null($actor)
-            && \is_null(Cache::get(
-                ActorSubscription::cacheKeys($actor, $group)['subscribed'],
-                fn () => DB::findOneBy('actor_subscription', [
-                    'subscriber_id' => $actor->getId(),
-                    'subscribed_id' => $group->getId(),
-                ], return_null: true),
-            ))
-        ) {
-            $subscribe_form = Form::create([['subscribe', SubmitType::class, ['label' => _m('Subscribe to this group')]]]);
-            $subscribe_form->handleRequest($request);
-            if ($subscribe_form->isSubmitted() && $subscribe_form->isValid()) {
-                DB::persist(ActorSubscription::create([
-                    'subscriber_id' => $actor->getId(),
-                    'subscribed_id' => $group->getId(),
-                ]));
-                DB::flush();
-                Cache::delete(E\Actor::cacheKeys($group->getId())['subscribers']);
-                Cache::delete(E\Actor::cacheKeys($actor->getId())['subscribed']);
-                Cache::delete(ActorSubscription::cacheKeys($actor, $group)['subscribed']);
-            }
-        }
-
-        $notes = !\is_null($group) ? DB::dql(
-            <<<'EOF'
-                select n from note n
-                    join activity a with n.id = a.object_id
-                    join group_inbox gi with a.id = gi.activity_id
-                where a.object_type = 'note' and gi.group_id = :group_id
-                order by a.created desc, a.id desc
-                EOF,
-            ['group_id' => $group->getId()],
-        ) : [];
-
-        return [
-            '_template'      => 'group/view.html.twig',
-            'actor'          => $group,
-            'nickname'       => $group?->getNickname() ?? $nickname,
-            'notes'          => $notes,
-            'subscribe_form' => $subscribe_form?->createView(),
-        ];
-    }
-
     /**
      * Page that allows an actor to create a new group
      *
      * @throws RedirectException
      * @throws ServerException
-     *
-     * @return array
      */
-    public function groupCreate(Request $request)
+    public function groupCreate(Request $request): array
     {
         if (\is_null($actor = Common::actor())) {
             throw new RedirectException('security_login');
         }
 
-        $create_form = Form::create([
+        $create_form = self::getGroupCreateForm($request, $actor);
-            ['group_nickname', TextType::class, ['label' => _m('Group nickname')]],
-            ['group_create', SubmitType::class, ['label' => _m('Create this group!')]],
-        ]);
-
-        $create_form->handleRequest($request);
-        if ($create_form->isSubmitted() && $create_form->isValid()) {
-            $data     = $create_form->getData();
-            $nickname = $data['group_nickname'];
-
-            Log::info(
-                _m(
-                    'Actor id:{actor_id} nick:{actor_nick} created the group {nickname}',
-                    ['{actor_id}' => $actor->getId(), 'actor_nick' => $actor->getNickname(), 'nickname' => $nickname],
-                ),
-            );
-
-            DB::persist($group = E\Actor::create([
-                'nickname' => $nickname,
-                'type'     => E\Actor::GROUP,
-                'is_local' => true,
-                'roles'    => UserRoles::BOT,
-            ]));
-            DB::persist(LocalGroup::create([
-                'group_id' => $group->getId(),
-                'nickname' => $nickname,
-            ]));
-            DB::persist(ActorSubscription::create([
-                'subscriber_id' => $group->getId(),
-                'subscribed_id' => $group->getId(),
-            ]));
-            DB::persist(GroupMember::create([
-                'group_id' => $group->getId(),
-                'actor_id' => $actor->getId(),
-                'is_admin' => true,
-            ]));
-            DB::flush();
-            Cache::delete(E\Actor::cacheKeys($actor->getId())['subscribers']);
-            Cache::delete(E\Actor::cacheKeys($actor->getId())['subscribed']);
-
-            throw new RedirectException();
-        }
 
         return [
             '_template'   => 'group/create.html.twig',
@@ -183,30 +80,104 @@ class Group extends FeedController
      * Settings page for the group with the provided nickname, checks if the current actor can administrate given group
      *
      * @throws ClientException
+     * @throws DuplicateFoundException
      * @throws NicknameEmptyException
+     * @throws NicknameException
      * @throws NicknameInvalidException
      * @throws NicknameNotAllowedException
      * @throws NicknameTakenException
      * @throws NicknameTooLongException
-     * @throws NoLoggedInUser
+     * @throws NotFoundException
      * @throws ServerException
-     *
-     * @return array
      */
-    public function groupSettings(Request $request, string $nickname)
+    public function groupSettings(Request $request, int $id): array
     {
-        $local_group = LocalGroup::getByNickname($nickname);
+        $local_group = DB::findOneBy(LocalGroup::class, ['actor_id' => $id]);
         $group_actor = $local_group->getActor();
         $actor       = Common::actor();
-        if (!\is_null($group_actor) && $actor->canAdmin($group_actor)) {
+        if (!\is_null($group_actor) && $actor->canModerate($group_actor)) {
             return [
                 '_template'          => 'group/settings.html.twig',
                 'group'              => $group_actor,
-                'personal_info_form' => ActorForms::personalInfo($request, $actor, $local_group)->createView(),
+                'personal_info_form' => ActorForms::personalInfo(request: $request, scope: $actor, target: $group_actor)->createView(),
                 'open_details_query' => $this->string('open'),
             ];
         } else {
-            throw new ClientException(_m('You do not have permission to edit settings for the group "{group}"', ['{group}' => $nickname]), code: 404);
+            throw new ClientException(_m('You do not have permission to edit settings for the group "{group}"', ['{group}' => $id]), code: 404);
         }
     }
+
+    /**
+     * Create a new Group FormInterface getter
+     *
+     * @throws RedirectException
+     * @throws ServerException
+     */
+    public static function getGroupCreateForm(Request $request, E\Actor $actor): FormInterface
+    {
+        $create_form = Form::create([
+            ['group_nickname', TextType::class, ['label' => _m('Group nickname')]],
+            ['group_type', ChoiceType::class, ['label' => _m('Type:'), 'multiple' => false, 'expanded' => false, 'choices' => [
+                _m('Group')        => 'group',
+                _m('Organisation') => 'organisation',
+            ]]],
+            ['group_scope', ChoiceType::class, ['label' => _m('Is this a private group:'), 'multiple' => false, 'expanded' => false, 'choices' => [
+                _m('No')  => 'public',
+                _m('Yes') => 'private',
+            ]]],
+            ['group_create', SubmitType::class, ['label' => _m('Create this group!')]],
+        ]);
+
+        $create_form->handleRequest($request);
+        if ($create_form->isSubmitted() && $create_form->isValid()) {
+            $data     = $create_form->getData();
+            $nickname = Nickname::normalize(
+                nickname: $data['group_nickname'],
+                check_already_used: true,
+                which: Nickname::CHECK_LOCAL_GROUP,
+                check_is_allowed: true,
+            );
+
+            $roles = ActorLocalRoles::VISITOR; // Can send direct messages to other actors
+
+            if ($data['group_scope'] === 'private') {
+                $roles |= ActorLocalRoles::PRIVATE_GROUP;
+            }
+
+            Log::info(
+                _m(
+                    'Actor id:{actor_id} nick:{actor_nick} created the ' . ($roles & ActorLocalRoles::PRIVATE_GROUP ? 'private' : 'public') . ' group {nickname}',
+                    ['{actor_id}' => $actor->getId(), 'actor_nick' => $actor->getNickname(), 'nickname' => $nickname],
+                ),
+            );
+
+            DB::persist($group = E\Actor::create([
+                'nickname' => $nickname,
+                'type'     => E\Actor::GROUP,
+                'is_local' => true,
+                'roles'    => $roles,
+            ]));
+            DB::persist(LocalGroup::create([
+                'actor_id' => $group->getId(),
+                'type'     => $data['group_type'],
+                'nickname' => $nickname,
+            ]));
+            DB::persist(ActorSubscription::create([
+                'subscriber_id' => $group->getId(),
+                'subscribed_id' => $group->getId(),
+            ]));
+            DB::persist(GroupMember::create([
+                'group_id' => $group->getId(),
+                'actor_id' => $actor->getId(),
+                // Group Owner
+                'roles' => ActorLocalRoles::OPERATOR | ActorLocalRoles::MODERATOR | ActorLocalRoles::PARTICIPANT | ActorLocalRoles::VISITOR,
+            ]));
+            DB::flush();
+            Cache::delete(E\Actor::cacheKeys($actor->getId())['subscribers']);
+            Cache::delete(E\Actor::cacheKeys($actor->getId())['subscribed']);
+
+            throw new RedirectException();
+        }
+        return $create_form;
+    }
 }

components/Group/Controller/GroupFeed.php

@@ -0,0 +1,130 @@
+<?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 Component\Group\Controller;
+
+use App\Core\Cache;
+use App\Core\DB\DB;
+use App\Core\Form;
+use function App\Core\I18n\_m;
+use App\Core\Router\Router;
+use App\Entity\Actor;
+use App\Entity as E;
+use App\Util\Common;
+use App\Util\Exception\ClientException;
+use App\Util\Exception\ServerException;
+use App\Util\HTML\Heading;
+use Component\Collection\Util\Controller\FeedController;
+use Component\Group\Entity\LocalGroup;
+use Component\Subscription\Entity\ActorSubscription;
+use Symfony\Component\Form\Extension\Core\Type\SubmitType;
+use Symfony\Component\HttpFoundation\Request;
+
+class GroupFeed extends FeedController
+{
+    /**
+     * @throws ServerException
+     */
+    public function groupView(Request $request, Actor $group): array
+    {
+        $actor          = Common::actor();
+        $subscribe_form = null;
+
+        if (!\is_null($actor)
+            && \is_null(Cache::get(
+                ActorSubscription::cacheKeys($actor, $group)['subscribed'],
+                fn () => DB::findOneBy('actor_subscription', [
+                    'subscriber_id' => $actor->getId(),
+                    'subscribed_id' => $group->getId(),
+                ], return_null: true),
+            ))
+        ) {
+            $subscribe_form = Form::create([['subscribe', SubmitType::class, ['label' => _m('Subscribe to this group')]]]);
+            $subscribe_form->handleRequest($request);
+            if ($subscribe_form->isSubmitted() && $subscribe_form->isValid()) {
+                DB::persist(ActorSubscription::create([
+                    'subscriber_id' => $actor->getId(),
+                    'subscribed_id' => $group->getId(),
+                ]));
+                DB::flush();
+                Cache::delete(E\Actor::cacheKeys($group->getId())['subscribers']);
+                Cache::delete(E\Actor::cacheKeys($actor->getId())['subscribed']);
+                Cache::delete(ActorSubscription::cacheKeys($actor, $group)['subscribed']);
+            }
+        }
+
+        $notes = DB::dql(<<<'EOF'
+            SELECT n FROM \App\Entity\Note AS n
+            WHERE n.id IN (
+                SELECT act.object_id FROM \App\Entity\Activity AS act
+                    WHERE act.object_type = 'note' AND act.id IN
+                        (SELECT att.activity_id FROM \Component\Notification\Entity\Notification AS att WHERE att.target_id = :id)
+                )
+            EOF, ['id' => $group->getId()]);
+
+        return [
+            '_template'        => 'group/view.html.twig',
+            'actor'            => $group,
+            'nickname'         => $group->getNickname(),
+            'notes'            => $notes,
+            'notes_feed_title' => (new Heading(1, [], $group->getNickname() . '\'s feed')),
+            'subscribe_form'   => $subscribe_form?->createView(),
+        ];
+    }
+
+    /**
+     * @throws ClientException
+     * @throws ServerException
+     */
+    public function groupViewId(Request $request, int $id): array
+    {
+        $group = Actor::getById($id);
+        if (\is_null($group) || !$group->isGroup()) {
+            throw new ClientException(_m('No such group.'), 404);
+        }
+        if ($group->getIsLocal()) {
+            return [
+                '_redirect' => Router::url('group_actor_view_nickname', ['nickname' => $group->getNickname()]),
+                'actor'     => $group,
+            ];
+        }
+        return $this->groupView($request, $group);
+    }
+
+    /**
+     * View a group feed by its nickname
+     *
+     * @param string $nickname The group's nickname to be shown
+     *
+     * @throws ClientException
+     * @throws ServerException
+     */
+    public function groupViewNickname(Request $request, string $nickname): array
+    {
+        $group = LocalGroup::getActorByNickname($nickname);
+        if (\is_null($group)) {
+            throw new ClientException(_m('No such group.'), 404);
+        }
+        return $this->groupView($request, $group);
+    }
+}

components/Group/Entity/GroupAlias.php

@@ -1,100 +0,0 @@
-<?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 Component\Group\Entity;
-
-use App\Core\Entity;
-use DateTimeInterface;
-
-/**
- * Entity for Group Alias
- *
- * @category  DB
- * @package   GNUsocial
- *
- * @author    Zach Copley <zach@status.net>
- * @copyright 2010 StatusNet Inc.
- * @author    Mikael Nordfeldth <mmn@hethane.se>
- * @copyright 2009-2014 Free Software Foundation, Inc http://www.fsf.org
- * @author    Hugo Sales <hugo@hsal.es>
- * @copyright 2020-2021 Free Software Foundation, Inc http://www.fsf.org
- * @license   https://www.gnu.org/licenses/agpl.html GNU AGPL v3 or later
- */
-class GroupAlias extends Entity
-{
-    // {{{ Autocode
-    // @codeCoverageIgnoreStart
-    private string $alias;
-    private int $group_id;
-    private DateTimeInterface $modified;
-
-    public function setAlias(string $alias): self
-    {
-        $this->alias = mb_substr($alias, 0, 64);
-        return $this;
-    }
-
-    public function getAlias(): string
-    {
-        return $this->alias;
-    }
-
-    public function setGroupId(int $group_id): self
-    {
-        $this->group_id = $group_id;
-        return $this;
-    }
-
-    public function getGroupId(): int
-    {
-        return $this->group_id;
-    }
-
-    public function setModified(DateTimeInterface $modified): self
-    {
-        $this->modified = $modified;
-        return $this;
-    }
-
-    public function getModified(): DateTimeInterface
-    {
-        return $this->modified;
-    }
-
-    // @codeCoverageIgnoreEnd
-    // }}} Autocode
-
-    public static function schemaDef(): array
-    {
-        return [
-            'name'   => 'group_alias',
-            'fields' => [
-                'alias'    => ['type' => 'varchar',   'length' => 64, 'not null' => true, 'description' => 'additional nickname for the group'],
-                'group_id' => ['type' => 'int',       'foreign key' => true, 'target' => 'Group.id', 'multiplicity' => 'many to one', 'not null' => true, 'description' => 'group id which this is an alias of'],
-                'modified' => ['type' => 'timestamp', 'not null' => true, 'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was modified'],
-            ],
-            'primary key' => ['alias'],
-            'indexes'     => [
-                'group_alias_group_id_idx' => ['group_id'],
-            ],
-        ];
-    }
-}

components/Group/Entity/GroupBlock.php

@@ -1,110 +0,0 @@
-<?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 Component\Group\Entity;
-
-use App\Core\Entity;
-use DateTimeInterface;
-
-/**
- * Entity for Group Block
- *
- * @category  DB
- * @package   GNUsocial
- *
- * @author    Zach Copley <zach@status.net>
- * @copyright 2010 StatusNet Inc.
- * @author    Mikael Nordfeldth <mmn@hethane.se>
- * @copyright 2009-2014 Free Software Foundation, Inc http://www.fsf.org
- * @author    Hugo Sales <hugo@hsal.es>
- * @copyright 2020-2021 Free Software Foundation, Inc http://www.fsf.org
- * @license   https://www.gnu.org/licenses/agpl.html GNU AGPL v3 or later
- */
-class GroupBlock extends Entity
-{
-    // {{{ Autocode
-    // @codeCoverageIgnoreStart
-    private int $group_id;
-    private int $blocked_actor;
-    private int $blocker_user;
-    private DateTimeInterface $modified;
-
-    public function setGroupId(int $group_id): self
-    {
-        $this->group_id = $group_id;
-        return $this;
-    }
-
-    public function getGroupId(): int
-    {
-        return $this->group_id;
-    }
-
-    public function setBlockedActor(int $blocked_actor): self
-    {
-        $this->blocked_actor = $blocked_actor;
-        return $this;
-    }
-
-    public function getBlockedActor(): int
-    {
-        return $this->blocked_actor;
-    }
-
-    public function setBlockerUser(int $blocker_user): self
-    {
-        $this->blocker_user = $blocker_user;
-        return $this;
-    }
-
-    public function getBlockerUser(): int
-    {
-        return $this->blocker_user;
-    }
-
-    public function setModified(DateTimeInterface $modified): self
-    {
-        $this->modified = $modified;
-        return $this;
-    }
-
-    public function getModified(): DateTimeInterface
-    {
-        return $this->modified;
-    }
-
-    // @codeCoverageIgnoreEnd
-    // }}} Autocode
-
-    public static function schemaDef(): array
-    {
-        return [
-            'name'   => 'group_block',
-            'fields' => [
-                'group_id'      => ['type' => 'int', 'foreign key' => true, 'target' => 'Group.id', 'multiplicity' => 'many to one', 'not null' => true, 'description' => 'group actor is blocked from'],
-                'blocked_actor' => ['type' => 'int', 'foreign key' => true, 'target' => 'Actor.id', 'multiplicity' => 'one to one', 'not null' => true, 'description' => 'actor that is blocked'],
-                'blocker_user'  => ['type' => 'int', 'foreign key' => true, 'target' => 'LocalUser.id', 'multiplicity' => 'one to one', 'not null' => true, 'description' => 'user making the block'],
-                'modified'      => ['type' => 'timestamp', 'not null' => true, 'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was modified'],
-            ],
-            'primary key' => ['group_id', 'blocked_actor'],
-        ];
-    }
-}

components/Group/Entity/GroupInbox.php

@@ -1,103 +0,0 @@
-<?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 Component\Group\Entity;
-
-use App\Core\Entity;
-use DateTimeInterface;
-
-/**
- * Entity for Group's inbox
- *
- * @category  DB
- * @package   GNUsocial
- *
- * @author    Zach Copley <zach@status.net>
- * @copyright 2010 StatusNet Inc.
- * @author    Mikael Nordfeldth <mmn@hethane.se>
- * @copyright 2009-2014 Free Software Foundation, Inc http://www.fsf.org
- * @author    Hugo Sales <hugo@hsal.es>
- * @copyright 2020-2021 Free Software Foundation, Inc http://www.fsf.org
- * @license   https://www.gnu.org/licenses/agpl.html GNU AGPL v3 or later
- */
-class GroupInbox extends Entity
-{
-    // {{{ Autocode
-    // @codeCoverageIgnoreStart
-    private int $group_id;
-    private int $activity_id;
-    private DateTimeInterface $created;
-
-    public function setGroupId(int $group_id): self
-    {
-        $this->group_id = $group_id;
-        return $this;
-    }
-
-    public function getGroupId(): int
-    {
-        return $this->group_id;
-    }
-
-    public function setActivityId(int $activity_id): self
-    {
-        $this->activity_id = $activity_id;
-        return $this;
-    }
-
-    public function getActivityId(): int
-    {
-        return $this->activity_id;
-    }
-
-    public function setCreated(DateTimeInterface $created): self
-    {
-        $this->created = $created;
-        return $this;
-    }
-
-    public function getCreated(): DateTimeInterface
-    {
-        return $this->created;
-    }
-
-    // @codeCoverageIgnoreEnd
-    // }}} Autocode
-
-    public static function schemaDef(): array
-    {
-        return [
-            'name'        => 'group_inbox',
-            'description' => 'Many-many table listing activities posted to a given group, or which groups a given activity was posted to',
-            'fields'      => [
-                'group_id'    => ['type' => 'int',      'foreign key' => true, 'target' => 'Group.id', 'multiplicity' => 'one to one', 'name' => 'group_inbox_group_id_fkey', 'not null' => true, 'description' => 'group receiving the activity'],
-                'activity_id' => ['type' => 'int',      'foreign key' => true, 'target' => 'Activity.id', 'multiplicity' => 'many to one', 'name' => 'group_inbox_activity_id_fkey', 'not null' => true, 'description' => 'activity received'],
-                'created'     => ['type' => 'datetime', 'not null' => true,    'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was created'],
-            ],
-            'primary key' => ['group_id', 'activity_id'],
-            'indexes'     => [
-                'group_inbox_activity_id_idx'                  => ['activity_id'],
-                'group_inbox_group_id_created_activity_id_idx' => ['group_id', 'created', 'activity_id'],
-                'group_inbox_created_idx'                      => ['created'],
-            ],
-        ];
-    }
-}

components/Group/Entity/GroupJoinQueue.php

@@ -76,7 +76,7 @@ class GroupJoinQueue extends Entity
             'description' => 'Holder for group join requests awaiting moderation.',
             'fields'      => [
                 'actor_id' => ['type' => 'int', 'foreign key' => true, 'target' => 'Actor.id', 'multiplicity' => 'many to one', 'name' => 'group_join_queue_actor_id_fkey', 'not null' => true, 'description' => 'remote or local actor making the request'],
-                'group_id' => ['type' => 'int', 'foreign key' => true, 'target' => 'Group.id',   'multiplicity' => 'many to one',   'name' => 'group_join_queue_group_id_fkey',   'not null' => true, 'description' => 'remote or local group to join, if any'],
+                'group_id' => ['type' => 'int', 'foreign key' => true, 'target' => 'Group.id', 'multiplicity' => 'many to one', 'name' => 'group_join_queue_group_id_fkey', 'not null' => true, 'description' => 'remote or local group to join, if any'],
             ],
             'primary key' => ['actor_id', 'group_id'],
             'indexes'     => [

components/Group/Entity/GroupMember.php

@@ -21,6 +21,7 @@ declare(strict_types = 1);
 
 namespace Component\Group\Entity;
 
+use App\Core\ActorLocalRoles;
 use App\Core\Entity;
 use DateTimeInterface;
 
@@ -44,8 +45,7 @@ class GroupMember extends Entity
     // @codeCoverageIgnoreStart
     private int $group_id;
     private int $actor_id;
-    private ?bool $is_admin = false;
+    private int $roles = ActorLocalRoles::VISITOR;
-    private ?string $uri    = null;
     private DateTimeInterface $created;
     private DateTimeInterface $modified;
 
@@ -71,26 +71,15 @@ class GroupMember extends Entity
         return $this->actor_id;
     }
 
-    public function setIsAdmin(?bool $is_admin): self
+    public function setRoles(int $roles): self
     {
-        $this->is_admin = $is_admin;
+        $this->roles = $roles;
         return $this;
     }
 
-    public function getIsAdmin(): ?bool
+    public function getRoles(): int
     {
-        return $this->is_admin;
+        return $this->roles;
-    }
-
-    public function setUri(?string $uri): self
-    {
-        $this->uri = \is_null($uri) ? null : mb_substr($uri, 0, 191);
-        return $this;
-    }
-
-    public function getUri(): ?string
-    {
-        return $this->uri;
     }
 
     public function setCreated(DateTimeInterface $created): self
@@ -123,18 +112,14 @@ class GroupMember extends Entity
         return [
             'name'   => 'group_member',
             'fields' => [
-                'group_id' => ['type' => 'int',       'foreign key' => true, 'target' => 'Actor.id', 'multiplicity' => 'one to one', 'name' => 'group_member_group_id_fkey', 'not null' => true,  'description' => 'foreign key to group table'],
+                'group_id' => ['type' => 'int', 'foreign key' => true, 'target' => 'Actor.id', 'multiplicity' => 'one to one', 'name' => 'group_member_group_id_fkey', 'not null' => true, 'description' => 'foreign key to group table'],
-                'actor_id' => ['type' => 'int',       'foreign key' => true, 'target' => 'Actor.id', 'multiplicity' => 'one to one', 'name' => 'group_member_actor_id_fkey', 'not null' => true,  'description' => 'foreign key to actor table'],
+                'actor_id' => ['type' => 'int', 'foreign key' => true, 'target' => 'Actor.id', 'multiplicity' => 'one to one', 'name' => 'group_member_actor_id_fkey', 'not null' => true, 'description' => 'foreign key to actor table'],
-                'is_admin' => ['type' => 'bool',      'default' => false,    'description' => 'is this actor an admin?'],
+                'roles'    => ['type' => 'int', 'not null' => true, 'default' => ActorLocalRoles::VISITOR,  'description' => 'Bitmap of permissions this actor has'],
-                'uri'      => ['type' => 'varchar',   'length' => 191,       'description' => 'universal identifier'],
+                'created'  => ['type' => 'datetime', 'not null' => true, 'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was created'],
-                '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'],
-                'modified' => ['type' => 'timestamp', 'not null' => true,    'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was modified'],
             ],
             'primary key' => ['group_id', 'actor_id'],
-            'unique keys' => [
+            'indexes'     => [
-                'group_member_uri_key' => ['uri'],
-            ],
-            'indexes' => [
                 'group_member_actor_id_idx'         => ['actor_id'],
                 'group_member_created_idx'          => ['created'],
                 'group_member_actor_id_created_idx' => ['actor_id', 'created'],

components/Group/Entity/LocalGroup.php

@@ -21,10 +21,8 @@ declare(strict_types = 1);
 
 namespace Component\Group\Entity;
 
-use App\Core\Cache;
 use App\Core\DB\DB;
 use App\Core\Entity;
-
 use App\Entity\Actor;
 use App\Util\Exception\NicknameEmptyException;
 use App\Util\Exception\NicknameException;
@@ -53,33 +51,45 @@ class LocalGroup extends Entity
 {
     // {{{ Autocode
     // @codeCoverageIgnoreStart
-    private int $group_id;
+    private int $actor_id;
-    private ?string $nickname = null;
+    private string $nickname;
+    private string $type = 'group';
     private DateTimeInterface $created;
     private DateTimeInterface $modified;
 
-    public function setGroupId(int $group_id): self
+    public function setActorId(int $actor_id): self
     {
-        $this->group_id = $group_id;
+        $this->actor_id = $actor_id;
         return $this;
     }
 
-    public function getGroupId(): int
+    public function getActorId(): int
     {
-        return $this->group_id;
+        return $this->actor_id;
     }
 
-    public function setNickname(?string $nickname): self
+    public function setNickname(string $nickname): self
     {
-        $this->nickname = \is_null($nickname) ? null : mb_substr($nickname, 0, 64);
+        $this->nickname = mb_substr($nickname, 0, 64);
         return $this;
     }
 
-    public function getNickname(): ?string
+    public function getNickname(): string
     {
         return $this->nickname;
     }
 
+    public function setType(string $type): self
+    {
+        $this->type = mb_substr($type, 0, 64);
+        return $this;
+    }
+
+    public function getType(): string
+    {
+        return $this->type;
+    }
+
     public function setCreated(DateTimeInterface $created): self
     {
         $this->created = $created;
@@ -107,19 +117,17 @@ class LocalGroup extends Entity
 
     public function getActor()
     {
-        return DB::find('actor', ['id' => $this->group_id]);
+        return DB::findOneBy(Actor::class, ['id' => $this->actor_id]);
     }
 
     public static function getByNickname(string $nickname): ?self
     {
-        $res = DB::findBy(self::class, ['nickname' => $nickname]);
+        return DB::findOneBy(self::class, ['nickname' => $nickname]);
-        return $res === [] ? null : $res[0];
     }
 
     public static function getActorByNickname(string $nickname): ?Actor
     {
-        $res = DB::findBy(Actor::class, ['nickname' => $nickname, 'type' => Actor::GROUP]);
+        return DB::findOneBy(Actor::class, ['nickname' => $nickname, 'type' => Actor::GROUP]);
-        return $res === [] ? null : $res[0];
     }
 
     /**
@@ -151,12 +159,13 @@ class LocalGroup extends Entity
             'name'        => 'local_group',
             'description' => 'Record for a user group on the local site, with some additional info not in user_group',
             'fields'      => [
-                'group_id' => ['type' => 'int',      'foreign key' => true, 'target' => 'Group.id', 'multiplicity' => 'one to one', 'name' => 'local_group_group_id_fkey', 'not null' => true, 'description' => 'group represented'],
+                'actor_id' => ['type' => 'int', 'foreign key' => true, 'target' => 'Group.id', 'multiplicity' => 'one to one', 'name' => 'local_group_group_id_fkey', 'not null' => true, 'description' => 'group represented'],
-                'nickname' => ['type' => 'varchar',  'length' => 64,        'description' => 'group represented'],
+                'nickname' => ['type' => 'varchar', 'not null' => true, 'length' => 64, 'description' => 'group represented'],
-                'created'  => ['type' => 'datetime', 'not null' => true,    'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was created'],
+                'type'     => ['type' => 'varchar', 'not null' => true, 'default' => 'group', 'length' => 64, 'description' => 'Group or Organisation'],
-                'modified' => ['type' => 'datetime', 'not null' => true,    'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was modified'],
+                'created'  => ['type' => 'datetime', 'not null' => true, 'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was created'],
+                'modified' => ['type' => 'datetime', 'not null' => true, 'default' => 'CURRENT_TIMESTAMP', 'description' => 'date this record was modified'],
             ],
-            'primary key' => ['group_id'],
+            'primary key' => ['actor_id'],
             'unique keys' => [
                 'local_group_nickname_key' => ['nickname'],
             ],

components/Group/Group.php

@@ -26,50 +26,61 @@ use function App\Core\I18n\_m;
 use App\Core\Modules\Component;
 use App\Core\Router\RouteLoader;
 use App\Core\Router\Router;
+use App\Entity\Activity;
 use App\Entity\Actor;
 use App\Util\Common;
 use App\Util\HTML;
 use App\Util\Nickname;
-use Component\Circle\Controller\SelfTagsSettings;
 use Component\Group\Controller as C;
 use Component\Group\Entity\LocalGroup;
+use Component\Notification\Notification;
 use Symfony\Component\HttpFoundation\Request;
 
 class Group extends Component
 {
     public function onAddRoute(RouteLoader $r): bool
     {
+        $r->connect(id: 'group_actor_view_id', uri_path: '/group/{id<\d+>}', target: [C\GroupFeed::class, 'groupViewId']);
+        $r->connect(id: 'group_actor_view_nickname', uri_path: '/!{nickname<' . Nickname::DISPLAY_FMT . '>}', target: [C\GroupFeed::class, 'groupViewNickname']);
         $r->connect(id: 'group_create', uri_path: '/group/new', target: [C\Group::class, 'groupCreate']);
-        $r->connect(id: 'group_actor_view_nickname', uri_path: '/!{nickname<' . Nickname::DISPLAY_FMT . '>}', target: [C\Group::class, 'groupViewNickname'], options: ['is_system_path' => false]);
+        $r->connect(id: 'group_actor_settings', uri_path: '/group/{id<\d+>}/settings', target: [C\Group::class, 'groupSettings']);
-        $r->connect(id: 'group_settings', uri_path: '/!{nickname<' . Nickname::DISPLAY_FMT . '>}/settings', target: [C\Group::class, 'groupSettings'], options: ['is_system_path' => false]);
         return Event::next;
     }
 
     /**
-     * Add an <a href=group_settings> to the profile card for groups, if the current actor can access them
+     * Enqueues a notification for an Actor (such as person or group) which means
+     * it shows up in their home feed and such.
+     */
+    public function onNewNotificationStart(Actor $sender, Activity $activity, array $targets = [], ?string $reason = null): bool
+    {
+        foreach ($targets as $target) {
+            if ($target->isGroup()) {
+                // The Group announces to its subscribers
+                Notification::notify(
+                    sender: $target,
+                    activity: $activity,
+                    targets: $target->getSubscribers(),
+                    reason: $reason,
+                );
+            }
+        }
+
+        return Event::next;
+    }
+
+    /**
+     * Add an <a href=group_actor_settings> to the profile card for groups, if the current actor can access them
      */
     public function onAppendCardProfile(array $vars, array &$res): bool
     {
         $actor = Common::actor();
         $group = $vars['actor'];
-        if (!\is_null($actor) && $group->isGroup() && $actor->canAdmin($group)) {
+        if (!\is_null($actor) && $group->isGroup()) {
-            $url   = Router::url('group_settings', ['nickname' => $group->getNickname()]);
+            if ($actor->canModerate($group)) {
-            $res[] = HTML::html(['a' => ['attrs' => ['href' => $url, 'title' => _m('Edit group settings'), 'class' => 'profile-extra-actions'], _m('Group settings')]]);
+                $url   = Router::url('group_actor_settings', ['id' => $group->getId()]);
-        }
+                $res[] = HTML::html(['a' => ['attrs' => ['href' => $url, 'title' => _m('Edit group settings'), 'class' => 'profile-extra-actions'], _m('Group settings')]]);
-        return Event::next;
+            }
-    }
+            $res[] = HTML::html(['a' => ['attrs' => ['href' => Router::url('blog_post', ['in' => $group->getId()]), 'title' => _m('Make a new blog post'), 'class' => 'profile-extra-actions'], _m('Post in blog')]]);
-
-    public function onPopulateSettingsTabs(Request $request, string $section, array &$tabs)
-    {
-        if ($section === 'profile' && $request->get('_route') === 'group_settings') {
-            $nickname = $request->get('nickname');
-            $group    = LocalGroup::getActorByNickname($nickname);
-            $tabs[]   = [
-                'title'      => 'Self tags',
-                'desc'       => 'Add or remove tags on this group',
-                'id'         => 'settings-self-tags',
-                'controller' => SelfTagsSettings::settingsSelfTags($request, $group, 'settings-self-tags-details'),
-            ];
         }
         return Event::next;
     }
@@ -79,27 +90,46 @@ class Group extends Component
      */
     private function getGroupFromContext(Request $request): ?Actor
     {
-        if (str_starts_with($request->get('_route'), 'group_actor_view_')) {
+        if (\is_array($request->get('post_note')) && \array_key_exists('_next', $request->get('post_note'))) {
-            if (!\is_null($id = $request->get('id'))) {
+            $next       = parse_url($request->get('post_note')['_next']);
-                return Actor::getById((int) $id);
+            $match      = Router::match($next['path']);
-            } elseif (!\is_null($nickname = $request->get('nickname'))) {
+            $route      = $match['_route'];
-                return LocalGroup::getActorByNickname($nickname);
+            $identifier = $match['id'] ?? $match['nickname'] ?? null;
+        } else {
+            $route      = $request->get('_route');
+            $identifier = $request->get('id') ?? $request->get('nickname');
+        }
+        if (str_starts_with($route, 'group_actor_view_')) {
+            switch ($route) {
+                case 'group_actor_view_nickname':
+                    return LocalGroup::getActorByNickname($identifier);
+                case 'group_actor_view_id':
+                    return Actor::getById((int) $identifier);
             }
         }
         return null;
     }
 
-    public function onPostingFillTargetChoices(Request $request, Actor $actor, array &$targets)
+    public function onPostingFillTargetChoices(Request $request, Actor $actor, array &$targets): bool
     {
         $group = $this->getGroupFromContext($request);
         if (!\is_null($group)) {
-            $nick           = '!' . $group->getNickname();
+            $nick           = "!{$group->getNickname()}";
             $targets[$nick] = $group->getId();
         }
         return Event::next;
     }
 
-    public function onPostingGetContextActor(Request $request, Actor $actor, ?Actor $context_actor)
+    /**
+     * Indicates the context in which Posting's form is to be presented. Passing on $context_actor to Posting's
+     * onAppendRightPostingBlock event, the Group a given $actor is currently browsing.
+     *
+     * Makes it possible to automagically fill in the targets (aka the Group which this $request route is connected to)
+     * in the Posting's form.
+     *
+     * @param null|Actor $context_actor Actor group, if current route is part of an existing Group set of routes
+     */
+    public function onPostingGetContextActor(Request $request, Actor $actor, ?Actor &$context_actor): bool
     {
         $ctx = $this->getGroupFromContext($request);
         if (!\is_null($ctx)) {

components/Group/templates/cards/group/create_widget.html.twig

@@ -0,0 +1,10 @@
+<details class="frame-section section-details-title">
+    <summary class="details-summary-title">
+        <strong>
+            {% trans %}Create a group{% endtrans %}
+        </strong>
+    </summary>
+    <form method="POST" class="section-form">
+        {{ form(create_form) }}
+    </form>
+</details>

components/Group/templates/group/settings.html.twig

@@ -1,16 +1,10 @@
 {% extends 'base.html.twig' %}
 
-{% import 'settings/macros.html.twig' as macros %}
+{% import 'cards/macros/settings.html.twig' as macros %}
-
-{% block stylesheets %}
-    {{ parent() }}
-    <link rel="preload" href="{{ asset('assets/default_theme/css/pages/settings.css') }}" as="style" type="text/css">
-    <link rel="stylesheet" href="{{ asset('assets/default_theme/css/pages/settings.css') }}">
-{% endblock stylesheets %}
 
 {% block body %}
     <nav class='section-settings'>
-        <h2>Settings</h2>
+        <h1>Settings</h1>
         <ul>
             <li>
                 {% set profile_tabs = [{'title': 'Personal Info', 'desc': 'Nickname, Homepage, Bio, Self Tags and more.', 'id': 'settings-personal-info', 'form': personal_info_form}] %}

components/Group/templates/group/view.html.twig

@@ -1,65 +1,16 @@
-{% extends 'stdgrid.html.twig' %}
+{% extends 'collection/notes.html.twig' %}
-{% import '/cards/note/view.html.twig' as noteView %}
 
 {% set nickname = nickname|escape %}
 {% block title %}{{ nickname }}{% endblock %}
 
-{% block stylesheets %}
-    {{ parent() }}
-    <link rel="stylesheet" href="{{ asset('assets/default_theme/css/pages/feeds.css') }}" type="text/css">
-{% endblock stylesheets %}
-
 {% block body %}
-
     {% if actor is defined and actor is not null %}
         {% block profile_view %}
-            {% include 'cards/profile/view.html.twig' with { 'actor': actor } only %}
+            {% include 'cards/blocks/profile.html.twig' with { 'actor': actor } only %}
         {% endblock profile_view %}
-
+        <hr>
         {% if notes is defined %}
-            <article>
+            {{ parent() }}
-                <header class="feed-header">
-                    {% if page_title is defined %}
-                        <h1 class="heading-no-margin">{{ page_title | trans }}</h1>
-                    {% else %}
-                        <h1 class="heading-no-margin">{{ 'Notes' | trans }}</h1>
-                    {% endif %}
-                    <nav class="feed-actions">
-                        <details class="feed-actions-details">
-                            <summary>
-                                {{ icon('filter', 'icon icon-feed-actions') | raw }} {# button-container #}
-                            </summary>
-                            <div class="feed-actions-details-dropdown">
-                                <menu>
-                                    {% for block in handle_event('AddFeedActions', app.request, notes is defined and notes is not empty) %}
-                                        {{ block | raw }}
-                                    {% endfor %}
-                                </menu>
-                            </div>
-                        </details>
-                    </nav>
-                </header>
-
-                {% if notes is not empty %}
-                    {# Backwards compatibility with hAtom 0.1 #}
-                    <section class="feed h-feed hfeed notes" tabindex="0" role="feed">
-                        {% for conversation in notes %}
-                            {% block current_note %}
-                                {% if conversation is instanceof('array') %}
-                                    {{ noteView.macro_note(conversation['note'], conversation['replies']) }}
-                                {% else %}
-                                    {{ noteView.macro_note(conversation) }}
-                                {% endif %}
-                                <hr tabindex="0" title="{{ 'End of note and replies.' | trans }}">
-                            {% endblock current_note %}
-                        {% endfor %}
-                    </section>
-                {% else %}
-                    <section class="feed h-feed hfeed notes" tabindex="0" role="feed">
-                        <strong>{% trans %}No notes yet...{% endtrans %}</strong>
-                    </section>
-                {% endif %}
-            </article>
         {% endif %}
     {% endif %}
 {% endblock body %}

components/Group/tests/Entity/GroupTest.php

@@ -19,10 +19,12 @@ declare(strict_types = 1);
 // along with GNU social.  If not, see <http://www.gnu.org/licenses/>.
 // }}}
 
-namespace App\Tests\Entity;
+namespace Component\Group\tests\Entity;
 
 use App\Core\DB\DB;
+use App\Entity\Actor;
 use App\Util\GNUsocialTestCase;
+use Component\Group\Entity\LocalGroup;
 use Jchook\AssertThrows\AssertThrows;
 
 class GroupTest extends GNUsocialTestCase
@@ -31,8 +33,8 @@ class GroupTest extends GNUsocialTestCase
 
     public function testGetActor()
     {
-        $group = DB::findOneBy('local_group', ['nickname' => 'taken_group']);
+        $group = DB::findOneBy(LocalGroup::class, ['nickname' => 'taken_public_group']);
-        $actor = DB::findOneBy('actor', ['nickname' => 'taken_group']);
+        $actor = DB::findOneBy(Actor::class, ['nickname' => 'taken_public_group']);
-        static::assertSame($actor, $group->getActor());
+        static::assertObjectEquals($actor, $group->getActor());
     }
 }

components/Language/Controller/Language.php

@@ -134,7 +134,7 @@ class Language extends Controller
                 // Stay on same page, but force update and prevent resubmission
                 throw new RedirectException('settings_sort_languages');
             } else {
-                throw new RedirectException('settings', ['open' => 'account', '_fragment' => 'save_account_info_languages']);
+                throw new RedirectException('person_actor_settings', ['id' => $user->getId(), 'open' => 'settings-language-details', '_fragment' => 'settings-language-details']);
             }
         }
 

components/Language/Entity/Language.php

@@ -116,7 +116,7 @@ class Language extends Entity
         return Cache::getHashMapKey(
             map_key: 'languages-id',
             key: (string) $id,
-            calculate_map: fn () => F\reindex(DB::dql('select l from language l'), fn (self $l) => (string) $l->getId()),
+            calculate_map: fn () => F\reindex(DB::dql('SELECT l FROM \Component\Language\Entity\Language AS l'), fn (self $l) => (string) $l->getId()),
         );
     }
 
@@ -125,7 +125,7 @@ class Language extends Entity
         return Cache::getHashMapKey(
             'languages',
             $locale,
-            calculate_map: fn () => F\reindex(DB::dql('select l from language l'), fn (self $l) => $l->getLocale()),
+            calculate_map: fn () => F\reindex(DB::dql('SELECT l FROM \Component\Language\Entity\Language AS l'), fn (self $l) => $l->getLocale()),
         );
     }
 
@@ -138,7 +138,7 @@ class Language extends Entity
     {
         $langs = Cache::getHashMap(
             'languages',
-            fn () => F\reindex(DB::dql('select l from language l'), fn (self $l) => $l->getLocale()),
+            fn () => F\reindex(DB::dql('SELECT l FROM \Component\Language\Entity\Language AS l'), fn (self $l) => $l->getLocale()),
         );
 
         return array_merge(...F\map(array_values($langs), fn ($l) => $l->toChoiceFormat()));

components/Language/Language.php

@@ -105,11 +105,25 @@ class Language extends Component
 
     public function onCollectionQueryAddJoins(QueryBuilder &$note_qb, QueryBuilder &$actor_qb): bool
     {
-        $note_qb->leftJoin('Component\Language\Entity\Language', 'note_language', Expr\Join::WITH, 'note.language_id = note_language.id')
+        $note_aliases = $note_qb->getAllAliases();
-            ->leftJoin('Component\Language\Entity\ActorLanguage', 'actor_language', Expr\Join::WITH, 'note.actor_id = actor_language.actor_id')
+        if (!\in_array('note_language', $note_aliases)) {
-            ->leftJoin('Component\Language\Entity\Language', 'note_actor_language', Expr\Join::WITH, 'note_actor_language.id = actor_language.language_id');
+            $note_qb->leftJoin('Component\Language\Entity\Language', 'note_language', Expr\Join::WITH, 'note.language_id = note_language.id');
-        $actor_qb->leftJoin('Component\Language\Entity\ActorLanguage', 'actor_language', Expr\Join::WITH, 'actor.id = actor_language.actor_id')
+        }
-            ->leftJoin('Component\Language\Entity\Language', 'language', Expr\Join::WITH, 'actor_language.language_id = language.id');
+        if (!\in_array('actor_language', $note_aliases)) {
+            $note_qb->leftJoin('Component\Language\Entity\ActorLanguage', 'actor_language', Expr\Join::WITH, 'note.actor_id = actor_language.actor_id');
+        }
+        if (!\in_array('note_actor_language', $note_aliases)) {
+            $note_qb->leftJoin('Component\Language\Entity\Language', 'note_actor_language', Expr\Join::WITH, 'note_actor_language.id = actor_language.language_id');
+        }
+
+        $actor_aliases = $note_qb->getAllAliases();
+        if (!\in_array('actor_language', $actor_aliases)) {
+            $actor_qb->leftJoin('Component\Language\Entity\ActorLanguage', 'actor_language', Expr\Join::WITH, 'actor.id = actor_language.actor_id');
+        }
+        if (!\in_array('language', $actor_aliases)) {
+            $actor_qb->leftJoin('Component\Language\Entity\Language', 'language', Expr\Join::WITH, 'actor_language.language_id = language.id');
+        }
+
         return Event::next;
     }
 }

components/Language/templates/language/sort.html.twig

@@ -1,8 +1,8 @@
 {% extends 'base.html.twig' %}
 
 {% block body %}
-  <div class="frame-section frame-section-padding">
+    <div class="frame-section frame-section-padding">
-    <h3>{{ 'Put the languages in the order you\'d like to see them in your language selection dropdown, when posting' | trans}}</h3>
+        <h3>{% trans %}Put the languages in the order you'd like to see them in your language selection dropdown, when posting{% endtrans %}</h3>
-    {{ form(form) }}
+        {{ form(form) }}
-  </div>
+    </div>
 {% endblock %}

components/LeftPanel/LeftPanel.php

@@ -42,7 +42,12 @@ class LeftPanel extends Component
         return Event::next;
     }
 
-    public function onAppendFeed(Actor $actor, string $title, string $route, array $route_params)
+    /**
+     * @throws \App\Util\Exception\DuplicateFoundException
+     * @throws \App\Util\Exception\ServerException
+     * @throws ClientException
+     */
+    public function onAppendFeed(Actor $actor, string $title, string $route, array $route_params): bool
     {
         $cache_key = Feed::cacheKey($actor);
         $feeds     = Feed::getFeeds($actor);
@@ -67,17 +72,4 @@ class LeftPanel extends Component
             return Event::stop;
         }
     }
-
-    /**
-     * Output our dedicated stylesheet
-     *
-     * @param array $styles stylesheets path
-     *
-     * @return bool hook value; true means continue processing, false means stop
-     */
-    public function onEndShowStyles(array &$styles, string $route): bool
-    {
-        $styles[] = 'components/LeftPanel/assets/css/view.css';
-        return Event::next;
-    }
 }

components/LeftPanel/templates/left_panel/edit_feeds.html.twig

@@ -2,7 +2,7 @@
 
 {% block stylesheets %}
   {{ parent() }}
-  <link rel="stylesheet" href="{{ asset('assets/default_theme/css/pages/feeds.css') }}" type="text/css">
+  <link rel="stylesheet" href="{{ asset('assets/default_theme/pages/feeds.css') }}" type="text/css">
 {% endblock stylesheets %}
 
 {% macro edit_feeds_form_row(child) %}
@@ -15,7 +15,7 @@
 {% block body %}
   <div class="frame-section">
     <form class="section-form" action="{{ path('edit_feeds') }}" method="post">
-        <h1 class="frame-section-title">{{ "Edit feed navigation links" | trans }}</h1>
+        <h1 class="frame-section-title">{% trans %}Edit feed navigation links{% endtrans %}</h1>
         {# Since the form is not separated into individual groups, this happened #}
         {{ form_start(edit_feeds) }}
         {{ form_errors(edit_feeds) }}

components/LeftPanel/templates/left_panel/view.html.twig

@@ -1,24 +1,24 @@
 {% block leftpanel %}
 <label class="panel-left-icon" for="toggle-panel-left" tabindex="-1">{{ icon('menu', 'icon icon-left') | raw }}</label>
-<a id="anchor-left-panel" class="anchor-hidden" tabindex="0" title="{{ 'Press tab followed by a space to access left panel' | trans }}"></a>
+<a id="anchor-left-panel" class="anchor-hidden" tabindex="0" title="{% trans %}Press tab followed by a space to access left panel{% endtrans %}"></a>
-<input type="checkbox" id="toggle-panel-left" tabindex="0" title="{{ 'Open left panel' | trans }}">
+<input type="checkbox" id="toggle-panel-left" tabindex="0" title="{% trans %}Open left panel{% endtrans %}">
 
 <aside class="section-panel section-panel-left">
     <section class="panel-content accessibility-target">
         {% if app.user %}
-            <section class='frame-section frame-section-padding' title="{{ 'Your profile information.' | trans }}">
+            <section class='frame-section frame-section-padding' title="{% trans %}Your profile information{% endtrans %}">
-                {% block profile_view %}{% include 'cards/profile/view.html.twig' with { actor: current_actor } %}{% endblock profile_view %}
+                {% block profile_view %}{% include 'cards/blocks/profile.html.twig' with { actor: current_actor } %}{% endblock profile_view %}
-                {{ block("profile_current_actor", "cards/navigation/view.html.twig") }}
+                {{ block("profile_current_actor", "cards/blocks/navigation.html.twig") }}
             </section>
         {% else %}
             <section>
-                {{ block("profile_security", "cards/navigation/view.html.twig") }}
+                {{ block("profile_security", "cards/blocks/navigation.html.twig") }}
             </section>
         {% endif %}
 
-        {{ block("feeds", "cards/navigation/view.html.twig") }}
+        {{ block("feeds", "cards/blocks/navigation.html.twig") }}
 
-        {{ block("footer", "cards/navigation/view.html.twig") }}
+        {{ block("footer", "cards/blocks/navigation.html.twig") }}
     </section>
 </aside>
 {% endblock leftpanel %}

components/Link/Entity/Link.php

@@ -126,7 +126,7 @@ class Link extends Entity
     {
         if (Common::isValidHttpUrl($url)) {
             // If the URL is a local one, do not create a Link to it
-            if (parse_url($url, \PHP_URL_HOST) === $_ENV['SOCIAL_DOMAIN']) {
+            if (parse_url($url, \PHP_URL_HOST) === Common::config('site', 'server')) {
                 Log::warning("It was attempted to create a Link to a local location {$url}.");
                 // Forbidden
                 throw new InvalidArgumentException(message: "A Link can't point to a local location ({$url}), it must be a remote one", code: 400);

components/Link/Link.php

@@ -38,14 +38,17 @@ class Link extends Component
     /**
      * Extract URLs from $content and create the appropriate Link and NoteToLink entities
      */
-    public function onProcessNoteContent(Note $note, string $content): bool
+    public function onProcessNoteContent(Note $note, string $content, string $content_type, array $process_note_content_extra_args = []): bool
     {
+        $ignore = $process_note_content_extra_args['ignoreLinks'] ?? [];
         if (Common::config('attachments', 'process_links')) {
             $matched_urls = [];
-            // TODO: This solution to ignore mentions when content is in html is far from ideal
+            preg_match_all($this->getURLRegex(), $content, $matched_urls);
-            preg_match_all($this->getURLRegex(), preg_replace('#<a href="(.*?)" class="u-url mention">#', '', $content), $matched_urls);
             $matched_urls = array_unique($matched_urls[1]);
             foreach ($matched_urls as $match) {
+                if (\in_array($match, $ignore)) {
+                    continue;
+                }
                 try {
                     $link_id = Entity\Link::getOrCreate($match)->getId();
                     DB::persist(NoteToLink::create(['link_id' => $link_id, 'note_id' => $note->getId()]));

components/Notification/Controller/Feed.php

@@ -53,9 +53,9 @@ class Feed extends Controller
             WHERE n.id IN (
                 SELECT act.object_id FROM \App\Entity\Activity AS act
                     WHERE act.object_type = 'note' AND act.id IN
-                        (SELECT att.activity_id FROM \Component\Notification\Entity\Notification AS att WHERE att.target_id = :id)
+                        (SELECT att.activity_id FROM \Component\Notification\Entity\Notification AS att WHERE att.target_id = :target_id)
                 )
-            EOF, ['id' => $user->getId()]);
+            EOF, [':target_id' => $user->getId()]);
         return [
             '_template'     => 'collection/notes.html.twig',
             'page_title'    => _m('Notifications'),

components/Notification/Entity/Attention.php

@@ -0,0 +1,105 @@
+<?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 Component\Notification\Entity;
+
+use App\Core\Entity;
+
+/**
+ * Entity for object attentions
+ *
+ * An attention is a form of persistent notification.
+ * It exists together and for as long as the object it belongs to.
+ * Creating an attention requires creating a Notification.
+ *
+ * @category  DB
+ * @package   GNUsocial
+ *
+ * @author    Zach Copley <zach@status.net>
+ * @copyright 2010 StatusNet Inc.
+ * @author    Mikael Nordfeldth <mmn@hethane.se>
+ * @copyright 2009-2014 Free Software Foundation, Inc http://www.fsf.org
+ * @author    Diogo Peralta Cordeiro <@diogo.site>
+ * @copyright 2021-2022 Free Software Foundation, Inc http://www.fsf.org
+ * @license   https://www.gnu.org/licenses/agpl.html GNU AGPL v3 or later
+ */
+class Attention extends Entity
+{
+    // {{{ Autocode
+    // @codeCoverageIgnoreStart
+    private string $object_type;
+    private int $object_id;
+    private int $target_id;
+
+    public function setObjectType(string $object_type): self
+    {
+        $this->object_type = mb_substr($object_type, 0, 32);
+        return $this;
+    }
+
+    public function getObjectType(): string
+    {
+        return $this->object_type;
+    }
+
+    public function setObjectId(int $object_id): self
+    {
+        $this->object_id = $object_id;
+        return $this;
+    }
+
+    public function getObjectId(): int
+    {
+        return $this->object_id;
+    }
+
+    public function setTargetId(int $target_id): self
+    {
+        $this->target_id = $target_id;
+        return $this;
+    }
+
+    public function getTargetId(): int
+    {
+        return $this->target_id;
+    }
+
+    // @codeCoverageIgnoreEnd
+    // }}} Autocode
+
+    public static function schemaDef(): array
+    {
+        return [
+            'name'        => 'attention',
+            'description' => 'Attentions to actors (these are not mentions)',
+            'fields'      => [
+                'object_type' => ['type' => 'varchar',  'length' => 32,     'not null' => true, 'description' => 'the name of the table this object refers to'],
+                'object_id'   => ['type' => 'int',      'not null' => true, 'description' => 'id in the referenced table'],
+                'target_id'   => ['type' => 'int', 'foreign key' => true, 'target' => 'Actor.id', 'multiplicity' => 'one to one', 'not null' => true, 'description' => 'actor_id for feed receiver'],
+            ],
+            'primary key' => ['object_type', 'object_id', 'target_id'],
+            'indexes'     => [
+                'attention_object_id_idx' => ['object_id'],
+                'attention_target_id_idx' => ['target_id'],
+            ],
+        ];
+    }
+}

components/Notification/Entity/Notification.php

@@ -28,17 +28,17 @@ use App\Entity\Actor;
 use DateTimeInterface;
 
 /**
- * Entity for attentions
+ * Entity for Notifications
+ *
+ * A Notification when isolated is a form of transient notification.
+ * When together with a persistent form of notification such as attentions or mentions,
+ * it records that the target was notified - which avoids re-notifying upon objects reconstructions.
  *
  * @category  DB
  * @package   GNUsocial
  *
- * @author    Zach Copley <zach@status.net>
+ * @author    Diogo Peralta Cordeiro <@diogo.site>
- * @copyright 2010 StatusNet Inc.
+ * @copyright 2021-2022 Free Software Foundation, Inc http://www.fsf.org
- * @author    Mikael Nordfeldth <mmn@hethane.se>
- * @copyright 2009-2014 Free Software Foundation, Inc http://www.fsf.org
- * @author    Hugo Sales <hugo@hsal.es>
- * @copyright 2020-2021 Free Software Foundation, Inc http://www.fsf.org
  * @license   https://www.gnu.org/licenses/agpl.html GNU AGPL v3 or later
  */
 class Notification extends Entity
@@ -121,7 +121,7 @@ class Notification extends Entity
      */
     public static function getNotificationTargetIdsByActivity(int|Activity $activity_id): array
     {
-        $notifications = DB::findBy('notification', ['activity_id' => \is_int($activity_id) ? $activity_id : $activity_id->getId()]);
+        $notifications = DB::findBy(self::class, ['activity_id' => \is_int($activity_id) ? $activity_id : $activity_id->getId()]);
         $targets       = [];
         foreach ($notifications as $notification) {
             $targets[] = $notification->getTargetId();
@@ -131,7 +131,16 @@ class Notification extends Entity
 
     public function getNotificationTargetsByActivity(int|Activity $activity_id): array
     {
-        return DB::findBy('actor', ['id' => $this->getNotificationTargetIdsByActivity($activity_id)]);
+        return DB::findBy(Actor::class, ['id' => $this->getNotificationTargetIdsByActivity($activity_id)]);
+    }
+
+    public static function getAllActivitiesTargetedAtActor(Actor $actor): array
+    {
+        return DB::dql(<<<'EOF'
+            SELECT act FROM \App\Entity\Activity AS act
+                WHERE act.object_type = 'note' AND act.id IN
+                    (SELECT att.activity_id FROM \Component\Notification\Entity\Notification AS att WHERE att.target_id = :id)
+            EOF, ['id' => $actor->getId()]);
     }
 
     public static function schemaDef(): array

components/Notification/Notification.php

@@ -26,14 +26,17 @@ use App\Core\Event;
 use function App\Core\I18n\_m;
 use App\Core\Log;
 use App\Core\Modules\Component;
+use App\Core\Queue\Queue;
 use App\Core\Router\RouteLoader;
 use App\Core\Router\Router;
 use App\Entity\Activity;
 use App\Entity\Actor;
 use App\Entity\LocalUser;
+use App\Util\Exception\ServerException;
 use Component\FreeNetwork\FreeNetwork;
-use Component\Group\Entity\GroupInbox;
 use Component\Notification\Controller\Feed;
+use Exception;
+use Throwable;
 
 class Notification extends Component
 {
@@ -43,7 +46,10 @@ class Notification extends Component
         return Event::next;
     }
 
-    public function onCreateDefaultFeeds(int $actor_id, LocalUser $user, int &$ordering)
+    /**
+     * @throws ServerException
+     */
+    public function onCreateDefaultFeeds(int $actor_id, LocalUser $user, int &$ordering): bool
     {
         DB::persist(\App\Entity\Feed::create([
             'actor_id' => $actor_id,
@@ -56,58 +62,122 @@ class Notification extends Component
     }
 
     /**
-     * Enqueues a notification for an Actor (user or group) which means
+     * Enqueues a notification for an Actor (such as person or group) which means
      * it shows up in their home feed and such.
+     * WARNING: It's highly advisable to have flushed any relevant objects before triggering this event.
+     * OBSERVATION: $sender->getSubscribers() will always be pre-included, thus why $targets=[] is normal
+     *
+     * $targets should be of the shape:
+     * ['source' => (int|Actor)[]] // Prefer Actor whenever possible
+     * Example of $targets:
+     * [[42, $actor_alice, $actor_bob]] // Avoid repeating actors or ids
+     *
+     * @param Actor       $sender   The one responsible for this activity, take care not to include it in targets
+     * @param Activity    $activity The activity responsible for the object being given to known to targets
+     * @param array       $targets  attentions, Mentions, any other source
+     * @param null|string $reason   An optional reason explaining why this notification exists
      */
-    public function onNewNotification(Actor $sender, Activity $activity, array $ids_already_known = [], ?string $reason = null): bool
+    public function onNewNotification(Actor $sender, Activity $activity, array $targets = [], ?string $reason = null): bool
     {
-        $targets = $activity->getNotificationTargets(ids_already_known: $ids_already_known, sender_id: $sender->getId());
+        // Ensure targets are all actor objects and unique
-        $this->notify($sender, $activity, $targets, $reason);
+        $effective_targets = [];
+        foreach ($sender->getSubscribers() as $subscriber) {
+            $effective_targets[$subscriber->getId()] = $subscriber;
+        }
+        foreach ($targets as $target) {
+            if (\is_int($target)) {
+                $target_id     = $target;
+                $target_object = null;
+            } else {
+                $target_id     = $target->getId();
+                $target_object = $target;
+            }
+            if (!\array_key_exists(key: $target_id, array: $effective_targets)) {
+                $target_object ??= Actor::getById($target_id);
+                $effective_targets[$target_id] = $target_object;
+            }
+        }
+        unset($targets);
 
+        if (Event::handle('NewNotificationStart', [$sender, $activity, $effective_targets, $reason]) === Event::next) {
+            self::notify($sender, $activity, $effective_targets, $reason);
+        }
+
+        Event::handle('NewNotificationEnd', [$sender, $activity, $effective_targets, $reason]);
         return Event::next;
     }
 
+    public function onQueueNotificationLocal(Actor $sender, Activity $activity, Actor $target, ?string $reason, array &$retry_args): bool
+    {
+        // TODO: use https://symfony.com/doc/current/notifier.html
+        return Event::stop;
+    }
+
+    public function onQueueNotificationRemote(Actor $sender, Activity $activity, array $targets, ?string $reason, array &$retry_args): bool
+    {
+        if (FreeNetwork::notify($sender, $activity, $targets, $reason)) {
+            return Event::stop;
+        } else {
+            return Event::next;
+        }
+    }
+
     /**
-     * Bring given Activity to Targets's attention
+     * Bring given Activity to Targets' knowledge.
+     * This will flush a Notification to DB.
+     *
+     * @return true if successful, false otherwise
      */
-    public function notify(Actor $sender, Activity $activity, array $targets, ?string $reason = null): bool
+    public static function notify(Actor $sender, Activity $activity, array $targets, ?string $reason = null): bool
     {
         $remote_targets = [];
         foreach ($targets as $target) {
             if ($target->getIsLocal()) {
-                if ($target->isGroup()) {
+                if ($target->hasBlocked($author = $activity->getActor())) {
-                    // FIXME: Make sure we check (for both local and remote) users are in the groups they send to!
+                    Log::info("Not saving notification to actor {$target->getId()} from sender {$sender->getId()} because receiver blocked author {$author->getId()}.");
-                    DB::persist(GroupInbox::create([
+                    continue;
-                        'group_id'    => $target->getId(),
+                }
-                        'activity_id' => $activity->getId(),
+                if (Event::handle('NewNotificationShould', [$activity, $target]) === Event::next) {
-                    ]));
+                    if ($sender->getId() === $target->getId()
-                } else {
+                        || $activity->getActorId() === $target->getId()) {
-                    if ($target->hasBlocked($activity->getActor())) {
+                        // The target already knows about this, no need to bother with a notification
-                        Log::info("Not saving reply to actor {$target->getId()} from sender {$sender->getId()} because of a block.");
                         continue;
                     }
                 }
-                if (Event::handle('NewNotificationShould', [$activity, $target]) === Event::next) {
+                Queue::enqueue(
-                    // TODO: use https://symfony.com/doc/current/notifier.html
+                    payload: [$sender, $activity, $target, $reason],
-                    // XXX: Unideal as in failures the rollback will leave behind a false notification,
+                    queue: 'NotificationLocal',
-                    // but most notifications (all) require flushing the objects first
+                    priority: true,
-                    // Should be okay as long as implementors bear this in mind
+                );
-                    DB::wrapInTransaction(fn() => DB::persist(Entity\Notification::create([
-                        'activity_id' => $activity->getId(),
-                        'target_id'   => $target->getId(),
-                        'reason'      => $reason,
-                    ])));
-                }
             } else {
                 // We have no authority nor responsibility of notifying remote actors of a remote actor's doing
                 if ($sender->getIsLocal()) {
                     $remote_targets[] = $target;
                 }
             }
+            // XXX: Unideal as in failures the rollback will leave behind a false notification,
+            // but most notifications (all) require flushing the objects first
+            // Should be okay as long as implementations bear this in mind
+            try {
+                DB::wrapInTransaction(fn () => DB::persist(Entity\Notification::create([
+                    'activity_id' => $activity->getId(),
+                    'target_id'   => $target->getId(),
+                    'reason'      => $reason,
+                ])));
+            } catch (Exception|Throwable $e) {
+                // We do our best not to record duplicate notifications, but it's not insane that can happen
+                Log::error('It was attempted to record an invalid notification!', [$e]);
+            }
         }
 
-        FreeNetwork::notify($sender, $activity, $remote_targets, $reason);
+        if ($remote_targets !== []) {
+            Queue::enqueue(
+                payload: [$sender, $activity, $remote_targets, $reason],
+                queue: 'NotificationRemote',
+                priority: false,
+            );
+        }
 
-        return Event::next;
+        return true;
     }
 }

components/Person/Controller/PersonFeed.php

@@ -0,0 +1,87 @@
+<?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 Component\Person\Controller;
+
+use function App\Core\I18n\_m;
+use App\Core\Router\Router;
+use App\Entity\Actor;
+use App\Entity as E;
+use App\Entity\LocalUser;
+use App\Util\Exception\ClientException;
+use App\Util\Exception\ServerException;
+use App\Util\HTML\Heading;
+use Component\Collection\Util\Controller\FeedController;
+use Symfony\Component\HttpFoundation\Request;
+
+class PersonFeed extends FeedController
+{
+    /**
+     * @throws ClientException
+     * @throws ServerException
+     */
+    public function personViewId(Request $request, int $id): array
+    {
+        $person = Actor::getById($id);
+        if (\is_null($person) || !$person->isPerson()) {
+            throw new ClientException(_m('No such person.'), 404);
+        }
+        if ($person->getIsLocal()) {
+            return [
+                '_redirect' => Router::url('person_actor_view_nickname', ['nickname' => $person->getNickname()]),
+                'actor'     => $person,
+            ];
+        }
+        return $this->personView($request, $person);
+    }
+
+    /**
+     * View a group feed by its nickname
+     *
+     * @param string $nickname The group's nickname to be shown
+     *
+     * @throws ClientException
+     * @throws ServerException
+     */
+    public function personViewNickname(Request $request, string $nickname): array
+    {
+        $user = LocalUser::getByNickname($nickname);
+        if (\is_null($user)) {
+            throw new ClientException(_m('No such person.'), 404);
+        }
+        $person = Actor::getById($user->getId());
+        return $this->personView($request, $person);
+    }
+
+    public function personView(Request $request, Actor $person): array
+    {
+        return [
+            '_template'        => 'actor/view.html.twig',
+            'actor'            => $person,
+            'nickname'         => $person->getNickname(),
+            'notes'            => E\Note::getAllNotesByActor($person),
+            'page_title'       => _m('{nickname}\'s profile', ['{nickname}' => $person->getNickname()]),
+            'notes_feed_title' => (new Heading(level: 2, classes: ['section-title'], text: 'Notes by ' . $person->getNickname())),
+        ];
+    }
+}

components/Person/Controller/PersonSettings.php

@@ -33,7 +33,7 @@ declare(strict_types = 1);
  * @license   https://www.gnu.org/licenses/agpl.html GNU AGPL v3 or later
  */
 
-namespace App\Controller;
+namespace Component\Person\Controller;
 
 // {{{ Imports
 
@@ -43,9 +43,16 @@ use App\Core\Event;
 use App\Core\Form;
 use function App\Core\I18n\_m;
 use App\Core\Log;
+use App\Entity\Actor;
 use App\Util\Common;
 use App\Util\Exception\AuthenticationException;
+use App\Util\Exception\NicknameEmptyException;
+use App\Util\Exception\NicknameInvalidException;
+use App\Util\Exception\NicknameNotAllowedException;
+use App\Util\Exception\NicknameTakenException;
+use App\Util\Exception\NicknameTooLongException;
 use App\Util\Exception\NoLoggedInUser;
+use App\Util\Exception\RedirectException;
 use App\Util\Exception\ServerException;
 use App\Util\Form\ActorArrayTransformer;
 use App\Util\Form\ActorForms;
@@ -64,29 +71,32 @@ use Symfony\Component\HttpFoundation\Request;
 
 // }}} Imports
 
-class UserPanel extends Controller
+class PersonSettings extends Controller
 {
     /**
      * Return main settings page forms
      *
-     * @throws \App\Util\Exception\NicknameEmptyException
+     * @throws \App\Util\Exception\ClientException
-     * @throws \App\Util\Exception\NicknameInvalidException
+     * @throws \App\Util\Exception\NicknameException
-     * @throws \App\Util\Exception\NicknameNotAllowedException
-     * @throws \App\Util\Exception\NicknameTakenException
-     * @throws \App\Util\Exception\NicknameTooLongException
-     * @throws \App\Util\Exception\RedirectException
      * @throws \Doctrine\DBAL\Exception
      * @throws AuthenticationException
+     * @throws NicknameEmptyException
+     * @throws NicknameInvalidException
+     * @throws NicknameNotAllowedException
+     * @throws NicknameTakenException
+     * @throws NicknameTooLongException
      * @throws NoLoggedInUser
+     * @throws RedirectException
      * @throws ServerException
      */
     public function allSettings(Request $request, LanguageController $language): array
     {
         // Ensure the user is logged in and retrieve Actor object for given user
-        $user  = Common::ensureLoggedIn();
+        $user = Common::ensureLoggedIn();
-        $actor = $user->getActor();
+        // Must be persisted
+        $actor = DB::findOneBy(Actor::class, ['id' => $user->getId()]);
 
-        $personal_form            = ActorForms::personalInfo($request, $actor, $user);
+        $personal_form            = ActorForms::personalInfo(request: $request, scope: $actor, target: $actor);
         $email_form               = self::email($request);
         $password_form            = self::password($request);
         $notifications_form_array = self::notifications($request);
@@ -115,8 +125,22 @@ class UserPanel extends Controller
         // TODO Add support missing settings
 
         $form = Form::create([
-            ['outgoing_email', TextType::class, ['label' => _m('Outgoing email'), 'required' => false, 'help' => _m('Change the email we use to contact you')]],
+            ['outgoing_email_sanitized', TextType::class,
-            ['incoming_email', TextType::class, ['label' => _m('Incoming email'), 'required' => false, 'help' => _m('Change the email you use to contact us (for posting, for instance)')]],
+                [
+                    'label'    => _m('Outgoing email'),
+                    'required' => false,
+                    'help'     => _m('Change the email we use to contact you'),
+                    'data'     => $user->getOutgoingEmail() ?: '',
+                ],
+            ],
+            ['incoming_email_sanitized', TextType::class,
+                [
+                    'label'    => _m('Incoming email'),
+                    'required' => false,
+                    'help'     => _m('Change the email you use to contact us (for posting, for instance)'),
+                    'data'     => $user->getIncomingEmail() ?: '',
+                ],
+            ],
             ['save_email', SubmitType::class, ['label' => _m('Save email info')]],
         ]);
 

components/Person/Person.php

@@ -19,23 +19,21 @@ declare(strict_types = 1);
 // along with GNU social.  If not, see <http://www.gnu.org/licenses/>.
 // }}}
 
-namespace Component\RightPanel;
+namespace Component\Person;
 
 use App\Core\Event;
 use App\Core\Modules\Component;
+use App\Core\Router\RouteLoader;
+use App\Util\Nickname;
+use Component\Person\Controller as C;
 
-class RightPanel extends Component
+class Person extends Component
 {
-    /**
+    public function onAddRoute(RouteLoader $r): bool
-     * Output our dedicated stylesheet
-     *
-     * @param array $styles stylesheets path
-     *
-     * @return bool hook value; true means continue processing, false means stop
-     */
-    public function onEndShowStyles(array &$styles, string $route): bool
     {
-        $styles[] = 'components/RightPanel/assets/css/view.css';
+        $r->connect(id: 'person_actor_view_id', uri_path: '/person/{id<\d+>}', target: [C\PersonFeed::class, 'personViewId']);
+        $r->connect(id: 'person_actor_view_nickname', uri_path: '/@{nickname<' . Nickname::DISPLAY_FMT . '>}', target: [C\PersonFeed::class, 'personViewNickname'], options: ['is_system_path' => false]);
+        $r->connect(id: 'person_actor_settings', uri_path: '/person/{id<\d+>}/settings', target: [C\PersonSettings::class, 'allSettings']);
         return Event::next;
     }
 }

components/Person/tests/Controller/PersonSettingsTest.php

@@ -0,0 +1,165 @@
+<?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 Component\Person\tests\Controller;
+
+use App\Core\DB\DB;
+use App\Core\Router\Router;
+use App\Entity\LocalUser;
+use App\Util\GNUsocialTestCase;
+use Jchook\AssertThrows\AssertThrows;
+
+class PersonSettingsTest extends GNUsocialTestCase
+{
+    use AssertThrows;
+
+    /**
+     * @covers \App\Controller\PersonSettings::allSettings
+     * @covers \App\Controller\PersonSettings::personalInfo
+     */
+    public function testPersonalInfo()
+    {
+        $client = static::createClient();
+        $user   = DB::findOneBy(LocalUser::class, ['nickname' => 'form_personal_info_test_user']);
+        $client->loginUser($user);
+
+        $client->request('GET', Router::url('person_actor_settings', ['id' => $user->getId()]));
+        $this->assertResponseIsSuccessful();
+        $crawler = $client->submitForm('Save personal info', [
+            'save_personal_info[nickname]'  => 'form_test_user_new_nickname',
+            'save_personal_info[full_name]' => 'Form User',
+            'save_personal_info[homepage]'  => 'https://gnu.org',
+            'save_personal_info[bio]'       => 'I was born at a very young age',
+            'save_personal_info[location]'  => 'right here',
+            //            'save_personal_info[phone_number]' => '+351908555842', // from fakenumber.net
+        ]);
+        $changed_user = DB::findOneBy(LocalUser::class, ['id' => $user->getId()]);
+        $actor        = $changed_user->getActor();
+        static::assertSame('form_test_user_new_nickname', $changed_user->getNickname());
+        static::assertSame('form_test_user_new_nickname', $actor->getNickname());
+        static::assertSame('Form User', $actor->getFullName());
+        static::assertSame('https://gnu.org', $actor->getHomepage());
+        static::assertSame('I was born at a very young age', $actor->getBio());
+        static::assertSame('right here', $actor->getLocation());
+//        static::assertSame('908555842', $changed_user->getPhoneNumber()->getNationalNumber());
+    }
+
+    /**
+     * @covers \App\Controller\PersonSettings::account
+     * @covers \App\Controller\PersonSettings::allSettings
+     */
+    public function testEmail()
+    {
+        $client = static::createClient();
+        $user   = DB::findOneBy(LocalUser::class, ['nickname' => 'form_account_test_user']);
+        $client->loginUser($user);
+
+        $client->request('GET', Router::url('person_actor_settings', ['id' => $user->getId()]));
+        $this->assertResponseIsSuccessful();
+        $crawler = $client->submitForm('Save email info', [
+            'save_email[outgoing_email_sanitized]' => 'outgoing@provider.any',
+            'save_email[incoming_email_sanitized]' => 'incoming@provider.any',
+        ]);
+
+        $changed_user = DB::findOneBy(LocalUser::class, ['id' => $user->getId()]);
+        static::assertSame($changed_user->getOutgoingEmail(), 'outgoing@provider.any');
+        static::assertSame($changed_user->getIncomingEmail(), 'incoming@provider.any');
+    }
+
+    /**
+     * @covers \App\Controller\PersonSettings::account
+     * @covers \App\Controller\PersonSettings::allSettings
+     */
+    public function testCorrectPassword()
+    {
+        $client = static::createClient();
+        $user   = DB::findOneBy(LocalUser::class, ['nickname' => 'form_account_test_user']);
+        $client->loginUser($user);
+
+        $client->request('GET', Router::url('person_actor_settings', ['id' => $user->getId()]));
+        $this->assertResponseIsSuccessful();
+        $crawler = $client->submitForm('Save new password', [
+            'save_password[old_password]'     => 'foobar',
+            'save_password[password][first]'  => 'this is some test password',
+            'save_password[password][second]' => 'this is some test password',
+        ]);
+
+        $changed_user = DB::findOneBy(LocalUser::class, ['id' => $user->getId()]);
+        static::assertTrue($changed_user->checkPassword('this is some test password'));
+    }
+
+    /**
+     * @covers \App\Controller\PersonSettings::account
+     * @covers \App\Controller\PersonSettings::allSettings
+     */
+    public function testAccountWrongPassword()
+    {
+        $client = static::createClient();
+        $user   = DB::findOneBy(LocalUser::class, ['nickname' => 'form_account_test_user']);
+        $client->loginUser($user);
+
+        $client->request('GET', Router::url('person_actor_settings', ['id' => $user->getId()]));
+        $this->assertResponseIsSuccessful();
+        $crawler = $client->submitForm('Save new password', [
+            'save_password[old_password]'     => 'some wrong password',
+            'save_password[password][first]'  => 'this is some test password',
+            'save_password[password][second]' => 'this is some test password',
+        ]);
+        $this->assertResponseStatusCodeSame(500); // 401 in future
+        $this->assertSelectorTextContains('.stacktrace', 'AuthenticationException');
+    }
+
+    // TODO: First actually implement this functionality
+//    /**
+//     * @covers \App\Controller\PersonSettings::allSettings
+//     * @covers \App\Controller\PersonSettings::notifications
+//     */
+//    public function testNotifications()
+//    {
+//        $client = static::createClient();
+//        $user   = DB::findOneBy(LocalUser::class, ['nickname' => 'form_account_test_user']);
+//        $client->loginUser($user);
+//
+//        $client->request('GET', Router::url('person_actor_settings', ['id' => $user->getId()]));
+//        $this->assertResponseIsSuccessful();
+//        $crawler = $client->submitForm('Save notification settings for Email', [
+//            'save_email[activity_by_subscribed]' => false,
+//            'save_email[mention]'                => true,
+//            'save_email[reply]'                  => false,
+//            'save_email[subscription]'           => true,
+//            'save_email[favorite]'               => false,
+//            'save_email[nudge]'                  => true,
+//            'save_email[dm]'                     => false,
+//            'save_email[enable_posting]'         => true,
+//        ]);
+//        $settings = DB::findOneBy('user_notification_prefs', ['user_id' => $user->getId(), 'transport' => 'email']);
+//        static::assertSame($settings->getActivityBySubscribed(), false);
+//        static::assertSame($settings->getMention(), true);
+//        static::assertSame($settings->getReply(), false);
+//        static::assertSame($settings->getSubscription(), true);
+//        static::assertSame($settings->getFavorite(), false);
+//        static::assertSame($settings->getNudge(), true);
+//        static::assertSame($settings->getDm(), false);
+//        static::assertSame($settings->getEnablePosting(), true);
+//    }
+}

components/Posting/Controller/Posting.php

@@ -0,0 +1,69 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace Component\Posting\Controller;
+
+use App\Core;
+use App\Core\Controller;
+use App\Core\Event;
+use function App\Core\I18n\_m;
+use App\Core\VisibilityScope;
+use App\Util\Common;
+use App\Util\Exception\ClientException;
+use Component\Posting\Form;
+use Symfony\Component\HttpFoundation\File\Exception\FormSizeFileException;
+use Symfony\Component\HttpFoundation\RedirectResponse;
+use Symfony\Component\HttpFoundation\Request;
+
+class Posting extends Controller
+{
+    public function onPost(Request $request): RedirectResponse
+    {
+        $actor = Common::actor();
+        $form  = Form\Posting::create($request);
+
+        $form->handleRequest($request);
+        if ($form->isSubmitted()) {
+            try {
+                if ($form->isValid()) {
+                    $data = $form->getData();
+                    Event::handle('PostingModifyData', [$request, $actor, &$data, $form]);
+
+                    if (empty($data['content']) && empty($data['attachments'])) {
+                        // TODO Display error: At least one of `content` and `attachments` must be provided
+                        throw new ClientException(_m('You must enter content or provide at least one attachment to post a note.'));
+                    }
+
+                    if (\is_null(VisibilityScope::tryFrom($data['visibility']))) {
+                        throw new ClientException(_m('You have selected an impossible visibility.'));
+                    }
+
+                    $extra_args = [];
+                    Event::handle('AddExtraArgsToNoteContent', [$request, $actor, $data, &$extra_args, $form]);
+
+                    if (\array_key_exists('in', $data) && $data['in'] !== 'public') {
+                        $target = $data['in'];
+                    }
+
+                    \Component\Posting\Posting::storeLocalNote(
+                        actor: $actor,
+                        content: $data['content'],
+                        content_type: $data['content_type'],
+                        locale: $data['language'],
+                        scope: VisibilityScope::from($data['visibility']),
+                        attentions: isset($target) ? [$target] : [],
+                        reply_to: \array_key_exists('reply_to_id', $data) ? $data['reply_to_id'] : null,
+                        attachments: $data['attachments'],
+                        process_note_content_extra_args: $extra_args,
+                    );
+
+                    return Core\Form::forceRedirect($form, $request);
+                }
+            } catch (FormSizeFileException $e) {
+                throw new ClientException(_m('Invalid file size given.'), previous: $e);
+            }
+        }
+        throw new ClientException(_m('Invalid form submission.'));
+    }
+}

components/Posting/Form/Posting.php

@@ -0,0 +1,103 @@
+<?php
+
+declare(strict_types = 1);
+
+namespace Component\Posting\Form;
+
+use App\Core\ActorLocalRoles;
+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\Actor;
+use App\Util\Common;
+use App\Util\Form\FormFields;
+use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
+use Symfony\Component\Form\Extension\Core\Type\FileType;
+use Symfony\Component\Form\Extension\Core\Type\HiddenType;
+use Symfony\Component\Form\Extension\Core\Type\SubmitType;
+use Symfony\Component\Form\Extension\Core\Type\TextareaType;
+use Symfony\Component\HttpFoundation\Request;
+use Symfony\Component\Validator\Constraints\Length;
+
+class Posting
+{
+    public static function create(Request $request)
+    {
+        $actor = Common::actor();
+
+        $placeholder_strings = ['How are you feeling?', 'Have something to share?', 'How was your day?'];
+        Event::handle('PostingPlaceHolderString', [&$placeholder_strings]);
+        $placeholder = $placeholder_strings[array_rand($placeholder_strings)];
+
+        $initial_content = '';
+        Event::handle('PostingInitialContent', [&$initial_content]);
+
+        $available_content_types = [
+            _m('Plain Text') => 'text/plain',
+        ];
+        Event::handle('PostingAvailableContentTypes', [&$available_content_types]);
+
+        $in_targets = [];
+        Event::handle('PostingFillTargetChoices', [$request, $actor, &$in_targets]);
+
+        $context_actor = null;
+        Event::handle('PostingGetContextActor', [$request, $actor, &$context_actor]);
+
+        $form_params = [];
+        if (!empty($in_targets)) { // @phpstan-ignore-line
+            // Add "none" option to the first of choices
+            $in_targets = array_merge([_m('Public') => 'public'], $in_targets);
+            // Make the context actor the first In target option
+            if (!\is_null($context_actor)) {
+                foreach ($in_targets as $it_nick => $it_id) {
+                    if ($it_id === $context_actor->getId()) {
+                        unset($in_targets[$it_nick]);
+                        $in_targets = array_merge([$it_nick => $it_id], $in_targets);
+                        break;
+                    }
+                }
+            }
+            $form_params[] = ['in', ChoiceType::class, ['label' => _m('In:'), 'multiple' => false, 'expanded' => false, 'choices' => $in_targets]];
+        }
+
+        $visibility_options = [
+            _m('Public')    => VisibilityScope::EVERYWHERE->value,
+            _m('Local')     => VisibilityScope::LOCAL->value,
+            _m('Addressee') => VisibilityScope::ADDRESSEE->value,
+        ];
+        if (!\is_null($context_actor) && $context_actor->isGroup()) { // @phpstan-ignore-line currently an open bug. See https://web.archive.org/web/20220226131651/https://github.com/phpstan/phpstan/issues/6234
+            if ($actor->canModerate($context_actor)) {
+                if ($context_actor->getRoles() & ActorLocalRoles::PRIVATE_GROUP) {
+                    $visibility_options = array_merge([_m('Group') => VisibilityScope::GROUP->value], $visibility_options);
+                } else {
+                    $visibility_options[_m('Group')] = VisibilityScope::GROUP->value;
+                }
+            }
+        }
+        $form_params[] = ['visibility', ChoiceType::class, ['label' => _m('Visibility:'), 'multiple' => false, 'expanded' => false, 'choices' => $visibility_options]];
+
+        $form_params[] = ['content', TextareaType::class, ['label' => _m('Content:'), 'data' => $initial_content, 'attr' => ['placeholder' => _m($placeholder)], 'constraints' => [new Length(['max' => Common::config('site', 'text_limit')])]]];
+        $form_params[] = ['attachments', FileType::class, ['label' => _m('Attachments:'), 'multiple' => true, 'required' => false, 'invalid_message' => _m('Attachment not valid.')]];
+        $form_params[] = FormFields::language($actor, $context_actor, label: _m('Note language'), help: _m('The selected language will be federated and added as a lang attribute, preferred language can be set up in settings'));
+
+        if (\count($available_content_types) > 1) {
+            $form_params[] = ['content_type', ChoiceType::class,
+                [
+                    'label'   => _m('Text format:'), 'multiple' => false, 'expanded' => false,
+                    'data'    => $available_content_types[array_key_first($available_content_types)],
+                    'choices' => $available_content_types,
+                ],
+            ];
+        } else {
+            $form_params[] = ['content_type', HiddenType::class, ['data' => $available_content_types[array_key_first($available_content_types)]]];
+        }
+
+        Event::handle('PostingAddFormEntries', [$request, $actor, &$form_params]);
+
+        $form_params[] = ['post_note', SubmitType::class, ['label' => _m('Post')]];
+
+        return Form::create($form_params, form_options: ['action' => Router::url(\Component\Posting\Posting::route)]);
+    }
+}

components/Posting/Posting.php

@@ -23,12 +23,13 @@ declare(strict_types = 1);
 
 namespace Component\Posting;
 
+use App\Core\Cache;
 use App\Core\DB\DB;
 use App\Core\Event;
-use App\Core\Form;
 use App\Core\GSFile;
 use function App\Core\I18n\_m;
 use App\Core\Modules\Component;
+use App\Core\Router\RouteLoader;
 use App\Core\Router\Router;
 use App\Core\VisibilityScope;
 use App\Entity\Activity;
@@ -40,165 +41,127 @@ use App\Util\Exception\ClientException;
 use App\Util\Exception\DuplicateFoundException;
 use App\Util\Exception\RedirectException;
 use App\Util\Exception\ServerException;
-use App\Util\Form\FormFields;
 use App\Util\Formatting;
 use App\Util\HTML;
 use Component\Attachment\Entity\ActorToAttachment;
 use Component\Attachment\Entity\AttachmentToNote;
 use Component\Conversation\Conversation;
 use Component\Language\Entity\Language;
-use Functional as F;
+use Component\Notification\Entity\Attention;
-use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
-use Symfony\Component\Form\Extension\Core\Type\FileType;
-use Symfony\Component\Form\Extension\Core\Type\SubmitType;
-use Symfony\Component\Form\Extension\Core\Type\TextareaType;
-use Symfony\Component\HttpFoundation\File\Exception\FormSizeFileException;
 use Symfony\Component\HttpFoundation\File\UploadedFile;
 use Symfony\Component\HttpFoundation\Request;
-use Symfony\Component\Routing\Exception\ResourceNotFoundException;
-use Symfony\Component\Validator\Constraints\Length;
 
 class Posting extends Component
 {
+    public const route = 'posting_form_action';
+
+    public function onAddRoute(RouteLoader $r): bool
+    {
+        $r->connect(self::route, '/form/posting', Controller\Posting::class);
+        return Event::next;
+    }
+
     /**
      * HTML render event handler responsible for adding and handling
      * the result of adding the note submission form, only if a user is logged in
      *
+     * @throws BugFoundException
      * @throws ClientException
+     * @throws DuplicateFoundException
      * @throws RedirectException
      * @throws ServerException
      */
-    public function onAppendRightPostingBlock(Request $request, array &$res): bool
+    public function onAddMainRightPanelBlock(Request $request, array &$res): bool
     {
-        if (\is_null($user = Common::user())) {
+        if (\is_null($user = Common::user()) || preg_match('(feed|conversation|group|view)', $request->get('_route')) === 0) {
             return Event::next;
         }
 
-        $actor = $user->getActor();
+        $res['post_form'] = Form\Posting::create($request)->createView();
-
-        $placeholder_strings = ['How are you feeling?', 'Have something to share?', 'How was your day?'];
-        Event::handle('PostingPlaceHolderString', [&$placeholder_strings]);
-        $placeholder = $placeholder_strings[array_rand($placeholder_strings)];
-
-        $initial_content = '';
-        Event::handle('PostingInitialContent', [&$initial_content]);
-
-        $available_content_types = [
-            _m('Plain Text') => 'text/plain',
-        ];
-        Event::handle('PostingAvailableContentTypes', [&$available_content_types]);
-
-        $in_targets = [];
-        Event::handle('PostingFillTargetChoices', [$request, $actor, &$in_targets]);
-
-        $context_actor = null;
-        Event::handle('PostingGetContextActor', [$request, $actor, &$context_actor]);
-
-        $form_params = [];
-        if (!empty($in_targets)) { // @phpstan-ignore-line
-            // Add "none" option to the top of choices
-            $in_targets    = array_merge([_m('Public') => 'public'], $in_targets);
-            $form_params[] = ['in', ChoiceType::class, ['label' => _m('In:'), 'multiple' => false, 'expanded' => false, 'choices' => $in_targets]];
-        }
-
-        // TODO: if in group page, add GROUP visibility to the choices.
-        $form_params[] = ['visibility', ChoiceType::class, ['label' => _m('Visibility:'), 'multiple' => false, 'expanded' => false, 'data' => 'public', 'choices' => [
-            _m('Public')    => VisibilityScope::EVERYWHERE->value,
-            _m('Local')     => VisibilityScope::LOCAL->value,
-            _m('Addressee') => VisibilityScope::ADDRESSEE->value,
-        ]]];
-        $form_params[] = ['content', TextareaType::class, ['label' => _m('Content:'), 'data' => $initial_content, 'attr' => ['placeholder' => _m($placeholder)], 'constraints' => [new Length(['max' => Common::config('site', 'text_limit')])]]];
-        $form_params[] = ['attachments', FileType::class, ['label' => _m('Attachments:'), 'multiple' => true, 'required' => false, 'invalid_message' => _m('Attachment not valid.')]];
-        $form_params[] = FormFields::language($actor, $context_actor, label: _m('Note language'), help: _m('The selected language will be federated and added as a lang attribute, preferred language can be set up in settings'));
-
-        if (\count($available_content_types) > 1) {
-            $form_params[] = ['content_type', ChoiceType::class,
-                [
-                    'label'   => _m('Text format:'), 'multiple' => false, 'expanded' => false,
-                    'data'    => $available_content_types[array_key_first($available_content_types)],
-                    'choices' => $available_content_types,
-                ],
-            ];
-        }
-
-        Event::handle('PostingAddFormEntries', [$request, $actor, &$form_params]);
-
-        $form_params[] = ['post_note', SubmitType::class, ['label' => _m('Post')]];
-        $form          = Form::create($form_params);
-
-        $form->handleRequest($request);
-        if ($form->isSubmitted()) {
-            try {
-                if ($form->isValid()) {
-                    $data = $form->getData();
-                    Event::handle('PostingModifyData', [$request, $actor, &$data, $form_params, $form]);
-
-                    if (empty($data['content']) && empty($data['attachments'])) {
-                        // TODO Display error: At least one of `content` and `attachments` must be provided
-                        throw new ClientException(_m('You must enter content or provide at least one attachment to post a note.'));
-                    }
-
-                    if (\is_null(VisibilityScope::tryFrom($data['visibility']))) {
-                        throw new ClientException(_m('You have selected an impossible visibility.'));
-                    }
-
-                    $content_type = $data['content_type'] ?? $available_content_types[array_key_first($available_content_types)];
-                    $extra_args   = [];
-                    Event::handle('AddExtraArgsToNoteContent', [$request, $actor, $data, &$extra_args, $form_params, $form]);
-
-                    if (\array_key_exists('in', $data) && $data['in'] !== 'public') {
-                        $target = $data['in'];
-                    }
-
-                    self::storeLocalNote(
-                        actor: $user->getActor(),
-                        content: $data['content'],
-                        content_type: $content_type,
-                        locale: $data['language'],
-                        scope: VisibilityScope::from($data['visibility']),
-                        target: $target ?? null, // @phpstan-ignore-line
-                        reply_to_id: $data['reply_to_id'],
-                        attachments: $data['attachments'],
-                        process_note_content_extra_args: $extra_args,
-                    );
-
-                    try {
-                        if ($request->query->has('from')) {
-                            $from = $request->query->get('from');
-                            if (str_contains($from, '#')) {
-                                [$from, $fragment] = explode('#', $from);
-                            }
-                            Router::match($from);
-                            throw new RedirectException(url: $from . (isset($fragment) ? '#' . $fragment : ''));
-                        }
-                    } catch (ResourceNotFoundException $e) {
-                        // continue
-                    }
-                    throw new RedirectException();
-                }
-            } catch (FormSizeFileException $e) {
-                throw new ClientException(_m('Invalid file size given'), previous: $e);
-            }
-        }
-
-        $res['post_form'] = $form->createView();
 
         return Event::next;
     }
 
+    /**
+     * @throws ClientException
+     * @throws DuplicateFoundException
+     * @throws ServerException
+     */
+    public static function storeLocalPage(
+        Actor $actor,
+        ?string $content,
+        string $content_type,
+        ?string $locale = null,
+        ?VisibilityScope $scope = null,
+        array $attentions = [],
+        null|int|Note $reply_to = null,
+        array $attachments = [],
+        array $processed_attachments = [],
+        array $process_note_content_extra_args = [],
+        bool $flush_and_notify = true,
+        ?string $rendered = null,
+        string $source = 'web',
+        ?string $title = null,
+    ): array {
+        [$activity, $note, $effective_attentions] = self::storeLocalNote(
+            actor: $actor,
+            content: $content,
+            content_type: $content_type,
+            locale: $locale,
+            scope: $scope,
+            attentions: $attentions,
+            reply_to: $reply_to,
+            attachments: $attachments,
+            processed_attachments: $processed_attachments,
+            process_note_content_extra_args: $process_note_content_extra_args,
+            flush_and_notify: false,
+            rendered: $rendered,
+            source: $source,
+        );
+        $note->setType('page');
+        $note->setTitle($title);
+
+        if ($flush_and_notify) {
+            // Flush before notification
+            DB::flush();
+            Event::handle('NewNotification', [
+                $actor,
+                $activity,
+                $effective_attentions,
+                _m('Actor {actor_id} created page {note_id}.', [
+                    '{actor_id}' => $actor->getId(),
+                    '{note_id}'  => $activity->getObjectId(),
+                ]),
+            ]);
+        }
+
+        return [$activity, $note, $effective_attentions];
+    }
+
     /**
      * Store the given note with $content and $attachments, created by
      * $actor_id, possibly as a reply to note $reply_to and with flag
      * $is_local. Sanitizes $content and $attachments
      *
-     * @param array $attachments                     Array of UploadedFile to be stored as GSFiles associated to this note
+     * @param Actor                $actor                           The Actor responsible for the creation of this Note
-     * @param array $processed_attachments           Array of [Attachment, Attachment's name] to be associated to this $actor and Note
+     * @param null|string          $content                         The raw text content
-     * @param array $process_note_content_extra_args Extra arguments for the event ProcessNoteContent
+     * @param string               $content_type                    Indicating one of the various supported content format (Plain Text, Markdown, LaTeX...)
+     * @param null|string          $locale                          Note's written text language, set by the default Actor language or upon filling
+     * @param null|VisibilityScope $scope                           The visibility of this Note
+     * @param array                $attentions                      Actor|int[]: In Group/To Person or Bot, registers an attention between note and target
+     * @param null|int|Note        $reply_to                        The soon-to-be Note parent's id, if it's a Reply itself
+     * @param array                $attachments                     UploadedFile[] to be stored as GSFiles associated to this note
+     * @param array                $processed_attachments           Array of [Attachment, Attachment's name][] to be associated to this $actor and Note
+     * @param array                $process_note_content_extra_args Extra arguments for the event ProcessNoteContent
+     * @param bool                 $flush_and_notify                True if the newly created Note activity should be passed on as a Notification
+     * @param null|string          $rendered                        The Note's content post RenderNoteContent event, which sanitizes and processes the raw content sent
+     * @param string               $source                          The source of this Note
      *
-     * @throws BugFoundException
      * @throws ClientException
      * @throws DuplicateFoundException
      * @throws ServerException
+     *
+     * @return array [Activity, Note, Effective Attentions]
      */
     public static function storeLocalNote(
         Actor $actor,
@@ -206,17 +169,18 @@ class Posting extends Component
         string $content_type,
         ?string $locale = null,
         ?VisibilityScope $scope = null,
-        null|Actor|int $target = null,
+        array $attentions = [],
-        ?int $reply_to_id = null,
+        null|int|Note $reply_to = null,
         array $attachments = [],
         array $processed_attachments = [],
         array $process_note_content_extra_args = [],
-        bool $notify = true,
+        bool $flush_and_notify = true,
         ?string $rendered = null,
         string $source = 'web',
-    ): Note {
+    ): array {
         $scope ??= VisibilityScope::EVERYWHERE; // TODO: If site is private, default to LOCAL
-        $mentions = [];
+        $reply_to_id = \is_null($reply_to) ? null : (\is_int($reply_to) ? $reply_to : $reply_to->getId());
+        $mentions    = [];
         if (\is_null($rendered) && !empty($content)) {
             Event::handle('RenderNoteContent', [$content, $content_type, &$rendered, $actor, $locale, &$mentions]);
         }
@@ -246,6 +210,17 @@ class Posting extends Component
         }
 
         DB::persist($note);
+        Conversation::assignLocalConversation($note, $reply_to_id);
+
+        // Update replies cache
+        if (!\is_null($reply_to_id)) {
+            Cache::incr(Note::cacheKeys($reply_to_id)['replies-count']);
+            // Not having them cached doesn't mean replies don't exist, but don't push it to the
+            // list, as that means they need to be re-fetched, or some would be missed
+            if (Cache::exists(Note::cacheKeys($reply_to_id)['replies'])) {
+                Cache::listPushRight(Note::cacheKeys($reply_to_id)['replies'], $note);
+            }
+        }
 
         // Need file and note ids for the next step
         $note->setUrl(Router::url('note_view', ['id' => $note->getId()], Router::ABSOLUTE_URL));
@@ -253,18 +228,18 @@ class Posting extends Component
             Event::handle('ProcessNoteContent', [$note, $content, $content_type, $process_note_content_extra_args]);
         }
 
+        // These are note attachments now, and not just attachments, ensure these relations are respected
         if ($processed_attachments !== []) {
             foreach ($processed_attachments as [$a, $fname]) {
-                if (DB::count('actor_to_attachment', $args = ['attachment_id' => $a->getId(), 'actor_id' => $actor->getId()]) === 0) {
+                // Most attachments should already be associated with its author, but maybe it didn't make sense
+                //for this attachment, or it's simply a repost of an attachment by a different actor
+                if (DB::count(ActorToAttachment::class, $args = ['attachment_id' => $a->getId(), 'actor_id' => $actor->getId()]) === 0) {
                     DB::persist(ActorToAttachment::create($args));
                 }
                 DB::persist(AttachmentToNote::create(['attachment_id' => $a->getId(), 'note_id' => $note->getId(), 'title' => $fname]));
-                $a->livesIncrementAndGet();
             }
         }
 
-        Conversation::assignLocalConversation($note, $reply_to_id);
-
         $activity = Activity::create([
             'actor_id'    => $actor->getId(),
             'verb'        => 'create',
@@ -274,29 +249,54 @@ class Posting extends Component
         ]);
         DB::persist($activity);
 
-        if (!\is_null($target)) {
+        $effective_attentions = [];
-            $target     = \is_int($target) ? Actor::getById($target) : $target;
+        foreach ($attentions as $target) {
-            $mentions[] = [
+            if (\is_int($target)) {
-                'mentioned'       => [$target],
+                $target_id                        = $target;
-                'type'            => match ($target->getType()) {
+                $add                              = !\array_key_exists($target_id, $effective_attentions);
-                    Actor::PERSON => 'mention',
+                $effective_attentions[$target_id] = $target;
-                    Actor::GROUP  => 'group',
+            } else {
-                    default       => throw new ClientException(_m('Unknown target type give in \'In\' field: {target}', ['{target}' => $target?->getNickname() ?? '<null>'])),
+                $target_id = $target->getId();
-                },
+                if ($add = !\array_key_exists($target_id, $effective_attentions)) {
-                'text' => $target->getNickname(),
+                    $effective_attentions[$target_id] = $target_id;
-            ];
+                }
+            }
+            if ($add) {
+                DB::persist(Attention::create(['object_type' => Note::schemaName(), 'object_id' => $note->getId(), 'target_id' => $target_id]));
+            }
         }
 
-        $mention_ids = F\unique(F\flat_map($mentions, fn (array $m) => F\map($m['mentioned'] ?? [], fn (Actor $a) => $a->getId())));
+        foreach ($mentions as $m) {
-
+            foreach ($m['mentioned'] ?? [] as $mentioned) {
-        // Flush before notification
+                $target_id = $mentioned->getId();
-        DB::flush();
+                if (!\array_key_exists($target_id, $effective_attentions)) {
-
+                    DB::persist(Attention::create(['object_type' => Note::schemaName(), 'object_id' => $note->getId(), 'target_id' => $target_id]));
-        if ($notify) {
+                }
-            Event::handle('NewNotification', [$actor, $activity, ['object' => $mention_ids], _m('{nickname} created a note {note_id}.', ['{nickname}' => $actor->getNickname(), '{note_id}' => $activity->getObjectId()])]);
+                $effective_attentions[$target_id] = $mentioned;
+            }
         }
 
-        return $note;
+        foreach ($actor->getSubscribers() as $subscriber) {
+            $target_id = $subscriber->getId();
+            DB::persist(Attention::create(['object_type' => Activity::schemaName(), 'object_id' => $activity->getId(), 'target_id' => $target_id]));
+            $effective_attentions[$target_id] = $subscriber;
+        }
+
+        if ($flush_and_notify) {
+            // Flush before notification
+            DB::flush();
+            Event::handle('NewNotification', [
+                $actor,
+                $activity,
+                $effective_attentions,
+                _m('Actor {actor_id} created note {note_id}.', [
+                    '{actor_id}' => $actor->getId(),
+                    '{note_id}'  => $activity->getObjectId(),
+                ]),
+            ]);
+        }
+
+        return [$activity, $note, $effective_attentions];
     }
 
     public function onRenderNoteContent(string $content, string $content_type, ?string &$rendered, Actor $author, ?string $language = null, array &$mentions = [])

components/RightPanel/templates/right_panel/view.html.twig

@@ -1,65 +1,97 @@
-{% block rightpanel %}
+{% macro posting(form) %}
-<label class="panel-right-icon" for="toggle-panel-right" tabindex="-1">{{ icon('chevron-left', 'icon icon-right') | raw }}</label>
+    <section class="section-form">
-<a id="anchor-right-panel" class="anchor-hidden" tabindex="0" title="{{ 'Press tab followed by a space to access right panel' | trans }}"></a>
+        {{ form_start(form) }}
-<input type="checkbox" id="toggle-panel-right" tabindex="0" title="{{ 'Open right panel' | trans }}">
+        {{ form_errors(form) }}
-
+        {% if form.in is defined %}
-<aside class="section-panel section-panel-right">
+            {{ form_row(form.in) }}
-    <section class="panel-content accessibility-target">
-        {% set prepend_right_panel = handle_event('PrependRightPanel', request) %}
-        {% for widget in prepend_right_panel %}
-            {{ widget | raw }}
-        {% endfor %}
-
-        {% set current_path = app.request.get('_route') %}
-        {% set blocks = handle_event('AppendRightPostingBlock', request) %}
-        {% if blocks['post_form'] is defined %}
-            <section class="frame-section" title="{{ 'Create a new note.' | trans }}">
-                <details class="section-details-title" open="open"
-                         title="{{ 'Expand if you want to access more options.' | trans }}">
-                    <summary class="details-summary-title">
-                        <h2>
-                            {% set current_path = app.request.get('_route') %}
-                            {% if current_path == 'conversation_reply_to' %}
-                                {{ "Reply to note" | trans }}
-                            {% else %}
-                                {{ "Create a note" | trans }}
-                            {% endif %}
-                        </h2>
-                    </summary>
-
-                    <section class="section-form">
-                        {{ form_start(blocks['post_form']) }}
-                        {{ form_errors(blocks['post_form']) }}
-                        {% if blocks['post_form'].in is defined %}
-                            {{ form_row(blocks['post_form'].in) }}
-                        {% endif %}
-                        {{ form_row(blocks['post_form'].visibility) }}
-                        {{ form_row(blocks['post_form'].content_type) }}
-                        {{ form_row(blocks['post_form'].content) }}
-                        {{ form_row(blocks['post_form'].attachments) }}
-
-                        <details class="section-details-subtitle">
-                            <summary class="details-summary-subtitle">
-                                <strong>
-                                    {{ "Additional options" | trans }}
-                                </strong>
-                            </summary>
-                            <section class="section-form">
-                                {{ form_row(blocks['post_form'].language) }}
-                                {{ form_row(blocks['post_form'].tag_use_canonical) }}
-                            </section>
-                        </details>
-                        {{ form_rest(blocks['post_form']) }}
-                        {{ form_end(blocks['post_form']) }}
-                    </section>
-                </details>
-            </section>
         {% endif %}
+        {{ form_row(form.visibility) }}
+        {{ form_row(form.content_type) }}
+        {{ form_row(form.content) }}
+        {{ form_row(form.attachments) }}
 
-        {% set extra_blocks = get_right_panel_blocks({'path': current_path, 'request': app.request, 'vars': (right_panel_vars | default)}) %}
+        <details class="section-details-subtitle frame-section">
-        {% for block in extra_blocks %}
+            <summary class="details-summary-subtitle">
-            {{ block | raw }}
+                <strong>
-        {% endfor %}
+                    {% trans %}Additional options{% endtrans %}
+                </strong>
+            </summary>
+            <section class="section-form">
+                {{ form_row(form.language) }}
+                {{ form_row(form.tag_use_canonical) }}
+            </section>
+        </details>
+        {{ form_rest(form) }}
+        {{ form_end(form) }}
     </section>
-</aside>
+{% endmacro %}
+
+{% macro posting_section_vanilla(widget) %}
+    <section class="frame-section" title="{% trans %}Create a new note{% endtrans %}">
+        <details class="section-details-title" open="open"
+                 title="{% trans %}Expand if you want to access more options{% endtrans %}">
+            <summary class="details-summary-title">
+                <span>
+                    {% trans %}Create a note{% endtrans %}
+                </span>
+            </summary>
+
+            {% import _self as forms %}
+            {{ forms.posting(widget) }}
+        </details>
+    </section>
+{% endmacro %}
+
+{% macro posting_section_reply(widget, extra) %}
+    <section class="frame-section" title="{% trans %}Create a new note{% endtrans %}">
+        <details class="section-details-title" open="open"
+                 title="{% trans %}Expand if you want to access more options{% endtrans %}">
+            <summary class="details-summary-title">
+                <span>
+                    {% trans %}Reply to note{% endtrans %}
+                </span>
+            </summary>
+
+            {% for block in extra %}
+                <section class="posting-extra">
+                    {{ block | raw }}
+                </section>
+            {% endfor %}
+
+            {% import _self as forms %}
+            {{ forms.posting(widget) }}
+        </details>
+    </section>
+{% endmacro %}
+
+{% block rightpanel %}
+    {% import _self as this %}
+    <label class="panel-right-icon" for="toggle-panel-right"
+           tabindex="-1">{{ icon('chevron-left', 'icon icon-right') | raw }}</label>
+    <a id="anchor-right-panel" class="anchor-hidden" tabindex="0"
+       title="{% trans %}Press tab followed by a space to access right panel{% endtrans %}"></a>
+       <input type="checkbox" id="toggle-panel-right" tabindex="0" title="{% trans %}Open right panel{% endtrans %}"
+           {% if app.request.get('_route') == 'conversation_reply_to' %}checked{% endif %}>
+
+    <aside class="section-panel section-panel-right">
+        {% set var_list = {'path': app.request.get('_route'), 'request': app.request, 'vars': right_panel_vars | default } %}
+        {% set blocks = add_right_panel_block('prepend', var_list) %}
+        {% set blocks = blocks|merge(add_right_panel_block('main', var_list)) %}
+        {% set blocks = blocks|merge(add_right_panel_block('append', var_list)) %}
+
+        <section class="panel-content accessibility-target">
+            {% for widget in blocks %}
+                {% if widget is iterable and widget.vars.id == 'post_note' %}
+                    {% if app.request.get('_route') == 'conversation_reply_to' %}
+                        {% set extra = handle_event('PrependPostingForm', request) %}
+                        {{ this.posting_section_reply(widget, extra) }}
+                    {% else %}
+                        {{ this.posting_section_vanilla(widget) }}
+                    {% endif %}
+                {% else %}
+                    {{ widget | raw }}
+                {% endif %}
+            {% endfor %}
+        </section>
+    </aside>
 {% endblock rightpanel %}

components/Search/Controller/Search.php

@@ -30,6 +30,7 @@ use App\Util\Exception\BugFoundException;
 use App\Util\Exception\RedirectException;
 use App\Util\Form\FormFields;
 use App\Util\Formatting;
+use App\Util\HTML\Heading;
 use Component\Collection\Util\Controller\FeedController;
 use Component\Search as Comp;
 use Symfony\Component\Form\Extension\Core\Type\CheckboxType;
@@ -135,6 +136,8 @@ class Search extends FeedController
             'search_form'         => Comp\Search::searchForm($request, query: $q, add_subscribe: !\is_null($actor)),
             'search_builder_form' => $search_builder_form->createView(),
             'notes'               => $notes ?? [],
+            'notes_feed_title'    => (new Heading(level: 3, classes: ['section-title'], text: 'Notes found')),
+            'actors_feed_title'   => (new Heading(level: 3, classes: ['section-title'], text: 'Actors found')),
             'actors'              => $actors ?? [],
             'page'                => 1, // TODO paginate
         ];

components/Search/Search.php

@@ -28,6 +28,7 @@ use App\Core\Form;
 use function App\Core\I18n\_m;
 use App\Core\Modules\Component;
 use App\Util\Common;
+use App\Util\Exception\ClientException;
 use App\Util\Exception\RedirectException;
 use App\Util\Formatting;
 use Symfony\Component\Form\Extension\Core\Type\SubmitType;
@@ -63,13 +64,22 @@ class Search extends Component
 
         if ($add_subscribe) {
             $form_definition[] = [
-                'title', TextType::class, ['label' => _m('Title'), 'required' => false, 'attr' => ['title' => _m('Title for this new feed in your left panel')]],
+                'title', TextType::class,
+                [
+                    'label'    => _m('Subscribe to search query'),
+                    'help'     => _m('By subscribing to a search query, a new feed link will be added to left panel\'s feed navigation menu'),
+                    'required' => false,
+                    'attr'     => [
+                        'title'       => _m('Title for this new feed in your left panel'),
+                        'placeholder' => _m('Input desired title...'),
+                    ],
+                ],
             ];
             $form_definition[] = [
                 'subscribe_to_search',
                 SubmitType::class,
                 [
-                    'label' => _m('Subscribe to this search'),
+                    'label' => _m('Subscribe'),
                     'attr'  => [
                         'title' => _m('Add this search as a feed in your feeds section of the left panel'),
                     ],
@@ -100,8 +110,11 @@ class Search extends Component
                     /** @var SubmitButton $subscribe */
                     $subscribe = $form->get('subscribe_to_search');
                     if ($subscribe->isClicked()) {
-                        // TODO ensure title is set
+                        if (!\is_null($data['title'])) {
-                        Event::handle('AppendFeed', [$actor, $data['title'], 'search', ['q' => $data['search_query']]]);
+                            Event::handle('AppendFeed', [$actor, $data['title'], 'search', ['q' => $data['search_query']]]);
+                        } else {
+                            throw new ClientException(_m('Empty title is not allowed.'));
+                        }
                         $redirect = true;
                     }
                 }
@@ -120,7 +133,7 @@ class Search extends Component
      *
      * @throws RedirectException
      */
-    public function onPrependRightPanel(Request $request, array &$elements)
+    public function onPrependRightPanelBlock(Request $request, array &$elements): bool
     {
         $elements[] = Formatting::twigRenderFile('cards/search/view.html.twig', ['search' => self::searchForm($request)]);
         return Event::next;

components/Search/templates/cards/search/view.html.twig

@@ -1,4 +1,4 @@
-<section class="section-form form-search" title="{{ 'Search for notes, actors, and beyond' | trans }}">
+<section class="section-form form-search" title="{% trans %}Search for notes, actors, and beyond{% endtrans %}">
 {{ form_start(search) }}
     <span>{{ form_row(search.search_query) }}{{ form_row(search.submit_search) }}</span>
     {{ form_rest(search) }}

components/Search/templates/search/view.html.twig

@@ -1,119 +1,130 @@
 {% extends 'collection/notes.html.twig' %}
 
+{% block search_query_simple %}
+<section>
+    <h1 class="section-title">{% trans %}Search{% endtrans %}</h1>
+
+    {{ form_start(search_form) }}
+    {{ form_errors(search_form) }}
+    {{ form_row(search_form.search_query) }}
+    {% if actor is not null %}
+        <details class="section-details-subtitle frame-section">
+            <summary class="details-summary-subtitle">
+                <strong>
+                    {% trans %}Extra options{% endtrans %}
+                </strong>
+            </summary>
+            <div class="section-form">
+                {{ form_row(search_form.title) }}
+                {{ form_row(search_form.subscribe_to_search) }}
+            </div>
+        </details>
+    {% endif %}
+    {{ form_row(search_form.submit_search) }}
+    {{ form_rest(search_form) }}
+    {{ form_end(search_form)}}
+</section>
+{% endblock search_query_simple %}
+
+{% block search_query_advanced %}
+    {{ form_start(search_builder_form) }}
+    <details class="section-details section-details-title frame-section">
+        <summary class="details-summary-title">
+            <span>{% trans %}Advanced search{% endtrans %}</span>
+        </summary>
+
+        <section class="frame-section-padding">
+            <details class="section-details-subtitle frame-section">
+                <summary class="details-summary-subtitle">
+                    <strong>{% trans %}People search options{% endtrans %}</strong>
+                </summary>
+                <div class="section-form">
+                    <div class="section-checkbox-flex">
+                        {{ form_row(search_builder_form.include_actors) }}
+                        {{ form_row(search_builder_form.include_actors_people) }}
+                        {{ form_row(search_builder_form.include_actors_groups) }}
+                        {{ form_row(search_builder_form.include_actors_lists) }}
+                        {{ form_row(search_builder_form.include_actors_businesses) }}
+                        {{ form_row(search_builder_form.include_actors_organizations) }}
+                        {{ form_row(search_builder_form.include_actors_bots) }}
+                    </div>
+                    <hr>
+                    {{ form_row(search_builder_form.actor_tags) }}
+                    <hr>
+                    {{ form_row(search_builder_form.actor_langs) }}
+                </div>
+            </details>
+
+            <details class="section-details-subtitle frame-section">
+                <summary class="details-summary-subtitle">
+                    <strong>{% trans %}Note search options{% endtrans %}</strong>
+                </summary>
+                <div class="section-form">
+                    <div class="section-checkbox-flex">
+                        {{ form_row(search_builder_form.include_notes) }}
+                        {{ form_row(search_builder_form.include_notes_text) }}
+                        {{ form_row(search_builder_form.include_notes_media) }}
+                        {{ form_row(search_builder_form.include_notes_polls) }}
+                        {{ form_row(search_builder_form.include_notes_bookmarks) }}
+                    </div>
+                    <hr>
+                    {{ form_row(search_builder_form.note_tags) }}
+                    <hr>
+                    {{ form_row(search_builder_form.note_langs) }}
+                    <hr>
+                    {{ form_row(search_builder_form.note_actor_tags) }}
+                    <hr>
+                    {{ form_row(search_builder_form.note_actor_langs) }}
+                </div>
+            </details>
+
+            {{ form_rest(search_builder_form) }}
+        </section>
+    </details>
+    {{ form_end(search_builder_form) }}
+{% endblock search_query_advanced %}
+
+{% block search %}
+    <section class="frame-section frame-section-padding">
+        {% if error is defined %}
+            <label class="alert alert-danger">
+                {{ error.getMessage() }}
+            </label>
+        {% endif %}
+        {{ block('search_query_simple') }}
+        <hr>
+        {{ block('search_query_advanced') }}
+    </section>
+{% endblock search %}
 
 {% block body %}
-    {% if error is defined %}
+    {{ block('search') }}
-        <label class="alert alert-danger">
+    <div class="frame-section frame-section-padding">
-            {{ error.getMessage() }}
+        <h1 class="section-title">{% trans %}Results{% endtrans %}</h1>
-        </label>
-    {% endif %}
 
-    <section class="frame-section frame-section-padding">
+        <section>
-        <h2>{% trans %}Search{% endtrans %}</h2>
-
-        {{ form_start(search_form) }}
-        <section class="frame-section section-form">
-            {{ form_errors(search_form) }}
-            {{ form_row(search_form.search_query) }}
-            {% if actor is not null %}
-                <details class="section-details-subtitle">
-                    <summary class="details-summary-subtitle">
-                        <strong>{% trans %}Other options{% endtrans %}</strong>
-                    </summary>
-
-                    <div class="section-form">
-                        <details class="section-details-subtitle">
-                            <summary class="details-summary-subtitle">
-                                <strong>
-                                    {% trans %}Save query as a feed{% endtrans %}
-                                </strong>
-                            </summary>
-                            <div class="section-form">
-                                {{ form_row(search_form.title) }}
-                                {{ form_row(search_form.subscribe_to_search) }}
-                            </div>
-                            <hr>
-                        </details>
-                    </div>
-                </details>
-            {% endif %}
-            {{ form_row(search_form.submit_search) }}
-        </section>
-        {{ form_end(search_form)}}
-
-        <section class="frame-section">
-            <details class="section-details-subtitle">
-                <summary class="details-summary-subtitle">
-                    <strong>{% trans %}Build a search query{% endtrans %}</strong>
-                </summary>
-
-                {{ form_start(search_builder_form) }}
-                <div class="section-form">
-                    {# actor options, display if first checked, with checkbox trick #}
-                    <details class="section-details-subtitle">
-                        <summary class="details-summary-subtitle">
-                            <strong>{% trans %}People search options{% endtrans %}</strong>
-                        </summary>
-                        <div class="section-form">
-                            {{ form_row(search_builder_form.include_actors) }}
-                            {{ form_row(search_builder_form.include_actors_people) }}
-                            {{ form_row(search_builder_form.include_actors_groups) }}
-                            {{ form_row(search_builder_form.include_actors_lists) }}
-                            {{ form_row(search_builder_form.include_actors_businesses) }}
-                            {{ form_row(search_builder_form.include_actors_organizations) }}
-                            {{ form_row(search_builder_form.include_actors_bots) }}
-                            {{ form_row(search_builder_form.actor_langs) }}
-                            {{ form_row(search_builder_form.actor_tags) }}
-                        </div>
-                        <hr>
-                    </details>
-
-                    <details class="section-details-subtitle">
-                        <summary class="details-summary-subtitle">
-                            <strong>{% trans %}Note search options{% endtrans %}</strong>
-                        </summary>
-                        <div class="section-form">
-                            {{ form_row(search_builder_form.include_notes) }}
-                            {{ form_row(search_builder_form.include_notes_text) }}
-                            {{ form_row(search_builder_form.include_notes_media) }}
-                            {{ form_row(search_builder_form.include_notes_polls) }}
-                            {{ form_row(search_builder_form.include_notes_bookmarks) }}
-                            {{ form_row(search_builder_form.note_langs) }}
-                            {{ form_row(search_builder_form.note_tags) }}
-                            {{ form_row(search_builder_form.note_actor_langs) }}
-                            {{ form_row(search_builder_form.note_actor_tags) }}
-                        </div>
-                        <hr>
-                    </details>
-                </div>
-                {{ form_end(search_builder_form) }}
-            </details>
-        </section>
-    </section>
-
-    <section class="frame-section frame-section-padding">
-        <h2>{% trans %}Results{% endtrans %}</h2>
-        <div class="frame-section frame-section-padding feed-empty">
             {% if notes is defined and notes is not empty %}
                 {{ parent() }}
             {% else %}
                 <h3>{% trans %}No notes found{% endtrans %}</h3>
                 <em>{% trans %}No notes were found for the specified query...{% endtrans %}</em>
             {% endif %}
-        </div>
+        </section>
-
+        <hr>
-        <div class="frame-section frame-section-padding feed-empty">
+        <section>
             <h3>{% trans %}Actors found{% endtrans %}</h3>
             {% if actors is defined and actors is not empty %}
                 {% for actor in actors %}
-                    {% include 'cards/profile/view.html.twig' with {'actor': actor} %}
+                    {% include 'cards/blocks/profile.html.twig' with {'actor': actor} %}
                 {% endfor %}
             {% else %}
                 <em>{% trans %}No Actors were found for the specified query...{% endtrans %}</em>
             {% endif %}
-        </div>
+        </section>
-    </section>
 
-    {{ "Page: " ~ page }}
+        <div class="frame-section-button-like">
+            {{ "Page: " ~ page }}
+        </div>
+    </div>
 {% endblock body %}
 

components/Subscription/Controller/Subscribers.php

@@ -32,7 +32,7 @@ use App\Entity\Actor;
 use App\Util\Common;
 use App\Util\Exception\ClientException;
 use App\Util\Exception\RedirectException;
-use Component\Collection\Util\ActorControllerTrait;
+use App\Util\Exception\ServerException;
 use Component\Collection\Util\Controller\CircleController;
 use Component\Subscription\Subscription as SubscriptionComponent;
 use Symfony\Component\Form\Extension\Core\Type\SubmitType;
@@ -43,30 +43,32 @@ use Symfony\Component\HttpFoundation\Request;
  */
 class Subscribers extends CircleController
 {
-    use ActorControllerTrait;
+    /**
-    public function subscribersByActorId(Request $request, int $id)
+     * @throws ServerException
+     */
+    public function subscribersByActor(Request $request, Actor $actor): array
     {
-        return $this->handleActorById(
+        return [
-            $id,
+            '_template'        => 'collection/actors.html.twig',
-            fn ($actor) => [
+            'title'            => _m('Subscribers'),
-                'actor' => $actor,
+            'empty_message'    => _m('No subscribers.'),
-            ],
+            'sort_form_fields' => [],
-        );
+            'page'             => $this->int('page') ?? 1,
+            'actors'           => $actor->getSubscribers(),
+        ];
     }
 
-    public function subscribersByActorNickname(Request $request, string $nickname)
+    /**
+     * @throws ClientException
+     * @throws ServerException
+     */
+    public function subscribersByActorId(Request $request, int $id): array
     {
-        return $this->handleActorByNickname(
+        $actor = Actor::getById($id);
-            $nickname,
+        if (\is_null($actor)) {
-            fn ($actor) => [
+            throw new ClientException(_m('No such actor.'), 404);
-                '_template'        => 'collection/actors.html.twig',
+        }
-                'title'            => _m('Subscribers'),
+        return $this->subscribersByActor($request, $actor);
-                'empty_message'    => _m('No subscribers.'),
-                'sort_form_fields' => [],
-                'page'             => $this->int('page') ?? 1,
-                'actors'           => $actor->getSubscribers(),
-            ],
-        );
     }
 
     /**

components/Subscription/Controller/Subscriptions.php

@@ -24,7 +24,9 @@ declare(strict_types = 1);
 namespace Component\Subscription\Controller;
 
 use function App\Core\I18n\_m;
-use Component\Collection\Util\ActorControllerTrait;
+use App\Entity\Actor;
+use App\Util\Exception\ClientException;
+use App\Util\Exception\ServerException;
 use Component\Collection\Util\Controller\CircleController;
 use Symfony\Component\HttpFoundation\Request;
 
@@ -33,29 +35,28 @@ use Symfony\Component\HttpFoundation\Request;
  */
 class Subscriptions extends CircleController
 {
-    use ActorControllerTrait;
+    /**
-    public function subscriptionsByActorId(Request $request, int $id)
+     * @throws ClientException
+     * @throws ServerException
+     */
+    public function subscriptionsByActorId(Request $request, int $id): array
     {
-        return $this->handleActorById(
+        $actor = Actor::getById($id);
-            $id,
+        if (\is_null($actor)) {
-            fn ($actor) => [
+            throw new ClientException(_m('No such actor.'), 404);
-                'actor' => $actor,
+        }
-            ],
+        return $this->subscriptionsByActor($request, $actor);
-        );
     }
 
-    public function subscriptionsByActorNickname(Request $request, string $nickname)
+    public function subscriptionsByActor(Request $request, Actor $actor)
     {
-        return $this->handleActorByNickname(
+        return [
-            $nickname,
+            '_template'        => 'collection/actors.html.twig',
-            fn ($actor) => [
+            'title'            => _m('Subscriptions'),
-                '_template'        => 'collection/actors.html.twig',
+            'empty_message'    => _m('Haven\'t subscribed anyone.'),
-                'title'            => _m('Subscriptions'),
+            'sort_form_fields' => [],
-                'empty_message'    => _m('Haven\'t subscribed anyone.'),
+            'page'             => $this->int('page') ?? 1,
-                'sort_form_fields' => [],
+            'actors'           => $actor->getSubscribers(),
-                'page'             => $this->int('page') ?? 1,
+        ];
-                'actors'           => $actor->getSubscribers(),
-            ],
-        );
     }
 }

components/Subscription/Entity/ActorSubscription.php

@@ -114,27 +114,6 @@ class ActorSubscription extends Entity
         ];
     }
 
-    /**
-     * @see Entity->getNotificationTargetIds
-     */
-    public function getNotificationTargetIds(array $ids_already_known = [], ?int $sender_id = null, bool $include_additional = true): array
-    {
-        if (!\array_key_exists('object', $ids_already_known)) {
-            $target_ids = [$this->getSubscribedId()]; // The object of any subscription is the one subscribed (or unsubscribed)
-        } else {
-            $target_ids = $ids_already_known['object'];
-        }
-
-        // Additional actors that should know about this
-        if ($include_additional && \array_key_exists('additional', $ids_already_known)) {
-            array_push($target_ids, ...$ids_already_known['additional']);
-        } else {
-            return $target_ids;
-        }
-
-        return array_unique($target_ids);
-    }
-
     public static function schemaDef(): array
     {
         return [

components/Subscription/Subscription.php

@@ -37,7 +37,7 @@ use App\Util\Common;
 use App\Util\Exception\DuplicateFoundException;
 use App\Util\Exception\NotFoundException;
 use App\Util\Exception\ServerException;
-use App\Util\Nickname;
+use Component\Notification\Entity\Attention;
 use Component\Subscription\Controller\Subscribers as SubscribersController;
 use Component\Subscription\Controller\Subscriptions as SubscriptionsController;
 
@@ -50,9 +50,7 @@ class Subscription extends Component
         $r->connect(id: 'actor_subscribe_add', uri_path: '/actor/subscribe/{object_id<\d+>}', target: [SubscribersController::class, 'subscribersAdd']);
         $r->connect(id: 'actor_subscribe_remove', uri_path: '/actor/unsubscribe/{object_id<\d+>}', target: [SubscribersController::class, 'subscribersRemove']);
         $r->connect(id: 'actor_subscriptions_id', uri_path: '/actor/{id<\d+>}/subscriptions', target: [SubscriptionsController::class, 'subscriptionsByActorId']);
-        $r->connect(id: 'actor_subscriptions_nickname', uri_path: '/@{nickname<' . Nickname::DISPLAY_FMT . '>}/subscriptions', target: [SubscriptionsController::class, 'subscriptionsByActorNickname']);
         $r->connect(id: 'actor_subscribers_id', uri_path: '/actor/{id<\d+>}/subscribers', target: [SubscribersController::class, 'subscribersByActorId']);
-        $r->connect(id: 'actor_subscribers_nickname', uri_path: '/@{nickname<' . Nickname::DISPLAY_FMT . '>}/subscribers', target: [SubscribersController::class, 'subscribersByActorNickname']);
         return Event::next;
     }
 
@@ -70,7 +68,7 @@ class Subscription extends Component
         $cache_subscriber = Cache::delete(Actor::cacheKeys($subscriber_id)['subscribed']);
         $cache_subscribed = Cache::delete(Actor::cacheKeys($subscribed_id)['subscribers']);
 
-        return [$cache_subscriber,$cache_subscribed];
+        return [$cache_subscriber, $cache_subscribed];
     }
 
     /**
@@ -100,20 +98,21 @@ class Subscription extends Component
         $subscription = DB::findOneBy(table: Entity\ActorSubscription::class, criteria: $opts, return_null: true);
         $activity     = null;
         if (\is_null($subscription)) {
-            DB::persist(Entity\ActorSubscription::create($opts));
+            DB::persist($subscription = Entity\ActorSubscription::create($opts));
             $activity = Activity::create([
                 'actor_id'    => $subscriber_id,
                 'verb'        => 'subscribe',
-                'object_type' => 'actor',
+                'object_type' => Actor::schemaName(),
                 'object_id'   => $subscribed_id,
                 'source'      => $source,
             ]);
             DB::persist($activity);
+            DB::persist(Attention::create(['object_type' => Activity::schemaName(), 'object_id' => $activity->getId(), 'target_id' => $subscribed_id]));
 
             Event::handle('NewNotification', [
                 \is_int($subject) ? $subject : Actor::getById($subscriber_id),
                 $activity,
-                ['object' => [$activity->getObjectId()]],
+                [$subscribed_id],
                 _m('{subject} subscribed to {object}.', ['{subject}' => $activity->getActorId(), '{object}' => $activity->getObjectId()]),
             ]);
         }
@@ -149,21 +148,22 @@ class Subscription extends Component
         if (!\is_null($subscription)) {
             // Remove Subscription
             DB::remove($subscription);
-            $previous_follow_activity = DB::findBy('activity', ['verb' => 'subscribe', 'object_type' => 'actor', 'object_id' => $subscribed_id], order_by: ['created' => 'DESC'])[0];
+            $previous_follow_activity = DB::findBy(Activity::class, ['verb' => 'subscribe', 'object_type' => Actor::schemaName(), 'object_id' => $subscribed_id], order_by: ['created' => 'DESC'])[0];
             // Store Activity
             $activity = Activity::create([
                 'actor_id'    => $subscriber_id,
                 'verb'        => 'undo',
-                'object_type' => 'activity',
+                'object_type' => Activity::schemaName(),
                 'object_id'   => $previous_follow_activity->getId(),
                 'source'      => $source,
             ]);
             DB::persist($activity);
 
+            DB::persist(Attention::create(['object_type' => Activity::schemaName(), 'object_id' => $activity->getId(), 'target_id' => $subscribed_id]));
             Event::handle('NewNotification', [
                 \is_int($subject) ? $subject : Actor::getById($subscriber_id),
                 $activity,
-                ['object' => [$previous_follow_activity->getObjectId()]],
+                [],
                 _m('{subject} unsubscribed from {object}.', ['{subject}' => $activity->getActorId(), '{object}' => $previous_follow_activity->getObjectId()]),
             ]);
         }

components/Subscription/templates/subscription/add_subscriber.html.twig

@@ -2,7 +2,7 @@
 
 {% block body %}
     {% block profile_view %}
-        {% include 'cards/profile/view.html.twig' with { actor: object } %}
+        {% include 'cards/blocks/profile.html.twig' with { actor: object } %}
     {% endblock profile_view %}
     {{ form(form) }}
 {% endblock body %}

components/Subscription/templates/subscription/remove_subscriber.html.twig

@@ -2,7 +2,7 @@
 
 {% block body %}
     {% block profile_view %}
-        {% include 'cards/profile/view.html.twig' with { actor: object } %}
+        {% include 'cards/blocks/profile.html.twig' with { actor: object } %}
     {% endblock profile_view %}
     {{ form(form) }}
 {% endblock body %}

components/Tag/Entity/NoteTag.php

@@ -50,13 +50,24 @@ class NoteTag extends Entity
 {
     // {{{ Autocode
     // @codeCoverageIgnoreStart
+    private int $note_id;
     private string $tag;
     private string $canonical;
-    private int $note_id;
     private bool $use_canonical;
     private ?int $language_id = null;
     private DateTimeInterface $created;
 
+    public function setNoteId(int $note_id): self
+    {
+        $this->note_id = $note_id;
+        return $this;
+    }
+
+    public function getNoteId(): int
+    {
+        return $this->note_id;
+    }
+
     public function setTag(string $tag): self
     {
         $this->tag = mb_substr($tag, 0, 64);
@@ -79,17 +90,6 @@ class NoteTag extends Entity
         return $this->canonical;
     }
 
-    public function setNoteId(int $note_id): self
-    {
-        $this->note_id = $note_id;
-        return $this;
-    }
-
-    public function getNoteId(): int
-    {
-        return $this->note_id;
-    }
-
     public function setUseCanonical(bool $use_canonical): self
     {
         $this->use_canonical = $use_canonical;

components/Tag/Tag.php

@@ -93,7 +93,7 @@ class Tag extends Component
                 'use_canonical' => $extra_args['tag_use_canonical'] ?? false,
                 'language_id'   => $lang_id,
             ]));
-            Cache::pushList("tag-{$canonical_tag}", $note);
+            Cache::listPushLeft("tag-{$canonical_tag}", $note);
             foreach (self::cacheKeys($canonical_tag) as $key) {
                 Cache::delete($key);
             }
@@ -220,8 +220,12 @@ class Tag extends Component
 
     public function onCollectionQueryAddJoins(QueryBuilder &$note_qb, QueryBuilder &$actor_qb): bool
     {
-        $note_qb->leftJoin(NoteTag::class, 'note_tag', Expr\Join::WITH, 'note_tag.note_id = note.id');
+        if (!\in_array('note_tag', $note_qb->getAllAliases())) {
-        $actor_qb->leftJoin(ActorTag::class, 'actor_tag', Expr\Join::WITH, 'actor_tag.tagger = actor.id');
+            $note_qb->leftJoin(NoteTag::class, 'note_tag', Expr\Join::WITH, 'note_tag.note_id = note.id');
+        }
+        if (!\in_array('actor_tag', $actor_qb->getAllAliases())) {
+            $actor_qb->leftJoin(ActorTag::class, 'actor_tag', Expr\Join::WITH, 'actor_tag.tagger = actor.id');
+        }
         return Event::next;
     }
 

components/Tag/templates/actor_tag_feed.html.twig

@@ -2,16 +2,16 @@
 
 {% block stylesheets %}
     {{ parent() }}
-    <link rel="stylesheet" href="{{ asset('assets/default_theme/css/pages/feeds.css') }}" type="text/css">
+    <link rel="stylesheet" href="{{ asset('assets/default_theme/pages/feeds.css') }}" type="text/css">
 {% endblock stylesheets %}
 
 {% block body %}
     {% if tag_name is defined and tag_name is not null %}
         {% if tag_name is instanceof('string') %}
-            <h2>{% trans %}Actors with tag: %tag_name%{% endtrans %}</h2>
+            <h1>{% trans %}Actors with tag: %tag_name%{% endtrans %}</h1>
         {% else %}
             {% set tags = tag_name|join(',') %} {# TODO Not ideal, hard to translate #}
-            <h2>{% trans %}Actors with tags: %tags%{% endtrans %}</h2>
+            <h1>{% trans %}Actors with tags: %tags%{% endtrans %}</h1>
         {% endif %}
     {% endif %}
 
@@ -20,10 +20,10 @@
     {% endfor %}
 
     {% for actor in results %}
-        {% block profile_view %}{% include 'cards/profile/view.html.twig' %}{% endblock profile_view %}
+        {% block profile_view %}{% include 'cards/blocks/profile.html.twig' %}{% endblock profile_view %}
     {% endfor %}
 
-    <div class="frame-section frame-section-padding">
+    <div class="frame-section-button-like">
         {{ "Page: " ~ page }}
     </div>
 {% endblock %}

components/Tag/templates/note_tag_feed.html.twig

@@ -1,18 +1,18 @@
 {% extends 'base.html.twig' %}
-{% import '/cards/note/view.html.twig' as noteView %}
+{% import "/cards/macros/note/factory.html.twig" as NoteFactory %}
 
 {% block stylesheets %}
     {{ parent() }}
-    <link rel="stylesheet" href="{{ asset('assets/default_theme/css/pages/feeds.css') }}" type="text/css">
+    <link rel="stylesheet" href="{{ asset('assets/default_theme/feeds.css') }}" type="text/css">
 {% endblock stylesheets %}
 
 {% block body %}
     {% if tag_name is defined and tag_name is not null %}
         {% if tag_name is instanceof('string') %}
-            <h2>{% trans %}Notes with tag: %tag_name%{% endtrans %}</h2>
+            <h1>{% trans %}Notes with tag: %tag_name%{% endtrans %}</h1>
         {% else %}
             {% set tags = tag_name|join(', ') %} {# TODO Not ideal, hard to translate #}
-            <h2>{% trans %}People with tags: %tags%{% endtrans %}</h2>
+            <h1>{% trans %}People with tags: %tags%{% endtrans %}</h1>
         {% endif %}
     {% endif %}
 
@@ -22,11 +22,12 @@
 
     {% for note in results %}
         {% block current_note %}
-            {{ noteView.macro_note(note) }}
+            {% set args = { 'type': 'vanilla_full', 'note': note, 'extra': { 'depth': 0 } } %}
+            {{ NoteFactory.constructor(args) }}
         {% endblock current_note %}
     {% endfor %}
 
-    <div class="frame-section frame-section-padding frame-section-paging">
+    <div class="frame-section-button-like">
         {{ "Page " ~ page }}
     </div>
 {% endblock %}

composer.json

@@ -19,45 +19,48 @@
     "lstrojny/functional-php": "^1.17",
     "masterminds/html5": "^2.7",
     "mf2/mf2": "^0.4.6",
+    "nyholm/psr7": "^1.4",
     "odolbeau/phone-number-bundle": "^3.1",
     "oro/doctrine-extensions": "^2.0",
     "php-ds/php-ds": "^1.2",
     "phpdocumentor/reflection-docblock": "^5.2",
-    "sensio/framework-extra-bundle": "^5.5",
+    "sensio/framework-extra-bundle": "6.*",
     "someonewithpc/memcached-polyfill": "^1.0",
     "someonewithpc/redis-polyfill": "dev-master",
-    "symfony/asset": "5.4.*",
+    "symfony/asset": "^6",
-    "symfony/cache": "5.4.*",
+    "symfony/cache": "^6",
-    "symfony/config": "5.4.*",
+    "symfony/config": "^6",
-    "symfony/console": "5.4.*",
+    "symfony/console": "^6",
-    "symfony/dom-crawler": "5.4.*",
+    "symfony/doctrine-messenger": "^6",
-    "symfony/dotenv": "5.4.*",
+    "symfony/dom-crawler": "^6",
-    "symfony/event-dispatcher": "5.4.*",
+    "symfony/dotenv": "^6",
-    "symfony/expression-language": "5.4.*",
+    "symfony/event-dispatcher": "^6",
-    "symfony/filesystem": "5.4.*",
+    "symfony/expression-language": "^6",
+    "symfony/filesystem": "^6",
     "symfony/flex": "^1.3.1",
-    "symfony/form": "5.4.*",
+    "symfony/form": "^6",
-    "symfony/framework-bundle": "5.4.*",
+    "symfony/framework-bundle": "^6",
-    "symfony/http-client": "5.4.*",
+    "symfony/http-client": "^6",
-    "symfony/intl": "5.4.*",
+    "symfony/intl": "^6",
-    "symfony/mailer": "5.4.*",
+    "symfony/mailer": "^6",
-    "symfony/messenger": "5.4.*",
+    "symfony/messenger": "^6",
-    "symfony/mime": "5.4.*",
+    "symfony/mime": "^6",
     "symfony/monolog-bundle": "^3.1",
-    "symfony/notifier": "5.4.*",
+    "symfony/notifier": "^6",
-    "symfony/process": "5.4.*",
+    "symfony/process": "^6",
-    "symfony/property-access": "5.4.*",
+    "symfony/property-access": "^6",
-    "symfony/property-info": "5.4.*",
+    "symfony/property-info": "^6",
-    "symfony/proxy-manager-bridge": "5.4.*",
+    "symfony/proxy-manager-bridge": "^6",
-    "symfony/security-bundle": "5.4.*",
+    "symfony/psr-http-message-bridge": "^2.1",
-    "symfony/serializer": "5.4.*",
+    "symfony/security-bundle": "^6",
-    "symfony/string": "5.4.*",
+    "symfony/serializer": "^6",
-    "symfony/translation": "5.4.*",
+    "symfony/string": "^6",
-    "symfony/twig-bundle": "5.4.*",
+    "symfony/translation": "^6",
-    "symfony/validator": "5.4.*",
+    "symfony/twig-bundle": "^6",
-    "symfony/var-exporter": "5.4.*",
+    "symfony/validator": "^6",
-    "symfony/web-link": "5.4.*",
+    "symfony/var-exporter": "^6",
-    "symfony/yaml": "5.4.*",
+    "symfony/web-link": "^6",
+    "symfony/yaml": "^6",
     "symfonycasts/reset-password-bundle": "^1.9",
     "symfonycasts/verify-email-bundle": "^1.0",
     "tgalopin/html-sanitizer-bundle": "^1.2",
@@ -68,26 +71,22 @@
     "wikimedia/composer-merge-plugin": "^2.0"
   },
   "require-dev": {
-    "codeception/codeception": "^4.1",
-    "codeception/module-phpbrowser": "^2.0",
-    "codeception/module-symfony": "^2.1",
     "doctrine/doctrine-fixtures-bundle": "^3.4",
     "friendsofphp/php-cs-fixer": "^3.2.1",
     "jchook/phpunit-assert-throws": "^1.0",
     "niels-de-blaauw/php-doc-check": "^0.2.2",
     "phpstan/phpstan": "dev-master",
     "phpunit/phpunit": "^9.5",
-    "symfony/browser-kit": "^5.4.",
+    "symfony/browser-kit": "^6.0",
-    "symfony/css-selector": "^5.4.",
+    "symfony/css-selector": "^6.0",
-    "symfony/debug-bundle": "^5.4.",
+    "symfony/debug-bundle": "^6.0",
-    "symfony/error-handler": "^5.4.",
+    "symfony/error-handler": "^6.0",
     "symfony/maker-bundle": "^1.14",
-    "symfony/phpunit-bridge": "^5.4.",
+    "symfony/phpunit-bridge": "^6.0",
-    "symfony/stopwatch": "5.4.*",
+    "symfony/stopwatch": "^6.0",
-    "symfony/web-profiler-bundle": "^5.4.",
+    "symfony/web-profiler-bundle": "^6.0",
     "ulrichsg/getopt-php": "*",
-    "wp-cli/php-cli-tools": "^0.11.13",
+    "wp-cli/php-cli-tools": "^0.11.13"
-    "codeception/module-asserts": "^1.0.0"
   },
   "config": {
     "preferred-install": {
@@ -108,11 +107,13 @@
       "App\\": "src/",
       "Plugin\\": "plugins/",
       "Component\\": "components/"
-    }
+    },
+    "exclude-from-classmap": ["/tests/"]
   },
   "autoload-dev": {
     "psr-4": {
-      "App\\Tests\\": "tests/"
+      "App\\Test\\Fixtures\\": "tests/fixtures/",
+      "App\\Test\\": "tests/"
     }
   },
   "replace": {
@@ -143,7 +144,7 @@
   "extra": {
     "symfony": {
       "allow-contrib": false,
-      "require": "5.4.*"
+      "require": "^6"
     },
     "merge-plugin": {
       "include": [
@@ -193,40 +194,26 @@
     {
       "type": "package",
       "package": {
-        "name": "codeception/codeception",
+        "name": "landrok/activitypub",
-        "version": "4.1.30",
+        "version": "0.5.7",
+        "require": {
+          "php": "^7.2|^8.0",
+          "guzzlehttp/guzzle": ">=6.3",
+          "monolog/monolog": "^1.12|^2.0",
+          "symfony/http-foundation": ">=3.4",
+          "phpseclib/phpseclib": "^3.0.7",
+          "psr/cache": "*",
+          "symfony/cache": ">=4.0"
+        },
         "autoload": {
           "psr-4": {
-            "Codeception\\": "src/Codeception",
+            "ActivityPhp\\": "src/ActivityPhp/"
-            "Codeception\\Extension\\": "ext"
+          }
-          },
-          "files": [
-            "functions.php"
-          ]
         },
-        "require": {
-          "php": ">=5.6.0 <9.0",
-          "ext-curl": "*",
-          "ext-json": "*",
-          "ext-mbstring": "*",
-          "codeception/lib-asserts": "^1.0 | 2.0.*@dev",
-          "guzzlehttp/psr7": "^1.4 | ^2.0",
-          "symfony/finder": ">=2.7 <6.0",
-          "symfony/console": ">=2.7 <6.0",
-          "symfony/event-dispatcher": ">=2.7 <6.0",
-          "symfony/yaml": ">=2.7 <6.0",
-          "symfony/css-selector": ">=2.7 <6.0",
-          "behat/gherkin": "^4.4.0",
-          "codeception/phpunit-wrapper": ">6.0.15 <6.1.0 | ^6.6.1 | ^7.7.1 | ^8.1.1 | ^9.0",
-          "codeception/stub": "^2.0 | ^3.0 | ^4.0"
-        },
-        "bin": [
-          "codecept"
-        ],
         "source": {
-          "url": "https://github.com/someonewithpc/Codeception.git",
+          "url": "https://github.com/landrok/activitypub.git",
           "type": "git",
-          "reference": "4.1"
+          "reference": "509dd3d"
         }
       }
     }

config/bundles.php

@@ -8,7 +8,7 @@ return [
     Twig\Extra\TwigExtraBundle\TwigExtraBundle::class                        => ['all' => true],
     Symfony\Bundle\WebProfilerBundle\WebProfilerBundle::class                => ['dev' => true, 'test' => true],
     Symfony\Bundle\MonologBundle\MonologBundle::class                        => ['all' => true],
-    Symfony\Bundle\DebugBundle\DebugBundle::class                            => ['dev' => true, 'test' => true],
+    Symfony\Bundle\DebugBundle\DebugBundle::class                            => ['dev' => true],
     Sensio\Bundle\FrameworkExtraBundle\SensioFrameworkExtraBundle::class     => ['all' => true],
     Doctrine\Bundle\DoctrineBundle\DoctrineBundle::class                     => ['all' => true],
     Doctrine\Bundle\MigrationsBundle\DoctrineMigrationsBundle::class         => ['all' => true],

config/packages/debug.yaml

@@ -0,0 +1,5 @@
+when@dev:
+    debug:
+        # Forwards VarDumper Data clones to a centralized server allowing to inspect dumps on CLI or in your browser.
+        # See the "server:dump" command to start a new server.
+        dump_destination: "tcp://%env(VAR_DUMPER_SERVER)%"

config/packages/dev/monolog.yaml

@@ -1,19 +0,0 @@
-monolog:
-    handlers:
-        main:
-            type: stream
-            path: "%kernel.logs_dir%/%kernel.environment%.log"
-            level: debug
-            channels: ["!event"]
-        # uncomment to get logging in your browser
-        # you may have to allow bigger header sizes in your Web server configuration
-        #firephp:
-        #    type: firephp
-        #    level: info
-        #chromephp:
-        #    type: chromephp
-        #    level: info
-        console:
-            type: console
-            process_psr_3_messages: false
-            channels: ["!event", "!doctrine", "!console"]

config/packages/dev/web_profiler.yaml

@@ -1,6 +0,0 @@
-web_profiler:
-    toolbar: true
-    intercept_redirects: false
-
-framework:
-    profiler: { only_exceptions: false }

config/packages/framework.yaml

@@ -1,7 +1,8 @@
+# see https://symfony.com/doc/current/reference/configuration/framework.html
 framework:
     secret: '%env(APP_SECRET)%'
     csrf_protection: true
-    #http_method_override: true
+    # http_method_override: false
 
     # Enables session support. Note that the session will ONLY be started if you read or write from it.
     # Remove or comment this section to explicitly disable session support.
@@ -9,8 +10,15 @@ framework:
         handler_id: null
         cookie_secure: auto
         cookie_samesite: lax
+        storage_factory_id: session.storage.factory.native
 
     #esi: true
     #fragments: true
     php_errors:
         log: true
+
+when@test:
+    framework:
+        test: true
+        session:
+            storage_factory_id: session.storage.factory.mock_file

config/packages/monolog.yaml

@@ -0,0 +1,61 @@
+monolog:
+    channels:
+        - deprecation # Deprecations are logged in the dedicated "deprecation" channel when it exists
+
+when@dev:
+    monolog:
+        handlers:
+            main:
+                type: stream
+                path: "%kernel.logs_dir%/%kernel.environment%.log"
+                level: debug
+                channels: ["!event"]
+            # uncomment to get logging in your browser
+            # you may have to allow bigger header sizes in your Web server configuration
+            #firephp:
+            #    type: firephp
+            #    level: info
+            #chromephp:
+            #    type: chromephp
+            #    level: info
+            console:
+                type: console
+                process_psr_3_messages: false
+                channels: ["!event", "!doctrine", "!console"]
+
+when@test:
+    monolog:
+        handlers:
+            main:
+                type: fingers_crossed
+                action_level: error
+                handler: nested
+                excluded_http_codes: [404, 405]
+                channels: ["!event"]
+            nested:
+                type: stream
+                path: "%kernel.logs_dir%/%kernel.environment%.log"
+                level: debug
+
+when@prod:
+    monolog:
+        handlers:
+            main:
+                type: fingers_crossed
+                action_level: error
+                handler: nested
+                excluded_http_codes: [404, 405]
+                buffer_size: 50 # How many messages should be saved? Prevent memory leaks
+            nested:
+                type: stream
+                path: php://stderr
+                level: debug
+                formatter: monolog.formatter.json
+            console:
+                type: console
+                process_psr_3_messages: false
+                channels: ["!event", "!doctrine"]
+            deprecation:
+                type: stream
+                channels: [deprecation]
+                path: php://stderr

config/packages/prod/monolog.yaml

@@ -1,24 +0,0 @@
-monolog:
-    handlers:
-        main:
-            type: fingers_crossed
-            action_level: error
-            handler: nested
-            excluded_http_codes: [404, 405]
-            buffer_size: 50 # How many messages should be saved? Prevent memory leaks
-        nested:
-            type: stream
-            path: "%kernel.logs_dir%/%kernel.environment%.log"
-            level: debug
-        console:
-            type: console
-            process_psr_3_messages: false
-            channels: ["!event", "!doctrine"]
-        deprecation:
-            type: stream
-            path: "%kernel.logs_dir%/%kernel.environment%.deprecations.log"
-        deprecation_filter:
-            type: filter
-            handler: deprecation
-            max_level: info
-            channels: ["php"]

config/packages/security.yaml

@@ -20,41 +20,28 @@ security:
         dev:
             pattern: ^/(_(profiler|wdt)|css|images|js)/
             security: false
-        api_apps:
-            pattern: ^/api/v1/apps$
-            security: false
-        api_token:
-            pattern: ^/oauth/token$
-            security: false
-        api:
-            provider: local_user
-            pattern: ^/api/
-            security: true
-            stateless: true
-
         main:
-            entry_point: App\Security\Authenticator
+            lazy: true
-            guard:
-              authenticators:
-                - App\Security\Authenticator
             provider: local_user
             form_login:
                 login_path: security_login
                 check_path: security_login
-                enable_csrf: true
+                default_target_path: root
             logout:
                 path: security_logout
                 # where to redirect after logout
                 target: root
 
-            remember_me:
+            # remember_me:
-                secret: '%kernel.secret%'
+            #     secret: '%kernel.secret%'
-                secure: true
+            #     secure: true
-                httponly: '%remember_me_httponly%'
+            #     httponly: '%remember_me_httponly%'
-                samesite: '%remember_me_samesite%'
+            #     samesite: '%remember_me_samesite%'
-                token_provider: 'Symfony\Bridge\Doctrine\Security\RememberMe\DoctrineTokenProvider'
+            #     token_provider: 'Symfony\Bridge\Doctrine\Security\RememberMe\DoctrineTokenProvider'
 
-            # activate different ways to authenticate
+            # custom_authenticator: 'App\Core\Security'
+
+              # activate different ways to authenticate
             # https://symfony.com/doc/current/security.html#firewalls-authentication
 
             # https://symfony.com/doc/current/security/impersonating_user.html
@@ -63,6 +50,5 @@ security:
     # Easy way to control access for large sections of your site
     # Note: Only the *first* access control that matches will be used
     access_control:
-        - { path: ^/admin, roles: ROLE_ADMIN }
+        - { path: ^/admin, roles: ROLE_OPERATOR }
-        - { path: ^/settings, roles: ROLE_USER }
+        - { path: ^/settings, roles: ROLE_VISITOR }
-        - { path: ^/authorize, roles: IS_AUTHENTICATED_REMEMBERED }

config/packages/test/framework.yaml

@@ -1,4 +0,0 @@
-framework:
-    test: true
-    session:
-        storage_id: session.storage.mock_file