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

aria.widgets can be undefined #462

Closed
wants to merge 1 commit into from

Conversation

flongo
Copy link
Collaborator

@flongo flongo commented Apr 16, 2013

When no class in aria.widgets package is loaded, aria.widgets is undefined.
Before checking if aria.widgets.AriaSkin is defined, the existence of aria.widgets has to be checked first.

flongo added a commit to flongo/ariatemplates that referenced this pull request Apr 16, 2013
When no class in aria.widgets package is loaded, aria.widgets is undefined.
Before checking if aria.widgets.AriaSkin is defined, the existence of aria.widgets has to be checked first.
@jakub-g
Copy link
Collaborator

jakub-g commented Apr 16, 2013

The same thing can happen also in aria.tools.contextual.ContextualMenu, line 141. Can you please update the pull request?

I thought I fixed that kinds of problems in de7e2ce but it turns out some of them were overlooked.

@flongo
Copy link
Collaborator Author

flongo commented Apr 16, 2013

I think it cannot happen in that case because aria.widgets.Template is in the dependencies of the class, which means thataria.widgets will be available.

Anyway, I will update the class.

flongo added a commit to flongo/ariatemplates that referenced this pull request Apr 16, 2013
When no class in aria.widgets package is loaded, aria.widgets is undefined.
Before checking if aria.widgets.AriaSkin is defined, the existence of aria.widgets has to be checked first.
When no class in aria.widgets package is loaded, aria.widgets is undefined.
Before checking if aria.widgets.AriaSkin is defined, the existence of aria.widgets has to be checked first.
@flongo flongo closed this in f754aa1 Apr 17, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants