search

bem / bh

BH template engine
http://bem.github.io/bh/
MIT License
69 stars 32 forks source link

Escape content when option `escapeContent` enabled #97

Closed mishanga closed 10 years ago

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.01%) when pulling 016c143281a668972d69d3d03687311199c5085c on escape into 2b33e9bb3b82ba43ec2fd449877112dc8c414b62 on master.

arikon commented 10 years ago

@mishanga Не хватает метода ctx.html() для консистентности

mishanga commented 10 years ago

giphy2

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.03%) when pulling 282a37557ba4c507ef57711f4173f3942abb82e8 on escape into 4a71930cadb0373e3ef48b310c36d2033082fa88 on master.

mishanga commented 10 years ago

@arikon добавил ctx.html() и тесты. Посмотри, пожалуйста.

mishanga commented 10 years ago

@dfilatov @greyevil @dench посмотрите, пожалуйста, тоже. Подробности здесь: https://nda.ya.ru/3QTTCZ

arikon commented 10 years ago

@mishanga На мой взгляд всё хорошо (за исключением мелких придирок к стилю)

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.02%) when pulling 71fcc48e825879be6064c6dc49185c0345666aac on escape into 034daeaceb906eb4bd296898b53ad422c166823c on master.

arikon commented 10 years ago

@mishanga Опцией сделать можно? disableContentEscaping, например.

mishanga commented 10 years ago

@arikon это еще +1 проверка на каждый узел. Не хочется делать.

arikon commented 10 years ago

@mishanga Падение в скорости на 7% же не в проверках, а в xmlEscape(), который работает со строками и от этого медленный.

Давай померяем?

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.0%) when pulling 4697b1bed55d9c4bcce47f2afe6e256c639085f5 on escape into bf286b8653546e7b297527e6ef394b46c5db46a8 on master.

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.01%) when pulling 933c4d6d70e8674bedbc0ecf16a502442c543e56 on escape into bf286b8653546e7b297527e6ef394b46c5db46a8 on master.