ocStore 2.3 - imagecreatetruecolor(): Invalid image dimensions

[markimax 1 490]

Когда будет устранена ?
 

Цитата

PHP Warning:  imagecreatetruecolor(): Invalid image dimensions in /var/www/autopazzle/data/www/new.autopazzle.ru/system/library/image.php on line 105

Я там видел на гите кто то пытался костылем закрыть - так вот костыль ошибочный.
Замечу: в оригинальном opencart 2.3.x такой ошибки нет
Первопричину надо найти, там где то намудрили с переменными настроек изображений темы, взяли со старой версии ocStore 2.1 куски кода
Ctrl-C -> Ctrl-V =  новый баг и ищи "свищи" потом по тысячам срок кода opencart, где вставили баг
 

[Matvei 46]

4 часа назад, markimax сказал:

Когда будет устранена ?
 

Я там видел на гите кто то пытался костылем закрыть - так вот костыль ошибочный.
Замечу: в оригинальном opencart 2.3.x такой ошибки нет
Первопричину надо найти, там где то намудрили с переменными настроек изображений темы, взяли со старой версии ocStore 2.1 куски кода
Ctrl-C -> Ctrl-V =  новый баг и ищи "свищи" потом по тысячам срок кода opencart, где вставили баг
 

так и есть! создание ocStore 2.3 осуществлялось через  Ctrl-C -> Ctrl-V  из версии 2.1

[toporchillo 436]

Такая ошибка возможна и в оригинальном opencart 2.3, 2.1.

В настройках магазина ставите разрешение любой картинки -100 x -100 и будет вам такая ошибка.

 

Стэк вызовов предоставьте.

Каждый может запилить модуль, который делает

$this->model_tool_image->resize($image, 'раз', 'два');

и получит такую же ошибку, которая указывает на system/library/image.php

 

[AlexDW 823]

6 часов назад, markimax сказал:

Замечу: в оригинальном opencart 2.3.x такой ошибки нет

потому что оригинальный opencart 2302 НЕ ВЫВОДИТ заглушку, когда нет главного изображения

достаточно глянуть его файл catalog\controller\product\product.php, строка 287:

$data['popup'] = '';

 

вывод заглушки - инициатива разработчиков ocStore, здесь обсуждалось

и да, для ее вывода скопировали строку от старой версии - отсюда и ошибка

$data['popup'] = $this->model_tool_image->resize('no_image.png', $this->config->get('config_image_popup_width'), $this->config->get('config_image_popup_height'));

 

в 2302, с учетом изменений в конфигах темы, тогда уж так:

$data['popup'] = $this->model_tool_image->resize('no_image.png', $this->config->get($this->config->get('config_theme') . '_image_popup_width'), $this->config->get($this->config->get('config_theme') . '_image_popup_height'));

 

[shoputils 230]

Господа, вам в гит религия не позволяет заглядывать? Это исправлено почти 3 месяца назад. https://github.com/myopencart/ocStore/commit/427e094b3d1c9fa5e92675a170a950ae8bb22f10#diff-382f353ca0870b9adb2df1c50fdcca75

[markimax 1 490]

53 минуты назад, shoputils сказал:

Господа, вам в гит религия не позволяет заглядывать?

А религия не позволяет за 3 месяца убрать архив с критичный багом из https://myopencart.com/downloads
Потому что пользователи постоянно спотыкаются там.
И кстати этот баг вылазит и после "патча" в других местах при "определенных" условиях. Замечено когда теме default меняешь папку

 

[nikifalex 333]

Достаточно иметь "стабильную" ветку, в которую постить только исправления вот таких багов. И соответственно собирать свежий архив.

Но что-то пошло не так.

[AlexDW 823]

32 минуты назад, markimax сказал:

А религия не позволяет за 3 месяца убрать архив с критичный багом из https://myopencart.com/downloads

 

28 минут назад, nikifalex сказал:

Достаточно иметь "стабильную" ветку, в которую постить только исправления вот таких багов. И соответственно собирать свежий архив.

предлагаете каждый день выпускать по релизу, типа ocStore 2302.48, 49, 50.. ? - это только внесет неразбериху

 

в целом, да - хотелось бы видеть выход релизов с исправленными багами чаще, чем раз в 3-4 месяца

но для этого нужно как минимум выработать политику/регламент со стороны администрации/разработчиков

 

плюс желательно вместо критики вносить свою лепту в участие, хотя бы на уровне тестирования

