Bug #30892: t3lib_befunc::getViewDomain returns wrong or no domain - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #30892

closed

t3lib_befunc::getViewDomain returns wrong or no domain

Added by Thomas Pilgaard Carlsen over 12 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Target version:

Start date:

2011-10-13

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

4.5

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

Issue 1:
If the TYPO3 BE hostname is not configured as a sys_domain record t3lib_befunc::getViewDomain will always return TYPO3 BE hostname and not the first domain of the root line of the page which is being previewed.

Issue 2:
If the TYPO3 BE hostname is configured as a sys_domain but the current root line does not contain a valid domain record t3lib_befunc::getViewDomain will return only 'http://' ie. no domain. I would think the logical thing would be to return the TYPO3 BE hostname.

Files

Download all files

class.t3lib_befunc.diff (1.03 KB) class.t3lib_befunc.diff		Jaume Prat, 2011-10-18 16:27
t3lib_befunc.diff (1.01 KB) t3lib_befunc.diff	Diff-patch	Jaume Prat, 2011-11-21 22:54

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Jaume Prat over 12 years ago

File class.t3lib_befunc.diff class.t3lib_befunc.diff added

I have a similar problem with my TYPO3 project. I needed to fix the class t3lib_befunc in core.

I'm looking forward to your feedback.

Actions

Copy link

Updated by Jaume Prat over 12 years ago

Assignee set to Christian Kuhn

Actions

Copy link

Updated by Christian Kuhn over 12 years ago

Assignee deleted (~~Christian Kuhn~~)

Actions

Copy link

Updated by Jaume Prat over 12 years ago

File t3lib_befunc.diff t3lib_befunc.diff added
Assignee set to TYPO3 Release Team
Target version set to 4.6.1

This patch is the correct and solves the problem. Please ignore the last one

Actions

Copy link

Updated by Jaume Prat over 12 years ago

Steps To Reproduce this issue:

With a multidomain Webpage(s) in one TYPO3-Instance:

add more than 1 domain in your Rootpage, for example:

www.domain1.com

www.domain2.com

put domain1 at the first position
in Backend with domain2 (www.domain2.com/typo3) try to preview a page.

You will be always redirect to domain1 (!)

With this patch you will be redirect to domain2 if you try to preview a page with domain2 and search if domain2 exist in your rootline.

Could anybody try this patch?

THX!

Actions

Copy link

Updated by Mr. Jenkins over 12 years ago

Status changed from New to Under Review

Patch set 1 of change I8118d2c932b2a9d6c3d2dae51c5707a6371784d8 has been pushed to the review server.
It is available at http://review.typo3.org/6830

Actions

Copy link

Updated by Steffen Gebert over 12 years ago

Assignee deleted (~~TYPO3 Release Team~~)
Target version deleted (~~4.6.1~~)

Actions

Copy link

Updated by Gerrit Code Review over 12 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/6830

Actions

Copy link

Updated by Georg Ringer over 12 years ago

is it possible that this is still not fully fixed?

My page tree:

. Tree 1 < -- Domain: fo.local
... Tree 1b
. Tree 2 < -- Domain: trunk.local
... Tree 2b

My backend domain: trunk.local and if i am on page "Tree 1b" I still get the trunc.local as preview domain but this should be fo.local as trunk.local is only valid for the other page tree.

problem is imo because ::getDomainStartPage() will deliver a result for trunk.local but first there should be a check if this domain is valid for the current rootline

Actions

Copy link

#10