note 124047 added to migration72.incompatible

From: 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

« previous php.notes (#211616) next »