note 124047 added to migration72.incompatible
| From: | jas at rephunter dot net | Date: | Thu, 18 Jul 2019 16:43:12 +0000 |
| Subject: | note 124047 added to migration72.incompatible | ||
| Groups: | php.notes | ||
| Request: | Send a blank email to php-notes+get-211616@lists.php.net to get a copy of this message | ||
The change in 7.2 to count() to throw an E_WARNING when the item is not countable means that every
use of count() has to be reviewed and some of them changed. I submit that it would have been better
to throw an E_NOTICE, which would prevent this need.
The problem is that many times the parameter to count() in our code can be null.
We have error_reporting = E_ALL & ~E_NOTICE, and don't feel a change to that is a good
idea. But now we have to have a separate test for null, or rework the logic.
----
Server IP: 208.43.231.9
Probable Submitter: 82.199.157.34 (proxied: 73.5.143.70)
----
Manual Page -- http://php.net/manual/en/migration72.incompatible.php
Edit -- https://master.php.net/note/edit/124047
Del: integrated -- https://master.php.net/note/delete/124047/integrated
Del: useless -- https://master.php.net/note/delete/124047/useless
Del: bad code -- https://master.php.net/note/delete/124047/bad+code
Del: spam -- https://master.php.net/note/delete/124047/spam
Del: non-english -- https://master.php.net/note/delete/124047/non-english
Del: in docs -- https://master.php.net/note/delete/124047/in+docs
Del: other reasons-- https://master.php.net/note/delete/124047
Reject -- https://master.php.net/note/reject/124047
Search -- https://master.php.net/manage/user-notes.php