Org Web Adapter

Editing hungryroot/jira/be_7381_move_orders_backward_view_returns_500_instead_of_400_on_malformed_post_data.org

:PROPERTIES:
:ID:       82ba470a-4323-4246-8ae3-6332d856edb2
:END:
#+title: BE-7381 move_orders_backward_view returns 500 instead of 400 on malformed POST data

- source :: https://hungryroot.atlassian.net/browse/BE-7381
- tags   :: [[id:3a477c02-e26d-40f7-a719-0aa1cd169dd8][Hungryroot API Project]]
- epic   :: [[id:86327fb4-0f66-4bc6-9068-c414cea77042][EP-411 Eng Parking Lot]]
- sprint :: [[id:9821b605-4d59-432e-b15d-95999d25cc39][HR BE Sprint 106]] [[id:c4663d81-db80-4eb0-a898-9f2c4aea9023][HR BE Sprint 107]]
- status :: [[id:4d41d978-e56f-45c1-88d1-e2c76510972e][Status - Complete]]
- priority :: [[id:d4e29db6-f9d5-4ca8-88a2-75c93323187f][Priority - Low]]
- points :: 0.25

* Tasks
** DONE Try running Cursor against bug ticket :work:hungryroot:bug:be7381:ep411:sprint106:cursor:
:PROPERTIES:
:ID:       90cc662e-354b-69f7-d0b6-af6c5c80e80d
:END:
<2026-03-30 Mon>

* Description
** Summary

The move_orders_backward_view in app/views/test.py raises an unhandled
ValueError (resulting in a 500 response) when the count POST parameter contains
a non-integer value (e.g. "1 with a stray quote character). This should return a
400 Bad Request instead, since the issue is malformed client input.

** Current Behavior

The view does int(request.POST.get("count", 2)) without any validation or error
handling. When a non-integer string is submitted, Python raises:

ValueError: invalid literal for int() with base 10: '"1'

This propagates up as an unhandled 500 Internal Server Error.

** Expected Behavior

The view should catch the ValueError and return a 400 Bad Request response with
a descriptive error message indicating the count parameter must be a valid
integer.

** Reproduction

POST to the move_orders_backward_view endpoint with count set to a non-integer
value such as "1.

** Suggested Fix

Wrap the int() conversion in a try/except block and return a JsonResponse with
status 400 on ValueError.

** File

app/views/test.py, line ~405