Skip to content

Missing use of required Expansion input variable in expand() #374

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

Closed
kasei opened this issue Feb 14, 2020 · 2 comments · Fixed by #377
Closed

Missing use of required Expansion input variable in expand() #374

kasei opened this issue Feb 14, 2020 · 2 comments · Fixed by #377

Comments

@kasei
Copy link
Contributor

kasei commented Feb 14, 2020

(Related to #265, #356)

expand() step 7:

Set expanded output to the result of using the Expansion algorithm, passing the active context, document from remote document or input if there is no remote document as element, documentUrl as base URL, and frameExpansion and and ordered flags from options.

Expansion 5.1.2 says active property is a required input variable, but it is not supplied at this callsite.

@gkellogg
Copy link
Member

Updated in #377.

@kasei
Copy link
Contributor Author

kasei commented Feb 15, 2020

@gkellogg #377 looks good. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants