Bug #102692
closed
Avoid type errors in `ResourceFactory` methods
100%
Description
With #92289 logic has been decoupled from `ResourceFactory`
into the `StorageRepository`. As a side change, native php
typeing has been added to a couple of methods.
`StorageRepository->findByUid()` paramter type has been set
to integer, which breaks the `null` fallback and retrieving
the default storage `uid: 0` in `ResourceFactory` method
`getStorageObjectFromCombinedIdentifier()`. Even worser is
the fact that `getObjectFromCombinedIdentifier()` does not
even try to cope with a invalid provided identifier.
Updated by Stefan Bürk 5 months ago
- Related to Task #92289: Decouple logic of ResourceFactory into StorageRepository added
Updated by Gerrit Code Review 5 months ago
- Status changed from New to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/82203
Updated by Gerrit Code Review 5 months ago
Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/82203
Updated by Gerrit Code Review 4 months ago
Patch set 1 for branch 12.4 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/82232
Updated by Gerrit Code Review 4 months ago
Patch set 1 for branch 11.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/82233
Updated by Stefan Bürk 4 months ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset ec79d5c517a4dfa9ef4f80d8706f328c5188fc2f.