Project

General

Profile

Actions

Bug #18432

closed

Linking between domains in the pagetree with checkRootline does not work with menus

Added by Benni Mack about 16 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Should have
Category:
Communication
Target version:
-
Start date:
2008-03-12
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.2
PHP Version:
5.0
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

Hey all,

the new solution by Dmitry only works for typoscript, but does not work for menus.

config.typolinkCheckRootline=1

This feels inconsistent to me: If people turn on this function, they should know that this works all the time.

(issue imported from #M7839)


Files

7839.diff (8.05 KB) 7839.diff Administrator Admin, 2008-03-27 14:17
7839-small.diff (7.29 KB) 7839-small.diff Administrator Admin, 2008-03-27 14:17
7839-v3-small.diff (9.05 KB) 7839-v3-small.diff Administrator Admin, 2008-03-28 13:21

Related issues 5 (0 open5 closed)

Related to TYPO3 Core - Bug #16147: Linking between domains in the pagetree ignores simulate static documentsClosedDmitry Dulepov2006-05-15

Actions
Related to TYPO3 Core - Bug #16182: Domain Substitution in external linkClosed2007-07-30

Actions
Related to TYPO3 Core - Bug #15202: links to pages with a different sys_domain recordClosedChristian Kuhn2005-11-15

Actions
Is duplicate of TYPO3 Core - Feature #14206: typolinkCheckRootline don't workClosed2004-06-16

Actions
Is duplicate of TYPO3 Core - Bug #17941: typolinkCheckRootline not working with multiple websites under one umbrella website.ClosedHelmut Hummel2007-12-19

Actions
Actions #1

Updated by Oliver Hader about 16 years ago

The reason is, that tslib_menu does not use $cObj->typoLink to create these links - a work around for TypoScript is the following:

NO {
doNotLinkIt = 1
stdWrap.typolink.parameter.field = uid
}

But, of course this should be fixed.

Actions #2

Updated by Steffen Kamper about 16 years ago

what's about processing $this->id at the beginning?

Actions #3

Updated by Benni Mack about 16 years ago

Looks like we need to add the config.typolinkEnableLinksAcrossDomains thing in

t3lib_tstemplate::linkData()

as well.

Actions #4

Updated by Oliver Hader about 16 years ago

Yepp, or to use typoLink by default - I don't see a reason why this wasn't done for menus as well... do you see a side-effect?

Actions #5

Updated by Benni Mack about 16 years ago

Actually, I just tried the workaround and it looks like it does not work as the page with a shortcut to another domain is not resolved (to its target). it's because the UID contains not the target ID but the source ID.

NO.doNotLinkIt = 1
NO.stdWrap.typolink.parameter.field = shortcut
NO.stdWrap.typolink.parameter.ifEmpty.field = uid

works, but not when the "shortcut_mode" is used.

I am also in favor of using the typolink function here. Don't know about the side-effects though.

Actions #6

Updated by Dmitry Dulepov about 16 years ago

The attached patch is the first work on this subject. Do not use it unless you are ready to risk.

Note: this will not work with RealURL/CoolURI because they need changes too.

Actions #7

Updated by Dmitry Dulepov about 16 years ago

Ok, v3 is the final version, it will work with RealURL (>=1.3.0) out of the box and CoolURI if/when CoolURI supports linking across domains.

Actions #8

Updated by Dmitry Dulepov about 16 years ago

rev 3495

Actions

Also available in: Atom PDF