bem / bh-php

PHP port of https://github.com/bem/bh. It's cool thing but better use this:
https://github.com/zxqfox/php-fpm-bem-demo-heroku
MIT License
34 stars 9 forks source link

WeakJSONParser: bug with recursive comments parsing #12

Open qfox opened 9 years ago

qfox commented 9 years ago

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

qfox commented 9 years ago

Need to deprecate it.

kompolom commented 9 years ago

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

qfox commented 9 years ago

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

qfox commented 9 years ago

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

kompolom commented 9 years ago

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

qfox commented 9 years ago

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

composer require zxqfox/WeakJsonParser
WeakJsonParser::parse(file_get_contents("some.bemjson.js"));
kompolom commented 9 years ago

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