Jump to content
Search In
  • More options...
Find results that contain...
Find results in...

Релиз ocStore 3.0.2.0

dinox

By dinox,
in Новости и анонсы

 Share

Recommended Posts

  • 1 month later...
thentru Mentor

thentru

Posted
Posted
2 часа назад, dewr1 сказал:

Всем доброго времени суток

использую ocStore 3.0.2.0  -  уже второй раз вижу конфликт ocfiltra с seopro - фильтр просто зависает 

откройте файл system\library\seopro.php
найдите после 

public function __construct($registry) {

строку $this->detectAjax(); и перенесите ее перед этой строкой $this->detectPostfix();
после сбросьте кеш и обновите модификаторы

  • +1 1
Link to comment
Share on other sites

dewr1 Newbie

dewr1

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

откройте файл system\library\seopro.php
найдите после 

public function __construct($registry) {

строку $this->detectAjax(); и перенесите ее перед этой строкой $this->detectPostfix();
после сбросьте кеш и обновите модификаторы

Спасибо

Link to comment
Share on other sites

chukcha Grand Master

chukcha

Posted
Posted
14 минут назад, multimen сказал:

OpenCart Alpha 3.1.0.0 вроде как вышла. Я так понимаю можно ждать осторе 3,1 к новому году или я тороплюсь ??

Download & host your own

v3.0.3.2 April 09, 2019. Release notes

 

 

3.1. еще не готова.. альфа не бета 

Link to comment
Share on other sites
1
1
1

  • 2 weeks later...
AlexDW Grand Master

AlexDW

Posted
Posted
7 часов назад, qwert5005 сказал:

Извиняюсь за глупый наверно вопрос, но где можно посмотреть демо админки OcStore 3.x. Демо админки Opencart 3.x нашел, а вот OcStore нет

 

на самом деле, отличный вопрос

потому что демо есть

но вот стандартная связка логин/пароль demo/demo не подходит

Link to comment
Share on other sites
1

costas Grand Master

costas

Posted
Posted

Приветствую.

В файле catalog/controller/startup/seo_url.php есть вызов 

		//seo_pro baseRewrite
		if($this->config->get('config_seo_pro')){		
			list($url, $data, $postfix) =  $this->seo_pro->baseRewrite($data, (int)$this->config->get('config_language_id'));	
		}

При этом нет проверки route на соответствие запроса, в результате любой url содержащий product_id получает url ссылающийся на страницу товара не зависимо от наличия 'product/product'

 

Например есть ссылка такого вида

Цитата

index.php?route=extension/module/help&product_id=72

на выходе после baseRewrite получаем ссылку на товар, что не верно.

 

Проблема в коде seopro.php, вот эта часть кода 

			foreach ($data as $key => $value) {
				switch ($key) {
					case 'product_id':
						$product_id = (int) $value;
						$queries[] = 'product_id=' . $product_id;
						$postfix = true;
						unset($data[$key]);
						break;
					case 'manufacturer_id':
						$manufacturer_id = (int) $value;
						$queries[] = 'manufacturer_id=' . $manufacturer_id;
						$postfix = true;
						unset($data[$key]);
						break;
					case 'category_id':
					.........

Не проверяет route, по хорошему нужно переписать конструкцию с использованием "if" вместо "switch case" c проверкой route на соответствие.

 

Я использовал костыль вида 

		$routes = array(
			'product/product',
			'product/product/review',
			'product/category',
			'product/manufacturer',
			'product/manufacturer/info',
			'information/information',
			'information/information/info',
			'information/information/agree',
			'blog/article',
			'blog/article/review',
			'blog/category'
		);
		
		if($route && in_array($route, $routes)) {

			foreach ($data as $key => $value) {
				switch ($key) {
					case 'product_id':
						$product_id = (int) $value;
						$queries[] = 'product_id=' . $product_id;
						$postfix = true;
						unset($data[$key]);
						break;
					case 'manufacturer_id':
						$manufacturer_id = (int) $value;
						$queries[] = 'manufacturer_id=' . $manufacturer_id;
						$postfix = true;
						unset($data[$key]);
						break;
					case 'category_id':
					case 'information_id':
						$information_id = (int) $value;
						$queries[] = 'information_id=' . $information_id;
						$postfix = true;
						unset($data[$key]);
						break;
					//blog	
					case 'blog_category_id':
						$blog_categories = explode('_', $value);
						foreach ($blog_categories as $blog_category_id) {
							$queries[] = 'blog_category_id=' . (int) $blog_category_id;
						}
						unset($data[$key]);
						break;
					case 'article_id':
						$article_id = (int) $value;
						$queries[] = 'article_id=' . $article_id;
						$postfix = true;
						unset($data[$key]);
						break;

					//blog	
					case 'path':
						$categories = explode('_', $value);
						foreach ($categories as $category_id) {
							$queries[] = 'category_id=' . (int) $category_id;
						}
						unset($data[$key]);
						break;
					default:
						break;
				}
			}
		}

 

  • +1 1
Link to comment
Share on other sites
1

optimlab Experienced

optimlab

Posted
Posted

@halfhope , лайк за пул-реквест!

Это получается при условии "Разрешить восстановление пароля - ДА", на всех демо-сайтах можно заблокировать на час аккаунт "demo"?

Link to comment
Share on other sites
1

AlexDW Grand Master

AlexDW

Posted
Posted
21 минуту назад, optimlab сказал:

Это получается при условии "Разрешить восстановление пароля - ДА", на всех демо-сайтах можно заблокировать на час аккаунт "demo"?

нет

Link to comment
Share on other sites
1

halfhope Veteran

halfhope

Posted
Posted
19 минут назад, optimlab сказал:

@halfhope , лайк за пул-реквест!

Это получается при условии "Разрешить восстановление пароля - ДА", на всех демо-сайтах можно заблокировать на час аккаунт "demo"?

 

Нет, читайте внимательно. Я говорю не об админке, а о каталоге, об аккаунтах покупателей.

  • +1 1
Link to comment
Share on other sites
1
1
1

chukcha Grand Master

chukcha

Posted
Posted
41 минуту назад, costas сказал:

вопрос защиты административной части никакого отношения к клиентской части не имеет.

В данном случае - защиты от подбора - имеет

И дело не в тролинге, как вы хотите думать, и не в том, как защищать админку, а в том, как отказать в обслуживании.

Link to comment
Share on other sites
1
1
1

optimlab Experienced

optimlab

Posted
Posted
3 часа назад, chukcha сказал:

Согласен с @halfhope  - прошла валидация  смены пароля. Подозрение на атаку  должно быть снято

Какие-то двойные стандарты применеы в ОС

т.е. атака в админку не валидируется., а пользовательская - да

Поддерживаю!

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

  • +1 1
Link to comment
Share on other sites
1

  • 2 weeks later...
AlexDW Grand Master

AlexDW

Posted
Posted
2 часа назад, Andronmakarkov сказал:

подскажите как ЧПУ включить?

 

здесь это немножко через #опу

 

плюс

Link to comment
Share on other sites
1

  • 2 months later...
dinox Veteran

dinox

Posted
  • Author
Posted

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

  • +1 2
Link to comment
Share on other sites

  • 2 weeks later...
ogomor Community Regular

ogomor

Posted
Posted

Здравствуйте! Нашёл на сайте вот такой пост 

 

Цитата

Администрация форума провела переговоры с командой OpenCart Pro и было принято решение о совместной разработке новых версий ocStore начиная с версии 3.0, все полезные решения которые были в OpenCart CMS мигрируют в ocStore. Дальнейшая разработка OpenCart CMS  будет прекращена, а OpenCart Pro будет в дальнейшем реализован на основе ocStore это позволит унифицировать наработки и упростит разработку дополнений.

 

Тут в описании никакой информации. Скажите пож. слияние не состоялось или что?

Link to comment
Share on other sites

  • 3 weeks later...
kamar87 Mentor

kamar87

Posted
Posted

Что-то не могу разобраться... В карточке товара пишет Производители хотя в языковом файле написано Производитель:

 

откуда оно его тянет?

Link to comment
Share on other sites

pawana Experienced

pawana

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

Тянет из языкового файла

Уже раз десять пересмотрел :)))

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

В шаблоне посмотреть название перменной

ото ж, все страндартно :(

Link to comment
Share on other sites

AlexDW Grand Master

AlexDW

Posted
Posted
3 часа назад, chukcha сказал:

Тянет из языкового файла
В шаблоне посмотреть название перменной

да из футера переписывает, та же фигня с языками что и про пустую корзину

сам же патч писал ;)

 

нужно бы его включить в следующий релиз

  • +1 1
Link to comment
Share on other sites
1

  • dinox unpinned this topic

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
 Share
Go to topic listing

Покупателям

Разработчикам

Полезная информация

Последние дополнения

×
×
  • Create New...

Important Information

On our site, cookies are used and personal data is processed to improve the user interface. To find out what and what personal data we are processing, please go to the link. If you click "I agree," it means that you understand and accept all the conditions specified in this Privacy Notice.

  I accept