Remove banner/avatar fields from channel/account

Deprecated since 4.2

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 77bf9171bd1642aac6b3bc84d69be562db577957..bce040970953ab4eb8f448fff8ec8695e8e00f53 100644 (file)
--- a/client/src/app/shared/shared-main/account/actor.model.ts
+++ b/client/src/app/shared/shared-main/account/actor.model.ts
@@ -13,9 +13,6 @@ export abstract class Actor implements ServerActor {
 
   createdAt: Date | string
 
-  // TODO: remove, deprecated in 4.2
-  avatar: never
-
   avatars: ActorImage[]
 
   isLocal: boolean

diff --git a/client/src/app/shared/shared-main/video-channel/video-channel.model.ts b/client/src/app/shared/shared-main/video-channel/video-channel.model.ts
index 62bd9434903ccd7d066e7ccc20b0da7a64e00bd2..db3e4b43fd4036bfcd1e35454c3930737ff96a4d 100644 (file)
--- a/client/src/app/shared/shared-main/video-channel/video-channel.model.ts
+++ b/client/src/app/shared/shared-main/video-channel/video-channel.model.ts
@@ -12,9 +12,6 @@ export class VideoChannel extends Actor implements ServerVideoChannel {
   nameWithHost: string
   nameWithHostForced: string
 
-  // TODO: remove, deprecated in 4.2
-  banner: never
-
   banners: ActorImage[]
 
   bannerUrl: string

diff --git a/client/src/app/shared/shared-main/video/video.model.ts b/client/src/app/shared/shared-main/video/video.model.ts
index 392dcadd0688cd1a077af6bd9c40e7d52174f659..1d077a6130377257d8e75e935a67b8a4c164aa2d 100644 (file)
--- a/client/src/app/shared/shared-main/video/video.model.ts
+++ b/client/src/app/shared/shared-main/video/video.model.ts
@@ -86,9 +86,6 @@ export class Video implements VideoServerModel {
     url: string
     host: string
 
-    // TODO: remove, deprecated in 4.2
-    avatar: ActorImage
-
     avatars: ActorImage[]
   }
 
@@ -99,9 +96,6 @@ export class Video implements VideoServerModel {
     url: string
     host: string
 
-    // TODO: remove, deprecated in 4.2
-    avatar: ActorImage
-
     avatars: ActorImage[]
   }
 

diff --git a/server/controllers/api/users/me.ts b/server/controllers/api/users/me.ts
index 4753308e8bb6370dac71e5c71bfc3082b789bbe7..26811136ed61b7a717ffa8baea730a9e65eea3a2 100644 (file)
--- a/server/controllers/api/users/me.ts
+++ b/server/controllers/api/users/me.ts
@@ -1,7 +1,6 @@
 import 'multer'
 import express from 'express'
 import { auditLoggerFactory, getAuditIdFromRes, UserAuditView } from '@server/helpers/audit-logger'
-import { getBiggestActorImage } from '@server/lib/actor-image'
 import { Hooks } from '@server/lib/plugins/hooks'
 import { pick } from '@shared/core-utils'
 import { ActorImageType, HttpStatusCode, UserUpdateMe, UserVideoQuota, UserVideoRate as FormattedUserVideoRate } from '@shared/models'
@@ -264,8 +263,6 @@ async function updateMyAvatar (req: express.Request, res: express.Response) {
   )
 
   return res.json({
-    // TODO: remove, deprecated in 4.2
-    avatar: getBiggestActorImage(avatars).toFormattedJSON(),
     avatars: avatars.map(avatar => avatar.toFormattedJSON())
   })
 }

diff --git a/server/controllers/api/video-channel.ts b/server/controllers/api/video-channel.ts
index 3d7ef31ee25dc4f29ff9125d089898f29b9a213d..18de5bf6a25684aab22624fd955634dbc3f75262 100644 (file)
--- a/server/controllers/api/video-channel.ts
+++ b/server/controllers/api/video-channel.ts
@@ -1,6 +1,5 @@
 import express from 'express'
 import { pickCommonVideoQuery } from '@server/helpers/query'
-import { getBiggestActorImage } from '@server/lib/actor-image'
 import { Hooks } from '@server/lib/plugins/hooks'
 import { ActorFollowModel } from '@server/models/actor/actor-follow'
 import { getServerActor } from '@server/models/application/application'
@@ -213,8 +212,6 @@ async function updateVideoChannelBanner (req: express.Request, res: express.Resp
   auditLogger.update(getAuditIdFromRes(res), new VideoChannelAuditView(videoChannel.toFormattedJSON()), oldVideoChannelAuditKeys)
 
   return res.json({
-    // TODO: remove, deprecated in 4.2
-    banner: getBiggestActorImage(banners).toFormattedJSON(),
     banners: banners.map(b => b.toFormattedJSON())
   })
 }
@@ -228,8 +225,6 @@ async function updateVideoChannelAvatar (req: express.Request, res: express.Resp
   auditLogger.update(getAuditIdFromRes(res), new VideoChannelAuditView(videoChannel.toFormattedJSON()), oldVideoChannelAuditKeys)
 
   return res.json({
-    // TODO: remove, deprecated in 4.2
-    avatar: getBiggestActorImage(avatars).toFormattedJSON(),
     avatars: avatars.map(a => a.toFormattedJSON())
   })
 }

diff --git a/server/lib/activitypub/actors/shared/object-to-model-attributes.ts b/server/lib/activitypub/actors/shared/object-to-model-attributes.ts
index d854bf2b53ddc41bc86579f22cebe3a67c35e489..3ce332681e3b12c021fd5279f105c3fcefd42cb9 100644 (file)
--- a/server/lib/activitypub/actors/shared/object-to-model-attributes.ts
+++ b/server/lib/activitypub/actors/shared/object-to-model-attributes.ts
@@ -32,7 +32,7 @@ function getActorAttributesFromObject (
 
 function getImagesInfoFromObject (actorObject: ActivityPubActor, type: ActorImageType) {
   const iconsOrImages = type === ActorImageType.AVATAR
-    ? actorObject.icons || actorObject.icon
+    ? actorObject.icon
     : actorObject.image
 
   return normalizeIconOrImage(iconsOrImages)

diff --git a/server/lib/activitypub/context.ts b/server/lib/activitypub/context.ts
index eba6d636d329aafc62400ae8dcbcc4fc9243e0fa..87eb498a3512bc71345f8fea66ea920cd08a2f33 100644 (file)
--- a/server/lib/activitypub/context.ts
+++ b/server/lib/activitypub/context.ts
@@ -27,9 +27,6 @@ const contextStore: { [ id in ContextType ]: (string | { [ id: string ]: string
     language: 'sc:inLanguage',
     identifier: 'sc:identifier',
 
-    // TODO: remove in a few versions, introduced in 4.2
-    icons: 'as:icon',
-
     isLiveBroadcast: 'sc:isLiveBroadcast',
     liveSaveReplay: {
       '@type': 'sc:Boolean',

diff --git a/server/models/account/account.ts b/server/models/account/account.ts
index 396959352c3792b9bae35f80f9e68e475e9f1cf7..8593f2f28f1667fc821706fbc677ba144120de57 100644 (file)
--- a/server/models/account/account.ts
+++ b/server/models/account/account.ts
@@ -433,10 +433,7 @@ export class AccountModel extends Model<Partial<AttributesOnly<AccountModel>>> {
       name: actor.name,
       url: actor.url,
       host: actor.host,
-      avatars: actor.avatars,
-
-      // TODO: remove, deprecated in 4.2
-      avatar: actor.avatar
+      avatars: actor.avatars
     }
   }
 

diff --git a/server/models/actor/actor.ts b/server/models/actor/actor.ts
index dccb47a10a5e254ad89d9cdc09dc53b9782a2149..e2e85f3d6f151759b1deae26deab58ef68807cd6 100644 (file)
--- a/server/models/actor/actor.ts
+++ b/server/models/actor/actor.ts
@@ -540,12 +540,7 @@ export class ActorModel extends Model<Partial<AttributesOnly<ActorModel>>> {
       url: this.url,
       name: this.preferredUsername,
       host: this.getHost(),
-      avatars: (this.Avatars || []).map(a => a.toFormattedJSON()),
-
-      // TODO: remove, deprecated in 4.2
-      avatar: this.hasImage(ActorImageType.AVATAR)
-        ? this.Avatars[0].toFormattedJSON()
-        : undefined
+      avatars: (this.Avatars || []).map(a => a.toFormattedJSON())
     }
   }
 
@@ -559,23 +554,16 @@ export class ActorModel extends Model<Partial<AttributesOnly<ActorModel>>> {
       followersCount: this.followersCount,
       createdAt: this.getCreatedAt(),
 
-      banners: (this.Banners || []).map(b => b.toFormattedJSON()),
-
-      // TODO: remove, deprecated in 4.2
-      banner: this.hasImage(ActorImageType.BANNER)
-        ? this.Banners[0].toFormattedJSON()
-        : undefined
+      banners: (this.Banners || []).map(b => b.toFormattedJSON())
     }
   }
 
   toActivityPubObject (this: MActorAPChannel | MActorAPAccount, name: string) {
-    let icon: ActivityIconObject
-    let icons: ActivityIconObject[]
+    let icon: ActivityIconObject[]
     let image: ActivityIconObject
 
     if (this.hasImage(ActorImageType.AVATAR)) {
-      icon = getBiggestActorImage(this.Avatars).toActivityPubObject()
-      icons = this.Avatars.map(a => a.toActivityPubObject())
+      icon = this.Avatars.map(a => a.toActivityPubObject())
     }
 
     if (this.hasImage(ActorImageType.BANNER)) {
@@ -613,7 +601,6 @@ export class ActorModel extends Model<Partial<AttributesOnly<ActorModel>>> {
       published: this.getCreatedAt().toISOString(),
 
       icon,
-      icons,
 
       image
     }

diff --git a/server/models/video/video-channel.ts b/server/models/video/video-channel.ts
index 19dd681a70787516141128455406461111d77ce3..2c38850d70447b5c3c7d0c468fe1d8642589cd80 100644 (file)
--- a/server/models/video/video-channel.ts
+++ b/server/models/video/video-channel.ts
@@ -779,10 +779,7 @@ export class VideoChannelModel extends Model<Partial<AttributesOnly<VideoChannel
       displayName: this.getDisplayName(),
       url: actor.url,
       host: actor.host,
-      avatars: actor.avatars,
-
-      // TODO: remove, deprecated in 4.2
-      avatar: actor.avatar
+      avatars: actor.avatars
     }
   }
 
@@ -821,10 +818,7 @@ export class VideoChannelModel extends Model<Partial<AttributesOnly<VideoChannel
       viewsPerDay,
       totalViews,
 
-      avatars: actor.avatars,
-
-      // TODO: remove, deprecated in 4.2
-      avatar: actor.avatar
+      avatars: actor.avatars
     }
 
     if (this.Account) videoChannel.ownerAccount = this.Account.toFormattedJSON()

diff --git a/server/tests/api/search/search-index.ts b/server/tests/api/search/search-index.ts
index 21473b6bf0a02e991996b7dadbd321204a490a36..cbe628cccc20eaf1d481d04d7919564016af9675 100644 (file)
--- a/server/tests/api/search/search-index.ts
+++ b/server/tests/api/search/search-index.ts
@@ -107,15 +107,11 @@ describe('Test index search', function () {
       expect(video.account.host).to.equal('framatube.org')
       expect(video.account.name).to.equal('framasoft')
       expect(video.account.url).to.equal('https://framatube.org/accounts/framasoft')
-      // TODO: remove, deprecated in 4.2
-      expect(video.account.avatar).to.exist
       expect(video.account.avatars.length).to.equal(2, 'Account should have one avatar image')
 
       expect(video.channel.host).to.equal('framatube.org')
       expect(video.channel.name).to.equal('joinpeertube')
       expect(video.channel.url).to.equal('https://framatube.org/video-channels/joinpeertube')
-      // TODO: remove, deprecated in 4.2
-      expect(video.channel.avatar).to.exist
       expect(video.channel.avatars.length).to.equal(2, 'Channel should have one avatar image')
     }
 
@@ -293,16 +289,12 @@ describe('Test index search', function () {
       const videoChannel = body.data[0]
       expect(videoChannel.url).to.equal('https://framatube.org/video-channels/bf54d359-cfad-4935-9d45-9d6be93f63e8')
       expect(videoChannel.host).to.equal('framatube.org')
-      // TODO: remove, deprecated in 4.2
-      expect(videoChannel.avatar).to.exist
       expect(videoChannel.avatars.length).to.equal(2, 'Channel should have two avatar images')
       expect(videoChannel.displayName).to.exist
 
       expect(videoChannel.ownerAccount.url).to.equal('https://framatube.org/accounts/framasoft')
       expect(videoChannel.ownerAccount.name).to.equal('framasoft')
       expect(videoChannel.ownerAccount.host).to.equal('framatube.org')
-      // TODO: remove, deprecated in 4.2
-      expect(videoChannel.ownerAccount.avatar).to.exist
       expect(videoChannel.ownerAccount.avatars.length).to.equal(2, 'Account should have two avatar images')
     }
 
@@ -369,15 +361,11 @@ describe('Test index search', function () {
       expect(videoPlaylist.ownerAccount.url).to.equal('https://peertube2.cpy.re/accounts/chocobozzz')
       expect(videoPlaylist.ownerAccount.name).to.equal('chocobozzz')
       expect(videoPlaylist.ownerAccount.host).to.equal('peertube2.cpy.re')
-      // TODO: remove, deprecated in 4.2
-      expect(videoPlaylist.ownerAccount.avatar).to.exist
       expect(videoPlaylist.ownerAccount.avatars.length).to.equal(2, 'Account should have two avatar images')
 
       expect(videoPlaylist.videoChannel.url).to.equal('https://peertube2.cpy.re/video-channels/chocobozzz_channel')
       expect(videoPlaylist.videoChannel.name).to.equal('chocobozzz_channel')
       expect(videoPlaylist.videoChannel.host).to.equal('peertube2.cpy.re')
-      // TODO: remove, deprecated in 4.2
-      expect(videoPlaylist.videoChannel.avatar).to.exist
       expect(videoPlaylist.videoChannel.avatars.length).to.equal(2, 'Channel should have two avatar images')
     }
 

diff --git a/shared/models/activitypub/activitypub-actor.ts b/shared/models/activitypub/activitypub-actor.ts
index efb6edec48c3255ae517c0a6d92ad3a0b141057d..b86bcb76477d87f38b1df10c66dd473d6e2e71c2 100644 (file)
--- a/shared/models/activitypub/activitypub-actor.ts
+++ b/shared/models/activitypub/activitypub-actor.ts
@@ -28,10 +28,7 @@ export interface ActivityPubActor {
   }
 
   image?: ActivityIconObject | ActivityIconObject[]
-
   icon?: ActivityIconObject | ActivityIconObject[]
-  // TODO: migrate to `icon`, introduced in 4.2
-  icons?: ActivityIconObject[]
 
   published?: string
 }

diff --git a/shared/models/actors/account.model.ts b/shared/models/actors/account.model.ts
index 60f4236d5e3558fe40298bb5bba0ed1d2d547b68..9fbec60ba035916c5c4122bbfc1c04e2fa38b03d 100644 (file)
--- a/shared/models/actors/account.model.ts
+++ b/shared/models/actors/account.model.ts
@@ -19,7 +19,4 @@ export interface AccountSummary {
   host: string
 
   avatars: ActorImage[]
-
-  // TODO: remove, deprecated in 4.2
-  avatar: ActorImage
 }

diff --git a/shared/models/actors/actor.model.ts b/shared/models/actors/actor.model.ts
index bf86a917f5e33d6f7a00f615f2c216a056565d65..ab07602632c12962c669245f81585aeebaa7e278 100644 (file)
--- a/shared/models/actors/actor.model.ts
+++ b/shared/models/actors/actor.model.ts
@@ -10,7 +10,4 @@ export interface Actor {
   createdAt: Date | string
 
   avatars: ActorImage[]
-
-  // TODO: remove, deprecated in 4.2
-  avatar: ActorImage
 }

diff --git a/shared/models/videos/channel/video-channel.model.ts b/shared/models/videos/channel/video-channel.model.ts
index 68e2f9c4c87ed7462c96c2f8afb437f2c5007665..ce5fc0e8d05b857d8975defb98e027fe9292c0df 100644 (file)
--- a/shared/models/videos/channel/video-channel.model.ts
+++ b/shared/models/videos/channel/video-channel.model.ts
@@ -21,9 +21,6 @@ export interface VideoChannel extends Actor {
   totalViews?: number
 
   banners: ActorImage[]
-
-  // TODO: remove, deprecated in 4.2
-  banner: ActorImage
 }
 
 export interface VideoChannelSummary {
@@ -34,7 +31,4 @@ export interface VideoChannelSummary {
   host: string
 
   avatars: ActorImage[]
-
-  // TODO: remove, deprecated in 4.2
-  avatar: ActorImage
 }