Bug #76147
closedcheckbox items which was created with itemsProcFunc ignored in datahandler
100%
Description
Is it not possible save dynamically added (itemsProcFunc) checkbox items. It is rendered correct in backend form, but values not saved in database.
There is code for radio element (function checkValueForRadio) -
if ($tcaFieldConf['itemsProcFunc'] && empty($res['value'])) {
$processingService = GeneralUtility::makeInstance(ItemProcessingService::class);
$processedItems = $processingService->getProcessingItems($table, $pid, $field, $this->checkValue_currentRecord,
$tcaFieldConf, $tcaFieldConf['items']);
foreach ($processedItems as $set) {
if ((string)$set[1] === (string)$value) {
$res['value'] = $value;
break;
}
}
}
Seems like similar code for checkboxes should be.
Updated by Christian Kuhn over 8 years ago
confirmed. there is a @todo in ext:styleguide about that, too:
tx_styleguide_elements_basic
'checkbox_6' => [ // @todo: Checking a checkbox that is added by itemsProcFunc is not persisted correctly. // @todo: HTML looks good, so this is probably an issue in DataHandler? 'label' => 'checkbox_6 itemsProcFunc', 'config' => [ 'type' => 'check', 'items' => [ ['foo', ''], ['bar', ''], ], 'itemsProcFunc' => 'TYPO3\\CMS\\Styleguide\\UserFunctions\\FormEngine\\TypeCheckbox8ItemsProcFunc->itemsProcFunc', ], ],
Updated by Gerrit Code Review over 8 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/48082
Updated by Gerrit Code Review over 8 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/48082
Updated by Gerrit Code Review over 8 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/48082
Updated by Gerrit Code Review over 8 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/48082
Updated by Anja Leichsenring over 8 years ago
- Sprint Focus set to On Location Sprint
Updated by Gerrit Code Review over 8 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/48082
Updated by Gerrit Code Review about 8 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/48082
Updated by Gerrit Code Review about 8 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/48082
Updated by Gerrit Code Review about 8 years ago
Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/48450
Updated by Anja Leichsenring about 8 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 8567387cd557456fccac7430a14e9ea2b41b3523.
Updated by Claus Due about 8 years ago
Same problem exists for "select" type - opening another issue.
Updated by Benni Mack almost 6 years ago
- Status changed from Resolved to Closed