Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Please make a note on the project page that this module may/will interfere with GlobalRedirect (which makes an effort to strip trailing slashes for improved SEO).
Comment | File | Size | Author |
---|---|---|---|
#5 | GlobalRedirect.GIF | 31.23 KB | Akaoni |
Comments
Comment #1
Akaoni CreditAttribution: Akaoni commentedGood pickup, Nicholas (and very quick too)!!
I've just installed GlobalRedirect so will test and address/note any issues.
Comment #2
Akaoni CreditAttribution: Akaoni commentedComment #3
Akaoni CreditAttribution: Akaoni commentedWith both Trailing Slash and GlobalRedirect > Deslash enabled, any requests to URLs with a trailing slash (eg. example.com/user/) currently redirect in an infinite loop.
This GlobalRedirect patch will fix this:
http://drupal.org/node/1130210
I've also added support to Trailing Slash for the Drupal 6.x custom_url_rewrite_outbound function.
This fixes the issue for current versions of GlobalRedirect.
This means however that when Trailing Slash is enabled, the GlobalRedirect > Deslash setting does nothing.
I'll note this on the Trailing Slash project page.
Would you also like me to add code that disables the GlobalRedirect > Deslash setting and explains why a user can't enable it?
Comment #4
nicholasThompsonHmm. You could either disable the option (ie force it) or use hook_requirements to check if it's enabled and flag a problem if it is?
Did you test against DEV or one of the "supported" tagged releases?
Comment #5
Akaoni CreditAttribution: Akaoni commentedI tested against GlobalRedirect version 7.x-1.3 with and without the above mentioned patch.
I've disabled the option and provided users with an explaination message that links to the Trailing Slash config page (see attached image).
Comment #6
Akaoni CreditAttribution: Akaoni commented