joomla-cms icon indicating copy to clipboard operation
joomla-cms copied to clipboard

[4.4] Fixing non-SEF URL in feed output

Open Hackwar opened this issue 1 year ago • 16 comments

Pull Request for Issue #39775.

Summary of Changes

When Joomla is installed in a subdirectory and URLs are set to be completely non-SEF, the URLs in a feed output are wrong. Specifically, the path to the root of Joomla is doubled, for example https://domain.tld/subdir/subdir/index.php instead of just https://domain.tld/subdir/index.php It works fine with SEF URLs enabled.

Testing Instructions

  1. Install Joomla in a subdirectory and disable all SEF URL options
  2. Go to a category view and look at the feed URL in the header.
  3. Download the feed and look into the output.

Actual result BEFORE applying this Pull Request

The link in for example <atom:link rel="self" type="application/rss+xml" is wrong

Expected result AFTER applying this Pull Request

The link in for example <atom:link rel="self" type="application/rss+xml" is correct.

Link to documentations

Please select:

  • [ ] Documentation link for docs.joomla.org:

  • [X] No documentation changes for docs.joomla.org needed

  • [ ] Pull Request link for manual.joomla.org:

  • [X] No documentation changes for manual.joomla.org needed

Hackwar avatar Feb 13 '24 11:02 Hackwar

@Hackwar Is there a reason why not fixing it for 4.4-dev?

richard67 avatar Feb 13 '24 11:02 richard67

Numerous of my PRs for 4.4 have been declared as new features even though they were bugfixes and rejected, so I stopped creating PRs for 4.4.

Hackwar avatar Feb 13 '24 11:02 Hackwar

@Hackwar New features go into 5.1-dev. Bug fixes for bugs which affect 4.4 and 5.x go into 4.4-dev. Into 5.0-dev only bug fixes go where the bug doesn’t affect 4.4.

richard67 avatar Feb 13 '24 12:02 richard67

I'm very well aware of that, but my time is not free and if I create bugfixes like #42693 and get told that this PHP error is not something which will be fixed in 4.4, then I'll stop creating fixes for 4.4. I don't have to waste my time and the time of other contributors by creating a PR for 4.4, organising people to test that PR, them investing the time to actually test, and then be told that an obvious bug is not a bug and that the PR is going to be rejected. So I'm skipping that whole part and create PRs against the next branch up.

Hackwar avatar Feb 13 '24 13:02 Hackwar

I have tested this item :white_check_mark: successfully on fda45a77a9424fe399465d9596ccfe9b8f4525ea


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42802.

Globulopolis avatar Feb 22 '24 10:02 Globulopolis

I have tested this item :white_check_mark: successfully on fda45a77a9424fe399465d9596ccfe9b8f4525ea


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42802.

viocassel avatar Feb 22 '24 16:02 viocassel

@Hackwar After our talk yesterday in the CMS Maintenance Team you said you will make this PR for 4.4-dev. Will you rebase this one, or will you make a new one? I'm asking because this one here has 2 good tests so it would be RTC (if not for the wrong branch).

richard67 avatar Feb 22 '24 17:02 richard67

I recreated the PR with 4.4-dev as base branch. The code is identical. Do we need new tests?

Hackwar avatar Feb 24 '24 13:02 Hackwar

@Globulopolis @viocassel Could you test this PR again on 4.4.x? Thanks in advance.

richard67 avatar Feb 24 '24 13:02 richard67

I have tested this item :white_check_mark: successfully on b3ef2726f8fe3ce94afee435c5dca30f495dd3b9

👌


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42802.

viocassel avatar Feb 24 '24 13:02 viocassel

I have not tested this item.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42802.

den1ska07 avatar Feb 24 '24 14:02 den1ska07

I have not tested this item.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42802.

den1ska07 avatar Feb 24 '24 14:02 den1ska07

I have not tested this item.

@den1ska07 If you submit a test result you have to select the right result before submitting.

richard67 avatar Feb 24 '24 15:02 richard67

I have tested this item :white_check_mark: successfully on b3ef2726f8fe3ce94afee435c5dca30f495dd3b9


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42802.

Globulopolis avatar Feb 24 '24 17:02 Globulopolis

I have tested this item :white_check_mark: successfully on b3ef2726f8fe3ce94afee435c5dca30f495dd3b9


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42802.

den1ska07 avatar Feb 24 '24 17:02 den1ska07

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/42802.

richard67 avatar Feb 24 '24 17:02 richard67

Thanks!

laoneo avatar Mar 04 '24 15:03 laoneo