bem/bh-php

WeakJSONParser: bug with recursive comments parsing

qfox opened this issue · 7 comments

qfox commented

Need to add an ability to fail on js code and skip comments parsing

qfox commented

Need to deprecate it.

Ты хочешь запретить комменты в нативном bemjson? под нативным, имею ввиду тот который на js

qfox commented

Я хочу выкинуть этот функционал в отдельный модуль, если он кому-то нужен, и делать сборку bemjson.js в bemjson.json при сборке ;-) Т.е. собирать bemjson.js нодой, потому что сейчас ни require, ни forEach не написать.

qfox commented

@kompolom А он нужен? Просто мы с перепугу сделали парсер js-like синтаксиса, но попарсить json без кавычек это одно, а выполнять js — несколько сложнее)

@zxqfox я часто пишу обычный bemjson и скармилваю bh.php

qfox commented

@kompolom Ты сможешь поставить себе модуль WeakJsonParser? ;-) И пропускать явно через него. Что-то типа:

composer require zxqfox/WeakJsonParser
WeakJsonParser::parse(file_get_contents("some.bemjson.js"));

@zxqfox не вопрос. Ты же в доку добавишь про модули. ))