Bug #76661

Suggest Wizard ajax response ignores columnsOverrides

Added by Daniel Siepmann over 3 years ago. Updated 4 months ago.

Status:
Under Review
Priority:
Should have
Assignee:
-
Category:
FormEngine aka TCEforms
Target version:
-
Start date:
2016-06-16
Due date:
% Done:

0%

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

Description

We override the records fields with different configuration.
That works on Rendering the BE-Form, but isn't taken into account while processing the ajax of suggest wizard. He only takes a look at the TCA, without the overwrites and is not usable. The only way to prevent insertions of invalid records is to disable the wizard via columnsOverrides.

Configuration is attached as screenshots, here is the example for copy and paste:

                        'columnsOverrides' => [
                            'records' => [
                                'config' => [
                                    'minitems' => 1,
                                    'foreign_table' => 'fe_users',
                                    'allowed' => 'fe_users',
                                    'wizards' => [
                                        'suggest' => [
                                            'default' => [
                                                'pidList' => '2',
                                                'searchCondition' => 'pid = 2',
                                                'searchWholePhrase' => 1,
                                                'additionalSearchFields' => 'username, city, country',
                                            ],
                                        ],
                                    ],
                                ],
                            ],
                        ],

For 7.6.9 the following line is used to fetch configuration:

        $fieldConfig = $GLOBALS['TCA'][$table]['columns'][$field]['config'];
        $this->overrideFieldNameAndConfigurationForFlexform($table, $field, $row, $fieldConfig);

        $wizardConfig = $fieldConfig['wizards']['suggest'];

        $queryTables = $this->getTablesToQueryFromFieldConfiguration($fieldConfig);
        $whereClause = $this->getWhereClause($fieldConfig);

sysext/backend/Classes/Form/Wizard/SuggestWizard.php

So the configuration is ignored.

Screen Shot 2016-06-16 at 15.04.24.png View - TYPO3 default records configuration (31.9 KB) Daniel Siepmann, 2016-06-16 15:09

Screen Shot 2016-06-16 at 15.04.27.png View - Overwrite of records configuration (37.2 KB) Daniel Siepmann, 2016-06-16 15:09


Related issues

Related to TYPO3 Core - Bug #86876: TCA columnsOverrides ignores default value New 2018-11-07

History

#1 Updated by Bjoern Jacob about 3 years ago

  • Category changed from Form Framework to FormEngine aka TCEforms
  • Sprint Focus set to On Location Sprint

#2 Updated by Susanne Moog over 2 years ago

  • Complexity set to medium

#3 Updated by Anja Leichsenring about 2 years ago

  • Sprint Focus deleted (On Location Sprint)

#4 Updated by R3 H6 over 1 year ago

Can confirm this bug in TYPO3 8.7

#5 Updated by Daniel Siepmann 9 months ago

We again hit this limitation in another project for current CMS v 8.7.20. Looks like the JavaScript needs adjustment, to provide the current Type (CType in case of tt_content). This way the suggest wizard can either simulate the data to use the existing FormDataProvider TcaColumnsOverrides, or can look up any overrides itself, which would lead to duplicate knowledge about structure.

Otherwise the type can be fetched from DB, in assumption that the record was already saved.

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

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

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

#9 Updated by Daniel Siepmann 8 months ago

  • Related to Bug #86876: TCA columnsOverrides ignores default value added

#10 Updated by Gerrit Code Review 4 months 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/c/Packages/TYPO3.CMS/+/59444

Also available in: Atom PDF