Интересно, а в какой момент beta станет стабильным релизом? Как ты для себя определяешь этот рубеж?

UPD 1.0.1-beta
— Добавлена в меню кнопка сканирования элементов

Спасибо большое! Первый вариант подошел, работает, по второму ошибка парсинга Unexpected token '?' Ругается на вопрос почему-то.

По данному варианту ошибка парсинга Unexpected token '?' Ругается на вопрос!

Спасибо, уже)

Warning: dirname() expects exactly 1 parameter, 2 given in
Поменяйте версию php на 7.0 или выше

В общем, все решилось сменой версии PHP. Хотя вроде проверял при установке.

А не проще, сделать проверку по ID?

{$vendor in list [6, 7] ? 'Цена по запросу' : $price}

Без разницы где велась разработка. Нужно проверять пути и права на файлы и папки.

разработка тестового сайта велась на поддомене другого домена. Скорее всего проблема в путях к minishop2?

Перемещал с помощью товарищей с рег ру, сейчас копаюсь в мануалах по переносу и нашел пару ошибок в путях core/config/config.inc.php, там был указан старый домен. Пока что вопрос не решился, продолжаю.

В путях. Правда непонятно как получилось запороть пути к конкретному компоненту. При переносе случайно ядро не перемещали на уровень выше? Ещё можно права на папки проверить.

Пара советов, если позволишь:

Не просто позволю, а ещё и благодарен буду)))

и далее в коде методов ты не будешь париться и обращаться к конфигу так:

Ну вообще запись станет длиннее this.config.someprop против this.someprop, но я доверюсь твоему опыту.

Обычно принято в мире js передавать селектор первым аргументом, а вторым конфиг плагина

Тут отдельная проблема. В конструктор надо передавать уникальный селектор, потому что в противном случае если на странице больше 1 вызова AjaxForm обработчики навешиваются столько раз сколько вызывается сниппет, а это плохо по понятным причинам. И передавать селектор в каждом вызове мне кажется неудобным. Я думаю надо установить селектор по умолчанию и дополнять его меткой времени и тоже самое делать с переданным селектором, чтобы на 100% исключить вероятность дублирования или есть варианты решения получше?

3. У тебя на 20-ой строке есть странность if (this.beforeSubmit(e.target)) { и судя по коду твой метод beforeSubmit всегда возвращает true, тогда я понятия не имею зачем тут проверка.

Это у меня он возвращает true, а если кто-то захочет свою логику, например не отправлять пока не прочитают политику работы с персональными данными?

Ну и можно по стандартам пройтись :|

Я самоучка, это не оправдание, но что значит пройтись по стандартам не понимаю.

И есть ещё один момент — уведомления

if (AjaxForm.Message != 'undefined') {
    AjaxForm.Message.close();
}

Это я оставил для обратной совместимости, но это объект, мне кажется правильнее было бы сделать показ сообщения методом или методами класса, но как туда впихнуть код выше? Тут предлагали сделать как @Николай Савин в minishop2, но у него это тоже объект. Может уведомления сделать отдельным классом? Добавить из коробки уведомления не требующие jQuery, только вот надо же ещё дать возможность подключать произвольные уведомления, но как это лучше сделать?

Вы решили проблему?

Заголовок и суть вопроса были отформатированы высокоразвитым искусственным интеллектом.

@Фарит Не надо писать такие заголовки, заголовок вопроса должен отражать суть вопроса и до ката также

{if $_pls['vendor.name'] in list ['nokia', 'iphone']}
    Цена по запросу
{else}
    {$price}
{/if}

{$_pls['vendor.name'] in list ['nokia', 'iphone'] ? 'Цена по запросу' : $price}

Ну и можно по стандартам пройтись :|

Опа, а это уже лучше. Пара советов, если позволишь:

1. Можно учитывать, что конфиг может по какой-то причине не будет доступен или он придёт неполным

constructor (config) {
    this.defaults = {
        someProp: true,
        anotherProp: 'long string',
    }

    this.config = Object.assign({}, this.defaults, config);
}

и далее в коде методов ты не будешь париться и обращаться к конфигу так:

this.config.someProp

2. Обычно принято в мире js передавать селектор первым аргументом, а вторым конфиг плагина, это намного удобнее, и я понимаю, что AjaxForm в оригинале такой и ты просто перевел код, но тут заодно можно и это исправить:

class AjaxForm {
    constructor (selector, config) {
        if (!selector) {
            console.error('Нет селектора');
        }
        
        this.defaults = {
            someProp: true,
            anotherProp: 'long string',
        }
    
        this.config = Object.assign({}, this.defaults, config);

        this.forms = document.querySelectorAll(selector);
    }
}

// И теперь можно вызывать и с конфигом и без
new AjaxForm(selector, config);
new AjaxForm(selector);

3. У тебя на 20-ой строке есть странность if (this.beforeSubmit(e.target)) { и судя по коду твой метод beforeSubmit всегда возвращает true, тогда я понятия не имею зачем тут проверка.

Супер, спасибо