На что ругается сайт?

Question

[Александр Коновалов]

Здравствуйте! В журнале ошибок opencart периодически выскакивают такие ошибки:
2019-04-13 16:53:39 - PHP Notice: Undefined index: product_id in /var/www/data/www/catalog/controller/module/viewed.php on line 15
2019-04-13 16:53:39 - PHP Notice: Undefined index: product_id in /var/www/data/www/catalog/model/module/criteo.php on line 17

viewed.php
https://pastebin.com/2s5UwgbR

criteo.php
https://pastebin.com/mhrgCYKT

Что ему не нравится?

Comments

[Александр Коновалов]

Владимир, прикрепил к вопросу

Answer 1

[InGhost]

Ему не нравится, что в $this->request->get отсутствует значение с ключом product_id.

Т.е. видимо в GET-запросе нет соответствующего аргумента.

Comments

[Александр Коновалов]

InGhost а как решить это?

[Александр Коновалов]

Дмитрий Большаков, спасибо! Это в каком файле надо прописать?

[InGhost]

Александр Коновалов,

Например, в условиях

viewed.php (строка 13)
if (isset($this->request->get['route']) && $this->request->get['route'] == 'product/product') {
заменить на
if (isset($this->request->get['route']) && $this->request->get['route'] == 'product/product' && isset($this->request->get['product_id'])) {

Во втором файле 13 строка
elseif (isset($this->request->get["route"]) && $this->request->get["route"] == "product/product")
на
elseif (isset($this->request->get["route"]) && $this->request->get["route"] == "product/product" && isset($this->request->get['product_id']))

[InGhost]

Владимир, пожалуйста :) Кэп всегда отвечает на очевидные вопросы :)

[Александр Коновалов]

InGhost, спасибо за помощь! Но появилась новая ошибка. Теперь вылезло:
2019-04-14 9:57:27 - PHP Warning: array_pop() expects parameter 1 to be array, null given in /var/www/data/www/catalog/model/module/criteo.php on line 48

Answer 2

[Дмитрий Большаков]

viewed.php
13 строка

if (isset($this->request->get['route']) && $this->request->get['route'] == 'product/product' && isset($_GET['product_id'])) {

upd: 17:30

Comments

[Александр Коновалов]

Дмитрий Большаков это нужно в viewed.php или criteo.php прописать или и там, и там?

[Дмитрий Большаков]

Александр Коновалов, viewed.php (15-ая строка)

[InGhost]

Дмитрий Большаков, не подойдет. Нужно в условие выше засунуть, иначе $product_id не будет иметь значения, и ошибки пойдут далее по скрипту

[Дмитрий Большаков]

Александр Коновалов,
viewed.php
13 строка

if (isset($this->request->get['route']) && $this->request->get['route'] == 'product/product' && isset($_GET['product_id'])) {

Вместо

if(isset($_GET['product_id'])){
$this->request->get['product_id'];
}

Оставляем на 15-ой
$this->request->get['product_id'];

[Дмитрий Большаков]

InGhost, в одно время писали :)

[Александр Коновалов]

Дмитрий Большаков, а вот это что такое? 2019-04-14 9:57:27 - PHP Warning: array_pop() expects parameter 1 to be array, null given in /var/www/data/www/catalog/model/module/criteo.php on line 48

Answer 3

[Mysterion]

$this->request->get['product_id']
Здесь пусто.