Почему бред? Все там правильно работает, по крайней мере в пределах текущей логики, которая основана на выборке значений через И как между атрибутами, так и между группами атрибутов. Другое дело, что можно было предусмотреть варианты переключения логики работы фильтра, но вы слишком много хотите от бесплатного встроенного фильтра.
И вам спасибо, я думал только dinox может принимать их. Насчет того, что осталось - я думаю нужно сначала с ними разобраться, а потом смотреть дальнейший todo, чтобы не накапливать пулл-реквесты. Так вот, там вроде критичного ничего нет. По номерам:
№35 (отправка почты покупателям конкретной позиции)
Проблема действительно есть, но я не понял, почему нельзя было просто скопировать реализацию из 1.5.4.1 или 1.5.5.1, где все работало (правда без проверки на не-пустой имейл). Получается, что предложенное изменение строки 757 в admin/model/sale/order.php из этого пулл-реквеста
$query = $this->db->query("SELECT DISTINCT email FROM `" . DB_PREFIX . "order` o LEFT JOIN " . DB_PREFIX . "order_product op ON (o.order_id = op.order_id) WHERE (" . implode(" OR ", $implode) . ") AND o.order_status_id <> '0'");
на вот это
$query = $this->db->query("SELECT DISTINCT email FROM `" . DB_PREFIX . "order` o LEFT JOIN " . DB_PREFIX . "order_product op ON (o.order_id = op.order_id) WHERE (" . implode(" OR ", $implode) . ") AND o.order_status_id <> '0' LIMIT " . $start . "," . $end);
с виду работает, но при этом не появляется нотис про успешную отправку письма (у меня не появился). В 1.5.4.1 эта строка выглядит так и прекрасно работает в таком виде:
$query = $this->db->query("SELECT DISTINCT email FROM " . DB_PREFIX . "order o LEFT JOIN " . DB_PREFIX . "order_product op ON (o.order_id = op.order_id) WHERE (" . implode(" OR ", $implode) . ") AND o.order_status_id <> '0'");
Та же ситуация со второй измененной строкой. Исходно, в мастере, она такая:
$query = $this->db->query("SELECT DISTINCT email FROM `" . DB_PREFIX . "order` o LEFT JOIN " . DB_PREFIX . "order_product op ON (o.order_id = op.order_id) WHERE (" . implode(" OR ", $implode) . ") AND o.order_status_id <> '0' LIMIT " . $start . "," . $end);
Ее предлагается изменить вот так
$query = $this->db->query("SELECT DISTINCT email FROM `" . DB_PREFIX . "order` o LEFT JOIN " . DB_PREFIX . "order_product op ON (o.order_id = op.order_id) WHERE (" . implode(" OR ", $implode) . ") AND o.order_status_id <> '0'");
В то время, как в 1.5.4.1 она выглядит следующим образом (и опять же, успешно работает):
$query = $this->db->query("SELECT COUNT(DISTINCT email) AS total FROM " . DB_PREFIX . "order o LEFT JOIN " . DB_PREFIX . "order_product op ON (o.order_id = op.order_id) WHERE (" . implode(" OR ", $implode) . ") AND o.order_status_id <> '0'");
И третье изменения по этому реквесту строки
return $query->row['total'];
на это
return $query->rows;
в 1.5.4.1 вообще отсутствует, там изначально возвращается $query->row['total'] и с ним все работает.
№33 (обрезание лишних пробелов в текстовых полях редактирования товара в админке)
Вообще не критично, но не знаю, нужно ли это - наверное хуже не будет (контент-менеджеры точно спасибо скажут), а с другой стороны это еще один шаг в сторону от оригинальной версии. Хотя ocstore и так раскурочен серьезно, и такие мелочи не особо его усложнят, так что я скорее за, чем против, этого предложения.
№30 (рекурсивная сортировка категорий)
Не понятно, что там должно улучшится, то ли лыжи не едут, то ли реквест не полностью указан.
№25 (сортировка файлов по дате изменения и увеличение макс. размера загружаемого файла)
Согласен с toporchillo, сортировка это дело вкуса, кому то удобнее стандартная, а увеличение лимита аж до 3 гб это перебор, может быть есть смысл поднять с 300 мб до 500 мб, но те, кто такие файлы грузит в магазин, обычно могут позволить себе нанять специалиста, который поднимет лимиты до нужного уровня.
№23 и №22 (скрытие описания категории и производителя на страницах, отличных от 1-й)
На мой взгляд, тут есть и свои плюсы, и свои минусы. Плюсы конечно для поисковиков - меньше дублей контента, лучше отношений к сайту в целом, но это не критично и влияние таких дублей на позиции в выдаче очень слабое, поисковики уже давно научились понимать, что у инет-магазинов уникальность контента не может быть такой же, как например у новостных или информационных ресурсов. А вот минусы у этого предложения посерьезнее, я например у себя делал в описании некоторых категорий акции и спец предложения с анонсами, кроме того там может быть важная информация для покупателей, и если этот покупатель попадет на 2-ю страницу категории из поиска, то он эту информацию уже не увидит (поскольку эта инфа будет видна лишь при переходе на 1-ю страницу), так что мое мнение - эти изменения могут принести больше вреда, чем пользы.
Конструктивные предложения приветствуются, кто хочет поспорить - подключайтесь, так мы быстрее получим стабильный релиз. Напоминаю, что открытые пулл-реквесты находятся здесь, мастер-версию можно скачать здесь, а ссылка на актуальный репозиторий - вот: https://github.com/myopencart/ocStore