Как решить ошибку на php 7.2?

Question

[Gopr]

: sizeof(): Parameter must be an array or an object that implements Countable in

public function notices() {

		if ( isset( $_GET['invalidkeyerror'] ) && '1' == $_GET['invalidkeyerror'] ) {
			appthemes_display_notice( 'error', __( 'Sorry, that key does not appear to be valid. Please try again.', APP_TD ) );
		}

		if ( isset( $this->error ) && sizeof( $this->error ) > 0 && $this->error->get_error_code() ) { //тут ошибка
			$error_html = '<ul class="errors">';
			foreach ( $this->error->errors as $error ) {
				$error_html .= '<li>' . $error[0] . '</li>';
			}
			$error_html .= '</ul>';
			appthemes_display_notice( 'error', $error_html );
		}
	}

Comments

[Сергей delphinpro]

По тексту ошибки и куску кода вангую, что там должно быть
sizeof( $this->error->errors )

[Ernest Faizullin]

Сергей delphinpro, а как раньше работало?))

[Сергей delphinpro]

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

!is_null( $this->error ) && $this->error->get_error_code()

Просто, понятно, и при переходе на 7,2 не вылезло бы ничего =))
В общем-то там ниже уже предложили заменить sizeof на is_object, годное решение. А можно просто выкинуть эту часть условия. Если конечно, ситуация не столь плачевная и $this->error не может внезапно становиться, например, строкой.

Answer 1

[Ernest Faizullin]

... && !empty($this->error) && count($this->error) ...

или просто
... && is_object($this->error) ...

корень проблемы в том что sizeof не может принимать строку или пустое значение

Comments

[Gopr]

if ( isset( $this->error ) && sizeof( $this->error ) > 0 && $this->error->get_error_code() ) {
заменил на
if ( isset( $this->error ) && ! empty($error) > 0 && $this->error->get_error_code() ) {

PHP ошибка пропала, но и предупреждение об ошибке исчезло...

[Gopr]

или просто
... && is_object($this->error) ...
Это подходит сработало, в комментариях дали почти такой же ответ

isset( $this->error ) && is_object($this->error) && $this->error->get_error_code()

Я оставил так:

isset( $this->error ) && is_object($this->error) > 0 && $this->error->get_error_code()

Или ноль лучше убрать?

[Ernest Faizullin]

Gopr, конечно убрать.
is_object возвращает true|false

Answer 2

[wagoodoogoo]

isset( $this->error ) && is_object($this->error) && $this->error->get_error_code()

Answer 3

[Андрей]

если $this->error строка, то \mb_strlen($this->error) > 0