From d0f266769135f886a77ef586893ef64a38a7a0c5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20B=C3=B6hmer?= <jan.h.boehmer@gmx.de>
Date: Fri, 15 Nov 2019 18:58:58 +0100
Subject: [PATCH] Only fetch database user from database once.

This should improve the performance a lot.
---
 src/Repository/UserRepository.php | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/Repository/UserRepository.php b/src/Repository/UserRepository.php
index 8e021bed..85a46004 100644
--- a/src/Repository/UserRepository.php
+++ b/src/Repository/UserRepository.php
@@ -33,20 +33,28 @@ use Symfony\Bridge\Doctrine\RegistryInterface;
  */
 class UserRepository extends ServiceEntityRepository
 {
+    protected $anonymous_user;
+
     public function __construct(RegistryInterface $registry)
     {
         parent::__construct($registry, User::class);
+        $this->anonymous_user = null;
     }
 
     /**
      * Returns the anonymous user.
+     * The result is cached, so the database is only called once, after the anonymous user was found.
      *
      * @return User|null
      */
     public function getAnonymousUser()
     {
-        return $this->findOneBy([
+        if ($this->anonymous_user === null) {
+            $this->anonymous_user = $this->findOneBy([
                 'id' => User::ID_ANONYMOUS,
             ]);
+        }
+
+        return $this->anonymous_user;
     }
 }