In field.api.php, the sample implementation of hook_field_widget_error() is:

<?php
  form_error
($element['value'], $error['message']);
?>

It should be:

<?php
  form_error
($element, $error['message']);
?>

I'll post a patch soon.

Files: 
CommentFileSizeAuthor
#8 hook_field_widget_error-1614468-8.patch912 bytesJorrit
PASSED: [[SimpleTest]]: [MySQL] 39,146 pass(es).
[ View ]
#1 hook_field_widget_error-1614468-1.patch924 bytesJorrit
PASSED: [[SimpleTest]]: [MySQL] 36,688 pass(es).
[ View ]

Comments

Assigned:Jorrit» Unassigned
Status:Active» Needs review
StatusFileSize
new924 bytes
PASSED: [[SimpleTest]]: [MySQL] 36,688 pass(es).
[ View ]

The patch

Status:Needs review» Reviewed & tested by the community

yup

Status:Reviewed & tested by the community» Fixed

Committed to 8.x. Thanks.

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

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

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

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

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

Status:Patch (to be ported)» Needs review
StatusFileSize
new912 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,146 pass(es).
[ View ]

Here it is.

Status:Needs review» Reviewed & tested by the community

yup v.7

Status:Reviewed & tested by the community» Fixed

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.

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.