Bug #76661
closedSuggest Wizard ajax response ignores columnsOverrides
100%
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.
Files
Updated by Björn Jacob over 8 years ago
- Category changed from Form Framework to FormEngine aka TCEforms
- Sprint Focus set to On Location Sprint
Updated by Anja Leichsenring about 7 years ago
- Sprint Focus deleted (
On Location Sprint)
Updated by Daniel Siepmann almost 6 years 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.
Updated by Gerrit Code Review almost 6 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/59444
Updated by Gerrit Code Review almost 6 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/59444
Updated by Gerrit Code Review almost 6 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/59444
Updated by Daniel Siepmann over 5 years ago
- Related to Bug #86876: TCA columnsOverrides ignores default value added
Updated by Gerrit Code Review over 5 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/c/Packages/TYPO3.CMS/+/59444
Updated by Gerrit Code Review about 5 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/c/Packages/TYPO3.CMS/+/59444
Updated by Gerrit Code Review over 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/c/Packages/TYPO3.CMS/+/59444
Updated by Gerrit Code Review over 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/c/Packages/TYPO3.CMS/+/59444
Updated by Daniel Siepmann over 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 8865b1cfa72064580a48c58a1b874f155b68ef55.
Updated by Daniel Siepmann over 4 years ago
- Status changed from Resolved to Closed