Bug #58352
closedInvalid localization overlay of cObject FILES
100%
Description
- l10n_mode is not considered
- additional translation table (e.g. for pages) is not considered correctly
Updated by Gerrit Code Review almost 11 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 https://review.typo3.org/29739
Updated by Oliver Hader almost 11 years ago
How to test the patch:
Requirements:- open typo3conf/extTables.php and put
$GLOBALS['TCA']['pages_language_overlay']['columns']['media']['l10n_mode'] = 'mergeIfNotBlank';
- clear TCA caches (e.g. "clear all" in Install Tool)
- create translation of a page, without translating the media field (on the resources tab)
- have some TypoScript to out the file or the name of the file, e.g.
@page = PAGE
page.10 = FILES - page.10.references = ...
page.10.renderObj = TEXT
page.10.renderObj.data = file:current:publicUrl
page.10.renderObj.wrap = |,
Tests:
1) TypoScript using the special "fieldName" propertypage.10.references.fieldName = media
2) TypoScript using stdWrappage.10.references.field = media
Updated by Frans Saris almost 11 years ago
@Olly:
This doesn't work:
page.10.references.field = media
In handleFileReferences() you explicit look for fieldName or else there is a early return.
Updated by Gerrit Code Review almost 11 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29739
Updated by Gerrit Code Review almost 11 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29739
Updated by Gerrit Code Review almost 11 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29739
Updated by Gerrit Code Review almost 11 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29739
Updated by Gerrit Code Review almost 11 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29739
Updated by Gerrit Code Review almost 11 years ago
Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29739
Updated by Gerrit Code Review over 10 years ago
Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29739
Updated by Tymoteusz Motylewski over 10 years ago
should be resolved with https://review.typo3.org/29739
Updated by Steffen Ritter over 10 years ago
- Status changed from Under Review to Resolved
Updated by Gerrit Code Review over 10 years ago
- Status changed from Resolved 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 https://review.typo3.org/30666
Updated by Gerrit Code Review over 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30666
Updated by Gerrit Code Review over 10 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/30666
Updated by Steffen Ritter over 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 90b0b4a3973697c2324c5a7ef8eec7db0a23d66e.