X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=client%2Fsrc%2Fapp%2Fshared%2Fshared-main%2Faccount%2Factor.model.ts;h=a54f51aa4c22897947346164bbacb888420c3a95;hb=5a8de57d574045c5f819b8c81fb0530a9e9a699f;hp=bda88bdeef44ccd61066d973046b8c1a3763f6f7;hpb=ec903c010ecc54ec2acad0bf2cf10e7fbf6a0fa2;p=github%2FChocobozzz%2FPeerTube.git

diff --git a/client/src/app/shared/shared-main/account/actor.model.ts b/client/src/app/shared/shared-main/account/actor.model.ts
index bda88bdee..a54f51aa4 100644
--- a/client/src/app/shared/shared-main/account/actor.model.ts
+++ b/client/src/app/shared/shared-main/account/actor.model.ts
@@ -1,35 +1,34 @@
-import { Actor as ActorServer, Avatar } from '@shared/models'
 import { getAbsoluteAPIUrl } from '@app/helpers'
+import { Actor as ServerActor, ActorImage } from '@shared/models'
 
-export abstract class Actor implements ActorServer {
+export abstract class Actor implements ServerActor {
   id: number
-  url: string
   name: string
+
   host: string
+  url: string
+
   followingCount: number
   followersCount: number
+
   createdAt: Date | string
-  updatedAt: Date | string
-  avatar: Avatar
 
-  avatarUrl: string
+  // TODO: remove, deprecated in 4.2
+  avatar: never
 
-  isLocal: boolean
+  avatars: ActorImage[]
 
-  static GET_ACTOR_AVATAR_URL (actor: { avatar?: { url?: string, path: string } }) {
-    if (actor?.avatar?.url) return actor.avatar.url
+  isLocal: boolean
 
-    if (actor && actor.avatar) {
-      const absoluteAPIUrl = getAbsoluteAPIUrl()
+  static GET_ACTOR_AVATAR_URL (actor: { avatars: { width: number, url?: string, path: string }[] }, size: number) {
+    const avatar = actor.avatars.sort((a, b) => a.width - b.width).find(a => a.width >= size)
 
-      return absoluteAPIUrl + actor.avatar.path
-    }
+    if (!avatar) return ''
+    if (avatar.url) return avatar.url
 
-    return this.GET_DEFAULT_AVATAR_URL()
-  }
+    const absoluteAPIUrl = getAbsoluteAPIUrl()
 
-  static GET_DEFAULT_AVATAR_URL () {
-    return window.location.origin + '/client/assets/images/default-avatar.png'
+    return absoluteAPIUrl + avatar.path
   }
 
   static CREATE_BY_STRING (accountName: string, host: string, forceHostname = false) {
@@ -41,33 +40,24 @@ export abstract class Actor implements ActorServer {
     return accountName + '@' + host
   }
 
-  protected constructor (hash: ActorServer) {
-    this.id = hash.id
-    this.url = hash.url
-    this.name = hash.name
-    this.host = hash.host
-    this.followingCount = hash.followingCount
-    this.followersCount = hash.followersCount
-
-    if (hash.createdAt) this.createdAt = new Date(hash.createdAt.toString())
-    if (hash.updatedAt) this.updatedAt = new Date(hash.updatedAt.toString())
-
-    this.avatar = hash.avatar
-
+  static IS_LOCAL (host: string) {
     const absoluteAPIUrl = getAbsoluteAPIUrl()
     const thisHost = new URL(absoluteAPIUrl).host
-    this.isLocal = this.host.trim() === thisHost
 
-    this.updateComputedAttributes()
+    return host.trim() === thisHost
   }
 
-  updateAvatar (newAvatar: Avatar) {
-    this.avatar = newAvatar
+  protected constructor (hash: Partial<ServerActor>) {
+    this.id = hash.id
+    this.url = hash.url ?? ''
+    this.name = hash.name ?? ''
+    this.host = hash.host ?? ''
+    this.followingCount = hash.followingCount
+    this.followersCount = hash.followersCount
 
-    this.updateComputedAttributes()
-  }
+    if (hash.createdAt) this.createdAt = new Date(hash.createdAt.toString())
 
-  private updateComputedAttributes () {
-    this.avatarUrl = Actor.GET_ACTOR_AVATAR_URL(this)
+    this.avatars = hash.avatars
+    this.isLocal = Actor.IS_LOCAL(this.host)
   }
 }