Notice: Undefined index: message in field_default_form_errors() (line 381 of /var/www/drupal/modules/field/field.form.inc).

Here's a patch to solve this.

Please review this patch.

Per the "hook_field_validate" API (http://api.drupal.org/api/drupal/modules!field!field.api.php/function/ho...), this is what we should change line 156 to:

$errors[$field['field_name']][$langcode][$delta][] = array(
  'error' => 'phone_invalid_number', 
  'message' => t($country['error'], array('%value' => $value)),
);

And we should add this:

/**
 * Implements hook_field_widget_error().
 */
function phone_field_widget_error($element, $error, $form, &$form_state) {
  form_error($element, $error['message']);
}

This is what I did and it solved the issue.

As suggested by seworthi, I changed the patch to fit correctly to the API.

Hi acissej, first you must checkout the latest version from repo (instructions here http://drupal.org/node/87303/git-instructions/7.x-1.x). Then you should download the patch and apply using the instruction in the link above in the section "applying a patch"

Hope it helps.

In fact this is the correct patch. Sorry.

Rerolled patch - no longer removes the extra newline at the end of the file and applies cleanly to 7.x-1.x-dev with or without git.

This patch fix the error, but validation only show error message and pass the wrong value....

Looks good. I just applied this to the git repo and tested it.

It's much better, but I got it because I wanted to put in an area code only into the default content.

It would be nice if the (613) were already there so folks just had to type in the 555-1212 after it.

Errors:

1. 1 in valid_int_phone_number() (line 42
2. message in field_default_form_errors() (line 381

Well, that's not good. What were you doing?

Can you give us some details so we can repeat it? This is for international phone numbers?

Rerolled patch #10 against branch 7.x.1.x

#15 Works well for me.

#15 works great for me, too.

Is this patch going to be committed at least to the dev version? It's urgent!

You shouldn't use any variables when calling t() function. Only string literals with placeholders.
See http://api.drupal.org/api/drupal/includes!bootstrap.inc/function/t/7

Hello. I am getting the same error as the OP when a user tries to register for my site including a South African phone #. I am validating against US & Canada though we have international users signup on our site and I didn't think they wouldn't be allowed to enter a non-US & Canada phone #. The phone # was written like this: 27215555555 (obviously different digits than the 5's).

According to the phone.info file, the version of phone-7.x-1.x-dev is from 2012-03-02.

I perused the README file and it does not confirm support for South African phone numbers though I notice in the phone.module file there is a line for South Africa.

This was brought to my attention by a user that emailed me through the contact form on our site. He gave me screenshots of his signup process and I went through each field to find the problem. I am very confident the problem lines with this module.

What do I need to do to solve this issue? Should I update to a newer dev version or apply one of these patches? Thank you.

Technically, I shouldn't be okay with having a variable inside t() like that, but since we're already doing it and we're getting rid of a notice, I can accept this for now. Over to rfsbsb for final review and commit.

The patch doesn't apply.

Because it's been applied!

Indeed this patch has already been applied http://drupalcode.org/project/phone.git/commit/ab36332

Sorry about that!

Thanks all! I spoke with cweagans on IRC and he applied the patch. I should have updated this thread.