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

CommentFileSizeAuthor
path_redirect_allow_space_in_url.patch2.32 KBrichard banks

Comments

HorsePunchKid’s picture

HorsePunchKid commented

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.

richard banks’s picture

richard banks commented

Yes this appears to be the same issue.

HorsePunchKid’s picture

HorsePunchKid commented
Version: 5.x-1.1 » 6.x-1.x-dev
Status: 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.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Patch (to be ported) » Fixed

Pretty sure this works now with the development going on with the module.

Status: Fixed » Closed (fixed)

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