ocStore 2.3 - финальное тестирование перед релизом

[chukcha]

  В 13.11.2016 в 17:04, ekaman сказав:

ошибка была на локалке

 

заменил $files = glob($path, GLOB_BRACE);

на $files = @glob($path, GLOB_BRACE);

ошибка ушла

 

правильно сделал или нет?

Это проблемы windows

[ekaman]

  chukcha писав:

Это проблемы windows

 

может это как-то сказываться на нормальной работе очистки кэша?

 

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

Змінено 13 листопада 2016 користувачем ekaman

[chukcha]

Да, конечно, в таком случае glob нужно заменить на

конструкцию вида (код не проверял)

$files = array();
$dir = opendir( $path );
while (false !== ($file = readdir($dir)))
{
        $files[] = $file;
}
closedir($dir);

[chukcha]

admin/model/catalog/product.php

        if (!empty($data['filter_category'])) {
            if (!empty($data['filter_sub_category'])) {
                $implode_data = array();
                $implode_data[] = "category_id = '" . (int)$data['filter_category'] . "'";
                $this->load->model('catalog/category');
                $categories = $this->model_catalog_category->getCategories($data['filter_category']);
                foreach ($categories as $category) {
                    $implode_data[] = "p2c.category_id = '" . (int)$category['category_id'] . "'";
                }
                $sql .= " AND (" . implode(' OR ', $implode_data) . ")";
            } else {
                $sql .= " AND p2c.category_id = '" . (int)$data['filter_category'] . "'";
            }
        }

'filter_sub_category' - ой!!! это ж вроде из фронта,

[globestar]

  В 13.11.2016 в 16:20, pashast сказав:

Вопрос не ко мне. Я не состою ни в администраторах, ни в octeam

Dinox нигде не отвечает, ни в скайпе ни в вк.  

[Sha]

  В 14.11.2016 в 08:52, globestar сказав:

Dinox нигде не отвечает, ни в скайпе ни в вк.  

И не говорите https://drive.google.com/a/webplaneta.com.ua/file/d/0B0-d6y2ivMdzTWlLc0ZmOGE2eVU/view?usp=drivesdk  :|

[globestar]

  В 16.11.2016 в 19:41, Sha сказав:

И не говорите https://drive.google.com/a/webplaneta.com.ua/file/d/0B0-d6y2ivMdzTWlLc0ZmOGE2eVU/view?usp=drivesdk  :|

А что надо было ответить, у меня нету модулей, был какойто один время работы типо, ито его не промодерировали и не опубликовали. Я прочитал пост, проинформирован. Удалите неопубликованый модуль. Хуже от етого никому не будет. 

[Sha]

оффтоп:

  В 17.11.2016 в 01:17, globestar сказав:

А что надо было ответить, у меня нету модулей, был какойто один время работы типо, ито его не промодерировали и не опубликовали. Я прочитал пост, проинформирован. Удалите неопубликованый модуль. Хуже от етого никому не будет. 

Это и нужно было ответить   :-)  Один какой-то тоже пришлось-бы поддерживать.

[dezik92]

скоро ли будет релиз ocStore 2.3? и есть ли смысл поставить на сервер уже то что тестируется?

[Sha]

  В 18.11.2016 в 16:15, dezik92 сказав:

...есть ли смысл поставить на сервер уже то что тестируется?

есть

[chukcha]

Очень прикольный баг, который тянется по всем версиям 2. связанный с работой filemanager

 

смотрим вызов

url: 'index.php?route=common/filemanager&token=' + getURLVar('token') + '&target=' + $(element).parent().find('input').attr('id') + '&thumb=' + $(element).attr('id'),

С одной стороны вроде ничего страшного

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

/admin/index.php?route=setting/setting&token=K5t3b00NDkxpAoFZYcZs2hCQJZ8W1FAv

 

/admin/index.php?route=setting/setting&token=K5t3b00NDkxpAoFZYcZs2hCQJZ8W1FAv#tab_tab-store

Ведь такая локальная ссылка имеет право на жизнь

Что имеем в проблеме

function getURLVar(key) {
	var value = [];

	var query = String(document.location).split('?');

вы получите в query

[0] = ''

[1]  = 'route=setting/setting&token=K5t3b00NDkxpAoFZYcZs2hCQJZ8W1FAv#tab_tab-store'

Далее разбиваем по параметрам

и ищем праметр, в данном случае tokken

В результате получаем

token = 'K5t3b00NDkxpAoFZYcZs2hCQJZ8W1FAv#tab_tab-store'

 

далее token подставляется в url, добавляются еще праметры

но!! hash должен быть последним!!!

а в результате получается

 

index.php?route=common/filemanager&token=K5t3b00NDkxpAoFZYcZs2hCQJZ8W1FAv#tab_tab-store&target=ЗДЕСЬ_УЖЕ_ВСЕ_РАВНО_ЧТО

 

 

Что делать

    var query = String(document.location).split('?');

здесь объект превращается в строку и разделяется

 

меняем на

    var query = document.location.search.split('?');

сразу строка и разделение.

[dinox]

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

[Pegalet]

Не переведена строка "Marketing" в админке, кажется в отчетах. Пул реквест на гитхабе + небольшие исправления с русским языком. Ещё текст статусов возврата обрезаются, issue так же отправлен на гитхаб.

[dinox]

  В 20.11.2016 в 23:54, Pegalet сказав:

Не переведена строка "Marketing" в админке, кажется в отчетах. Пул реквест на гитхабе + небольшие исправления с русским языком. Ещё текст статусов возврата обрезаются, issue так же отправлен на гитхаб.

Спасибо за проделанную работу, Ваш реквест принят

Спасибо все кто принял участие в тестировании и занимался доработкой ocStore! Релиз ocStore выпущен!

Данная тема закрывается все дальнейшее обсуждение в теме релиза