Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Зацикливается при попытке создать несуществующий блок #49

Closed
alexeyten opened this Issue · 4 comments

3 participants

@alexeyten
Collaborator

Такой код приводит к зависанию браузера, если не объявлен блок someblock.

<div data-nb="someblock" class="_init">

Метод Factory.get уходит в бесконечную рекурсию. Нужно проверять, что «сложный класс» действительно сложный, т.е. содержит пробелы.

@pasaran pasaran was assigned
@pasaran
Owner

Я щитаю это нормально )
Это баг в твоем коде явно — все используемые блоки должны быть объявлены.
Конечно, лучше бы оно с эксепшеном падало, но в целом и так неплохо )

Ну ок, будем ошибку кидать, хотя это как-то нехорошо.

@alexeyten
Collaborator

Конечно код нужно писать без ошибок, но в процессе разработки я предпочитаю видеть ошибку в консоли, а не прибивать зависший браузер.

@alexeyten
Collaborator

Я у себя пока добавил строчку

if (names.length < 2) throw "Undefined block  '" + name + "'";
@alexeyten alexeyten referenced this issue from a commit in alexeyten/nanoblocks
@alexeyten alexeyten Fix for #49
Бросаем исключение при попытке создать необъявленный блок.
Всё лучше, чем зависание браузера.
1f0ef72
@chestozo
Collaborator

Это уже давно в общей ветке.

@chestozo chestozo closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.