Требуется реальный пример как исправить ошибку с Warning: count(): Parameter must be an array or an object that implements Countable in ....?

Question

[Алексей]

Прошу помочь старику. В логах сервера очень частая ошибка на сайте с joomla 3.8 & PHP 7.3
Уточню, на всякий случай, текст ошибки и фрагмент проблемного кода:

Warning: count(): Parameter must be an array or an object that implements Countable in /..../components/com_weblinks/models/categories.php on line 95, referer: http...

/**
	 * Redefine the function and add some properties to make the styling more easy
	 *
	 * @return  mixed  An array of data items on success, false on failure.
	 */
	public function getItems()
	{
!!!Это строка 95 --->		if (!count($this->_items))
		{
			$app = JFactory::getApplication();
			$menu = $app->getMenu();
			$active = $menu->getActive();
			$params = new JRegistry;

			if ($active)
			{
				$params->loadString($active->params);
			}

			$options = array();
			$options['countItems'] = $params->get('show_cat_num_links', 1) || !$params->get('show_empty_categories_cat', 0);
			$categories = JCategories::getInstance('Weblinks', $options);
			$this->_parent = $categories->get($this->getState('filter.parentId', 'root'));

			if (is_object($this->_parent))
			{
				$this->_items = $this->_parent->getChildren();
			}
			else
			{
				$this->_items = false;
			}
		}

		return $this->_items;
	}

Дорогие гуру, как исправить его, чтобы работало?
Заранее благодарю.

Прошу прощения, я в PHP полный нолик, только в сравнении примеров дохожу как надо. Вот накопал такой вариант, он для моего случая подойдет?
if (isset($this->_items) && !empty($this->_items))

или

if (isset($this->_items) && count($this->_items))

Comments

[maksam07]

Поменять проверку на empty ?

Answer 1

[Алексей]

найдено решение:

if ($this->_items === null)

после замены, сайт работает как обычно, ошибка больше не выводилась в логах.
Спасибо всем за проявленное терпение и принятое участие в решении проблемы.

Comments

[Ипатьев]

Я понимаю, что мы все лоханулись, глядя не не исходную строчку, а на ваши нелепые потуги исправления.
Но скажите, как, КАК вам пришло в голову исправлять if(!count($this->_items)) на if (count($this->_items))?!

[Алексей]

Ипатьев, Вы не поверите, полностью доверился Яндексу с Алисой, среди её ответов попались и эти примеры, вот я и взял их за основу. А до сегодня для меня наличие или отсутствие восклицательного знака в коде - ничего не говорило....

Век живи, век учись ... а всё равно дураком помрёшь

.

Answer 2

[Ипатьев]

if (is_countable($this->_items) && count($this->_items))

ну уж в таком-то варианте должно сработать

Comments

[Алексей]

К сожалению Ваш вариант, ни мои два не работают. Ошибка в логе сервера пропадает, а функционирование сайта наоборот ограничивается, некоторые запросы страниц уходят в 404 ошибку. Пишет, что не найден каталог. С оригинальным файлом работает нормально, но вот в логе несколько строк с ошибкой в час.

[alexalexes]

Алексей, значит не хватает вызова метода, предшествующего вызову getItems, который наполнит $this->_items.
Если ошибка заставляет исправлять сам класс, то предварительно проверьте, а правильно ли используются методы этого класса по стеку вызова вверх.

[Алексей]

alexalexes, спасибо за участие, что не оставили без внимания. Решение найдено, оно от разработчиков:
if ($this->_items === null)
Тему закрываю, т.к. вопрос решен.
Спасибо еще раз всем.

[Ипатьев]

alexalexes, да не, там всё проще. Если бы только этот недотёпа не нарисовал в вопросе свои нелепые потуги на решение, ему тут же бы ответили.

[Алексей]

Ипатьев, я не обижаюсь на критику в мой адрес, ну что же, я же писал, что я полный НОЛЬ в PHP, а в 61 начинать учиться довольно проблематично. Т.ч. извиняйте, если кого ввел в искушение.