Предрелиз ocStore 2.3.0.2.4b для тестирования

[dinox]

15 минут назад, mpn2005 сказал:

Ещё не помешает чистка лога модификаторов, при применении. Т.к. на живых магазинах не редкость логи на десятки Мб.

И при каждом заходе в модифкаторы, это всё грузит браузер.

 

В файле:

admin/controller/extension/modification.php

После первого вхождения:

if ($this->validate()) {

Добавить:

$handle = fopen(DIR_LOGS . 'ocmod.log', 'w+');
fclose($handle);

 

На то они и логи чтобы их анализировать на протяжении определенного периода, не вижу целесообразным это применять для дистрибутива. Можно например рассмотреть это в виде дополнительной настройки в вкладке сервер и желающие пусть очищают.  Если есть время, можете сделать и отправить pull реквест с этой правкой, с выносом в настройки я думаю можно это принять в дистрибутив. 

[mpn2005]

Только что, dinox сказал:

На то они и логи чтобы их анализировать на протяжении определенного периода, не вижу целесообразным это применять для дистрибутива. Можно например рассмотреть это в виде дополнительной настройки в вкладке сервер и желающие пусть очищают.  Если есть время, можете сделать и отправить pull реквест с этой правкой, с выносом в настройки я думаю можно это принять в дистрибутив. 

Логи старых применений по сути не актуальны. Т.к. там уже и номера строк, которых нет.

И как правило, нужен именно лог последнего применения, а из админки его просто нереально посмотреть, там получается простыня.

Нет ротации логов, поэтому хранить лог модификаторов с момента запуска магазина - довольно странное дело.

Приходится лезть на фтп, чистить лог, потом уже можно что-то глянуть.

[dinox]

8 минут назад, mpn2005 сказал:

Ещё для новых версий mysql желательно в файлах:

install/model/install/install.php

install/cli_install.php

 

Это :

$db->query("SET @@session.sql_mode = 'MYSQL40'");

 

Заменить на это:

$db->query("SET @@session.sql_mode = ''");

Иначе на более свежих версиях MySQL будет падать с ошибкой

Спасибо, забрал в коммит

[mpn2005]

6 минут назад, dinox сказал:

На то они и логи чтобы их анализировать на протяжении определенного периода, не вижу целесообразным это применять для дистрибутива. Можно например рассмотреть это в виде дополнительной настройки в вкладке сервер и желающие пусть очищают.  Если есть время, можете сделать и отправить pull реквест с этой правкой, с выносом в настройки я думаю можно это принять в дистрибутив. 

Или надо сделать как с системным логом. Хотя бы ограничение, по которому его не грузить.

Текущий код просто свалит страницу в ошибку 500, если не хватит памяти для загрузки.

И из админки его вообще никак даже не почистить.

Вот что имеем:

		// Log
		$file = DIR_LOGS . 'ocmod.log';

		if (file_exists($file)) {
			$data['log'] = htmlentities(file_get_contents($file, FILE_USE_INCLUDE_PATH, null));
		} else {
			$data['log'] = '';
		}

 

[spectre]

может вообще убрать загрузку лога на странице модификаторов? я не знаю кто хоть раз туда заходил в здравом уме и затем чтобы посмотреть этот лог, оно только браузера память жрет или 500 выдаёт 

 

если сильно его надо видеть из админки можно сделать по аналогии с обычным логом

 

[chukcha]

Регулярно пользуюсь

Очистил, обновил
открыл
Скопировал
Посмотрел свою часть
Не всегда прошу доступы к файлам

9 часов назад, mpn2005 сказал:

И из админки его вообще никак даже не почистить.

легко
admin/index.php?route=marketplace/modification/clearlog

 

а вот сделать или ротацию, или просто очищать перед запуском, как мне кажется, имеет смысл

[RGB]

10 часов назад, dinox сказал:

Реквест принят! Спасибо

Добавил еще один, т.к. с правами не до конца исправлена ситуация, на просмотр-то мы их даем, а на редактирование забыли  

[RGB]

С валютами еще проблема, статус в списке Курсы валют неправильно выводится, у меня там всегда Отключено:

Спойлер

И по синхронизации самих курсов вопрос - добавил гривну с кодом UAH (установка чистая), жму Обновить:

Спойлер

Курс обновляется, но выводится непонятно на каком основании. Если рубль - валюта по-умолчанию, то мы должны считать от него, как с долларом (в 1 usd 74.4 rub), а выходит, что в 1 uah аж 26.8 rub, что больше похоже на курс гривны к доллару. Или я чего-то не понимаю?

[dinox]

Все модули реализованы, по логике работы модуля центробанка ЕС, который там тоже есть в списке. В момент когда вы выбираете Валюту по умолчанию Рубль, должен стоят Банк Центробанк РФ, если основная валюта грн, Должен стоять Нацбанк Украины, если Выбираете Цетробанк ЕС основной валютой должен быть Евро. Такая логика работы модулей с ожидаемой версии Опенкарт 4, я не менял концепт, просто адаптировал.  По поводу статуса что включено отключена, это общий баг большинства модулей там в названии переменной должно быть спереди слово module_ тогда этот статус будет отрабатыватся

[chukcha]

1 минуту назад, dinox сказал:

это общий баг большинства модулей там в названии переменной должно быть спереди слово module_ тогда этот статус будет отрабатыватся

так здесь тема о 2.3

 

 

[dinox]

2 минуты назад, chukcha сказал:

так здесь тема о 2.3

 

 

Ну так, если есть время и желание пройдитесь адаптируйте названия переменных. В данном контексте мне было Важно запустить сам фунционал конвертации валют, максимально не меняя механику, ну и упустил момент с этим статусом. Для этого мы и тестируем тут все

[RGB]

38 минут назад, dinox сказал:

По поводу статуса что включено отключена, это общий баг большинства модулей там в названии переменной должно быть спереди слово module_ тогда этот статус будет отрабатыватся

Насколько я вижу, спереди не хватало 'currency_' . 

https://github.com/myopencart/ocStore/pull/206

К слову, еще до установки цбрф он уже включен, так и задумано?

Спойлер

 

И насчет этого  

38 минут назад, dinox сказал:

В момент когда вы выбираете Валюту по умолчанию Рубль, должен стоят Банк Центробанк РФ, если основная валюта грн, Должен стоять Нацбанк Украины, если Выбираете Цетробанк ЕС основной валютой должен быть Евро.

Все равно странно работает, вот что у меня сейчас выбрано:

Спойлер

Включаю один единственный источник - ЦБРФ:

Спойлер

Жму обновить (рубль - валюта по-умолчанию с курсом 1):

Спойлер

Грн все равно считает неправильно

[Starychenko]

Есть очень полезный функционал от авторов форума, который можно было бы добавить в релиз с Вашего согласия и согласия авторов

1. Сортировка модулей в макетов через Drag and Drop от @PaulKravchenko
2. Порядок выполнения модификаторов от @AlexDW
3. Добавить OCMOD Editor, есть на OpenCart.com модуль бесплатный https://www.opencart.com/index.php?route=marketplace/extension/info&extension_id=22015&filter_search=ocmod+editor
4. Добавить при установке Украинский язык, сейчас есть только Русский, Английский. Есть на форуме от @PaulKravchenko
 

Spoiler

 

 

 

 

Змінено 5 березня 2021 користувачем Starychenko

[Tom]

Тема стала скатываться в сторону стола заказов.

Единственное, что точно нужно, это украинская локализация. Остальное каждый вправе сам выбрать и установить.

Кстати самая полная локализация есть у @PaulKravchenko

[Starychenko]

Вот тоже к примеру, я так понимаю для этого релиза тоже актуально ?
 

 

[Starychenko]

Just now, Tom said:

Тема стала скатываться в сторону стола заказов.

Единственное, что точно нужно, это украинская локализация. Остальное каждый вправе сам выбрать и установить.

Кстати самая полная локализация есть у @PaulKravchenko

Вы правы, но я думаю имел бы смысл сделать сбор средств для развития проекта.
Почему бы не сделать голосование за нужны функционал и развить сборку ещё лучше ?

[Tom]

Потому, что выйдет, как в анекдоте "А сейчас мы со всей этой @ернёй попробуем взлететь".

[AlexDW]

@dinox

 

уберите уже художество применением модификаторов по дате

https://github.com/myopencart/ocStore/blob/ocStore2/upload/admin/controller/extension/modification.php

 

[Yakaya]

16 часов назад, dinox сказал:

Добавлено в гит, можете потестить. Добавлена настройка вывода тегов canonical, prev, next, сделали ее  более гибкой. Теперь есть возможность делать тег canonical на любые страницы с  любыми хвостами, а так же оверрайдить его на все страницы, в том числе и главную, а так же по желанию отключать и включать prev next при включенном способе ocStore

 

С модулями по типу [SEO] Full IndeX, SEO мультиязык | мультирегион будет совместимо или какие-то другие решения?

[dinox]

1 час назад, RGB сказал:

Насколько я вижу, спереди не хватало 'currency_' . 

https://github.com/myopencart/ocStore/pull/206

К слову, еще до установки цбрф он уже включен, так и задумано?

  Показать контент

 

И насчет этого  

Все равно странно работает, вот что у меня сейчас выбрано:

  Показать контент

Включаю один единственный источник - ЦБРФ:

  Скрыть контент

Жму обновить (рубль - валюта по-умолчанию с курсом 1):

  Скрыть контент

Грн все равно считает неправильно

А в настройках что стоит как источник?

[Prooksius]

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

А в настройках что стоит как источник?

Там в выдаче курсов ЦБРФ надо номинал учесть еще. То есть получается курс = Value / Nominal

Змінено 5 березня 2021 користувачем Prooksius

[AndreyQ]

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

[Prooksius]

Вот еще есть ошибка одна, если не исправили.

Если есть мультимагазин и мы в одном из магазинов (не дефолтном) добавляем/изменяем локации.
файл admin\controller\setting\store.php
 

Вот этот код с ошибкой:

		$this->load->model('localisation/location');

		$data['locations'] = $this->model_localisation_location->getLocations();

		if (isset($this->request->post['config_location'])) {
			$data['config_location'] = $this->request->post['config_location'];
		} elseif (isset($store_info['config_location'])) {
			$data['config_location'] = $this->config->get('config_location');
		} else {
			$data['config_location'] = array();
		}

нужно:

		$this->load->model('localisation/location');

		$data['locations'] = $this->model_localisation_location->getLocations();

		if (isset($this->request->post['config_location'])) {
			$data['config_location'] = $this->request->post['config_location'];
		} elseif (isset($store_info['config_location'])) {
			$data['config_location'] = $store_info['config_location'];
		} else {
			$data['config_location'] = array();
		}

 

[RGB]

1 час назад, dinox сказал:

А в настройках что стоит как источник?

На 2-м скрине же - ЦБРФ:

Спойлер

 

[dinox]

2 часа назад, Prooksius сказал:

Там в выдаче курсов ЦБРФ надо номинал учесть еще. То есть получается курс = Value / Nominal

 

1 час назад, RGB сказал:

На 2-м скрине же - ЦБРФ:

 

Спасибо @Prooksius, что обратили внимание, это и есть проблема некорректного отображения гривны, отправил в гит, правку. @RGB, посмотрите исправило ли проблему у Вас?