Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Inserttags: Fix domain for article_url, news_url, event_url and faq_url inserttags #8212

Closed
fatcrobat opened this issue Feb 8, 2016 · 2 comments

Comments

@fatcrobat
Copy link

Hi,

i was wondering why the article_url, news_url, event_url and faq_url inserttags does not have the $blnFixDomain Parameter set within Controller::generateFrontendUrl.

All their archives belong to an jumpTo Page, so it should be possible to add an check like the following:

$objTarget = $objPid->loadDetails();
$blnFixDomain = $objTarget->domain != '' && $objTarget->domain != \Environment::get('host')

Maybe i missed somethin, but please take into consideration.

Sheers.

@fritzmg
Copy link
Contributor

fritzmg commented Feb 8, 2016

👍

As I said before, this should be fixed eveywhere, since it is simply a bug in my oppinion. There are multiple instances where Contao generates the wrong URL for pages that are either on a different domain or a different language than the current one. See also #3765, #6743, #7922 and #8204.

@leofeyer
Copy link
Member

See #8204.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants