dinox

Релиз ocStore 2.3.0.2

Рекомендуемые сообщения

chukcha    891
defined('HTTPS_SERVER') ? HTTPS_SERVER : HTTP_SERVER,

упс.. как всегда о мултимагазинах забыто

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
jvz    119

Ну, это информационная строка.

просто пример

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
markimax    1 432

off вернет только в одном случае

Не забываем балансировщики, внутренние прокси и т.п. настройки серверов

Там "чево хошь" может вернуться и откуда "хошь"

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
chukcha    891

по правильному

 

Нужно :)

 

isset($this->request->server['HTTPS'])

 

в самом начале определиться...

и тупо сделать так как надо

 

$this->request->server['HTTPS'] = ((isset($_SERVER['HTTPS']) && ($_SERVER['HTTPS'] == 'on' || $_SERVER['HTTPS'] == '1')) || (!empty($_SERVER['HTTP_X_FORWARDED_PROTO']) && (strtolower($_SERVER['HTTP_X_FORWARDED_PROTO']) == 'https') || (!empty($_SERVER['HTTP_X_FORWARDED_SSL']) && $_SERVER['HTTP_X_FORWARDED_SSL'] == 'on')))

 

Я не ошибся в скобках?

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
ArtemPitov    371

давайте тогда подумаем как это все поправить, что бы и красиво и в ocStore добавить, есть 33 места во все проекте где это юзаеться 

https://github.com/myopencart/ocStore/search?p=1&q=HTTP_CATALOG&utf8=✓

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
markimax    1 432

Вообще в opencart не мешало бы Даниэлю сделать в настройках что то типа такого как СТАНДАРТ

 

6711856034.png

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
ArtemPitov    371

Я не ошибся в скобках?
 

Нет, все в порядке 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
ArtemPitov    371

Вообще в opencart не мешало бы Даниэлю сделать в настройках что то типа такого как СТАНДАРТ

 

6711856034.png

Ага - дождемся от него, так может что то такое впились в OC сборку и забыть, а то пока "Даня" родит это 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
markimax    1 432

Ага - дождемся от него, так может что то такое впились в OC сборку и забыть, а то пока "Даня" родит это 

Еще есть проблема в модулях - это должно быть стандартом, иначе все напрасно

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

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
ArtemPitov    371

Еще есть проблема в модулях - это должно быть стандартом, иначе все напрасно

Особенно в модулях с разными точками входа

а если грубо её вот так

 

if ($this->config->get('ssl_settings'))

$this->request->server['HTTPS'] = 1;

else  

$this->request->server['HTTPS'] = 0;

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
RGB    1 004

Проверяли ли обновление с ocstore 2.1.0.2.1 на новый релиз? У меня не получалось нормально обновиться на чистом опенкарте, и, насколько помню курение оф. форума, там у людей тоже корректного обновления движка почти ни у кого не происходило

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
markimax    1 432

а если грубо её вот так

 

if ($this->config->get('ssl_settings'))

$this->request->server['HTTPS'] = 1;

else  

$this->request->server['HTTPS'] = 0;

Да, для google самое то...

А как тем кому надо переехать в yandex с http на https, где магазин должен еще как минимум месяц (до полной индексации) отдаваться как по http так и по https ?

Да yandex тормозит "прогресс" конкретно, да вообще он тормоз

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
ArtemPitov    371

Да, для google самое то...

А как тем кому надо переехать в yandex с http на https, где магазин должен еще как минимум месяц (до полной индексации) отдаваться как по http так и по https ?

Да yandex тормозит "прогресс" конкретно, да вообще он тормоз

редирект ? 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
markimax    1 432

 

Автогенерация SEO URL есть?

 

Для этого есть специализированные модули

Нельзя впихивать в сборки "все подряд"

Главное для сборки - это стабильность

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
AlexDW    781

согласен с Марком

чем больше впихнуть - тем больше шансов потом получить конфликт с каким-нибудь модулем/дополнением

 

в дополнениях смотрите, например такое

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
ArtemPitov    371

Что-то делаем с SSL или так и оставляем ? 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
AWARO    550

Что-то делаем с SSL или так и оставляем ?

а если доп папки добавить с разными вариантами?

залил заменил кому какой надо

или оставил как есть

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
ArtemPitov    371

а если доп папки добавить с разными вариантами?

залил заменил кому какой надо

или оставил как есть

Самый оптимальный вариант сделать все это модификатором и тогда кому надо ссл установит себе 

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
pantagruel964    218

Релиз зарелизили, а на композер болт положили?

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
ReManager    9

Я хз, баг это или я что-то не так сделал.. Но установил... создал товар. БЕЗ картинки... и  на странице с товаром выскочили ошибки:

 

Warning: imagecreatetruecolor(): Invalid image dimensions in /var/www/***/data/www/***/system/library/image.php on line 105Warning: imagealphablending() expects parameter 1 to be resource, boolean given in /var/www/***/data/www/***/system/library/image.php on line 108Warning: imagesavealpha() expects parameter 1 to be resource, boolean given in /var/www/***/data/www/***/system/library/image.php on line 109Warning: imagecolorallocatealpha() expects parameter 1 to be resource, boolean given in /var/www/***/data/www/***/system/library/image.php on line 110Warning: imagecolortransparent() expects parameter 1 to be resource, boolean given in /var/www/***/data/www/***/system/library/image.php on line 111Warning: imagefilledrectangle() expects parameter 1 to be resource, boolean given in /var/www/***/data/www/***/system/library/image.php on line 116Warning: imagecopyresampled() expects parameter 1 to be resource, boolean given in /var/www/***/data/www/***/system/library/image.php on line 118

 

 

no_image.png не отобразилось на странице с товаром... Как исправить подскажите, пожалуйста? я в этом не силен... :(

 

Добавил картинку к товары - ошибки исчезли...Но это не выход...

 

UPD. Я вообще не знаю.. у меня ещё и картинки загружать не хочет. Ошибка SyntaxError выскакивает... :( день не задался, для начала создания магаза ..

Изменено пользователем ReManager

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
AlexDW    781

создал товар. БЕЗ картинки... и на странице с товаром выскочили ошибки:

здесь про это писал, но партия не одобрила :ugeek:

 

как вариант - в файле catalog\controller\product\product.php

 

строку

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

заменить на

$data['popup'] = '';

и обновить кеш модификаторов для верности

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
ReManager    9

про это писал, но партия не одобрила

Спасибо! Помогло решение... Но.. Как всё же сделать заглушку?))

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
AlexDW    781

а нафига показывать ее на странице товара, если изображения нет?

 

чтобы потом для таких товаров без изображений весь инет пестрел картинками-заглушками? :ugeek:

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
pantagruel964    218

а нафига показывать ее на странице товара, если изображения нет?

 

чтобы потом для таких товаров без изображений весь инет пестрел картинками-заглушками? :ugeek:

Достаточно добавить в robots.txt

User-agent: Googlebot-Image
Disallow: /images/zaglushka.jpg

User-agent: YandexImages
Disallow: /images/zaglushka.jpg

Поделиться сообщением


Ссылка на сообщение
Поделиться на другие сайты
Гость
Эта тема закрыта для публикации ответов.

  • Последние посетители   0 пользователей онлайн

    Ни одного зарегистрированного пользователя не просматривает данную страницу