Add an option for case-insensitive redirects

jcarnett - October 28, 2009 - 20:43
Project:Path redirect
Version:6.x-1.0-beta4
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:needs review
Description

The attached patch adds an option within the Advanced fieldset to make the redirect case insensitive so it behaves like Apache mod_rewrite's [NC] flag.

AttachmentSize
path_redirect.ignore_case.patch4.35 KB

#1

jcarnett - October 28, 2009 - 23:08

I just remembered that MySQL string comparisons are case-insensitive by default, so this is really only applicable to PostgreSQL folks.

I suppose that leads to another question then: should the path lookups be done in a case-sensitive manner in MySQL? I think initially I compared this module to Apache redirects which are case sensitive, but I'd like to know what everyone else thinks.

#2

Dave Reid - November 4, 2009 - 02:58

I don't see any reason why we should be allowing case-sensitive redirections. Let's make the default behavior case-insensitive.

 
 

Drupal is a registered trademark of Dries Buytaert.