Bug #75040

placeholder pointing to column of related record shows values from wrong language

Added by Philipp Wrann over 5 years ago. Updated almost 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
FormEngine aka TCEforms
Target version:
Start date:
2016-03-14
Due date:
% Done:

100%

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

Description

Hope this is not a duplicte.

If i create a file Reference on default language it happens that the placeholder is calculated from some translation.

In my case i added a media-field and handle inline relations to sys_file_reference records. When i edit the default language the meta title|alternative of the first translation record is shown.

The data is saved correctly, so the aggregate and the sys_file_reference both have sys_language_uid set to 0 - but still the wrong metadata is used as placeholder.

The language setting should be taken into account there.


Related issues

Related to TYPO3 Core - Bug #76048: IRRE: Child records - New record doesn't get language of parent (non localized records but using new button)ClosedJo Hasenau2016-05-04

Actions
Related to TYPO3 Core - Bug #48883: Wrong sys_language_uid in sys_file_referenceClosed2013-06-05

Actions
Related to TYPO3 Core - Bug #85711: File language field is rendered twice and sys_language_uid is never rendered in sys_file_metadataClosed2018-08-01

Actions
Has duplicate TYPO3 Core - Bug #78395: SysFileReference shows localized metadata for default languageClosed2016-10-24

Actions
Has duplicate TYPO3 Core - Bug #78897: file references consider wrong language uid for metadataClosed2016-12-06

Actions
Has duplicate TYPO3 Core - Bug #82193: FAL relation: Default file meta data values shown in wrong languageClosed2017-08-24

Actions
Has duplicate TYPO3 Core - Bug #83150: Sys_file_reference in default language showing metadata of translationClosed2017-11-29

Actions
#1

Updated by Florian Baumann over 5 years ago

I can confirm this. It always takes the last created translation of the record.

In FE, the correct translation is taken for the rendering.

#2

Updated by Peter Linzenkirchner over 5 years ago

I can confirm it for the current dev (8.1.0-dev).

#3

Updated by Jeffrey Nellissen about 5 years ago

I also can confirm this bug in the current stable version 7.6.9. It is really confusing for our editors and it is not only happened in the default language. The placeholder for the title, caption,... will always taken his content from the last created language version of a sys_file, not from the choosen sys_language. Like Florian wrote, in FE it works like a charm.

#4

Updated by Andreas Wolf about 5 years ago

  • Status changed from New to Accepted

I can also confirm this. My use case was a relation to sys_file, which resolves the placeholders from sys_file_metadata. The problem here is that the sys_language_uid is not available when resolving the records, which happens in TcaInline::resolveRelatedRecords(). Therefore, a list of the sys_file_metadata records for all languages is fetched, from which one is semi-randomly chosen (it always uses the first one returned).

My idea for a fix is to add the language to use to $result, which then needs to pass it on. In general, it would be good to pass on this context in a more robust way, both here and in DataHandler.

#5

Updated by Tymoteusz Motylewski over 4 years ago

can somebody provide a test extension to ease reproducing and fixing the issue?
is the https://forge.typo3.org/issues/78395 a duplicate?

#6

Updated by Florian Baumann over 4 years ago

There is no need for an extension to reproduce that.

Do the following on an instance with more than one language installed:

1) Add a new file to the FAL
2) Add the file's FAL record translations (fill in the title) for all the available languages
3) Add a new text & media content element and import the newly created file there
4) Take a look at the title field. You will notice the last created translation of the file as a default placeholder there

I just tested this on a TYPO3 7.6.12. The issue still persists.

#7

Updated by Tymoteusz Motylewski over 4 years ago

Hi Florian
in the step 2) you're referring to translating the file in th filelist module?

#8

Updated by Tymoteusz Motylewski over 4 years ago

I've managed to reproduce it. The issue is similar to #76048 - we lack the language context in there too.

#9

Updated by Thomas Deinhamer over 4 years ago

Happens also in TYPO3 8.4

#10

Updated by Helmut Hummel about 4 years ago

  • Related to deleted (Bug #78897: file references consider wrong language uid for metadata)
#11

Updated by Helmut Hummel about 4 years ago

  • Has duplicate Bug #78897: file references consider wrong language uid for metadata added
#12

Updated by Gerrit Code Review about 4 years ago

  • Status changed from Accepted 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/53483

#13

Updated by Gerrit Code Review about 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/53483

#14

Updated by Gerrit Code Review about 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/53483

#15

Updated by Gerrit Code Review almost 4 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/53483

#16

Updated by Gerrit Code Review almost 4 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/53483

#17

Updated by Gerrit Code Review almost 4 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/53483

#18

Updated by Gerrit Code Review almost 4 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/53483

#19

Updated by Christian Kuhn over 3 years ago

  • Has duplicate Bug #82193: FAL relation: Default file meta data values shown in wrong language added
#20

Updated by Gerrit Code Review over 3 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54686

#21

Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54686

#22

Updated by Gerrit Code Review over 3 years ago

Patch set 3 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54686

#23

Updated by Gerrit Code Review over 3 years ago

Patch set 4 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54686

#24

Updated by Gerrit Code Review over 3 years ago

Patch set 5 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54686

#25

Updated by Gerrit Code Review over 3 years ago

Patch set 6 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54686

#26

Updated by Gerrit Code Review over 3 years ago

Patch set 7 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54686

#27

Updated by Gerrit Code Review over 3 years ago

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/54722

#28

Updated by Gerrit Code Review over 3 years ago

Patch set 8 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54686

#29

Updated by Gerrit Code Review over 3 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/54722

#30

Updated by Nicole Cordes over 3 years ago

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

Updated by Markus Klein over 3 years ago

  • Status changed from Resolved to New
#32

Updated by Markus Klein over 3 years ago

  • % Done changed from 100 to 0

master patch had to be reverted

#33

Updated by Gerrit Code Review over 3 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/54831

#34

Updated by Gerrit Code Review over 3 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/54831

#35

Updated by Christian Kuhn over 3 years ago

  • Has duplicate Bug #83150: Sys_file_reference in default language showing metadata of translation added
#36

Updated by Gerrit Code Review over 3 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/54831

#37

Updated by Gerrit Code Review over 3 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/54831

#38

Updated by Gerrit Code Review over 3 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/54831

#39

Updated by Gerrit Code Review over 3 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/54831

#40

Updated by Gerrit Code Review over 3 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/54831

#41

Updated by Gerrit Code Review over 3 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/54831

#42

Updated by Nicole Cordes over 3 years ago

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

Updated by Gerrit Code Review over 3 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/54888

#44

Updated by Nicole Cordes over 3 years ago

  • Status changed from Under Review to Resolved
#45

Updated by Helmut Hummel almost 3 years ago

  • Related to Bug #85711: File language field is rendered twice and sys_language_uid is never rendered in sys_file_metadata added
#46

Updated by Benni Mack almost 3 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF