Posted by SubOracle on April 14, 2008 at 3:26pm
Jump to:
| Project: | Path redirect |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
This patch transparently encodes and decodes the "From" path so that spaces may be included in the path.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| path_redirect_allow_space_in_url.patch | 2.32 KB | Ignored: Check issue status. | None | None |
Comments
#1
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
Yes this appears to be the same issue.
#3
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 likea b c?page=1, and it seems to work fine. Nonetheless, I'd appreciate it if you could test out the dev version.#4
Pretty sure this works now with the development going on with the module.
#5
Automatically closed -- issue fixed for 2 weeks with no activity.