User tests: Successful: Unsuccessful:
Fixed https base tag generation.
Discussion is here:
#4961 (comment)
Easy | No | ⇒ | Yes |
Labels |
Added:
?
|
Category | ⇒ | Libraries |
Title |
|
Rel_Number | 0 | ⇒ | 4961 |
Relation Type | ⇒ | Pull Request for |
Title |
|
I think the issue might be in libraries/cms/application/site.php
Which has:
if ($router->getMode() == JROUTER_MODE_SEF)
{
$document->setBase(htmlspecialchars(JUri::current()));
}
Shouldn't that be?
if ($router->getMode() == JROUTER_MODE_SEF)
{
$document->setBase(htmlspecialchars(JUri::base()));
}
Peter provided solution, but it wasn't included in Joomla 3.4.3. Please add this patch in next Joomla version. Thank you.
If someone opens a pull request with his suggested change and it gets tested then it can be included in a future release. Until then, there isn't much that can be done. http://magazine.joomla.org/issues/issue-sept-2015/item/2837-did-they-fix-it-yet offers some good insight on how you can help ensure issues that you are having with the core software get resolved
Michael, thanks for the link. Sure, it's important to test all patches.
Well, I've tested this patch on various sites with SSL and Varnish - it works. Since the status of this fix is marked as checked I thought it's already revised.
The checks block just above the comment box here shows the status of our automated test suites, mainly to make sure that unit tests and codestyle checks continue to pass. Unless a pull request has the RTC
label or has already been merged, then it still needs human review and testing.
Ok, thanks for explanation. How many reviews are usually required to accept the patch?
Usually two.
@GregoryRusakov can you update your PR to @nonumber solution? Seems that it is the right one.
@b2z I sent PR #7902 which implements solution suggested by @nonumber. @GregoryRusakov please test mentioned PR.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-09-17 23:43:51 |
Closed_By | ⇒ | zero-24 |
Reading the code, it seems as if the base is set wrong somewhere. If you don't use getBase(), we can't influence this from for example a plugin. So why are you not fixing the place where the base in JDocument is set to a false value and instead replace the call with the static JURI::base()?