Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 13 additions & 7 deletions src/sentry/api/endpoints/broadcast_details.py
Original file line number Diff line number Diff line change
Expand Up @@ -29,18 +29,27 @@ class BroadcastDetailsEndpoint(Endpoint):
}
permission_classes = (SentryIsAuthenticated,)

def _get_broadcast(self, request: Request, broadcast_id):
def convert_args(
self,
request: Request,
broadcast_id: int | str,
*args,
**kwargs,
):
if request.access.has_permission("broadcasts.admin"):
queryset = Broadcast.objects.all()
else:
queryset = Broadcast.objects.filter(
Q(date_expires__isnull=True) | Q(date_expires__gt=timezone.now()), is_active=True
)
try:
return queryset.get(id=int(broadcast_id))
broadcast = queryset.get(id=int(broadcast_id))
except (Broadcast.DoesNotExist, ValueError):
raise ResourceDoesNotExist

kwargs["broadcast"] = broadcast
return args, kwargs

def _get_validator(self, request: Request):
if request.access.has_permission("broadcasts.admin"):
return AdminBroadcastValidator
Expand All @@ -55,15 +64,12 @@ def _serialize_response(self, request: Request, broadcast):
serializer_cls = self._get_serializer(request)
return self.respond(serialize(broadcast, request.user, serializer=serializer_cls()))

def get(self, request: Request, broadcast_id) -> Response:
broadcast = self._get_broadcast(request, broadcast_id)
def get(self, request: Request, broadcast: Broadcast) -> Response:
return self._serialize_response(request, broadcast)

def put(self, request: Request, broadcast_id) -> Response:
def put(self, request: Request, broadcast: Broadcast) -> Response:
if not request.user.is_authenticated:
return Response(status=400)

broadcast = self._get_broadcast(request, broadcast_id)
validator = self._get_validator(request)(data=request.data, partial=True)
if not validator.is_valid():
return self.respond(validator.errors, status=400)
Expand Down
Loading