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.
Just discovered the following notices after adding a 2 different phone fields in a dummy content type.
Notice : Undefined index: country in phone_field_instance_settings_form() (line 115 in /docroot/sites/all/modules/contrib/phone/phone.module).
Notice : Undefined index: country in phone_field_instance_settings_form() (line 132 in /docroot/sites/all/modules/contrib/phone/phone.module).
Notice : Undefined index: country in phone_field_widget_form() (line 236 in /docroot/sites/all/modules/contrib/phone/phone.module).
Comment | File | Size | Author |
---|---|---|---|
#1 | phone-notice-1713148-2.patch | 2.1 KB | mgifford |
Comments
Comment #1
mgiffordCan confirm this. Here's a patch that should address it.
Comment #2
rfsbsbHow can I reproduce this?
Comment #3
anavarreIt's been a while but trying to reproduce now didn't show anything wrong. Can't remember exactly how I managed to get these errors in first place. @mgifford, maybe you remember?
Comment #4
rfsbsbI thought it could be related to some migration problem or even some buggy installation. Another situation is an old option for empty country that doesn't exists anymore.
Anyway, I will wait for @mgifford answer. If we found out it's not happening anymore, we declare it fixed.
Comment #5
mgiffordSorry.. Was too long ago.
Comment #6
rfsbsbI will consider it fixed. If someone else find a way to reproduce this, we can reopen.
Comment #8
seanrThis is not fixed. Just downloaded phone onto a fresh Panopoly instance and tried to add a phone field to a content type. Got the same three errors the original author got. I'm guessing there's some config setting it's expecting?
The default value for the country field in the phone field settings for should try to default to the site's country field and fall back to international if necessary, IMO.
Comment #9
Matt Habermehl CreditAttribution: Matt Habermehl commentedI can confirm that this is an issue on Panopoly installs.