diff options
| author | Mar0xy <marie@kaifa.ch> | 2023-10-06 02:32:09 +0200 |
|---|---|---|
| committer | Mar0xy <marie@kaifa.ch> | 2023-10-06 02:32:09 +0200 |
| commit | 4e6439763544f7b96009dd1411035343fb561d2a (patch) | |
| tree | 6f0d183391c27f383297ee1713e821ade4e60d20 /packages/backend/src | |
| parent | merge: add speakAsCat extension and non-nya speech option (diff) | |
| download | sharkey-4e6439763544f7b96009dd1411035343fb561d2a.tar.gz sharkey-4e6439763544f7b96009dd1411035343fb561d2a.tar.bz2 sharkey-4e6439763544f7b96009dd1411035343fb561d2a.zip | |
add: profile backgrounds
Diffstat (limited to 'packages/backend/src')
7 files changed, 91 insertions, 3 deletions
diff --git a/packages/backend/src/core/DriveService.ts b/packages/backend/src/core/DriveService.ts index cecbec9638..4b4872a2d6 100644 --- a/packages/backend/src/core/DriveService.ts +++ b/packages/backend/src/core/DriveService.ts @@ -423,6 +423,10 @@ export class DriveService { q.andWhere('file.id != :bannerId', { bannerId: user.bannerId }); } + if (user.backgroundId) { + q.andWhere('file.id != :backgroundId', { backgroundId: user.backgroundId }); + } + //This selete is hard coded, be careful if change database schema q.addSelect('SUM("file"."size") OVER (ORDER BY "file"."id" DESC ROWS BETWEEN UNBOUNDED PRECEDING AND CURRENT ROW)', 'acc_usage'); q.orderBy('file.id', 'ASC'); diff --git a/packages/backend/src/core/activitypub/ApRendererService.ts b/packages/backend/src/core/activitypub/ApRendererService.ts index 155aee39a9..c862bffce5 100644 --- a/packages/backend/src/core/activitypub/ApRendererService.ts +++ b/packages/backend/src/core/activitypub/ApRendererService.ts @@ -454,9 +454,10 @@ export class ApRendererService { const id = this.userEntityService.genLocalUserUri(user.id); const isSystem = user.username.includes('.'); - const [avatar, banner, profile] = await Promise.all([ + const [avatar, banner, background, profile] = await Promise.all([ user.avatarId ? this.driveFilesRepository.findOneBy({ id: user.avatarId }) : undefined, user.bannerId ? this.driveFilesRepository.findOneBy({ id: user.bannerId }) : undefined, + user.backgroundId ? this.driveFilesRepository.findOneBy({ id: user.backgroundId }) : undefined, this.userProfilesRepository.findOneByOrFail({ userId: user.id }), ]); @@ -496,6 +497,7 @@ export class ApRendererService { summary: profile.description ? this.mfmService.toHtml(mfm.parse(profile.description)) : null, icon: avatar ? this.renderImage(avatar) : null, image: banner ? this.renderImage(banner) : null, + backgroundUrl: background ? this.renderImage(background) : null, tag, manuallyApprovesFollowers: user.isLocked, discoverable: user.isExplorable, @@ -650,6 +652,9 @@ export class ApRendererService { // Firefish firefish: "https://joinfirefish.org/ns#", speakAsCat: "firefish:speakAsCat", + // Sharkey + sharkey: "https://joinsharkey.org/ns#", + backgroundUrl: "sharkey:backgroundUrl", // vcard vcard: 'http://www.w3.org/2006/vcard/ns#', }, diff --git a/packages/backend/src/core/activitypub/models/ApPersonService.ts b/packages/backend/src/core/activitypub/models/ApPersonService.ts index bbb362646d..639d11add3 100644 --- a/packages/backend/src/core/activitypub/models/ApPersonService.ts +++ b/packages/backend/src/core/activitypub/models/ApPersonService.ts @@ -225,8 +225,8 @@ export class ApPersonService implements OnModuleInit { return null; } - private async resolveAvatarAndBanner(user: MiRemoteUser, icon: any, image: any): Promise<Pick<MiRemoteUser, 'avatarId' | 'bannerId' | 'avatarUrl' | 'bannerUrl' | 'avatarBlurhash' | 'bannerBlurhash'>> { - const [avatar, banner] = await Promise.all([icon, image].map(img => { + private async resolveAvatarAndBanner(user: MiRemoteUser, icon: any, image: any): Promise<Pick<MiRemoteUser, 'avatarId' | 'bannerId' | 'backgroundId' | 'avatarUrl' | 'bannerUrl' | 'backgroundUrl' | 'avatarBlurhash' | 'bannerBlurhash' | 'backgroundBlurhash'>> { + const [avatar, banner, background] = await Promise.all([icon, image].map(img => { if (img == null) return null; if (user == null) throw new Error('failed to create user: user is null'); return this.apImageService.resolveImage(user, img).catch(() => null); @@ -235,10 +235,13 @@ export class ApPersonService implements OnModuleInit { return { avatarId: avatar?.id ?? null, bannerId: banner?.id ?? null, + backgroundId: background?.id ?? null, avatarUrl: avatar ? this.driveFileEntityService.getPublicUrl(avatar, 'avatar') : null, bannerUrl: banner ? this.driveFileEntityService.getPublicUrl(banner) : null, + backgroundUrl: background ? this.driveFileEntityService.getPublicUrl(background) : null, avatarBlurhash: avatar?.blurhash ?? null, bannerBlurhash: banner?.blurhash ?? null, + backgroundBlurhash: background?.blurhash ?? null }; } diff --git a/packages/backend/src/core/entities/UserEntityService.ts b/packages/backend/src/core/entities/UserEntityService.ts index 714459d76b..dbad82b1fe 100644 --- a/packages/backend/src/core/entities/UserEntityService.ts +++ b/packages/backend/src/core/entities/UserEntityService.ts @@ -308,6 +308,14 @@ export class UserEntityService implements OnModuleInit { bannerBlurhash: banner.blurhash, }); } + if (user.backgroundId != null && user.backgroundUrl === null) { + const background = await this.driveFilesRepository.findOneByOrFail({ id: user.backgroundId }); + user.backgroundUrl = this.driveFileEntityService.getPublicUrl(background); + this.usersRepository.update(user.id, { + backgroundUrl: user.backgroundUrl, + backgroundBlurhash: background.blurhash, + }); + } const meId = me ? me.id : null; const isMe = meId === user.id; @@ -385,6 +393,8 @@ export class UserEntityService implements OnModuleInit { lastFetchedAt: user.lastFetchedAt ? user.lastFetchedAt.toISOString() : null, bannerUrl: user.bannerUrl, bannerBlurhash: user.bannerBlurhash, + backgroundUrl: user.backgroundUrl, + backgroundBlurhash: user.backgroundBlurhash, isLocked: user.isLocked, isSilenced: this.roleService.getUserPolicies(user.id).then(r => !r.canPublicNote), isSuspended: user.isSuspended ?? falsy, @@ -429,6 +439,7 @@ export class UserEntityService implements OnModuleInit { ...(opts.detail && isMe ? { avatarId: user.avatarId, bannerId: user.bannerId, + backgroundId: user.backgroundId, isModerator: isModerator, isAdmin: isAdmin, injectFeaturedNote: profile!.injectFeaturedNote, diff --git a/packages/backend/src/models/User.ts b/packages/backend/src/models/User.ts index 08c8243421..c5670f635b 100644 --- a/packages/backend/src/models/User.ts +++ b/packages/backend/src/models/User.ts @@ -124,6 +124,19 @@ export class MiUser { @JoinColumn() public banner: MiDriveFile | null; + @Column({ + ...id(), + nullable: true, + comment: 'The ID of background DriveFile.', + }) + public backgroundId: MiDriveFile['id'] | null; + + @OneToOne(type => MiDriveFile, { + onDelete: 'SET NULL', + }) + @JoinColumn() + public background: MiDriveFile | null; + @Column('varchar', { length: 512, nullable: true, }) @@ -135,6 +148,11 @@ export class MiUser { public bannerUrl: string | null; @Column('varchar', { + length: 512, nullable: true, + }) + public backgroundUrl: string | null; + + @Column('varchar', { length: 128, nullable: true, }) public avatarBlurhash: string | null; @@ -144,6 +162,11 @@ export class MiUser { }) public bannerBlurhash: string | null; + @Column('varchar', { + length: 128, nullable: true, + }) + public backgroundBlurhash: string | null; + @Index() @Column('varchar', { length: 128, array: true, default: '{}', diff --git a/packages/backend/src/models/json-schema/user.ts b/packages/backend/src/models/json-schema/user.ts index a8fb34acb1..bea2922f5a 100644 --- a/packages/backend/src/models/json-schema/user.ts +++ b/packages/backend/src/models/json-schema/user.ts @@ -122,6 +122,15 @@ export const packedUserDetailedNotMeOnlySchema = { type: 'string', nullable: true, optional: false, }, + backgroundUrl: { + type: 'string', + format: 'url', + nullable: true, optional: false, + }, + backgroundBlurhash: { + type: 'string', + nullable: true, optional: false, + }, isLocked: { type: 'boolean', nullable: false, optional: false, @@ -304,6 +313,11 @@ export const packedMeDetailedOnlySchema = { nullable: true, optional: false, format: 'id', }, + backgroundId: { + type: 'string', + nullable: true, optional: false, + format: 'id', + }, injectFeaturedNote: { type: 'boolean', nullable: true, optional: false, diff --git a/packages/backend/src/server/api/endpoints/i/update.ts b/packages/backend/src/server/api/endpoints/i/update.ts index 93897b9c8f..1011a8d31a 100644 --- a/packages/backend/src/server/api/endpoints/i/update.ts +++ b/packages/backend/src/server/api/endpoints/i/update.ts @@ -60,6 +60,12 @@ export const meta = { id: '0d8f5629-f210-41c2-9433-735831a58595', }, + noSuchBackground: { + message: 'No such background file.', + code: 'NO_SUCH_BACKGROUND', + id: '0d8f5629-f210-41c2-9433-735831a58582', + }, + avatarNotAnImage: { message: 'The file specified as an avatar is not an image.', code: 'AVATAR_NOT_AN_IMAGE', @@ -72,6 +78,12 @@ export const meta = { id: '75aedb19-2afd-4e6d-87fc-67941256fa60', }, + backgroundNotAnImage: { + message: 'The file specified as a background is not an image.', + code: 'BACKGROUND_NOT_AN_IMAGE', + id: '75aedb19-2afd-4e6d-87fc-67941256fa40', + }, + noSuchPage: { message: 'No such page.', code: 'NO_SUCH_PAGE', @@ -133,6 +145,7 @@ export const paramDef = { lang: { type: 'string', enum: [null, ...Object.keys(langmap)] as string[], nullable: true }, avatarId: { type: 'string', format: 'misskey:id', nullable: true }, bannerId: { type: 'string', format: 'misskey:id', nullable: true }, + backgroundId: { type: 'string', format: 'misskey:id', nullable: true }, fields: { type: 'array', minItems: 0, @@ -300,6 +313,21 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint- updates.bannerBlurhash = null; } + if (ps.backgroundId) { + const background = await this.driveFilesRepository.findOneBy({ id: ps.backgroundId }); + + if (background == null || background.userId !== user.id) throw new ApiError(meta.errors.noSuchBackground); + if (!background.type.startsWith('image/')) throw new ApiError(meta.errors.backgroundNotAnImage); + + updates.backgroundId = background.id; + updates.backgroundUrl = this.driveFileEntityService.getPublicUrl(background); + updates.backgroundBlurhash = background.blurhash; + } else if (ps.backgroundId === null) { + updates.backgroundId = null; + updates.backgroundUrl = null; + updates.backgroundBlurhash = null; + } + if (ps.pinnedPageId) { const page = await this.pagesRepository.findOneBy({ id: ps.pinnedPageId }); |