MushroomObserver / mushroom-observer / 31915816277

99%

main: 99%

Pull #5088

github

Fix field-slip review loop: duplicate _method made Turbo submit POST

Superform emits a `_method` hidden unconditionally -- `post` for a
non-persisted model -- and the review form added its own
`_method=patch` after it. A plain browser POST survives that (Rack
keeps the last value), but turbo-rails' encodeMethodIntoRequestBody
reads only the FIRST `_method` in the body and then strips them all,
so once the form went Turbo the save arrived as a bare POST: routed
to `create`, which re-queued the extraction and bounced back to the
review page instead of saving.

ApplicationForm now suppresses the `_method` field when its value is
`post` (POST needs no override; Rails' own form_with omits it too),
and the review form declares `method: :patch` instead of hand-rolling
a second hidden.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

3 of 3 new or added lines in 1 file covered. (100.0%)

55165 of 55561 relevant lines covered (99.29%)

858.61 hits per line