Do not rewrite <front> in redirect rules

jcarnett - October 28, 2009 - 20:31
Project:Path redirect
Version:6.x-1.0-beta4
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Currently when you use in the redirect destination it is resolved to the real path of the site's front page. This seems like a bug to me, as does not mean the same thing as the "real" path. For example, if you later changed the site's front page, your redirects would be going to the wrong page. The redirects work perfectly if the rewriting is removed, and administration feels a bit more consistent to boot. Patch attached.

This is somewhat related to http://drupal.org/node/454518

AttachmentSize
path_redirect.front_.patch569 bytes

#1

jcarnett - October 28, 2009 - 20:35

Look at me forgetting to escape the < and >. Corrections:

  1. ... when you use <front> in the redirect destination ...
  2. ... as <front> does not mean the same thing ...

#2

Dave Reid - November 4, 2009 - 06:00
Status:needs review» fixed

Thanks for the patch. I basically committed the same thing to CVS, but with an upgrade path if it finds any redirections to the current homepage.

#3

System Message - November 18, 2009 - 06:10
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.