zaaarf/upub
1
0
Fork 0
forked from alemi/upub
Code Pull requests Activity

fix: skip lemmy community hashtags

Browse source
This commit is contained in:
əlemi 2024-07-06 05:27:35 +02:00
parent 05738cccf7
commit 0934a84de9
Signed by: alemi
GPG key ID: A4895B84D311642C
1 changed files with 12 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
upub/core/src/traits/normalize.rs
View file

@ -113,6 +113,18 @@ impl Normalizer for crate::Context {
let hashtag = l.name() let hashtag = l.name()
.unwrap_or_else(|_| l.href().unwrap_or_default().split('/').last().unwrap_or_default()) // TODO maybe just fail? .unwrap_or_else(|_| l.href().unwrap_or_default().split('/').last().unwrap_or_default()) // TODO maybe just fail?
.replace('#', ""); .replace('#', "");
// TODO lemmy added a "fix" to make its communities kind of work with mastodon:
// basically they include the community name as hashtag. ughhhh, since we handle
// hashtags and audience it means our hashtags gets clogged with posts from lemmy
// communities. it kind of make sense to include them since they fit the hashtag
// theme, but nonetheless it's annoying and i'd rather not have the two things
// mixed. maybe it's just me and this should go instead? maybe this has other
// issues and it's just not worth fixing this tiny lemmy kink? idkk
if let Some(ref audience) = object_model.audience {
if audience.ends_with(&hashtag) {
continue;
}
}
let model = crate::model::hashtag::ActiveModel { let model = crate::model::hashtag::ActiveModel {
internal: NotSet, internal: NotSet,
object: Set(object_model.internal), object: Set(object_model.internal),