| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.2.5 |
| Bug #3953 | HTML_Common::_updateAttrArray() does not return array | ||
|---|---|---|---|
| Submitted: | 2005-03-24 23:18 UTC | ||
| From: | herojoker at nexgo dot de | Assigned: | thesaur |
| Status: | Closed | Package: | HTML_Common |
| PHP Version: | Irrelevant | OS: | Irrelevant |
| Roadmaps: | (Not assigned) | ||
[2005-03-24 23:18 UTC] herojoker at nexgo dot de
Description:
------------
Hello!
As described in the summary HTML_Common::_updateAttrArray() does not return the updated array.
Since it works with the reference and you don't use the return value (which would be null), you should update the docs (remove @return array).
Furthermore you should documentate that it returns false when $attr2 is not an array. (Eventually you should make use of that in your code)
Hero Wanders
Comments
[2005-03-24 23:25 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2005-03-24 23:43 UTC] herojoker at nexgo dot de
I knew that. I just wanted to assure code quality.
Hero Wanders
[2005-03-24 23:54 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!
[2005-09-01 10:51 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!