Posted by mherchel on November 1, 2010 at 5:34pm
9 followers
| Project: | Phone |
| Version: | 6.x-2.18 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
Issue Summary
Feeds is a great module, but I need a mapper for the Phone (CCK) module (which defines a phone number CCK field and formats it) http://drupal.org/project/phone
I would do it myself, but I am not a developer and don't have a clue where to start.
Comments
#1
This should be really easy. I suggest you take a look at the content mapper and take it from there. This may also be able to be rolled as a patch to the content mapper.
#2
Postponing as there are no plans to work on this. I'd be happy to accept a patch though, in this case reset the issue status when you post.
#3
Attached is a draft of a phone_mapper patch. Just to repeat, this patch assumes you are using the "phone" module for CCK fields, I didnt explore the location module phones, etc.
I tested it in a non-vanilla install, using a xml processor, and it works fine, maps well 1->n fields, etc.
Note: I also have no idea how to write patches, (just a plumber, not a pro coder). All I did is hack the working patch for the email_mapper, and make some changes, apologies for any problems. Otherwise seems a straightforward patch (works with the latest release of the phone module, 6.x-2.18), no idea about the D7 ports, sorry.
Draft patch attached, Link to draft patch file here
#4
Can I activate this thread?
#5
There is an additional quotation that is causing the patch to break in line 57 ('int_phone''). Removed in file attached.
#6
Feeds should only support core module feed mappings. This support should live in Phone module.
#7
subcribing
need this patch for the 7.X dev version
don't know finally if I have to create the same feature request for the 7 dev .. ?
thx all
#8
gazwai, see this patch.