Bug #42789
closedrequestUpdate doesn't work if fields are separated by a space
100%
Description
I set my TCA like:
$TCA['table']['ctrl']['requestUpdate'] = 'field1, field2';
If I change the value of 'field1' in BE, a reload is requested. If I change the value of 'field2' in BE, there's no reload requested. I expect a reload request in this case too.
I noticed that if I remove the space between the fields, it works.
$TCA['table']['ctrl']['requestUpdate'] = 'field1,field2';
Updated by Francois Suter over 11 years ago
- Status changed from New to Accepted
- Assignee set to Francois Suter
- Target version set to next-patchlevel
Indeed, this is due to \TYPO3\CMS\Core\Utility\GeneralUtility::inList() not tolerating spaces. The behaviour of that method cannot be changed easily for fear of breaking existing uses or of hurting performance (this method is used very widely). After discussion in the core mailing list, a good solution seems to be to remove spaces from "requestUpdate" upon handling it, since no spaces can be expected in that place.
I'll prepare a patch for this.
Updated by Gerrit Code Review over 11 years ago
- Status changed from Accepted to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22174
Updated by Gerrit Code Review over 11 years ago
Patch set 1 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/22185
Updated by Gerrit Code Review over 11 years ago
Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/22186
Updated by Francois Suter over 11 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 064b39ef81a60ca45aa1ecf37a0165513b0c228a.
Updated by Gerrit Code Review over 11 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/22187
Updated by Gerrit Code Review over 11 years ago
Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/22188
Updated by Francois Suter over 11 years ago
- Status changed from Under Review to Resolved
Applied in changeset 7d714560d402080b7e6591b366ddca7ce57b8e83.