Closed (duplicate)
Project:
Pathauto
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
10 Nov 2009 at 13:34 UTC
Updated:
16 Mar 2010 at 15:39 UTC
Jump to comment: Most recent file
Comments
Comment #1
xenophyle commentedI am also seeing this problem. It looks like taxonomy_pathauto_bulkupdate() is testing whether an alias for the taxonomy term exists before creating the feed alias, rather than checking if the the feed alias exists.
If the term alias already exists, you won't get your feed alias.
Comment #2
xenophyle commentedI've made a patch that takes a rather timid/conservative approach but seems to do the job. In taxonomy_pathauto_bulkupdate, it was checking whether the taxonomy term existed, and if not it called _taxonomy_pathauto_alias to create the taxonomy alias and, if requested, the feed alias as well. I added a condition to call _taxonomy_pathauto_alias if the feed alias does not exist. I also noticed that the count of how many aliases were created is off, since it doesn't count the taxonomy aliases and the taxonomy feed aliases separately, but I made no attempt to fix it.
A more aggressive solution might be to divide the _taxonomy_pathauto_alias function to handle creating the two types of aliases separately. This could fix the count issue too.
Comment #3
dave reidTagging all the bulk alias issues for #713238: RFC: Pathauto Bulk module.
Comment #4
hawleyal commentednot compatible with postgres
Comment #5
dave reidThis will be fixed with #201151: Use batch API to perform path alias bulk updates. As such I'm going to mark it as a duplicate of that issue.