Now that #717386: Stop supporting the user/x/contact paths and defer to Sub-path URL aliases is in, I think we should remove our 'feed' sub-path alias support as well. We should leave this as well to sub-path URL alias alter. Plus this helps lighten things up even more and removes what is a really confusing feature for users.

Comments

dave reid’s picture

Title: Stop supporting feed aliases » Stop supporting feed aliases (because they're sub-paths too!)
dave reid’s picture

Status: Active » Needs review
StatusFileSize
new9.04 KB
dave reid’s picture

Priority: Normal » Major

Bumping to major to get fixed before 7.0 official.

dave reid’s picture

Re-rolled against latest code.

greggles’s picture

Makes sense and looks good to me.

The variable_del isn't really complete. I don't care about it much, but if someone did care...a more complete technique would be to look for content type names in the node table to see what content types might have existed at some point and combine that with data from node_type.type.

dave reid’s picture

Status: Needs review » Reviewed & tested by the community

Let's just go with it for now then. Interpreting #5 as an RTBC.

dave reid’s picture

Version: 7.x-1.x-dev » 6.x-2.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed #4 to CVS.
http://drupal.org/cvs?commit=479416

Needs to be backported to 6.x-2.x

dave reid’s picture

Status: Patch (to be ported) » Fixed
dave reid’s picture

Version: 6.x-2.x-dev » 7.x-1.x-dev

Status: Fixed » Closed (fixed)

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