Нелегальная строка смещения PHP 5.6

Question

У нас есть старая версия CakePHP, что мы переехали на новый сервер PHP 5.6, и мы начали Получать эту ошибку при добавлении продукта в корзину:

предупреждение (2): Illegal строка смещения '' AddBasketData [APP/контроллеры/personalisation_controller.php, строка 848]

Вот строка 848 в файле:

if (is_array($this->data['AddBasketData'])) { 
    $personalisation_data['Personalise'] = $this->data['AddBasketData']; 
}else { 
     $personalisation_data['Personalise'] = array(); 
} 

Может ли кто-нибудь пролить свет на это, я думаю, что это до определенной версии PHP, в которой мы сейчас работаем, но любая помощь будет отличной.

Благодаря

Answer 1

Как отметил @ roberto06 в комментариях к вашему вопросу, почему вы получаете ошибку, потому что вы пытаетесь лечить строковое значение как массив.
Причина этого конкретного сообщения об ошибке заключается в том, что вы можете использовать обозначение смещения массива для извлечения одного символа из строки. Как и в строковых массивах C. Но это поддерживает только числовые индексы, а не строковый индекс, как показано в коде, который вы опубликовали.

Теперь простой способ остановить возникшую ошибку - просто проверить тип данных и определить, действительно ли данный индекс существует.
Как так:

if (is_array ($this->data) && !empty ($this->data['AddBasketData'])) { 
    $personalisation_data['Personalise'] = $this->data['AddBasketData']; 
} else { 
    $personalisation_data['Personalise'] = array(); 
} 

Однако, видя, как вы не вдруг получаю эту ошибку после обновления намекает на что-то еще является проблемой. Тем более, что введенный вами код ожидает, что значение, хранящееся в индексе Personalise, будет массивом. Я проследил бы, где член $this->data будет установлен и изменен, чтобы узнать, можете ли вы найти основную причину, почему это кажущееся изменение в поведении. В конце концов, это может быть побочным эффектом более гнусной тонкой ошибки.

Answer 2

Преобразование мои комментарии в ответ:

Проблема здесь, кажется, что $this->data является string и не array. Вы должны проверить это первое, а затем проверить, если смещение AddBasketData существует, и, наконец, если смещение AddBasketData является array:

if (is_array($this->data) && isset($this->data['AddBasketData']) && is_array($this->data['AddBasketData'])) { 
    $personalisation_data['Personalise'] = $this->data['AddBasketData']; 
} else { 
    $personalisation_data['Personalise'] = array(); 
} 

Конечно, это будет только исправить симптомы (что поднятое предупреждение), вам может потребоваться некоторая отладка кода, чтобы выяснить, почему $this->data является string вместо array.