Bug #86554

Field value does not get converted to array if using overrideVals

Added by Andreas Wolf 10 months ago. Updated 2 months ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Backend User Interface
Start date:
2018-10-03
Due date:
% Done:

0%

TYPO3 Version:
9
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

When a field value override is specified, the TCA configuration of that field gets reset to type = "hidden".

The unwished side effect of this is that e.g. the TcaSelectItems data provider does not process the field anymore. Therefore, the value stays as-is and is not converted to an array (by processDatabaseFieldValue()).

This is e.g. relevant for pages:sys_language_uid, which is overridden in the "edit" button in the page module. This leads to e.g. the wrong URL slug being shown (if your sys_language_uid is >= 10).

How to reproduce

  • have a site with sys_language_uid 1 and one sys_language_uid X (10..19)
  • create a translation of a page to languages 1 and X
  • edit the page in language X => the slug will be the one from language 1

Possible fix

The DatabaseRecordOverrideValues processor should only set renderType = hidden, and that must be respected by the code rendering the forms.


Related issues

Related to TYPO3 Core - Bug #86754: OverrideVals are not always set for editing pages Closed 2018-10-26
Duplicates TYPO3 Core - Bug #86734: not possible to edit pages with sys_language_uid > 9 with SiteHandling Closed 2018-10-24

Associated revisions

Revision c26facec (diff)
Added by Daniel Goerz 9 months ago

[BUGFIX] Also process array in InputSlugElement

It is possible that the incoming $row contains an array
at the $languageField. This patch assures that the
$languageId can always be determined.

Resolves: #86734
Resolves: #86554
Relates: #86754
Releases: master
Change-Id: Ia2c8621cd935788f33a9b625ec0226a22e21b123
Reviewed-on: https://review.typo3.org/58700
Reviewed-by: Susanne Moog <>
Tested-by: Susanne Moog <>
Tested-by: TYPO3com <>
Reviewed-by: Oliver Hader <>
Tested-by: Oliver Hader <>
Reviewed-by: Anja Leichsenring <>
Tested-by: Anja Leichsenring <>

History

#1 Updated by Gerrit Code Review 10 months 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/58565

#2 Updated by Susanne Moog 9 months ago

  • Duplicates Bug #86734: not possible to edit pages with sys_language_uid > 9 with SiteHandling added

#3 Updated by Susanne Moog 9 months ago

  • Status changed from Under Review to Closed

Issue is/will be fixed with the duplicated issue.

#4 Updated by Daniel Goerz 9 months ago

  • Related to Bug #86754: OverrideVals are not always set for editing pages added

#5 Updated by Gerrit Code Review 9 months ago

  • Status changed from Closed to Under Review

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

#6 Updated by Gerrit Code Review 9 months 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/58700

#7 Updated by Anonymous 9 months ago

  • Status changed from Under Review to Resolved

#8 Updated by Benni Mack 2 months ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF