Closed (won't fix)
Project:
Pathauto
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
14 Apr 2008 at 22:14 UTC
Updated:
10 Jan 2010 at 00:42 UTC
If one entered a value in Internal feed alias text and generated internal feed aliases, those are not deleted if the value removed and left blank.
Comments
Comment #1
xolotl commentedOn further consideration, perhaps better UI integration of path bulk delete and pathauto would address this issue, as per http://drupal.org/node/246658.
Perhaps a note something like the following could be
"The text to use for aliases for RSS feeds. Examples are "feed" and "0/feed". To remove existing feed aliases, remove text here and delete aliases for all content. You can then regenerate aliases for all content by checking the "Bulk generate aliases" box above and using your site's cron processes to regenerate all missing aliases."
Comment #2
gregglesThere is code now that attempts to delete feeds even if that feature is disabled. Can you clarify what you put into the "Internal feed alias" text box? I think that perhaps the code that attempts to delete the feed aliases needs to be made more robust...
Comment #3
gregglesI re-read this issue and understand the problem a little better, I think. When you blanked out the "/feed" box you expected all of the /feed aliases to be deleted but they weren't.
Do you think this is still necessary now that the Pathauto settings are better integrated into the paths area and are near the bulk deletion?
Comment #4
gregglesbetter status...
Comment #5
xolotl commentedhmm: I see nice pathauto integration in the paths bulk delete area in 5.x.2.x.dev, but NOT for feed aliases.
I guess I was seeking a way to bulk delete all feed aliases. It makes sense to locate that in the path bulk delete page along with all the other pathauto bulk delete controls.
My original idea of feed aliases being deleted when one emptied the "Internal feed alias text" field is probably not as good a solution.
Comment #6
gregglesWell, this can become a feature request for that then. Good idea.
It should probably be in 6.x-2.x-dev.
Comment #7
xolotl commentedSounds like the right solution...thanks for sticking with this.
Comment #8
gregglesI no longer feel this is worthwhile to add to the UI cruft.