Merge pull request #856 from taigaio/fixing-upvote-downvote-api
Fixing upvote downvote apiremotes/origin/issue/4795/notification_even_they_are_disabled
commit
10d1b3f672
|
@ -23,6 +23,8 @@ from django.db import transaction as tx
|
||||||
from django.apps import apps
|
from django.apps import apps
|
||||||
from django.contrib.auth import get_user_model
|
from django.contrib.auth import get_user_model
|
||||||
|
|
||||||
|
from django_pglocks import advisory_lock
|
||||||
|
|
||||||
from .models import Votes, Vote
|
from .models import Votes, Vote
|
||||||
|
|
||||||
|
|
||||||
|
@ -37,6 +39,7 @@ def add_vote(obj, user):
|
||||||
:param user: User adding the vote. :class:`~taiga.users.models.User` instance.
|
:param user: User adding the vote. :class:`~taiga.users.models.User` instance.
|
||||||
"""
|
"""
|
||||||
obj_type = apps.get_model("contenttypes", "ContentType").objects.get_for_model(obj)
|
obj_type = apps.get_model("contenttypes", "ContentType").objects.get_for_model(obj)
|
||||||
|
with advisory_lock("vote-{}-{}".format(obj_type.id, obj.id)):
|
||||||
vote, created = Vote.objects.get_or_create(content_type=obj_type, object_id=obj.id, user=user)
|
vote, created = Vote.objects.get_or_create(content_type=obj_type, object_id=obj.id, user=user)
|
||||||
if not created:
|
if not created:
|
||||||
return
|
return
|
||||||
|
@ -58,6 +61,7 @@ def remove_vote(obj, user):
|
||||||
:param user: User removing her vote. :class:`~taiga.users.models.User` instance.
|
:param user: User removing her vote. :class:`~taiga.users.models.User` instance.
|
||||||
"""
|
"""
|
||||||
obj_type = apps.get_model("contenttypes", "ContentType").objects.get_for_model(obj)
|
obj_type = apps.get_model("contenttypes", "ContentType").objects.get_for_model(obj)
|
||||||
|
with advisory_lock("vote-{}-{}".format(obj_type.id, obj.id)):
|
||||||
qs = Vote.objects.filter(content_type=obj_type, object_id=obj.id, user=user)
|
qs = Vote.objects.filter(content_type=obj_type, object_id=obj.id, user=user)
|
||||||
if not qs.exists():
|
if not qs.exists():
|
||||||
return
|
return
|
||||||
|
|
Loading…
Reference in New Issue