Bug #26612
closedPreview does not include L parameter if not default language
100%
Description
Using T3 4.5.2 and no templavoila, i have the problem that the preview icon link is not correct.
I am in my workspace and in page module. I switch to another language via the dropdown in the top toolbar.
My link looks like this:
http://example.com/typo3/mod.php?id=14 &tx_workspaces_web_workspacesworkspaces[action]=index &tx_workspaces_web_workspacesworkspaces[controller]=Tx_Workspaces_Controller_PreviewController &tx_workspaces_web_workspacesworkspaces[controller]=Preview &M=web_WorkspacesWorkspaces &id=14
Well, id is included twice but no L parameter. SOMEHOW this works with templavoila, i have no idea why...
Updated by Oliver Hader over 13 years ago
- Status changed from New to Accepted
- Assignee set to Oliver Hader
- TYPO3 Version set to 4.5
Updated by Stefan Neufeind about 13 years ago
code-path for further investigations:
Classes/Service/Workspaces.php, generateWorkspaceSplittedPreviewLink()
which calls the UriBuilder from Extbase:
Classes/MVC/Web/Routing/UriBuilder.php
It seems that the UriBuilder already generates the id=... at beginning of this link and we could avoid adding it later. Also the controller-parameter is generated in the first place and then manually added again (second parameter, probably overwriting the first one) in the URL mentioned.
The language-parameter is added in
t3lib/class.t3lib_befunc.php, viewOnClick()
But for some reason it doesn't add the L-parameter there ...
The $viewScript-parameter itself is created in the preProcess-hook called from viewOnClick() and implemented in the workspace-package.
Updated by Gerrit Code Review almost 12 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch 4.5 has been pushed to the review server.
It is available at https://review.typo3.org/17277
Updated by Gerrit Code Review almost 12 years ago
Patch set 2 for branch 4.5 has been pushed to the review server.
It is available at https://review.typo3.org/17277
Updated by Gerrit Code Review almost 12 years ago
Patch set 3 for branch 4.5 has been pushed to the review server.
It is available at https://review.typo3.org/17277
Updated by Gerrit Code Review almost 12 years ago
Patch set 4 for branch 4.5 has been pushed to the review server.
It is available at https://review.typo3.org/17277
Updated by Gerrit Code Review almost 12 years ago
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/17278
Updated by Gerrit Code Review almost 12 years ago
Patch set 1 for branch 4.6 has been pushed to the review server.
It is available at https://review.typo3.org/17346
Updated by Gerrit Code Review almost 12 years ago
Patch set 1 for branch workspaces_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/17347
Updated by Gerrit Code Review almost 12 years ago
Patch set 1 for branch workspaces_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/17349
Updated by Oliver Hader almost 12 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset commit:c6d19bc0e7417ff2bce7da587d812a7daa7a3473.
Updated by Michael Stucki almost 11 years ago
- Project changed from 624 to TYPO3 Core
- Category changed from Workspaces to Workspaces