Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.This field cannot be imported via Feeds.
Is this possible by creating a mapper for it?
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | cck_phone_feeds_1160186_7.patch | 2.25 KB | franz |
| #7 | cck_phone_feeds_1160186_7.patch | 2.18 KB | Anonymous (not verified) |
| #4 | cck_phone_feeds-1160186-4.patch | 2.69 KB | Anonymous (not verified) |
| #2 | cck_phone_feeds-1160186-2.patch | 2.67 KB | Anonymous (not verified) |
Comments
Comment #1
ckngWelcome any patch on this.
Comment #2
Anonymous (not verified) commentedAttached is a patch to add feed support to CCK Phone.
Comment #3
ckngI checked #1139676: feeds_alter doesn't support hook_module_implements_alter, hook_hook_info and hook_hook_info_alter is now fixed. Could you reroll and test it on your side?
Don't use feeds importer ATM, so will have to trust you on that, but the code looks fine.
Comment #4
Anonymous (not verified) commentedRerolled patch. Have tested in two different install profiles and this patch works well.
Comment #5
Anonymous (not verified) commentedChanging to RTBC.
Comment #6
ckng@kevee
I'm referring to this piece of code
Do you still need this since #1139676: feeds_alter doesn't support hook_module_implements_alter, hook_hook_info and hook_hook_info_alter is fixed?
Comment #7
Anonymous (not verified) commentedSorry, wrong file. Correct, we no longer need that line if the user is on Feed 2.x-dev, but that has not been implemented in a numbered release yet.
Comment #8
franzI think it's useful to add an strtolower() on Country code. In my use case, I use the same country code for and Addressfield, and it requires it to be uppercase, so with this version of the patch, I was able to use the same value on both targets.
Other than that, it works great!
Comment #9
franzComment #10
jibus commentedHi !
Tested (#8) with an import of 100 phone members. I used country code & phone number in Feeds.
Countries phone tested: Malaysia, France, Italy, Austria, Portugal, Spain, United Kingdom, Germany, Belgium, Mexico, Uruguay, Hong Kong, South Africa, Taiwan, Switzerland, Israel, Brazil, Argentina, Russian Federation, China, Latvia, Netherlands, Croatia, South Korea, Thailand, Colombia, Japan, Greece, Poland, Czech Republic, Slovenia, Indonesia, Romania, Australia, Ghana, Morocco, Lebanon, Serbia, Sweden, India, Denmark, United States, New Zealand, Singapore, Viet Nam, Hungary, Chile and Canada
Worked like a charm, thanks !
Comment #11
jibus commentedThere is a new "stable" version of feeds 7.x-2.0-alpha5 (2012-May-28), maybe it's time to apply this patch against the 7.x-1.x-dev ?
Comment #12
johnvJibus, this patch is release-independent, as is contains is a new file, with a 'plugin' with a pre-defined interface.
Comment #13
jibus commented@johnv
I don't understand why this feature (which I think is very useful) couldn't be part of the module. Some users who are not very familiar in patching could be interereted.
Comment #14
johnvOK, then I misunderstood. I thought you asked for a new version of the patch, but you meant to "commit this patch against the 7.x-1.x-dev of Phone Number ?"
My bad.
Comment #15
jibus commentedYes yes ! Sorry my comment wasn't very clear: my english is not perfect ^^
Comment #16
franzComment #17
dddbbb commentedSo, did this patch get committed to 7.x-1.x-dev or do I still need to patch?
Comment #18
jibus commentedNope, you have to apply this patch against the current dev version if you want to import data into phone number fields
Comment #19
ckngThanks for all the works, committed to D7.