Счетная проблема для PHP 7.2
У меня есть клиент, который годами полагался на плагин WP, от которого автор отказался несколько лет назад. Я смог внести некоторые исправления, чтобы заставить его работать с обновлением до PHP 7.2, но это меня озадачивает. Чем больше я исследую это, тем меньше я понимаю, как это исправить.
Предупреждение ниже:
Warning: count(): Parameter must be an array or an object that implements Countable in /app/public/wp-content/plugins/ozh-admin-drop-down-menu/inc/core.php on line 311 (first line below is 311)
if (!count($wp_ozh_adminmenu)) {
$wp_ozh_adminmenu = (array)get_option('ozh_adminmenu');
unset($wp_ozh_adminmenu[0]);
}
и какова ценность $wp_ozh_adminmenu? Я предполагаю, что это строка. как насчет теста типа... if (!is_array($wp_ozh_adminmenu) || count($wp_ozh_adminmenu) === 0)
Не уверен, что понял, но просто завернул приведенный выше код в: if (!is_array($wp_ozh_adminmenu) || count($wp_ozh_adminmenu) === 0) { } Это не помогло.
Ответы 1
До PHP 7.2 при передаче немассива в count() он возвращал 1.
Теперь, как вы можете видеть в критические изменения, count() выдает предупреждение, когда вы даете ему не массив.
Простой обходной путь — проверить, является ли это массивом, прежде чем использовать его. Это может потребовать немного больше проверок, если вы хотите, например, разрешить массивы и строки.
<?php
if (!is_array($wp_ozh_adminmenu) || !count($wp_ozh_adminmenu)) {
$wp_ozh_adminmenu = (array)get_option('ozh_adminmenu');
unset($wp_ozh_adminmenu[0]);
}
Это исправило это. Может потребоваться мне немного больше исследований, чтобы понять, как/почему. Спасибо огромное!
@Ray Исследования должны в основном касаться плагина, какие типы переменных он предоставляет вашей функции. Он может использовать несколько типов (например, массив или строку), и слабая типизация PHP позволяет ему это делать. С критическим изменением PHP 7.2 вам нужно немного адаптироваться. Если вы используете только is_array() и больше не разрешаете какой-либо другой тип, вы можете столкнуться с некоторыми проблемами в будущем, когда плагин попытается использовать вместо этого строку.
Является ли
$wp_ozh_adminmenuмассивом? Вы можете убедиться в этом?