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.