[#12210] - Change base (URL) for JDocument to meet specifications
- Closed
- 29 Sep 2016
- Medium
- Build: staging
- # 12210
- Diff
- matrikular:patch-14
- Success continuous-integration/drone the build was successful Details
- Success continuous-integration/travis-ci/pr The Travis CI build passed Details
- Pending JTracker/HumanTestResults Human Test Results: 1 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Summary of Changes
Currently, if SEF was enabled, the base tag that gets rendered in the head of HTML and Feed documents shows the current address. According to the specification, it should be the base path of the installation only.
Testing Instructions
Open a source view (window or tab) for your site and look for the tag within the head section. Where for an installation on localhost (no vhost, installed in a sub-folder) the base address for the frontpage should be something similar to this: <base href="http://localhost/myjoomla/" />.
For an online installation this could be: http[s]://[subdomain].domain.tld/" />, which would be fine either way.
Now, if you create an article called Test and a menu item to that article, the base tag without the patch looks like this: http[s]://[subdomain].domain.tld/test"[.suffix] /> or, if you linked the article via a category: http[s]://[subdomain].domain.tld/test/1-test[.suffix]" />.
After applying the patch the base tag should only include the base or "root" path / address of the site. This, thanks to @zero-24 who did some tests, also works for Joomla! sites that have been installed into a (nested) subfolder.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Category | ⇒ | Libraries |
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-09-29 20:02:00 |
| Closed_By | ⇒ | matrikular |
Maybe we could ask for this change in J4, and other code that needs to be updated, e.g. multi-page articles can be updated too
(if it can be done without problems, with bookmarked URLs and google indexed pages)
I have tested this item✅ successfully on 53b058f
works here. Thanks
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/12210.