Project

General

Profile

Actions

Bug #18764

closed

Linking between domains in the pagetree with checkRootline does not work with menus and normal text links

Added by Tobias Hunziker over 16 years ago. Updated about 6 years ago.

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

0%

Estimated time:
TYPO3 Version:
4.1
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

This is the same issue as reported in 0007839. One of our customers doesn't wants to upgrade to Typo3 version 4.2. Therefore we are looking for a working patch for Typo3 version 4.1.6.

We donate for a working patch on our system.
(issue imported from #M8356)


Files

8356.diff (9.91 KB) 8356.diff Administrator Admin, 2008-05-12 13:11
Actions #1

Updated by Benni Mack over 16 years ago

Hey,

there is a workaround for this for 4.1 and 4.0 to do this over TypoScript. Would this work for you?
I'll see if I can find this one for you again.

Actions #2

Updated by Dmitry Dulepov over 16 years ago

Benny, are you sure it is possible to enable links across domains using TS?

Actions #3

Updated by Dmitry Dulepov over 16 years ago

The code patch for 4.1.6 is attached. It applies ok to 4.1.6:

~/Projects/TYPO3/TYPO3_4-1-6 $ patch -p0 --dry-run < ../8356.diff 
patching file typo3/sysext/cms/tslib/class.tslib_fe.php
patching file typo3/sysext/cms/tslib/class.tslib_menu.php
patching file typo3/sysext/cms/tslib/class.tslib_content.php

I made it some time ago but it did not went to 4.1 and there is no chance that it can get into 4.1 because it is a new feature. But you can apply it yourself.

Actions #4

Updated by Tobias Hunziker over 16 years ago

Hi Dmitry, i tested already the patch from issue 0007839 in our system, but there were some errors patching the file. I will test your patch 8356.diff on wednesday and give you a feedback about that.
@Benny: I think a patch is the better solution for our customer.

Actions #5

Updated by Dmitry Dulepov over 16 years ago

7839 is for 4.2, it will not apply to 4.1 :)

Actions #6

Updated by Tobias Hunziker over 16 years ago

Hello Dmitry,
first thank you very much for your patch. We installed it on our test enviroment without problems.
The Situation now is as fallows:
- I enabled typolinkCheckRootline and typolinkEnableLinksAccrosDomains.
- Text-links (from text-content elements > RTE) seems to work now correct between domains!
Links in navigations (TMENU) pointing to an other domain does still not work.

Do you have any tips for me to get that working? you can contact me via mail

Actions #7

Updated by Dmitry Dulepov over 16 years ago

All domains must have "Is root of web site" set in page properties for the domain's root page.

Actions #8

Updated by Tobias Hunziker over 16 years ago

I copied the original src-files again into the directory and patched them once again without errors. But today I thinked i check the patched files if they are really patched and realized, that the files were not changed (Patching didn't work). Therefore I changed the files manually and checked the links again. Now all links are working correctly.

Please send me your paypal mail-address that we can transfer the money.

Actions #9

Updated by Dmitry Dulepov over 16 years ago

PayPal address is . I have problems with making invoices (not sure if it is important to you). If you do not need formal invoice, then all fine.

Actions #10

Updated by Dmitry Dulepov over 16 years ago

Tobias, I do not have your e-mail but you can contact me at gmail address or at .

Actions #11

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF