Features: 1) Add validation to ensure order_product_uuid is required in download_digital_asset_view.
Fixes: 1) Remove unnecessary blank line in `core/viewsets.py`. Extra: 1) Minor cleanup in `core/views.py` to improve clarity.
This commit is contained in:
parent
0bfc4c2984
commit
00e94a2b29
2 changed files with 4 additions and 2 deletions
|
|
@ -304,7 +304,10 @@ class BuyAsBusinessView(APIView):
|
||||||
def download_digital_asset_view(request: HttpRequest, *args, **kwargs) -> FileResponse | JsonResponse:
|
def download_digital_asset_view(request: HttpRequest, *args, **kwargs) -> FileResponse | JsonResponse:
|
||||||
try:
|
try:
|
||||||
logger.debug(f"download_digital_asset_view: {kwargs}")
|
logger.debug(f"download_digital_asset_view: {kwargs}")
|
||||||
uuid = urlsafe_base64_decode(str(kwargs.get("order_product_uuid"))).decode("utf-8")
|
op_uuid = str(kwargs.get("order_product_uuid"))
|
||||||
|
if not op_uuid:
|
||||||
|
raise BadRequest(_("order_product_uuid is required"))
|
||||||
|
uuid = urlsafe_base64_decode(op_uuid).decode("utf-8")
|
||||||
|
|
||||||
download = DigitalAssetDownload.objects.get(order_product__uuid=uuid)
|
download = DigitalAssetDownload.objects.get(order_product__uuid=uuid)
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -769,7 +769,6 @@ class OrderViewSet(EvibesViewSet):
|
||||||
except ValueError as ve:
|
except ValueError as ve:
|
||||||
return Response(status=status.HTTP_400_BAD_REQUEST, data={"detail": str(ve)})
|
return Response(status=status.HTTP_400_BAD_REQUEST, data={"detail": str(ve)})
|
||||||
|
|
||||||
|
|
||||||
@action(detail=True, methods=["post"], url_path="bulk_remove_order_products")
|
@action(detail=True, methods=["post"], url_path="bulk_remove_order_products")
|
||||||
def bulk_remove_order_products(self, request: Request, *args, **kwargs) -> Response:
|
def bulk_remove_order_products(self, request: Request, *args, **kwargs) -> Response:
|
||||||
serializer = BulkRemoveOrderProductsSerializer(data=request.data)
|
serializer = BulkRemoveOrderProductsSerializer(data=request.data)
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue