Bug #53928
closed
itemsProcFunc for Radio-Buttons doesn't work correctly
100%
Description
As noone seems to answer this issue, I recreate it. And to quote myself: If a radio button is filled with itemsProcFunc, the value check (function checkValue_radio in DataHandler.php) fails as it only test the items-array ...
I've come up with kind of a solution (patch to current master attached): if the tca config has a itemsProcFunc and the selected value doesn't match one in the items-array, just accept it. I don't know whether this solution is sufficient for you, but at least it works.
Files
Updated by Stefan Galinski over 10 years ago
Hi Uwe,
Please upload your patch to the review system. You can find detailed instructions in the TYPO3 wiki on the page http://wiki.typo3.org/Contribution_Walkthrough_(Git).
Updated by Stefan Galinski over 10 years ago
- TYPO3 Version changed from 6.2 to 4.5
Updated by Uwe Wiebach over 10 years ago
Done (https://review.typo3.org/25867). I hope I did everything correctly (my first contribution) ...
Updated by Gerrit Code Review about 10 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/26992
Updated by Uwe Wiebach about 10 years ago
As you changed the TYPO3 version to 4.5 I pushed a patch for this branch to Git ...
Updated by Alexander Opitz about 10 years ago
You pushed it to master and not into the right branch.
Also your review https://review.typo3.org/#/c/25867/ seams to have a little problem (You didn't used "Resolves" in commit message, fixed that), as it isn't shown here.
The change of the TYPO3 Version only means, that this issue exists since this version.
In #21043 was no answer, as it was closed (course there was no response).
By the way, I don't think that this issue will be backported to 4.5 or 4.7 as they will only receive security fixes.
Updated by Gerrit Code Review about 10 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/25867
Updated by Gerrit Code Review over 9 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/25867
Updated by Gerrit Code Review almost 9 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/25867
Updated by Gerrit Code Review almost 9 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/25867
Updated by Andreas Wolf almost 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 13b6775f772d9bbfa579fcd01eb6aada883657a9.
Updated by Gerrit Code Review almost 9 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/41642
Updated by Gerrit Code Review over 8 years ago
Patch set 2 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/41642
Updated by Mathias Schreiber about 8 years ago
- Status changed from Under Review to New
Updated by Mathias Schreiber about 8 years ago
- Status changed from New to Closed
Use FormEngine DataProcessors :)