Use proper casing of username in avatar filesystem setup

* before you could request an avatar for User instead of user
  which sets up the filesystem for that user twice causing
  the sharing codes collision detection to detect a lot of
  collisions

Signed-off-by: Morris Jobke <hey@morrisjobke.de>

diff --git a/lib/private/AvatarManager.php b/lib/private/AvatarManager.php
index e461a70608b362090b67f539e8a004af581af953..96b9e7b3bd9206e91981ba1d729f88adad60a5bc 100644 (file)
--- a/lib/private/AvatarManager.php
+++ b/lib/private/AvatarManager.php
@@ -84,6 +84,8 @@ class AvatarManager implements IAvatarManager {
                if (is_null($user)) {
                        throw new \Exception('user does not exist');
                }
+               // sanitize userID - fixes casing issue (needed for the filesystem stuff that is done below)
+               $userId = $user->getUID();
 
                /*
                 * Fix for #22119