FeedAPI Mapper
guix - June 13, 2009 - 15:40
| Project: | CCK Redirection |
| Version: | 6.x-1.1 |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | postponed |
Description
Great module, thanks !
I'd love to see a mapping to CCK Redirection.
I'll try to submit a patch when I got a chance.
#1
Here it is, in a .zip since it's just a new file in the mappers directory.
#2
I'd love to see this going into the CCK redirection module as it's not a widely used module like date or cck or taxonomy.
#3
#4
I should have added if (module_exists('cck_redirection')) too, would it be better like that ? But it will be as you like, just confirm me and I'll try my chance in the CCK redirection module.
#5
guix: it's not about module_exists() or any other technical question, it's rather that I'm trying to limit mappers that are included with feedapi_mapper to the core module + popular contrib modules. Otherwise I'm left with a plethora of third party module integrations to maintain...
Moving this to CCK Redirection now. Patch needs work to integrate with CCK redirection.
#6
No pb Alex, I understand your reasons. I'm gonna update the patch to include it in CCK Redirection. And thanks for the module, it helps a lot in my project !
#7
Let's first clean up and fix CCK Redirection and have a new release before adding stuff.
#8
I can provide a patch as soon as we have an URI validation (see #445894: URI validation and cleanup).
But Robin, would you agree to include this as a sub-module ? Or should I create a new module like "FeedAPI Mapper contrib" for this one and maybe others that won't make it into FeedAPI Mapper ?