Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.In field.api.php, the sample implementation of hook_field_widget_error() is:
form_error($element['value'], $error['message']);
It should be:
form_error($element, $error['message']);
I'll post a patch soon.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | hook_field_widget_error-1614468-8.patch | 912 bytes | Jorrit |
| #1 | hook_field_widget_error-1614468-1.patch | 924 bytes | Jorrit |
Comments
Comment #1
Jorrit CreditAttribution: Jorrit commentedThe patch
Comment #2
bleen CreditAttribution: bleen commentedyup
Comment #3
Dries CreditAttribution: Dries commentedCommitted to 8.x. Thanks.
Comment #4
David_Rothstein CreditAttribution: David_Rothstein commentedThis was for Drupal 7 too, wasn't it?
Comment #5
Jorrit CreditAttribution: Jorrit commentedYes. The patch is similar, only without the core/ directory obviously. If you want a new patch, I can make one.
Comment #6
Jorrit CreditAttribution: Jorrit commentedBTW: Dries: what about the Git attribution? :)
Comment #7
David_Rothstein CreditAttribution: David_Rothstein commentedYeah, if you can reroll a new patch, that would be great.
Comment #8
Jorrit CreditAttribution: Jorrit commentedHere it is.
Comment #9
bleen CreditAttribution: bleen commentedyup v.7
Comment #10
David_Rothstein CreditAttribution: David_Rothstein commentedCommitted to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/43f6142
Comment #11
Jorrit CreditAttribution: Jorrit commentedThanks for committing. Unfortunately you did not follow the Git commit attribution guidelines, I would have appreciated it to see my name attached to a Drupal core commit.
Comment #12
David_Rothstein CreditAttribution: David_Rothstein commentedI'm just using the standard guidelines at http://drupal.org/node/52287 which give credit inline.
We haven't really been using the Git attribution method for Drupal core patches (except for maybe some of the Drupal 8 initiatives); I think if we switched to that method for regular patches it would need to be a wholesale switch.