- 🇷🇺Russia kala4ek 🇷🇺 Novosibirsk
A little bit adjusted patch from #16.
Now it also tries to display the actual field name from nested paragraph. - Status changed to Needs review
over 1 year ago 10:06am 22 June 2023 - last update
over 1 year ago 178 pass, 2 fail - last update
over 1 year ago 178 pass, 2 fail The last submitted patch, 15: 3027525-14.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.The last submitted patch, 17: 3027525_17.patch, failed testing. View results →
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.- Status changed to Needs work
over 1 year ago 11:51pm 20 September 2023 - 🇧🇷Brazil gfbarbosa
#17 works fine here when a required field inside a collapsed paragraph have a null value
- 🇳🇱Netherlands Maico de Jong
- Encountered an error when validation fails on a field without a label property. Fixed by using the getLabel() method.
- Update test to check for the improved error message contents.
- last update
about 1 year ago 180 pass - Status changed to Needs review
about 1 year ago 8:56pm 3 October 2023 - Status changed to RTBC
8 months ago 9:04pm 8 May 2024 - 🇺🇸United States imaninotsalami
Tested on Drupal 10.3.0-dev and Paragraphs 8.x-1.17
Paragraph label is displayed in the error message. Collapsing a required paragraph also triggers validation. We can confirm that the error message is triggered at both the node level and paragraph level.
RTBC
- last update
8 months ago Patch Failed to Apply - last update
8 months ago Patch Failed to Apply - last update
8 months ago Patch Failed to Apply - Merge request !130Issue #3027525: Show the paragraph and field label of fields that fail validation → (Merged) created by berdir
- Status changed to Fixed
5 months ago 7:24pm 6 August 2024 - 🇨🇭Switzerland berdir Switzerland
Converted to a MR, did manual testing, fixed a fatal error that could happen if the target type is no longer allowed by loading the paragraph type directly to get its label.
There's still room for improvement, specifically when you have multiple nested paragraphs, then the delta is not unique, but it's obviously a big improvement. Merged.
- 🇩🇪Germany broon Potsdam
Just as a note for anyone looking into that change: By adding a sixth parameter to the widget class, this breaks functionality for modules extending Paragraphs, for example "Default Paragraphs".
- 🇨🇭Switzerland berdir Switzerland
That's unfortunate, missed that, I'd accept a patch to make it conditional with fallback in a new issue.
Automatically closed - issue fixed for 2 weeks with no activity.