Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

Add isDefined method #8

Closed
mdevils opened this issue Jul 9, 2013 · 3 comments
Closed

Add isDefined method #8

mdevils opened this issue Jul 9, 2013 · 3 comments

Comments

@mdevils
Copy link
Contributor

mdevils commented Jul 9, 2013

No matter if it will be sync or async. Choose the better way.

@dfilatov
Copy link
Member

dfilatov commented Jul 9, 2013

isDefined?

@dfilatov
Copy link
Member

dfilatov commented Jul 9, 2013

Мне тут чего-то кажется, что второй колбэк в modules.require будет поконсистентнее как-то, так как будет учитывать все модули, объявленные в пределах поколения. Или опиши поподробнее задачу, для которой тебе это понадобилось.

@mdevils
Copy link
Contributor Author

mdevils commented Jul 9, 2013

Я хочу получить класс по переданному имени модуля. А если такого нет, то вставить вместо него другой класс. Сейчас при любой ошибке о ненахождении модуля (если я пишу require в коллбеке другого require), modules падает в бесконечный асинхронный цикл, кидающий ошибку, которую не поймать.

dfilatov added a commit that referenced this issue Jul 10, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants