From 108e547fb6beed903ff0e96fd4aec086f639d103 Mon Sep 17 00:00:00 2001 From: zyoshoka Date: Sat, 25 Nov 2023 21:34:40 +0900 Subject: [PATCH] chore: rename 'FeedType' to 'FeedFormat' --- .../backend/src/server/web/ClientServerService.ts | 12 ++++++------ packages/backend/src/server/web/FeedService.ts | 14 +++++++------- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/packages/backend/src/server/web/ClientServerService.ts b/packages/backend/src/server/web/ClientServerService.ts index 5213549671..2d3f46401a 100644 --- a/packages/backend/src/server/web/ClientServerService.ts +++ b/packages/backend/src/server/web/ClientServerService.ts @@ -37,7 +37,7 @@ import { deepClone } from '@/misc/clone.js'; import { bindThis } from '@/decorators.js'; import { FlashEntityService } from '@/core/entities/FlashEntityService.js'; import { RoleService } from '@/core/RoleService.js'; -import { FeedService, FeedType } from './FeedService.js'; +import { FeedService, FeedFormat } from './FeedService.js'; import { UrlPreviewService } from './UrlPreviewService.js'; import { ClientLoggerService } from './ClientLoggerService.js'; import type { FastifyInstance, FastifyPluginOptions, FastifyReply } from 'fastify'; @@ -417,11 +417,11 @@ export class ClientServerService { fastify.get<{ Querystring: { url: string; lang: string; } }>('/url', (request, reply) => this.urlPreviewService.handle(request, reply)); // Feed - const feedTypes: FeedType[] = ['atom', 'rss', 'json']; - for (const feedType of feedTypes) { - fastify.get<{ Params: { user: string; } }>(`/@:user.${feedType}`, this.feedService.handle(feedType)); - fastify.get<{ Params: { user: string; } }>(`/@:user.with_replies.${feedType}`, this.feedService.handle(feedType, { withReplies: true })); - fastify.get<{ Params: { user: string; } }>(`/@:user.with_files.${feedType}`, this.feedService.handle(feedType, { withFiles: true })); + const feedFormats: FeedFormat[] = ['atom', 'rss', 'json']; + for (const feedFormat of feedFormats) { + fastify.get<{ Params: { user: string; } }>(`/@:user.${feedFormat}`, this.feedService.handle(feedFormat)); + fastify.get<{ Params: { user: string; } }>(`/@:user.with_replies.${feedFormat}`, this.feedService.handle(feedFormat, { withReplies: true })); + fastify.get<{ Params: { user: string; } }>(`/@:user.with_files.${feedFormat}`, this.feedService.handle(feedFormat, { withFiles: true })); } //#region SSR (for crawlers) diff --git a/packages/backend/src/server/web/FeedService.ts b/packages/backend/src/server/web/FeedService.ts index 4873b148c3..af87f02c8b 100644 --- a/packages/backend/src/server/web/FeedService.ts +++ b/packages/backend/src/server/web/FeedService.ts @@ -18,7 +18,7 @@ import { IdService } from '@/core/IdService.js'; import { FunoutTimelineService } from '@/core/FunoutTimelineService.js'; import type { FastifyReply, FastifyRequest } from 'fastify'; -export type FeedType = 'atom' | 'rss' | 'json'; +export type FeedFormat = 'atom' | 'rss' | 'json'; @Injectable() export class FeedService { @@ -125,7 +125,7 @@ export class FeedService { @bindThis public handle( - feedType: FeedType, + feedFormat: FeedFormat, options?: { withReplies?: boolean; withFiles?: boolean; @@ -145,14 +145,14 @@ export class FeedService { const feed = user && await this.packFeed(user, options); if (feed) { - const subtype = feedType === 'atom' || feedType === 'rss' - ? `${feedType}+xml` - : feedType; + const subtype = feedFormat === 'atom' || feedFormat === 'rss' + ? `${feedFormat}+xml` + : feedFormat; reply.header('Content-Type', `application/${subtype}; charset=utf-8`); - if (feedType === 'atom') return feed.atom1(); - else if (feedType === 'rss') return feed.rss2(); + if (feedFormat === 'atom') return feed.atom1(); + else if (feedFormat === 'rss') return feed.rss2(); else return feed.json1(); } else { reply.code(404);