summaryrefslogtreecommitdiff
path: root/packages/backend/src/server/api/endpoints/blocking/create.ts
diff options
context:
space:
mode:
authordakkar <dakkar@thenautilus.net>2024-07-13 10:39:24 +0100
committerdakkar <dakkar@thenautilus.net>2024-07-13 10:53:40 +0100
commit9a9570ab6245df69d9d89c1f8733c1eb49da2995 (patch)
tree8aec5172effd8f3c69b7c729f20beca26bb02637 /packages/backend/src/server/api/endpoints/blocking/create.ts
parentremove fedidb link (diff)
downloadsharkey-9a9570ab6245df69d9d89c1f8733c1eb49da2995.tar.gz
sharkey-9a9570ab6245df69d9d89c1f8733c1eb49da2995.tar.bz2
sharkey-9a9570ab6245df69d9d89c1f8733c1eb49da2995.zip
mute users when blocking them - fixes #558
we already unfollow them, and make them unfollow us, so muting them is a very sensible thing to want since the mute is handled separately, users can un-mute the people they block (same as they can re-follow them)
Diffstat (limited to 'packages/backend/src/server/api/endpoints/blocking/create.ts')
-rw-r--r--packages/backend/src/server/api/endpoints/blocking/create.ts21
1 files changed, 19 insertions, 2 deletions
diff --git a/packages/backend/src/server/api/endpoints/blocking/create.ts b/packages/backend/src/server/api/endpoints/blocking/create.ts
index 5066215749..fd6c164449 100644
--- a/packages/backend/src/server/api/endpoints/blocking/create.ts
+++ b/packages/backend/src/server/api/endpoints/blocking/create.ts
@@ -6,9 +6,10 @@
import ms from 'ms';
import { Inject, Injectable } from '@nestjs/common';
import { Endpoint } from '@/server/api/endpoint-base.js';
-import type { UsersRepository, BlockingsRepository } from '@/models/_.js';
+import type { UsersRepository, BlockingsRepository, MutingsRepository } from '@/models/_.js';
import { UserEntityService } from '@/core/entities/UserEntityService.js';
import { UserBlockingService } from '@/core/UserBlockingService.js';
+import { UserMutingService } from '@/core/UserMutingService.js';
import { DI } from '@/di-symbols.js';
import { GetterService } from '@/server/api/GetterService.js';
import { ApiError } from '../../error.js';
@@ -69,9 +70,13 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
@Inject(DI.blockingsRepository)
private blockingsRepository: BlockingsRepository,
+ @Inject(DI.mutingsRepository)
+ private mutingsRepository: MutingsRepository,
+
private userEntityService: UserEntityService,
private getterService: GetterService,
private userBlockingService: UserBlockingService,
+ private userMutingService: UserMutingService,
) {
super(meta, paramDef, async (ps, me) => {
const blocker = await this.usersRepository.findOneByOrFail({ id: me.id });
@@ -99,7 +104,19 @@ export default class extends Endpoint<typeof meta, typeof paramDef> { // eslint-
throw new ApiError(meta.errors.alreadyBlocking);
}
- await this.userBlockingService.block(blocker, blockee);
+ await Promise.all([
+ this.userBlockingService.block(blocker, blockee),
+ this.mutingsRepository.exists({
+ where: {
+ muterId: blocker.id,
+ muteeId: blockee.id,
+ },
+ }).then(exists => {
+ if (!exists) {
+ this.userMutingService.mute(blocker, blockee, null);
+ }
+ }),
+ ]);
return await this.userEntityService.pack(blockee.id, blocker, {
schema: 'UserDetailedNotMe',