Download & Extend
Feeds » Issues

Path alias target when using pathauto

Posted by franz on June 29, 2012 at 1:36pm
Project:Feeds
Version:7.x-2.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

#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.

Login or register to post comments

Comments

#1

Posted by twistor on June 29, 2012 at 5:05pm

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

In setTargetElement() should do it.

#2

Posted by franz on July 3, 2012 at 11:52am
Status:active» needs review

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

AttachmentSizeStatusTest resultOperations
feeds-pathauto.patch1.48 KBIgnored: Check issue status.NoneNone

#3

Posted by twistor on July 3, 2012 at 4:24pm
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.

#4

Posted by franz on July 3, 2012 at 4:55pm
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

#5

Posted by System Message on July 17, 2012 at 5:01pm
Status:fixed» closed (fixed)

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