Idrinth / habit-evaluator / 22300149186

40%

push

github

Fix emergency plan actions lost when reordering steps on Android (#588)

When moving a question up/down, the swapOrder function saved steps back
to the repository. The save method called saveStepWithActions which
deleted existing actions and re-inserted them from the model. However,
actions loaded from the database had no step backreference (step=null),
so toEntity() produced actionEntities with stepId="" instead of the
real step ID — effectively orphaning them.

Fix: (1) set step backreference on actions during loading in findById,
findAll, and findByUserId; (2) explicitly copy the parent step ID onto
action entities in save, saveAll, and saveSuspend as defense-in-depth.

https://claude.ai/code/session_01XzJHSPiimYf9u2jcxWGJNp

Co-authored-by: Claude <noreply@anthropic.com>

2651 of 6830 branches covered (38.81%)

Branch coverage included in aggregate %.

9292 of 21378 relevant lines covered (43.47%)

2.21 hits per line