summaryrefslogtreecommitdiff
path: root/packages/backend/src/server/api
diff options
context:
space:
mode:
authorHazelnoot <acomputerdog@gmail.com>2024-10-21 14:20:30 -0400
committerHazelnoot <acomputerdog@gmail.com>2024-10-21 17:55:06 -0400
commit04654b2f843b081f31ce8e5fe35b83213c0c014c (patch)
tree445ab3bef2f2603e389700d56a8b471d64dd4fe2 /packages/backend/src/server/api
parentreturn error when calling following feed with both includeReplies and filesOnly (diff)
downloadsharkey-04654b2f843b081f31ce8e5fe35b83213c0c014c.tar.gz
sharkey-04654b2f843b081f31ce8e5fe35b83213c0c014c.tar.bz2
sharkey-04654b2f843b081f31ce8e5fe35b83213c0c014c.zip
add "followers" tab to following feed
Diffstat (limited to 'packages/backend/src/server/api')
-rw-r--r--packages/backend/src/server/api/endpoints/notes/following.ts49
1 files changed, 41 insertions, 8 deletions
diff --git a/packages/backend/src/server/api/endpoints/notes/following.ts b/packages/backend/src/server/api/endpoints/notes/following.ts
index 10c211b1fc..b6604b9798 100644
--- a/packages/backend/src/server/api/endpoints/notes/following.ts
+++ b/packages/backend/src/server/api/endpoints/notes/following.ts
@@ -4,12 +4,14 @@
*/
import { Inject, Injectable } from '@nestjs/common';
+import { ObjectLiteral, SelectQueryBuilder } from 'typeorm';
import { SkLatestNote, MiFollowing } from '@/models/_.js';
import type { NotesRepository } from '@/models/_.js';
import { Endpoint } from '@/server/api/endpoint-base.js';
import { NoteEntityService } from '@/core/entities/NoteEntityService.js';
import { DI } from '@/di-symbols.js';
import { QueryService } from '@/core/QueryService.js';
+import { ApiError } from '@/server/api/error.js';
export const meta = {
tags: ['notes'],
@@ -34,13 +36,19 @@ export const meta = {
code: 'BOTH_INCLUDE_REPLIES_AND_FILES_ONLY',
id: '91c8cb9f-36ed-46e7-9ca2-7df96ed6e222',
},
+ bothWithFollowersAndIncludeNonPublic: {
+ message: 'Specifying both list:followers and includeNonPublic is not supported',
+ code: 'BOTH_LIST_FOLLOWERS_AND_INCLUDE_NON_PUBLIC',
+ id: '7a1b9cb6-235b-4e58-9c00-32c1796f502c',
+ },
},
} as const;
export const paramDef = {
type: 'object',
properties: {
- mutualsOnly: { type: 'boolean', default: false },
+ list: { type: 'string', enum: ['following', 'followers', 'mutuals'], default: 'following' },
+
filesOnly: { type: 'boolean', default: false },
includeNonPublic: { type: 'boolean', default: false },
includeReplies: { type: 'boolean', default: false },
@@ -67,6 +75,7 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
) {
super(meta, paramDef, async (ps, me) => {
if (ps.includeReplies && ps.filesOnly) throw new ApiError(meta.errors.bothWithRepliesAndWithFiles);
+ if (ps.list === 'followers' && ps.includeNonPublic) throw new ApiError(meta.errors.bothWithFollowersAndIncludeNonPublic);
const query = this.notesRepository
.createQueryBuilder('note')
@@ -107,14 +116,15 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
.leftJoinAndSelect('reply.user', 'replyUser')
.leftJoinAndSelect('renote.user', 'renoteUser')
.leftJoinAndSelect('note.channel', 'channel')
+ ;
- // Limit to followers
- .innerJoin(MiFollowing, 'following', 'latest.user_id = following."followeeId"')
- .andWhere('following."followerId" = :me');
-
- // Limit to mutuals, if requested
- if (ps.mutualsOnly) {
- query.innerJoin(MiFollowing, 'mutuals', 'latest.user_id = mutuals."followerId" AND mutuals."followeeId" = :me');
+ // Select the appropriate collection of users
+ if (ps.list === 'followers') {
+ addFollower(query);
+ } else if (ps.list === 'following') {
+ addFollowee(query);
+ } else {
+ addMutual(query);
}
// Limit to files, if requested
@@ -143,3 +153,26 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
});
}
}
+
+/**
+ * Limit to followers (they follow us)
+ */
+function addFollower<T extends SelectQueryBuilder<ObjectLiteral>>(query: T): T {
+ return query.innerJoin(MiFollowing, 'follower', 'follower."followerId" = latest.user_id AND follower."followeeId" = :me');
+}
+
+/**
+ * Limit to followees (we follow them)
+ */
+function addFollowee<T extends SelectQueryBuilder<ObjectLiteral>>(query: T): T {
+ return query.innerJoin(MiFollowing, 'followee', 'followee."followerId" = :me AND followee."followeeId" = latest.user_id');
+}
+
+/**
+ * Limit to mutuals (they follow us AND we follow them)
+ */
+function addMutual<T extends SelectQueryBuilder<ObjectLiteral>>(query: T): T {
+ addFollower(query);
+ addFollowee(query);
+ return query;
+}