- First commit to issue fork.
- 🇨🇦Canada ezeedub
Here's the first of two possible approaches. This one skips validation when the element's access is explicitly disabled. Before writing test coverage, I would appreciate some guidance on this approach (vs the next one).
- 🇨🇦Canada ezeedub
Here's another approach, more to the point, which removes validation for an element that the translation handler should not be on the form.
- Status changed to RTBC
over 1 year ago 6:52pm 18 August 2023 - 🇨🇦Canada efrainh
I have a multilingual D9 site and I had the same issue with an entity reference (media) field, it is an untranslatable and required field. I tested the patch #11 and it worked fine. Thanks @ezeedub.
I noticed that this was fixed on Drupal 10, because while upgrading the site (on another branch) and without the patch we didn't have this issue.
- last update
over 1 year ago 30,341 pass - last update
over 1 year ago 30,341 pass - last update
over 1 year ago 30,341 pass - last update
over 1 year ago 30,341 pass - last update
about 1 year ago 30,341 pass - last update
about 1 year ago 30,341 pass - last update
about 1 year ago 30,341 pass - Status changed to Needs review
about 1 year ago 5:30am 31 August 2023 - 🇳🇿New Zealand quietone
I'm triaging RTBC issues → . I read the IS and the comments.
According to the Issue Summary and #10 and #11 the approach to resolve this has yet to be decided. I also expect that this will need a test.
This also needs to be on 11.x now, changing version.I am setting this to NR for discussion on the approach to use.
Leaving at RTBC.
- Status changed to Needs work
about 1 year ago 4:25pm 4 September 2023 - 🇺🇸United States smustgrave
Per #14 adding issue summary update and tests.
Moving to NW for the tests.
- 🇩🇪Germany kle
In my case even the patch will not work with a Media-Element where the image-field is required but not translatable.
- First commit to issue fork.
- Merge request !7058Draft: [Test only] Validation failing for required untranslatable field → (Open) created by mohit_aghera
- 🇮🇳India mohit_aghera Rajkot
I agree with @efrainh's comment in #13
This doesn't seems to be reproducible in latest 11.x.
I even manually tried in 10.2.x branch and doesn't seem to be reproducible there either.To confirm that, I wrote one simple test case based on steps to reproduce.
Tests are coming green, so probably issue doesn't exist anymore or we need proper steps to reproduce the issue.One can refer this PR https://git.drupalcode.org/project/drupal/-/merge_requests/7058 for the confirmation of test.