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.

CommentFileSizeAuthor
#8 hook_field_widget_error-1614468-8.patch912 bytesJorrit
#1 hook_field_widget_error-1614468-1.patch924 bytesJorrit

Comments

Jorrit’s picture

Jorrit commented
Assigned: Jorrit » Unassigned
Status: Active » Needs review
StatusFileSize
new hook_field_widget_error-1614468-1.patch924 bytes

The patch

bleen’s picture

bleen commented
Status: Needs review » Reviewed & tested by the community

yup

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to 8.x. Thanks.

David_Rothstein’s picture

David_Rothstein commented
Version: 8.x-dev » 7.x-dev
Status: Fixed » Patch (to be ported)

This was for Drupal 7 too, wasn't it?

Jorrit’s picture

Jorrit commented

Yes. The patch is similar, only without the core/ directory obviously. If you want a new patch, I can make one.

Jorrit’s picture

Jorrit commented

BTW: Dries: what about the Git attribution? :)

David_Rothstein’s picture

David_Rothstein commented

Yeah, if you can reroll a new patch, that would be great.

Jorrit’s picture

Jorrit commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new hook_field_widget_error-1614468-8.patch912 bytes

Here it is.

bleen’s picture

bleen commented
Status: Needs review » Reviewed & tested by the community

yup v.7

David_Rothstein’s picture

David_Rothstein commented
Status: Reviewed & tested by the community » Fixed

Committed to 7.x - thanks! http://drupalcode.org/project/drupal.git/commit/43f6142

Jorrit’s picture

Jorrit commented

Thanks 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.

David_Rothstein’s picture

David_Rothstein commented

I'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.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.