Project

General

Profile

Actions

Bug #22542

closed

Problem regarding sys_domain records handling in typolinks

Added by Gabriel Kaufmann over 14 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2010-04-28
Due date:
% Done:

100%

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

Description

If there are multiple sys_domain records used in one page-tree (same Folder/Pid) every link rendered within the page will be forced to use the first sys_domain record (=1st domain given) for all links (Typolinks).

This situation may be worse if there is for example a "One-Tree" concept for f.e. different customer-domains or different languages (domain.de / domain.com).
In my opinion it would make sense to leave all links on that domain the page has been opened with (even if there are multiple records for that tree within the same page). The behaviour that is given now could and should in my opinion be switched by the "force-prefix" checkbox listed for sys_domains records. The normal handling should be that there is no domain-prefix in the page even if there are multiple sys_domains so that the page-admin can decide what should happen. A check if the current domain exists as sys_domain record (for that tree) should be enough to check against any security issues.

I noticed this behaving is switched in "tslib/class.tslib_content.php" within the function "typoLink".

I would like to know if there any reason to force the first domain to be the default without letting any choise by Typoscript (f.e. config.baseURL). It would make sense to give any chance for site-admins to choose this as any class extension or hard php-hack would not be a clean solution.

has patch: not yet

I would like to know if this feature request has any chance to become a core feature before investing time and money into making a patch.
(issue imported from #M14242)


Files

class.tslib_content_4.3.3.php.patch (1.42 KB) class.tslib_content_4.3.3.php.patch Administrator Admin, 2010-04-28 18:38
Actions

Also available in: Atom PDF