Bug #34698

t3lib/class.t3lib_befunc.php -> getViewDomain

Added by Thomas Layh almost 10 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2012-03-09
Due date:
% Done:

100%

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

Description

The changed function getViewDomain should always take the firstDomain from the tree first instead of using the currently called backend domain.

I try to explain the problem with the following example tree.

- root1 (www.mydomain.com)
-- home
-- more pages
- root2 (www1.mydomain.com)

I login using the domain www1.mydomain.com. If I click on preview on the home page, I want to see the page www.mydomain.com/home.html. Currently, if a valid www1 domain is set, it is used instead of the domain in the root line. The behavior before (#30892) was that for the preview domain the rootline domain was used, now the www1 domain has a higher priority which is imho wrong.

Going to push the patch to gerrit.


Files

class.t3lib_befunc.php.patch (911 Bytes) class.t3lib_befunc.php.patch Thomas Layh, 2012-03-09 18:20
#1

Updated by Gerrit Code Review almost 10 years ago

  • Status changed from New to Under Review

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

#2

Updated by Gerrit Code Review almost 10 years ago

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at http://review.typo3.org/9538

#3

Updated by Gerrit Code Review almost 10 years ago

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at http://review.typo3.org/9539

#4

Updated by Thomas Layh almost 10 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#5

Updated by Robert Heel over 9 years ago

The preview domain has to be the same domain as the backend login, otherwise disabled pages or pages in workspace wouldn't be shown, because the BE cookie only exists in domain from backend login.

#6

Updated by Benni Mack over 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF