Bug #46465

ObjectAccess::getProperty() breaks on \stdClass due to accessor cache

Added by Karsten Dambekalns over 2 years ago. Updated over 2 years ago.

Status:Resolved Start date:2013-03-20
Priority:Must have Due date:
Assigned To:Karsten Dambekalns % Done:

100%

Category:Reflection
Target version:TYPO3 Flow Base Distribution - 2.1
PHP Version: Complexity:
Has patch:No Affected Flow version:Git master

Description

When using getProperty() on \stdClass instances, the accessor cache leads to false results - because the class name alone in this case does not tell anything about the availability of a property.

Associated revisions

Revision d1fdd2a3
Added by Karsten Dambekalns over 2 years ago

[BUGFIX] ObjectAccess::getProperty() breaks on \stdClass due to cache

When using getProperty() on \stdClass instances, the accessor cache
leads to false results - because the class name alone in this case does
not tell anything about the availability of a property.

With this change getProperty() does not use the cache for \stdClass
anymore.

Change-Id: I85aab26ef0975d9af9ec4559cf079014334439a7
Fixes: #46465
Releases: master

Revision 9daf3906
Added by Karsten Dambekalns over 2 years ago

[BUGFIX] ObjectAccess::getProperty() breaks on \stdClass due to cache

When using getProperty() on \stdClass instances, the accessor cache
leads to false results - because the class name alone in this case does
not tell anything about the availability of a property.

With this change getProperty() does not use the cache for \stdClass
anymore.

Change-Id: I85aab26ef0975d9af9ec4559cf079014334439a7
Fixes: #46465
Releases: master

History

#1 Updated by Karsten Dambekalns over 2 years ago

  • Target version changed from 2.0 to 2.1
  • Affected Flow version changed from Git 2.0 to Git master

#2 Updated by Gerrit Code Review over 2 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19082

#3 Updated by Gerrit Code Review over 2 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/19082

#4 Updated by Karsten Dambekalns over 2 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#5 Updated by Gerrit Code Review over 2 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch composer has been pushed to the review server.
It is available at https://review.typo3.org/19376

#6 Updated by Karsten Dambekalns over 2 years ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF