?

User tests: Successful: Unsuccessful:

avatar jo-sf
jo-sf
22 Sep 2014

If the URL "index.php" was given, it was converted into "http://http://<servername>/index.php", and an URL starting with "index.php" (e.g. "index.php/sample-sites") was converted into (in this example) "http://index.php/sample-sites". Now "index.php" and e.g. "index.php/sample-sites" are treated identically, they are converted into "http://<servername>/index.php" resp. "http://<servername>/index.php/sample-sites".

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

@test
This works here.

avatar b2z
b2z - comment - 24 Sep 2014

I suppose that this should be added to 3.x also?

avatar jo-sf
jo-sf - comment - 25 Sep 2014

@b2z Initially I made this change as part of a greater change ("Allow relative URLs for links A/B/C in articles") both for 2.5.x and 3.x, but them infograf768 informed me that this is a new feature which will not be added to the 2.5.x branch, and he instructed me to only add the bug fix regarding index.php handling as a new PR to the 2.5.x branch.

So you find this bug fix as part of the PR #4310 for the 3.x branch. If desirable I could split that PR into one with the new feature ("Allow relative URLs ...") and another similar to PR #4321 ("correct index.php handling"), both for the 3.x branch.

avatar b2z
b2z - comment - 25 Sep 2014

@jo-sf thank you for explanation.

If desirable I could split that PR into one with the new feature ("Allow relative URLs ...") and another similar to PR #4321 ("correct index.php handling"), both for the 3.x branch.

I think that there is no need. #4310 is ok ;)

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

avatar infograf768 infograf768 - change - 25 Sep 2014
Labels Added: ?
avatar infograf768 infograf768 - close - 25 Sep 2014
avatar infograf768 infograf768 - change - 25 Sep 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-09-25 13:42:18
avatar zero-24 zero-24 - change - 14 Oct 2015
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment