Bug #61226

Web-View Module does not respect TCEMAIN.previewDomain

Added by Georg Kühnberger about 5 years ago. Updated 12 months ago.

Status:
Closed
Priority:
Should have
Category:
Backend User Interface
Target version:
Start date:
2014-08-27
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
5.3
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

https://forge.typo3.org/projects/typo3cms-core/repository/revisions/f2b91c8e50a4fe62045f125431b48a2aa18f59cd
Introduced the feature to set an alternative domain for preview with TCEMAIN.previewDomain = example.com in PageTS.

However previewDomain is NOT respected in the Module Web>View,
resulting in: Web>View behaves different than Page>View.


Related issues

Related to TYPO3 Core - Bug #30889: It isn't posible to set an alternative Domain for Preview Closed 2011-10-13

Associated revisions

Revision 2ad3dbd3 (diff)
Added by Georg Kühnberger about 5 years ago

[BUGFIX] Let Web-View respect TCEMAIN.previewDomain

Added previewDomain in order to have Web-View functioning in the
same way like Page-Preview does.

Fixes: #61226
Related: #30889
Releases: 6.3, 6.2
Change-Id: I872651aeac1bda27f7a8a034fc30a21ff7162df0
Reviewed-on: http://review.typo3.org/32510
Reviewed-by: Oliver Gassner <>
Tested-by: Oliver Gassner <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

Revision 89fddd12 (diff)
Added by Georg Kühnberger about 5 years ago

[BUGFIX] Let Web-View respect TCEMAIN.previewDomain

Added previewDomain in order to have Web-View functioning in the
same way like Page-Preview does.

Change-Id: I7193cd93a2b9b2d175e162d3fab0b837aefd2487
Fixes: #61226
Related: #30889
Releases: 6.3, 6.2
Reviewed-on: http://review.typo3.org/32512
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

History

#1 Updated by Georg Kühnberger about 5 years ago

  • % Done changed from 0 to 100

Implemented FIX in:

https://github.com/plan2net/TYPO3.CMS/commit/23dd39e3aacb2fdec92c326cb8b54ee7058896d1

Sent Pull Request.
Please try to get the fix into the next Mantainance-Release.
Thanks G

#2 Updated by Markus Klein about 5 years ago

Georg, can you please push the fix to Gerrit? (I know you're able to ;-))

#3 Updated by Alexander Opitz about 5 years ago

  • Status changed from New to Needs Feedback

#4 Updated by Markus Klein about 5 years ago

  • Status changed from Needs Feedback to New

#5 Updated by Gerrit Code Review about 5 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32510

#6 Updated by Gerrit Code Review about 5 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32510

#7 Updated by Gerrit Code Review about 5 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32510

#8 Updated by Gerrit Code Review about 5 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32510

#9 Updated by Gerrit Code Review about 5 years ago

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/32512

#10 Updated by Georg Kühnberger about 5 years ago

  • Status changed from Under Review to Resolved

#11 Updated by Benni Mack 12 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF