mirror of
https://github.com/ad-aures/castopod.git
synced 2026-04-03 23:09:18 +02:00
fix(notifications): remove cache inconsistencies when marking notification as read
This commit is contained in:
parent
2e7b462d94
commit
46d70541d3
4 changed files with 4 additions and 34 deletions
|
|
@ -22,15 +22,6 @@ class ActorModel extends FediverseActorModel
|
|||
|
||||
public function getActorById(int $id): ?Actor
|
||||
{
|
||||
$cacheName = config('Fediverse')
|
||||
->cachePrefix . "actor#{$id}";
|
||||
if (! ($found = cache($cacheName))) {
|
||||
$found = $this->find($id);
|
||||
|
||||
cache()
|
||||
->save($cacheName, $found, DECADE);
|
||||
}
|
||||
|
||||
return $found;
|
||||
return $this->find($id);
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -85,10 +85,7 @@ class NotificationController extends BaseController
|
|||
|
||||
$post = (new PostModel())->getPostById($this->notification->post_id);
|
||||
|
||||
return redirect()->route(
|
||||
'post',
|
||||
[esc((new PodcastModel())->getPodcastByActorId($this->notification->actor_id)->handle), $post->id]
|
||||
);
|
||||
return redirect()->route('post', [$this->podcast->handle, $post->id]);
|
||||
}
|
||||
|
||||
public function markAllAsRead(): RedirectResponse
|
||||
|
|
|
|||
|
|
@ -61,16 +61,7 @@ class ActorModel extends BaseModel
|
|||
|
||||
public function getActorById(int $id): ?Actor
|
||||
{
|
||||
$cacheName = config('Fediverse')
|
||||
->cachePrefix . "actor#{$id}";
|
||||
if (! ($found = cache($cacheName))) {
|
||||
$found = $this->find($id);
|
||||
|
||||
cache()
|
||||
->save($cacheName, $found, DECADE);
|
||||
}
|
||||
|
||||
return $found;
|
||||
return $this->find($id);
|
||||
}
|
||||
|
||||
/**
|
||||
|
|
|
|||
|
|
@ -89,16 +89,7 @@ class PostModel extends BaseUuidModel
|
|||
|
||||
public function getPostById(string $postId): ?Post
|
||||
{
|
||||
$cacheName = config('Fediverse')
|
||||
->cachePrefix . "post#{$postId}";
|
||||
if (! ($found = cache($cacheName))) {
|
||||
$found = $this->find($postId);
|
||||
|
||||
cache()
|
||||
->save($cacheName, $found, DECADE);
|
||||
}
|
||||
|
||||
return $found;
|
||||
return $this->find($postId);
|
||||
}
|
||||
|
||||
public function getPostByUri(string $postUri): ?Post
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue