User: Fix Resolution of Avatar

srsolutionsag · Nov 22, 2023 · 35345e9 · 35345e9
1 parent bf65020
commit 35345e9
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/Services/User/classes/Avatar/class.ilUserAvatarResolver.php b/Services/User/classes/Avatar/class.ilUserAvatarResolver.php
@@ -170,12 +170,12 @@ public function getAvatar(bool $name_as_set_as_text_closely = false): Avatar
 
     public function getUserPictureForVCard(): array
     {
-        if (!$this->hasProfilePicture()) {
+        if (!$this->hasProfilePicture()
+            || $this->rid === '-') {
             return [null, null];
         }
 
         if ($this->rid !== null
-            && $this->rid !== '-'
             && ($identification = $this->irss->manage()->find($this->rid)) !== null) {
             $flavour_streams = $this->irss->flavours()
                 ->get($identification, $this->flavour_definition)
@@ -214,9 +214,8 @@ public function getUserPictureForVCard(): array
      */
     public function getLegacyPictureURL(): string
     {
-        if ($this->hasProfilePicture()) {
-            // IRSS
-            if ($this->rid !== null && $this->rid !== '-') {
+        if ($this->hasProfilePicture() && $this->rid !== '-') {
+            if ($this->rid !== null) {
                 return $this->resolveProfilePicturePath();
             }