Bug #82130

Inline record for NEW tt_content element becomes sys_language_uid=-1

Added by Martin Horniak almost 2 years ago. Updated about 1 year ago.

Status:
New
Priority:
Must have
Assignee:
-
Category:
FormEngine aka TCEforms
Target version:
-
Start date:
2017-08-18
Due date:
% Done:

0%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
medium
Is Regression:
Sprint Focus:

Description

By creating a new tt_content element with included inline records, is not possible for FormInlineAjaxController to determine the current parent (tt_content-element) language. Therefore become all included inline records -1 as sys_language_uid.
For the previously saved tt_content elements woks the patch from Daniel Maier (https://review.typo3.org/#/c/53206/).

My recommendation is to adapt the DOM Object ID for InlineStackProcessor:
- Include the current parent language in the AJAX call and determine the right localizationMode in TcaInlineConfiguration::initializeLocalizationMode for this case not from $result['defaultLanguageRow'], but from this prepared parent language field (probably set in $formDataCompilerInputForParent).
- Extend the patch mentioned above: get the language directly from InlineStack (parent) data instead of DB record.

This issue is based on TYPO3 version 7.6.20. Not verified for other versions.

2017-08-17_15-03-52.jpg View (78.6 KB) Martin Horniak, 2017-08-18 16:19

History

#1 Updated by Martin Horniak almost 2 years ago

  • Project changed from forge.typo3.org to TYPO3 Core
  • Category deleted (Development)
  • TYPO3 Version set to 7
  • Complexity set to medium

#2 Updated by Martin Horniak almost 2 years ago

  • Category set to FormEngine aka TCEforms

#4 Updated by Christian Kuhn over 1 year ago

  • Status changed from New to Needs Feedback

Is this still valid? The patch to cope with the language uid has been released with 7.6.22 and was not yet included in 7.6.20.

#5 Updated by Martin Horniak over 1 year ago

Can I review the merge, which may fix this issue? I didn't found any change (7.6.20 -> 7.6.22) on review.typo3.org, which handles the language for new inline records in new content element. Thanks.

#7 Updated by Martin Horniak over 1 year ago

Riccardo De Contardi wrote:

Is it maybe: https://review.typo3.org/#/c/53206/ ?

No, it's not: MathUtility::canBeInterpretedAsInteger($parent['uid']) can't become true on NEW<hash> ids and parent record is still not persisted at this time.

#8 Updated by Alexander Opitz about 1 year ago

  • Status changed from Needs Feedback to New

Also available in: Atom PDF