Bug #75647
closedStory #69617: FormEngine bugs
hardcoded "hidden" col in backend/form creates wrong SQL
100%
Description
Hello Core-Team,
if you have a record with an inline-relation to be_users and that inline record is collapsed (<---notloaded--> in HTML-comment) and you save that record, then a SQL-Error appears: col "hidden" not found in be_users.
In InlineRecordContainer.php you're working with hardcoded "hidden" cols:
PS: If you save the record with expanded inline-records it works like expected.
if (!$data['isInlineChildExpanded'] && !empty($GLOBALS['TCA'][$foreignTable]['ctrl']['enablecolumns']['disabled']) ) { $checked = !empty($record['hidden']) ? ' checked="checked"' : ''; $html .= '<input type="checkbox" data-formengine-input-name="data' . htmlspecialchars($appendFormFieldNames) . '[hidden]" value="1"' . $checked . ' />'; $html .= '<input type="input" name="data' . htmlspecialchars($appendFormFieldNames) . '[hidden]" value="' . htmlspecialchars($record['hidden']) . '" />'; }
Stefan
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/47888
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/47888
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/47888
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/47888
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/47888
Updated by Gerrit Code Review over 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/47889
Updated by Wouter Wolters over 8 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 968be676db270a9c5aa347b55e83fcae1e4f8590.
Updated by Gerrit Code Review over 8 years ago
- Status changed from Resolved 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/47892
Updated by Gerrit Code Review over 8 years ago
Patch set 2 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/47889
Updated by Christian Kuhn over 8 years ago
- Status changed from Under Review to Resolved
Applied in changeset b4bfa8e9a07ae5e7613f8452247602dec6bbf4d4.
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed