Александр

Спасибо, хорошее решение! Только таким способом вы отрезаете 2 события: msOnBeforeValidateOrderValue и msOnValidateOrderValue, и вызываться, насколько я понимаю, будут эти события у родителя, то есть валидация будет происходить по стандарту в miniShop2. Конечно, это только если использовать это в плагинах, но тем не менее, почему не сделать так?

public function validate($key, $value)
    {
        if ($key != 'comment') {
            $value = preg_replace('/\s+/', ' ', trim($value));
        }

        $response = $this->ms2->invokeEvent('msOnBeforeValidateOrderValue', array(
            'key' => $key,
            'value' => $value,
        ));
        $value = $response['data']['value'];

        switch ($key) {
            case 'receiver':
                // Transforms string from "nikolaj -  coster--Waldau jr." to "Nikolaj Coster-Waldau Jr."
                $tmp = preg_replace(
                    array('/[^-a-zР°-СЏС‘РЃС–С—С”Т‘СћвЂ™Кј`\s\.\']/iu', '/\s+/', '/\-+/', '/\.+/', '/(\'|вЂ™|Кј|`)+/'),
                    array('', ' ', '-', '.', "'"),
                    $value
                );
                $tmp = preg_split('/\s/', $tmp, -1, PREG_SPLIT_NO_EMPTY);
                $tmp = array_map(array($this, 'ucfirst'), $tmp);
                $value = preg_replace('/\s+/', ' ', implode(' ', $tmp));
                if (empty($value)) {
                    $value = false;
                }
                break;
            default:
                return parent::validate($key, $value);
        }

        $response = $this->ms2->invokeEvent('msOnValidateOrderValue', array(
            'key' => $key,
            'value' => $value,
        ));
        $value = $response['data']['value'];

        return $value;
    }

Готовые решения

miniShop2 удаляет апостроф, украинские и белорусские буквы 13

Александр

28 мая 2018, 14:27

Василий, похоже я неправильно выразился и суть моего сообщения была неверно понята.

Речь как раз о том случае, когда автор согласился с тем, что баг есть, утвердительно написал, что решит в ближайшее время, но это время всё никак не наступает.

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

А про прихоти и собак речи и не было, не знаю, зачем Вы это добавили.
Я может что-то не понимаю, но по-моему в этой ситуации выигрывают оба. Автор получает обратную связь, баг-репорт, практически, о работе своего расширения, что позволяет ему улучшить своё расширение, получить отзывы от реальных проектов, как кто использует его продукт, узнать какие функции интересны людям, чтобы сделать и продукт более функциональным, и привлечь большую аудиторию. А мотивирует его то, что отвечать за свои слова надо вовремя всего лишь статистикой: «Неотвеченные тикеты: 2/150».
И от этого в выигрыше все: довольные покупатели, которые получили корректно работающее то, что хотели, и автор расширения, возможно получив новый опыт, улучшив, исправив свой продукт, сообщество MODX в целом от новой возможности, которую можно внедрить на свой сайт.

Для разработчиков

Защита дополнений в деталях 180

Александр

28 мая 2018, 14:04

Судя по тексту ошибки, скрипт не может переименовать изображение: не нравится ему слеш / в названии, он его экранирует и файл далее не видит из-за этого.

Вопросы

[РЕШЕНО] Swift + ms2Gallery - ошибка ресайза на Селектел 3

Александр

27 мая 2018, 15:45

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

Есть смысл сделать возможность установки тикету статус «решенный» при каждом комментировании, после которого тикет стане только для чтения. И автоматически проставлять этот статус через N-ное количество времени у тикетов, на которые автор дополнения дал ответ, а клиент статус не поставил.

Для разработчиков

Защита дополнений в деталях 180

Александр

27 мая 2018, 15:37

Отвеченный тикет != решенный тикет.
Пример.
— Нашёл баг, не работает….
— Да, есть баг, исправлю.
Проходит 3 месяца… тишина.

Для разработчиков

Защита дополнений в деталях 180

Александр

23 мая 2018, 14:26

Здравствуйте. Не проверял, но нужно дополнительно присоединить таблицу с TV для родителя. Что-то такое:

{'msProducts' | snippet : [
	'templates' => 3,
	'limit' => 0,
	'parents' => 0,
	'tpl' => 'yandexProductsAndPricesOffer',
	'leftJoin' => [
		'parentCategory' => [
			'class' => 'msCategory',
			'on' => 'msProduct.parent = parentCategory.id'
		],
		'parentCategoryTV' => [
			'class' => 'modTemplateVar',
			'on' => 'parentCategoryTV.id=parentCategoryTVvalue.tmplvarid'
		],
		'parentCategoryTVvalue' => [
			'class' => 'modTemplateVarResource',
			'on' => 'parentCategoryTV.contentid=parentCategory.id'
		]
	],
	'select' => [
		'msProduct' => '*',
		'parentCategory' => '*',
		'parentCategoryTVvalue' => '*'
	],
	'where' => [
		'parentCategoryTVvalue.categoryYandexProductsAndPricesEnabled:=' => 1
	],
	'showLog' => 1
]}

Вопросы

Вывести товары с определённым значением TV их категорий 3

Александр

21 мая 2018, 23:25

1) Создать класс в core/components/minishop2/custom/delivery (инструкция)
2) Зарегистрировать созданный класс (инструкция)
3) Указать в нужных методах доставки созданный класс

Вопросы

Бесплатная доставка курьером от определенной суммы 27

Александр

29 апреля 2018, 15:40

1) Не работает с последним msOptionsPrice2. Думаю это как-то связано с:

2.5.2 beta (24.04.2018)
Change «saveProductModification» [msopModification]
Change «isWorkingClassKey, isWorkingTemplates, loadControllerJsCss» [msoptionsprice]
2.5.1 beta (23.04.2018)
Change «msOptionsPriceMsOnGetProductPrice,msOptionsPriceMsOnGetProductWeight,msOptionsPriceMsOnBeforeAddToCart» [msOptionsPricePlugin]
Add «msoptionsprice_mid» [js]

2) При установке даже на чистый MODX на демо modhost.pro не загружаются лексиконы. В меню вместо «cityFields» написано «cityfields_action». Также вместо ошибок выводятся названия лексиконов.

3) Можно ли как-то интегрировать это решение с msOptionsPrice2? Т.е. города и информацию для городов добавлять через cityFields, а цены регулировать msOptionsPrice2?
Например, если товар «iPhone 5» в двух цветах: красный и белый. 2 региона: Москва и Санкт-Петербург. В таком виде, как это сделано сейчас, можно лишь изменить отдельно цену для региона и отдельно цену для цвета, а если добавить как модификатор, то можно будет и для разных регионов и разных опций выставлять цену. К тому же, можно будет выставлять старую цену отдельно для каждой модификации.

Новые дополнения и их версии

cityFields. Города, информация и цены для каждого 92

Александр

11 апреля 2018, 16:33

PHP 7.2
MODX 2.6.2
Хостинг modhost

(ERROR @ /home/***/www/core/components/mschart/model/mschart/mschart.class.php: 120) PHP warning: count(): Parameter must be an array or an object that implements Countable

Новые дополнения и их версии

[msChart] Графики для miniShop2 17

Александр

01 февраля 2018, 12:17