vkz / bemhtml-syntax

10 stars 4 forks source link

Regexp is broken after convert #28

Open blond opened 8 years ago

blond commented 8 years ago

Input file suggest-item_type_nav with BEMHTML:

block suggest-item, mod type nav {
    default: {
        var url = this.ctx.data[4];
        applyNext({_url: (url.match(/^\w[\w-]*:\/\//g) ? '' : 'http://') + url});
    }
}

Run bemhtml-syntax@0.0.25:

$ bemhtml-syntax -i suggest-item_type_nav

Actual:

block('suggest-item').mod('type', 'nav')(
    def()(function() {
        var url = this.ctx.data[4];
        return applyNext({
            _url: (url.match('^\w[\w-]*:\/\/' // regexp turned to string
                g) ? '' : 'http://') + url 
        });
    }))

Expected:

block('suggest-item').mod('type', 'nav')(
    def()(function() {
        var url = this.ctx.data[4];
        return applyNext({
            _url: (url.match(/^\w[\w-]*:\/\//g) ? '' : 'http://') + url
        });
    }))
collapsus commented 7 years ago

проблема актуальна, хотим использовать конвертер для bemtree.xjst, и там довольно много регулярок

qfox commented 7 years ago

@collapsus Смотрите в сторону https://github.com/bem/bem-xjst/issues/353 Правда, похоже, этот баг всё равно будет болеть и надо будет чинить. cc @miripiruni

miripiruni commented 7 years ago

Этот репозиторий переехал в https://github.com/bem/bem-templates-converter если что. Вряд ли Влад будет что-то здесь делать.