search

yandex-ui / noscript

Noscript: JavaScript MVC Framework for building SPA
http://yandex-ui.github.io/noscript/
MIT License
34 stars 22 forks source link

Поломанный ключ в случае наличия управляющих символов в значениях параметров #529

Closed vitkarpov closed 9 years ago

vitkarpov commented 9 years ago

Фикс для #528

vitkarpov commented 9 years ago

@doochik Заиспользовал encodeURIComponent

doochik commented 9 years ago

Я бы еще написал тест, которые бы проверял, что починилась проблема из тикета

vitkarpov commented 9 years ago

@doochik Добавил тест в update: проверяю, что нода вьюшки появилась, по аналогии с другими тестами

doochik commented 9 years ago

Убрал твой фикс, тест апдейта не завалился. Ну и вообще лучше тесты делать отдельными и изолированными, а не балковыми.

doochik commented 9 years ago

Если надо, я тебе могу помочь с тестом

vitkarpov commented 9 years ago

@doochik

Написал два теста. Первый тест очевидно упадет без фикса, а второй должен проходить в любом случае. Так вот без фикса он упадет, в случае наличия в ключе управляющих символов.

Собственно об этом и баг.

Посмотри, пожалуйста, на тесты. Если считаешь, что нужен другой тест (например, что нужно запустить целиком апдейт и проверить, что не будет исключений в _updateHTML) — давай поговорим об этом.

doochik commented 9 years ago

Переписал тесты на коллекцию, баг был именно там

vitkarpov commented 9 years ago

:+1: Точняк

doochik commented 9 years ago

Поскваш коммиты, чтобы убрать мусор

vitkarpov commented 9 years ago

https://github.com/yandex-ui/noscript/pull/529