?

User tests: Successful: Unsuccessful:

avatar jo-sf
jo-sf
9 Sep 2014

Suppose you have a web server using Joomla where this web server is both reachable via http://www.example.org/ and http:/example.org/. With the current implementation of the redirection plugin you need to define two redirections within the redirect manager component for every page you want to automatically redirect from its old location to its new location:

old page: http://www.example.org/path/to/old/page.html
=> new page: http://www.example.org/path/to/new/page.html

old page: http://example.org/path/to/old/page.html
=> new page: http://example.org/path/to/new/page.html

With my proposed file change it would be sufficient to define only one redirection in this case:

old server-relative page: /path/to/old/page.html
=> new server-relative page: /path/to/new/page.html

avatar jo-sf jo-sf - open - 9 Sep 2014
avatar jissues-bot jissues-bot - change - 9 Sep 2014
Labels Added: ?
avatar jo-sf
jo-sf - comment - 9 Sep 2014

zero-24, I made the changes in my patch as you suggested, but please keep in mind that I simply mimicked the code style you find some lines above my insertion: the { are often not in a line for themselves and the string concatenation in the lines 55-57 is also written without spaces. Maybe these lines should be changed too in order to have a consistent code style here?

avatar zero-24
zero-24 - comment - 9 Sep 2014

@jo-sf

Maybe these lines should be changed too in order to have a consistent code style here?

Oh sorry this are the 2.5.x PR. There we not use all types of CS. But if you send the seccond PR against the staging branche yes this lines should be changed too but the most is allready done there :)
https://github.com/joomla/joomla-cms/blob/staging/plugins/system/redirect/redirect.php

avatar jo-sf
jo-sf - comment - 9 Sep 2014

@zero-24

I changed my patch for 2.5.x as well according to the patch built against staging since the same problem might arise here as well (see #4255).

avatar zero-24
zero-24 - comment - 10 Sep 2014

thanks @jo-sf it also work ok on j2.5 using the last Nightly Build

This comment was created with the J!Tracker Application at http://issues.joomla.org/.

avatar zero-24 zero-24 - change - 10 Sep 2014
Category Front End Plugins
avatar zero-24 zero-24 - change - 10 Sep 2014
Easy No Yes
avatar b2z
b2z - comment - 17 Sep 2014

@test ok on version 2.5.24

This comment was created with the J!Tracker Application at http://issues.joomla.org/.

avatar zero-24
zero-24 - comment - 17 Sep 2014

thanks @b2z moving to RTC now

This comment was created with the J!Tracker Application at http://issues.joomla.org/.

avatar zero-24 zero-24 - change - 17 Sep 2014
Status Pending Ready to Commit
avatar phproberto phproberto - change - 8 Oct 2014
Labels Added: ?
avatar phproberto
phproberto - comment - 10 Oct 2014

Fixed in 2.5.x e087b5b

Thanks!!

avatar phproberto phproberto - close - 10 Oct 2014
avatar phproberto phproberto - change - 10 Oct 2014
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2014-10-10 12:21:14

Add a Comment

Login with GitHub to post a comment