Closed (fixed)
Project:
Address Field Phone
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Mar 2016 at 11:35 UTC
Updated:
15 Apr 2017 at 17:22 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
msamavi commentedComment #3
agon024 commentedSame thing with me just downloaded this to use it for the first time and I am getting Notice: Undefined index: fax in addressfield_format_phone_generate() (line 69 of /foo/sites/all/modules/addressfield_phone/plugins/format/phone.inc).
Comment #4
cybermache commentedperhaps changing it to match the other settings?
so instead of this
change line 69 to this
Comment #5
nattsThanks cybermache, #4 works for me.
Will this fix be rolled into the next version of this module?
Comment #6
brainhell commentedThanks cybermache, #4 works for me very well. Another question: how can i change the phone fields order?
For example:
Mobile Phone number
Phone number
phone number extension
instead
Phone number
Phone number extension
Mobile phone number
Thanks advanced.......
Comment #7
cybermache commented@natts, I'm not a contributor on this module so can't say when or if this will be rolled into the next release. I believe the process is first a patch needs to be submitted then that patch needs to be confirmed by the community. Since this is one line and possibly something overlooked during the last update, it happens to me on occasion, it might just get added the next time around. I'm not going to hold my breath though.
@brainhell I would suggest opening a request issue for your question. From my experience it's best to keep these issues focused on the initial topic. Otherwise an issue could potentially never be resolved. Plus, it helps keep things clear for those trying to fix the original issue. I would also suggest clarifying where you are looking to change this order, when the field values are being displayed or entered.
Comment #8
balintcsaba commentedComment #9
neograph734Works great. Let's get this in the next dev.
Comment #10
vasikei confirm both : the issue and the patch solution from #8
and i think this is a bug.
Comment #11
coreykck commentedBug solved by #8
Tnx
Comment #12
madelyncruz commentedPatch #8 works. Thank you.
Comment #13
c_lehel commentedPatch #8 works fine. Thank you.
Comment #14
ollie222 commentedAnother thumbs up for patch #8.
Can we get this committed please?
Comment #16
jcisio commentedThanks. Committed and pushed.
Comment #17
jcisio commentedComment #19
br0kenSomeone, please, create a release with this fix! Tests for other module, which depends on this, are failing. :(
Comment #20
stijnbousard commentedNumber #4 works for me. Thanks!