eliseuamaro/gnu-social
1
0
Fork 0
forked from GNUsocial/gnu-social
Code Issues Pull Requests Releases Wiki Activity

[Avatar] Add default avatar route and improve url getter

Browse Source
This commit is contained in:
Diogo Peralta Cordeiro 2021-09-20 17:01:36 +01:00
parent 0ef151edac
commit 85969a8cff
Signed by: diogo
GPG Key ID: 18D2D35001FBFAB0
3 changed files with 27 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
components/Avatar/Avatar.php
View File

@ -38,8 +38,9 @@ class Avatar extends Component
public function onAddRoute($r): bool public function onAddRoute($r): bool
{ {
$r->connect('avatar', '/actor/{actor_id<\d+>}/avatar/{size<full|big|medium|small>?full}', [Controller\Avatar::class, 'avatar_view']); $r->connect('avatar_actor', '/actor/{actor_id<\d+>}/avatar/{size<full|big|medium|small>?full}', [Controller\Avatar::class, 'avatar_view']);
$r->connect('settings_avatar', '/settings/avatar', [Controller\Avatar::class, 'settings_avatar']); $r->connect('avatar_default', '/avatar/default/{size<full|big|medium|small>?full}', [Controller\Avatar::class, 'default_avatar_view']);
$r->connect('avatar_settings', '/settings/avatar', [Controller\Avatar::class, 'settings_avatar']);
return Event::next; return Event::next;
} }
@ -62,9 +63,13 @@ class Avatar extends Component
public function onAvatarUpdate(int $actor_id): bool public function onAvatarUpdate(int $actor_id): bool
{ {
Cache::delete('avatar-' . $actor_id); foreach (['full', 'big', 'medium', 'small'] as $size) {
Cache::delete('avatar-url-' . $actor_id); foreach ([Router::ABSOLUTE_PATH, Router::ABSOLUTE_URL] as $type) {
Cache::delete('avatar-file-info-' . $actor_id); Cache::delete("avatar-{$actor_id}-{$size}-{$type}");
Cache::delete("avatar-url-{$actor_id}-{$size}-{$type}");
Cache::delete("avatar-file-info-{$actor_id}-{$size}-{$type}");
}
}
return Event::next; return Event::next;
} }
@ -89,11 +94,13 @@ class Avatar extends Component
/** /**
* Get the cached avatar associated with the given Actor id, or the current user if not given * Get the cached avatar associated with the given Actor id, or the current user if not given
*/ */
public static function getAvatarUrl(int $actor_id, string $size = 'full'): string public static function getAvatarUrl(int $actor_id, string $size = 'full', int $type = Router::ABSOLUTE_PATH): string
{ {
return Cache::get("avatar-url-{$actor_id}", function () use ($actor_id, $size) { try {
return Router::url('avatar', ['actor_id' => $actor_id, 'size' => $size]); return self::getAvatar($actor_id)->getUrl($size, $type);
}); } catch (NoAvatarException) {
return Router::url('avatar_default', ['size' => $size], $type);
}
} }
/** /**

5
components/Avatar/Controller/Avatar.php
View File

@ -45,6 +45,11 @@ use Symfony\Component\HttpFoundation\Response;
class Avatar extends Controller class Avatar extends Controller
{ {
public function default_avatar_view(Request $request, string $size): Response
{
return $this->avatar_view($request, 0, $size);
}
/** /**
* @throws Exception * @throws Exception
*/ */

8
components/Avatar/Entity/Avatar.php
View File

@ -21,6 +21,7 @@
namespace Component\Avatar\Entity; namespace Component\Avatar\Entity;
use App\Core\Cache;
use App\Core\DB\DB; use App\Core\DB\DB;
use App\Core\Entity; use App\Core\Entity;
use App\Core\Router\Router; use App\Core\Router\Router;
@ -117,9 +118,12 @@ class Avatar extends Entity
private ?Attachment $attachment = null; private ?Attachment $attachment = null;
public function getUrl(): string public function getUrl(string $size = 'full', int $type = Router::ABSOLUTE_PATH): string
{ {
return Router::url('avatar', ['actor_id' => $this->actor_id]); $actor_id = $this->getActorId();
return Cache::get("avatar-url-{$actor_id}-{$size}-{$type}", function () use ($actor_id, $size, $type) {
return Router::url('avatar_actor', ['actor_id' => $actor_id, 'size' => $size], $type);
});
} }
public function getAttachment(): Attachment public function getAttachment(): Attachment