Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WeakJSONParser: bug with recursive comments parsing #12

Open
qfox opened this issue Dec 29, 2014 · 7 comments
Open

WeakJSONParser: bug with recursive comments parsing #12

qfox opened this issue Dec 29, 2014 · 7 comments
Labels

Comments

@qfox
Copy link
Member

qfox commented Dec 29, 2014

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

@qfox qfox added the bug label Dec 29, 2014
@qfox
Copy link
Member Author

qfox commented Apr 20, 2015

Need to deprecate it.

@kompolom
Copy link

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

@qfox
Copy link
Member Author

qfox commented Apr 24, 2015

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

@qfox
Copy link
Member Author

qfox commented Jul 18, 2015

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

@kompolom
Copy link

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

@qfox
Copy link
Member Author

qfox commented Jul 20, 2015

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

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

@kompolom
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants