- πΊπΈUnited States dcam
This issue went out of scope in a big hurry. The issue summary states that the the problem is that the
LinkWidget
validation error messages don't contain the field name so you can identify which one had the problem. Then the patches very quickly became about preventing validation errors for external URL Link fields if HTML5 form validation is bypassed, such as in browsers Drupal no longer supports.We're going to re-scope this issue back to its original purpose: adding the field name to error messages. I checked for other Core fields that have validation callbacks to see what they do. The only other one left is in
OptionsWidgetBase
and it does include the field name. We can take that as a best practice.The error messages in the
LinkWidget
validation callback functions should be updated to include the field name. You can usecore/lib/Drupal/Core/Field/Plugin/Field/FieldWidget/OptionsWidgetBase.php
as an example. Since this should be an easy task I'm tagging this as being a Novice issue. Do not reroll the existing patches. Start fresh with a new MR and focus on implementing the proposed resolution in the issue summary.