FoundKeyGang/FoundKey
FoundKeyGang/FoundKey
7
19
Fork 22
Code Issues 37 Pull requests 4 Releases Activity

Narrow type of isPureRenote
All checks were successful
ci/woodpecker/push/lint-backend Pipeline was successful
Details
ci/woodpecker/push/lint-foundkey-js Pipeline was successful
Details
ci/woodpecker/push/lint-client Pipeline was successful
Details
ci/woodpecker/push/build Pipeline was successful
Details
ci/woodpecker/push/test Pipeline was successful
Details

Browse source 
As side effect of that, a non-null assertion can be removed.

Co-authored-by: Johann150 <johann.galle@protonmail.com>
This commit is contained in:
Michcio 2022-09-25 15:33:29 +02:00 committed by Johann150
parent 47b2f619a6
commit c282ed7683
Signed by: Johann150
GPG key ID: 9EE6577A2A06F8F1
2 changed files with 2 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
packages/backend/src/misc/renote.ts
View file

@ -1,5 +1,5 @@
import { Note } from '@/models/entities/note.js';
export function isPureRenote(note: Note): boolean {
export function isPureRenote(note: Note): note is Note & { renoteId: string, text: null, fileIds: null | never[], hasPoll: false } {
return note.renoteId != null && note.text == null && (note.fileIds == null || note.fileIds.length === 0) && !note.hasPoll;
}

6
packages/backend/src/services/note/delete.ts
View file

@ -43,11 +43,7 @@ export default async function(user: { id: User['id']; uri: User['uri']; host: Us
// if deleted note is renote
if (isPureRenote(note)) {
renote = await Notes.findOneBy({
// isPureRenote checks if note.renoteId is null already, so renoteId should be non-null.
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
id: note.renoteId!,
});
renote = await Notes.findOneBy({ id: note.renoteId });
}
const content = renderActivity(renote