AkkomaGang/akkoma-fe
4
13
Fork 56
Code Issues 74 Pull requests 14 Projects Releases 1 Wiki Activity

[feat] apply wordfilters to attachment alt-texts #384

Open
sn0w wants to merge 1 commit from sn0w/akkoma-fe:feature/alt-text-filters into develop
pull from: sn0w/akkoma-fe:feature/alt-text-filters
merge into: AkkomaGang:develop
Conversation 0 Commits 1 Files changed 1 +10 -1
1 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 6f48569e59 - Show all commits

11
src/services/status_parser/status_parser.js
View file

@ -3,8 +3,17 @@ import { filter } from 'lodash'
export const muteWordHits = (status, muteWords) => {
const statusText = status.text.toLowerCase()
const statusSummary = status.summary.toLowerCase()
const hits = filter(muteWords, (muteWord) => {
return statusText.includes(muteWord.toLowerCase()) || statusSummary.includes(muteWord.toLowerCase())
muteWord = muteWord.toLowerCase()
let res = statusText.includes(muteWord) || statusSummary.includes(muteWord)
for (let attachment of status.attachments) {
res |= attachment.description?.toLowerCase().includes(muteWord)
}
Oneric commented 2024-05-04 01:09:51 +00:00
Review
Copy link

nit: this could directly return as soon as something matches the word

nit: this could directly return as soon as something matches the word
floatingghost commented 2024-05-04 02:57:31 +00:00
Review
Copy link

it could also be rewritten as a .some call to remove the loop entirely

res |= status.attachments.some((attachment) => ...)

it could also be rewritten as a `.some` call to remove the loop entirely res |= status.attachments.some((attachment) => ...)
return res
})
return hits