-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
[REGRESSION] In 1.13.2 "TypeError: Cannot read property 'key' of null at advanceToKey" #11504
Comments
I believe this is related to @cibernox's issue mentioned in tildeio/htmlbars@eeb4bf8#commitcomment-11742721. |
@rwjblue Ah yes- that seek issue is what I'm seeing :) thanks dude! |
@rwjblue I'll let you decide to close this out /or use this to track the regression :) |
Will leave it open here to track. I think we need a reproduction to properly track this down (and get whatever is going wrong under test). |
Keep this open. It's the same issue. My scenario is complex and not easy to reproduce, but I'll try anyway. |
Seems to be the same issue as #11500. |
@btecu - Yep, agreed. I still need help reproducing so we can fix... |
Here is a jsbin reproducing the error: http://jsbin.com/tuxugupoxe/edit?html,js,console,output If you remove the |
I don't have a JSBin of it, but I've run into it while working on an addon. If you pull down knownasilya/pagination-pager#14 and click through the pagination with ellipses (click 50, 49, 48, 47, 46, 45), you'll get this error. See also #11500 |
#11525 provides a useful error in this case until we can fix it upstream in HTMLBars. |
Yesterday I upgraded from 1.12.1 to 1.13.1 w/ no issue. Today I upgraded from 1.13.1 to 1.13.2 and get the error
This occurs as I transition from a detail route to the parent that has a simple each loop like so
I have a special sort mixin that rolls through and sorts by a special attr
I realize a full jsbin is more useful here - I'll see if I can reproduce it for clarity. In the meantime I wanted to get this up quick for others who might be having the same issue
The text was updated successfully, but these errors were encountered: