I am trying to add the redirect to a path that contains the following:

http://www.example.com:8080/newpage

No matter how hard I tried, the module just ignore the :8080 port number.

Can you fix this?

Thanks in advance.

CommentFileSizeAuthor
#5 path_redirect.admin_.inc_.patch771 bytestimmillwood

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Title: Add the port number to the redirected path » Port numbers stripped from redirection URL
Version: 6.x-1.0-beta6 » 6.x-1.x-dev
Assigned: Unassigned » dave reid
Category: feature » bug

Good find, this is a bug that I need to fix.

hoobuba’s picture

hoobuba commented

Is there any update on this? - thank you!

Mantis544’s picture

Mantis544 commented

Any news ?

http://drupal.org/node/1014358

I have the same problem... Dave ?

mastervissie’s picture

mastervissie commented
Status: Active » Needs review

This problem can be fixed to change 1 single line of code: change in file path_redirect.admin.inc line 443 (version 6.x-1.0-rc2) to:
$form_state['values']['redirect'] = (isset($parsed['scheme']) ? $parsed['scheme'] . '://'. $parsed['host'] : '') . (isset($parsed['port']) ? ':' . $parsed['port'] : '') . $parsed['path'];

This added the port number (if exists) to the redirect variable.

timmillwood’s picture

timmillwood
Primary language English
Location 🏴󠁧󠁢󠁷󠁬󠁳󠁿 Wales, UK
commented
StatusFileSize
new path_redirect.admin_.inc_.patch771 bytes

Here's this as a real life patch.

Status: Needs review » Needs work

The last submitted patch, path_redirect.admin_.inc_.patch, failed testing.

timmillwood’s picture

timmillwood
Primary language English
Location 🏴󠁧󠁢󠁷󠁬󠁳󠁿 Wales, UK
commented
Status: Needs work » Reviewed & tested by the community

Seems to be working.