Bug #41809
ConfigurationBuilder throws an error on classes without methods
Status: | Resolved | Start date: | 2012-10-09 | |
---|---|---|---|---|
Priority: | Should have | Due date: | ||
Assigned To: | Rens Admiraal | % Done: | 100% |
|
Category: | - | |||
Target version: | 2.0 beta 1 | |||
Affected Flow version: | Git 1.2 (master) |
Description
The ConfigurationBuilder does a foreach over the result of get_class_methods($class). This throws a fatal error if the class does not have methods.
Related issues
Associated revisions
[BUGFIX] Throw proper exception in ConfigurationBuilder
Add a check if the return value of get_class_methods() used in
autowireProperties() actually is an array. Throw an exception if it is
not, as this should not happen.
Change-Id: I2fdbec9d4513ee27bcb18c87f858d5b17eb1695b
Fixes: #41809
Releases: 1.2
History
#1 Updated by Gerrit Code Review almost 3 years ago
- Status changed from New to Under Review
Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15459
#2 Updated by Gerrit Code Review almost 3 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15459
#3 Updated by Gerrit Code Review almost 3 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15459
#4 Updated by Gerrit Code Review almost 3 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15459
#5 Updated by Rens Admiraal almost 3 years ago
little update, it's not when a class has no methods, just when the class does not exist or another error occurs during fetching the methods
#6 Updated by Gerrit Code Review over 2 years ago
Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15459
#7 Updated by Gerrit Code Review over 2 years ago
Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15459
#8 Updated by Gerrit Code Review over 2 years ago
Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15459
#9 Updated by Gerrit Code Review over 2 years ago
Patch set 8 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/15459
#10 Updated by Rens Admiraal over 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset commit:d488398654c2a1a88ec736aa38d1c8cb2aaa615c.