Allow spaces in from path

SubOracle - April 14, 2008 - 15:26
Project:Path Redirect
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:patch (to be ported)
Description

This patch transparently encodes and decodes the "From" path so that spaces may be included in the path.

AttachmentSize
path_redirect_allow_space_in_url.patch2.32 KB

#1

HorsePunchKid - April 14, 2008 - 16:02

Is this the same issue described here?

I'll test this out tonight, since I'd like to get 5.x-1.2 ready to release by tomorrow. No promises, but the patch does look pretty straightforward.

#2

SubOracle - April 14, 2008 - 22:08

Yes this appears to be the same issue.

#3

HorsePunchKid - April 16, 2008 - 02:54
Version:5.x-1.1» 6.x-1.x-dev
Status:patch (code needs review)» patch (to be ported)

I had to update this a bit to work with the latest dev version. In particular, it needed to account for the fact that the "from" path can include a query string, and urlencode()ing that is not good. You can view the diff here. I tested it out with some "from" paths like a b c?page=1, and it seems to work fine. Nonetheless, I'd appreciate it if you could test out the dev version.

 
 

Drupal is a registered trademark of Dries Buytaert.