Task #85610
closedRemove skipped extbase text for argument not being array
100%
Description
typo3/sysext/extbase/Tests/Unit/Mvc/RequestTest.php::setArgumentThrowsExceptionIfTheGivenArgumentValueIsAnObject
is skipped since a long time.
Investigating why it would be advisable to deny the usage of objects in request arguments, I got this conclusions:
there are two places reading the incoming request arguments and using them.
- typo3/sysext/extbase/Classes/Mvc/Controller/AbstractController.php::mapRequestArgumentsToControllerArguments uses the argument to set it to the controller arguments and in this process, deals with incoming objects just fine.
- typo3/sysext/indexed_search/Classes/Controller/AdministrationController.php::processRequest uses the arguments to write the userSettings in writeUC(). This function takes incoming values and serializes them to store them in DB, is therefore able to handle objects just fine.
-> there is no reason to deny the usage of objects in extbase requests, therefor the (formely skipped) test can be deleted.
Updated by Gerrit Code Review over 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/57649
Updated by Gerrit Code Review over 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/57649
Updated by Anja Leichsenring over 6 years ago
- Subject changed from Remove unnecessary skipped extbase text to Remove skipped extbase text for argument not being array
Updated by Anonymous over 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 34eb9737c58ae5f0a86f079a78969add064d6161.