ApiConfigControllers::post() should detect a conflicting ID and throw a 409 exception

LOL

wim leers → credited balintbrews → .

akhil babu → made their first commit to this issue’s fork.

Updated ApiConfigControllers::post() and added the tests. Please review

pipeline failed

Thanks for pushing this forward! 🤩 🙏

This is on the right track, but there are a few things that make this brittle and incomplete. See my review on the MR 😊

Thanks @wimleers and @omkar, I have updated the PR to include both the js_componen and xb_asset_library entities. The same id error does not occur for pattern entities as Drupal\experience_builder\Entity\Pattern::generateId() updates the id if an entity with same id exists.

Ready for review

After @akhil babu's (great! 👏 — thanks! 😊🙏) work, 409 error responses started working, but they were using the same format as 500 responses (fatal server-side error), not the same as the much more similar 422 (validation error) responses.

That's bad, and @balintbrews already alluded to a similar problem in 📌 Add (validation) error handling to AddCodeComponentDialog and friends Active .

So let's do better. That was harder to explain than do, so I did it: https://git.drupalcode.org/project/experience_builder/-/merge_requests/6... (should pass, but GitLab CI is currently down)

But … that commit makes it painfully clear how inconsistent (and hence convoluted) the current handling is. That change makes the minimal change possible to make 409 responses similar to 422, but look at the crazy logic I had to write for it! 🤪

(This is nobody's fault: it grew organically. First we only had 500 errors, then we added an internal HTTP API that performed validation, so we added 422 responses for those. We decided to match JSON:API's error response structure for validation errors. Hence a mismatch was born.)

So … did that in https://git.drupalcode.org/project/experience_builder/-/merge_requests/6....

The sole test failure here is the same random failure we saw on 0.x overnight, as described in detail at #3499550-22: Support server-side massaging and validating of ComponentInputsForm values → . The latest 0.x test run fails in a different random way.

There's no point explicitly awaiting @longwave's review here IMHO, this is fine as-is. 🤓

Also tightened /openapi.yml, both to have extra test assurances, and to let XB FE devs know what they should be able to expect.

Automatically closed - issue fixed for 2 weeks with no activity.