Project

General

Profile

Actions

Bug #87887

open

"disablePrependAtCopy" does not work properly in combination with "allowLanguageSynchronization"

Added by Benedikt Imminger about 5 years ago. Updated 3 months ago.

Status:
New
Priority:
Should have
Assignee:
-
Category:
Localization
Target version:
-
Start date:
2019-03-11
Due date:
% Done:

0%

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

Description

For the assets-field allowLanguageSynchronization is set to 'true' and disablePrependAtCopy is active for sys_file_reference .

This configuration works as expected when the content element is translated for the first time. [Translate to XY] is not visible in the translated content element.

However, when I add another image to the content element after the initial translation, [Translate to XY] is visible.

I think this might be a bug because it works as expected on the initial translation, but not correctly afterwards.

Steps to reproduce:
  1. Set allowLanguageSynchronization to true for 'assets'
  2. Activate disablePrependAtCopy for sys_file_reference
  3. For sys-lang 0: Add image to a content element which uses the assets field (e.g. text-media).
  4. Add metadata for image (e.g. title).
  5. Translate content element to sys_lang 1 ([Translate to XY] is not visible as expected).
  6. Go back to sys-lang 0: add a second image to content element and save the changes.
  7. Switch to sys-lang 1: metadata of new file now contains [Translate to XY]

Files

PrependAtCopy.png (288 KB) PrependAtCopy.png Benedikt Imminger, 2019-03-11 18:10
Actions #1

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 https://review.typo3.org/c/Packages/TYPO3.CMS/+/60266

Actions #2

Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/60266

Actions #3

Updated by Gerrit Code Review almost 4 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/c/Packages/TYPO3.CMS/+/60266

Actions #4

Updated by Philip Heidenreich over 2 years ago

The ticket refers to TYPO3 v9, however changes in v10 were made. Nevertheless, these changes do not work. Even if the prefix is not set due to the conditions, it is set in the following line 1352-1354

if (empty($prefix)) {
    $prefix = 'Translate to ' . $languageRecord['title'] . ':';
}

Therefore, the condition if the table allows the prefix need to be moved to the loop, or everything after the edited condition needs to be placed within that IF statement.

Actions #5

Updated by Georg Ringer over 2 years ago

  • Status changed from Under Review to New
Actions #6

Updated by Felix Nagel about 2 years ago

I'm able to confirm this for TYPO3 10.4.27 with the pages.media field.

Actions #7

Updated by Joey Bouten 3 months ago

  • TYPO3 Version changed from 9 to 10

Still encounter this issue. We do not want any labels anywhere so we just XClasses it to return the data immediatly

Actions

Also available in: Atom PDF