а то получается, что про найденные баги пишут практически новички, а предлагающих коммиты/исправления - вообще можно пересчитать по пальцам одной руки

[markimax 1 490]

52 минуты назад, AlexDW сказал:

 

предлагаете каждый день выпускать по релизу, типа ocStore 2302.48, 49, 50.. ? - это только внесет неразбериху

 

в целом, да - хотелось бы видеть выход релизов с исправленными багами чаще, чем раз в 3-4 месяца

но для этого нужно как минимум выработать политику/регламент со стороны администрации/разработчиков

плюс желательно вместо критики вносить свою лепту в участие, хотя бы на уровне тестирования

а то получается, что про найденные баги пишут практически новички, а предлагающих коммиты/исправления - вообще можно пересчитать по пальцам одной руки

За три месяца ? Один супер критичный баг ?
Не убрали, с официальной "ссылки" скачивания.
За последнее время я встречал единицы кто поставил ocStore 2.3
Вот вам стратегия "ваша".
Понятное дело - скачивают с официальной страницы "релиз", а там критичный баг трехмесячной давности, который вылазит сразу. Кто то после этого поставит эту сборку боевой?
И только и вижу теперь ставят, или opencart.cms/pro или "русская сборка" 2.3

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

[levit 9]

Честно, не понимаю дальнейшее развитие проекта ocStore, если из коробки OpenCart данной ошибки нет. Получается, что сразу же при установке ocStore мы (пользователи) получаем данную ошибку!!!! Баг как я понял тянется с выходом второй версии и на него просто тупо забили... 

[ekaman 4]

В 10.02.2017 в 15:40, levit сказал:

Честно, не понимаю дальнейшее развитие проекта ocStore, если из коробки OpenCart данной ошибки нет. Получается, что сразу же при установке ocStore мы (пользователи) получаем данную ошибку!!!! Баг как я понял тянется с выходом второй версии и на него просто тупо забили... 

 

причина описана тут

 

[levit 9]

Цитата

потому что оригинальный opencart 2302 НЕ ВЫВОДИТ заглушку, когда нет главного изображения достаточно глянуть его файл..

Так адекватного решения нет все равно, если нет изображения! а если товаров десятки тысяч, как быть? Постоянно подтирать ошибки?
Мелочи конечно, но в готовой версии таких багов не должно быть. Так же и не решено в ocStore 2.3 с поиском. Когда включен seoPro поиск не ищет в категории товаров, а редиректит просто в выбранную категорию. надо предупреждать об этом при опубликовании версии до конечного пользователя.

Изменено 14 февраля пользователем levit

[LexusZP 1]

Прочитал дебаты, не очень понял, вопрос таки решен или нет с ошибкой  PHP Warning: imagecreatetruecolor(): Invalid image dimensions in /home/lexuszp/public_html/system/library/image.php on line 105/108/109/110...

у меня версия 2.3 в ней вылазит если есть файл \image\no_image.png если его убрать, то ошибка пропадает, но соответственно пропадает "заглушка" в других местах где нет картинки, конкретно эта ошибка вываливается только в карточке товара.

Изменено 12 марта пользователем LexusZP

[AlexDW 823]

12 часов назад, LexusZP сказал:

Прочитал дебаты, не очень понял, вопрос таки решен или нет с ошибкой  PHP Warning: imagecreatetruecolor():

 

тему с самого начала прочитайте, внимательно

а вообще, лучше обновить ocStore 2302 до версии 23021 - там это исправлено, как и многое другое

[Marianna 56]

В 10.02.2017 в 09:58, AlexDW сказал:

потому что оригинальный opencart 2302 НЕ ВЫВОДИТ заглушку, когда нет главного изображения

достаточно глянуть его файл catalog\controller\product\product.php, строка 287:

$data['popup'] = '';

 

вывод заглушки - инициатива разработчиков ocStore, здесь обсуждалось

и да, для ее вывода скопировали строку от старой версии - отсюда и ошибка

$data['popup'] = $this->model_tool_image->resize('no_image.png', $this->config->get('config_image_popup_width'), $this->config->get('config_image_popup_height'));

 

в 2302, с учетом изменений в конфигах темы, тогда уж так:

$data['popup'] = $this->model_tool_image->resize('no_image.png', $this->config->get($this->config->get('config_theme') . '_image_popup_width'), $this->config->get($this->config->get('config_theme') . '_image_popup_height'));

 

а для категории как надо?