mirrors/bookwyrm
1
0
Fork 1
mirror of https://github.com/bookwyrm-social/bookwyrm.git synced 2024-09-24 20:40:18 +00:00
Code Issues Projects Releases Wiki Activity

Don't save duplicate boosts

Browse source
This commit is contained in:
Mouse Reeve 2021-04-22 19:36:27 -07:00
parent 9148f36719
commit b457446f2f
2 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
bookwyrm/models/activitypub_mixin.py
View file

@ -204,10 +204,8 @@ class ObjectMixin(ActivitypubMixin):
created = created or not bool(self.id) created = created or not bool(self.id)
# first off, we want to save normally no matter what # first off, we want to save normally no matter what
super().save(*args, **kwargs) super().save(*args, **kwargs)
if ( if not broadcast or (
not broadcast hasattr(self, "status_type") and self.status_type == "Announce"
or hasattr(self, "status_type")
and self.status_type == "Announce"
): ):
return return

8
bookwyrm/models/status.py
View file

@ -351,6 +351,14 @@ class Boost(ActivityMixin, Status):
def save(self, *args, **kwargs): def save(self, *args, **kwargs):
""" save and notify """ """ save and notify """
# This constraint can't work as it would cross tables.
# class Meta:
# unique_together = ('user', 'boosted_status')
if Boost.objects.filter(
boosted_status=self.boosted_status, user=self.user
).exists():
return
super().save(*args, **kwargs) super().save(*args, **kwargs)
if not self.boosted_status.user.local or self.boosted_status.user == self.user: if not self.boosted_status.user.local or self.boosted_status.user == self.user:
return return