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

[Glimmer2] Runtime exception when yielding values within partial #14642

Closed
simonihmig opened this issue Nov 27, 2016 · 4 comments
Closed

[Glimmer2] Runtime exception when yielding values within partial #14642

simonihmig opened this issue Nov 27, 2016 · 4 comments
Labels

Comments

@simonihmig
Copy link
Contributor

This is related to #14263, which was fixed by @chancancode recently. But a new error could be observed, that seems to be very much related.

After the changes from #14633 hasBlock worked again, but I started to get an exception of TypeError: Cannot read property 'child' of null in my build: https://travis-ci.org/kaliber5/ember-bootstrap/builds/179128906#L559. This is coming from VM.pushChildScope, so within Glimmer itself it seems.

Took me quite some time to isolate this, but it seems it happens when yielding values from the partial. Here is a Twiddle that reproduces this: https://ember-twiddle.com/5a29e54ab543a2668f1d725375d2e0ea

@simonihmig
Copy link
Contributor Author

Just added a (failing) test: #14643

@locks locks added the Glimmer2 label Nov 27, 2016
@chancancode
Copy link
Member

Oops. Thank you for reporting! I am looking into it

chancancode added a commit to glimmerjs/glimmer-vm that referenced this issue Nov 27, 2016
- Move the "prelude" compilation into the DSL
- Ensure component layouts using partials always get the caller scope

Fixes emberjs/ember.js#14642
chancancode added a commit that referenced this issue Nov 27, 2016
@chancancode chancancode reopened this Nov 27, 2016
@chancancode
Copy link
Member

chancancode commented Nov 27, 2016

@simonihmig should be fixed now!

chancancode pushed a commit that referenced this issue Nov 27, 2016
(cherry picked from commit 8ec69ba)
@simonihmig
Copy link
Contributor Author

Confirmed, this finally fixes the remaining failing tests for Glimmer2 in ember-bootstrap, all green now! Thanks a lot! 👍

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

No branches or pull requests

3 participants