-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Conversation
public function toString() | ||
{ | ||
return parent::__toString(); | ||
} | ||
|
||
/** | ||
* @return string | ||
*/ | ||
public function __toString() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this method only calls parent::__toString()
, it has no use and can be removed.
@Martin-P tnx for your comments. I 'm working in an other branch to clean this up. There are several methods in the zf-code component that where not covered by tests and methods like these tostring that are useless. I didn't want to mix those changes, so this one only contains docblock changes. |
The to string methods are possibly added for the same reason why this was done in the FunctionReflection class There are no tests for this method and I could reproduce it by adding a test? |
@ralphschindler You wrote this comment. Can you elaborate on this and provide a link to the bug you were referring to? |
@Martin-P IIRC, we have to override the |
fix some of the documentation errors
…ock_fixes fix some of the documentation errors
…ock_fixes fix some of the documentation errors
No description provided.