Bug #55861
closed
New PropertyMapper must accept not found arguments
100%
Description
A typical controller call looks like this
/**
* Single view of a news record
*
* @param Tx_News_Domain_Model_News $news news item
* @param integer $currentPage current page for optional pagination
* @return void
*/
public function detailAction(Tx_News_Domain_Model_News $news = NULL, $currentPage = 1) {
}
As long as no news property or a correct uid is given everything is ok but if the record can't be found, the exception
Exception while property mapping at property path "":Object with identity "3" not found.is thrown.
There is currently no way for an extension, to handle this error in the own code! It is very normal, that records get deleted, set to hidden or what so ever. Therefore it must be up to the extension if a default value is set (like NULL) to return NULL by the property mapper!
This behaviour has been introduced with the new property mapper, disabling it makes it work fine again (which is of course no choice at all)
Updated by Gerrit Code Review almost 11 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/27535
Updated by Gerrit Code Review almost 11 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/27535
Updated by Andreas Kießling over 10 years ago
Any news on this one? IMHO a breaking change with the new property mapper!
The fix from Bastian only made it to Flow so far, but we really need this one too :/
Updated by
Updated by Andreas Kießling over 10 years ago
Ah, yes... You're right!
I'd still prefer your patch, as it does not seem to need any new code to handle the issue.
Updated by Florian Schwaiger about 10 years ago
Is this fixed yet or do I have to merge the patch in my project manually?
Updated by Marc Hirdes about 10 years ago
I agree to Florain: Is this fixed yet or do I have to merge the patch in my project manually?
Updated by Henrik Ziegenhain about 10 years ago
Hi all,
nope, this Issue is not merged or fixed by another one.
The patch from above is working for my specific problem.
Updated by Sven Burkert over 9 years ago
Please fix this soon, this bug is really annoying and it applies to very much extensions.
Updated by Johann Carstens over 9 years ago
Would be nice if this could be fixed in the next 6.2.xx Version. I just ran into the same problem.
Updated by Klaus Bitto over 9 years ago
This is a high priority issue also for us.
Is there any official status on the matter?
Updated by Fabian Schöner over 9 years ago
Im also running in this issue almost daily.
Any official status update or better a quick fix would be highly appreciated!
Updated by Marc Bastian Heinrichs over 9 years ago
- Target version changed from 6.2.0 to next-patchlevel
The first patch set is abandoned now.
Please test the new WIP Patch https://review.typo3.org/#/c/39946
Updated by Gerrit Code Review over 9 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/39946
Updated by Gerrit Code Review over 9 years ago
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/42663
Updated by Anja Leichsenring over 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset ba31c035080015cd00c782b54a07e72135926f04.
Updated by