Posted by berenddeboer on December 5, 2011 at 4:53am
13 followers
| Project: | Phone |
| Version: | 7.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | reviewed & tested by the community |
Issue Summary
Any phone field (http://drupal.org/project/phone) does not show up in the mapper.
Comments
#1
This is a duplicate of #959106: Feeds mapper for phone CCK fields, which was for 6.x.
Mapper support in feeds is only for drupal core fields. This request should be in the phone module. Perhaps a second feature request for 7.x mapper is in order also.
Here is a list of mappers and mapper requests: [#856780]
#2
Here's the patch.
#3
Applied this here, and it seems to work fine. The field shows up in the mapper, and appears to import correctly.
#4
#2 worked great...thanks!
#5
Dev version of the feeds 2.x module allows to simply add MODULE_NAME.feeds.inc file in the root of the module folder and it will load it automatically.
So I have refactored the patch from comment #2 to use this feature. Please review it.
#6
Patch in #5 works for me.
Thanks!
#7
i can confirm that #5 works. thanks!
#8
Patch applied and feeds brought in data.
My only complaint is to do with the fact that the phone field requires a default country. If I'm importing mexican, candadian, and US phone numbers, two of the three will fail validation. Sad, but fixable.
Josh
#9
The patch in #5 works, I'm not sure why the @file refers to link.module though.
#10
#5 works for me as well. I had to save the file, name it as "phone.inc" and put it in modules/feeds/mappers before it would work. I am reassigning it to the feeds project, as it would need to be applied there I believe. If I'm wrong, please switch it back to "Phone".
Thanks!
#11
@deggertsen - You'll need to use 7.x-2.x-dev of the Feeds module for now.
#12
Just confirming that #5 works for me as well.
#13
It worked for me too, but there is one little side effect. When using Views, and adding fields to a view, there are now TWO phone fields to choose from:
Content: Phone Number
Appears in: node:dealers.
Content: Phone Number (field_dealer_phone_number:delta)
Delta - Appears in: node:dealers.
I had to pick the first one to get anything to show.
#14
JSCSJSCS: That's not from this patch. Multiple-value fields provide that to views so you have access to the index of the value within the field.