dinox

Релиз ocStore 2.3.0.2

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

Anatohi    12

Спасибо большое Всем кто принимал участие в выходе этого релиза! Ребята от души низкий Вам всем поклон

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


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

Приемного благодарен.

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


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

Дождались ) Спасибо огромное

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


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

Супер, спасибо!

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


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

Если в процессе роботы\тестирования будут выявлены баги - добавляйте пулл-реквесты на гитхаб https://github.com/myopencart/ocStore/tree/dev

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


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

При установке на хостинг в магазине нельзя переключить ни валюту ни язык магазина, список ошибок пуст. На локалке все работает отлично.

 

 

Сори, разобрался. Проблема была в браузере.

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

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


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

Спасибо за релиз! Очень ждал. :eek:

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


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

Огромное спасибо за развитие OcStore!!! :eek:

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

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


Ссылка на сообщение
Поделиться на другие сайты
markimax    1 432
  • Добавлена проверка SSL сертификата в catalog/controller/startup/startup.php

 

$_SERVER['HTTPS'] = 'off';

?

if ($this->request->server['HTTPS']) { ...

$_SERVER['HTTPS'] = 'off';

 

Не, не встречали ? А я частенько. Что будет с кодом выполнения IF ? Да, баг, к сожалению

 

Настройки https у разных хостеров могут диаметрально отличаться. У 50% хостеров они вообще криво - левые. Может вообще не быть $_SERVER['HTTPS], да и такое часто бывает

 

Так что код не зачтен = баг

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


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

Марк, поделитесь своим решением https://github.com/m...cStore/tree/dev

ArtemPitov - однозначно правильного решения нет

Есть на 99%. T.е. при 10`000 скачиваний ocStore 100  магазинов все равно словят баг связанный с https

Но

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

Имеет и того меньший процент ~ 50%

Да еще и notice получат, так как у многих вместо $_SERVER['HTTPS'] используется $_SERVER['HTTP_X_FORWARDED_PROTO'] или $_SERVER['HTTP_X_FORWARDED_SSL']

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


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

Марк, поделитесь своим решением https://github.com/m...cStore/tree/dev

Код на 99%

if ((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

 

Код на 99%

if ((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'))) {
...
}

Поправим  :-)

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


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

Но все это можно чаcтично решить строкой в htaccess

 

RewriteEngine On
RewriteCond %{HTTP:X-Forwarded-Proto} https
RewriteRule .* - [E=HTTPS:on]
 

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


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

Поправим  :-)

Толку... никакого  - там в opencart еще туча багов связанных с "https"

Я промолчу еще сколько их в модулях. Все равно на выходе получается "каша"

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


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

Толку... никакого  - там в opencart еще туча багов связанных с "https"

Я промолчу еще сколько их в модулях. Все равно на выходе получается "каша"

нужно все собирать и фиксить и только тогда мы все получим хороший продукт 

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


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

Не, а что делать если разработчик вообще не думает

 

а тупо пишет HTTP_CATALOG . "

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


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

Не, а что делать если разработчик вообще не думает

 

а тупо пишет HTTP_CATALOG . "

https://github.com/myopencart/ocStore/blob/dev/upload/admin/controller/extension/feed/google_base.php

66 строка  :-D

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


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

Не, а что делать если разработчик вообще не думает

 

а тупо пишет HTTP_CATALOG . "

Та да. вот и я о том же :(

И это помимо багов opencart в стиле

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

Где может запросто быть $_SERVER['HTTPS'] = 'off'

Там куда не кинь взгляд с https везде баги

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


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

Да в opencart они на каждом шагу

 

order.php !!!

 

Что будет при $_SERVER['HTTPS'] = 'off'  или отсутствии $_SERVER['HTTPS']

if ($order_info) {
                $this->load->language('extension/payment/ocstore_payeer');
                if (($this->config->get('ocstore_payeer_order_later_status_id') == $this->request->post['order_status_id']) && ($order_info['payment_code'] == 'ocstore_payeer')) {
                        $onpay_url = sprintf('%sindex.php/?route=extension/payment/ocstore_payeer/laterpay&order_id=%s&order_tt=%s',
                                                defined('HTTPS_SERVER') ? HTTPS_SERVER : HTTP_SERVER,
                                                $order_info['order_id'],
                                                $order_info['total']
                        );
                        $this->request->post['comment'] .= "\n" . sprintf($this->language->get('text_payeer_onpay'), $onpay_url);
                }

                $this->load->language('extension/payment/ocstore_yk');
                if (($this->config->get('ocstore_yk_order_later_status_id') == $this->request->post['order_status_id']) && (strpos($order_info['payment_code'], 'ocstore_yk') !== false)) {
                    $onpay_url = sprintf('%sindex.php/?route=extension/payment/ocstore_yk/laterpay&order_id=%s&order_ttl=%s&paymentType=%s',
                                                                isset($this->request->server['HTTPS']) && $this->request->server['HTTPS'] ? $this->config->get('config_ssl') : $this->config->get('config_url'),
                                                                $order_info['order_id'],
                                                                $order_info['total'],
                                                                $order_info['payment_code']
                                                             );
                    $this->request->post['comment'] .= "\n" . sprintf($this->language->get('text_yk_onpay'), $onpay_url);
                }

                $this->load->language('api/order');

                $this->model_checkout_order->addOrderHistory($order_id, $this->request->post['order_status_id'], $this->request->post['comment'], $this->request->post['notify'], $this->request->post['override']);

                $json['success'] = $this->language->get('text_success');
            } else {
                $json['error'] = $this->language->get('error_not_found');
            }

И так везде в opencart

Не могли уже сделать ОДНУ точку входа  этой определяемой переменной (или константы), чтобы потом через define('HTTPS', ...функция определения...); и т п

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


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

Та да. вот и я о том же :(

И это помимо багов opencart в стиле

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

Где может запросто быть $_SERVER['HTTPS'] = 'off'

Там куда не кинь взгляд с https везде баги

 

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

 

 

Обратите внимание, что при использовании ISAPI с IIS значение будет off, если запрос не был произведен через протокол HTTPS.

 

Но самое интересное в том что есть 33 вставки по коду HTTP_CATALOG

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


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

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

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


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

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

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