Bug #91784
closed
Extbase ObjectAcces::isPropertyGettable ArrayAccess logic breaks usability
100%
Description
Hi there,
Since commit 2a4dcbad895 the logic to check if a property is gettable is somehow reversed...
This implies that a model class implementing ArrayAccess solely has properties which are accessible as array offsets:
if (($object instanceof \ArrayAccess) && !$object->offsetExists($propertyName)) {
return false;
}
This makes it impossible to use the ArrayAccess Interface only for a few properties like in earlier versions. This was handy i.e. to implement some "magic" getters in Fluid.
Updated by
Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/65234
Updated by Gerrit Code Review over 4 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/c/Packages/TYPO3.CMS/+/65234
Updated by Gerrit Code Review about 4 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65234
Updated by Gerrit Code Review about 4 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/65234
Updated by Anonymous about 4 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 43ca70300229e3973f98898f085e5eba30f04d91.
Updated by Gerrit Code Review about 4 years ago
- Status changed from Resolved to Under Review
Patch set 1 for branch 10.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/+/65756
Updated by Gerrit Code Review about 4 years ago
Patch set 2 for branch 10.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/+/65756
Updated by Gerrit Code Review about 4 years ago
Patch set 3 for branch 10.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/+/65756
Updated by Riccardo De Contardi over 1 year ago
- Status changed from Under Review to Closed
Abandoned for 10.4 -> closing it
If you think that this is the wrong decision or experience the issue again, please reopen it or open a new issue with a reference to this one. Thank you.