#1001590: Path alias mapping target added Path alias mapper, but Pathauto module overwrites it when saving the node. I'm working on a patch for this, as pathauto is a widely used module.

CommentFileSizeAuthor
#7 feeds-1665450.patch1.21 KBkaustubhb
#2 feeds-pathauto.patch1.48 KBfranz

Comments

twistor’s picture

twistor commented 
$target_node->path['pathauto'] = FALSE;

In setTargetElement() should do it.

franz’s picture

franz
any
Location Montréal
commented
Status: Active » Needs review
StatusFileSize
new feeds-pathauto.patch1.48 KB

Yes, I had tested this and it's working fine. If anyone can just sign-off, I'll commit it.

twistor’s picture

twistor commented
Status: Needs review » Reviewed & tested by the community

This should be revisited if config for mappers gets in. Making the pathauto behavior configurable would be nice, but this seems like the most sane default behavior.

franz’s picture

franz
any
Location Montréal
commented
Status: Reviewed & tested by the community » Fixed

+1 to make this configurable later. I'll open the issue so we don't forget it: #1670006: Path mapper config

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

cayenne’s picture

cayenne commented

Is this patch committed and working? I'm importing a csv with urls specified, but the URL alias is getting overwritten. I hate to reopen the bug if there's some other thing going on. Using the dev version.

Update: Never mind. Was using "url" rather than Path alias. My error...

kaustubhb’s picture

kaustubhb commented
Issue summary: View changes
StatusFileSize
new feeds-1665450.patch1.21 KB

Hi I am using the Feeds 2.0 but I don't see these changes in the code base. I have added a patch in my setup submitting it here.

I have a use case where I want to import the aliases along with the node data. Can we re-open the this bug?