Skip to content

twice the amount of expected elements are created inside a nested if / repeat #14460

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
sethladd opened this issue Oct 25, 2013 · 7 comments
Closed
Assignees
Milestone

Comments

@sethladd
Copy link
Contributor

Please see the repro here: https://gist.github.com/sethladd/7161294

Also, here's the public thread: https://groups.google.com/a/dartlang.org/forum/#!topic/web-ui/JY46fZIKZ98

@sethladd
Copy link
Contributor Author

Sorry, here's the details:

I have a three element list. I repeat over that list. The custom element has a ready() callback that prints when ready() is called.

I see 6 printed lines in the console.

I am a null (ID: 177863854) and I am ready
I am a null (ID: 924578920) and I am ready
I am a null (ID: 394322034) and I am ready
I am a null (ID: 854832235) and I am ready
I am a null (ID: 587923908) and I am ready
I am a null (ID: 763736301) and I am ready

I was expecting to see only three lines in console output.

@sigmundch
Copy link
Member

Added this to the Later milestone.

@jmesserly
Copy link

Set owner to @jmesserly.
Added Accepted label.

@sigmundch
Copy link
Member

I'm stealing this from you =)

I've been looking at another bug that has some similarities with this (see issue #14459), but I'm not sure if they are the same


Set owner to @sigmundch.

@sigmundch
Copy link
Member

I just tried this again, and it seems to be running ok. This might have been fixed with all the updates John did to node-bind and template-binding in 0.8.8.

Similarly the other bug I found last week no longer reproduces.

I'll mark as fixed, but please reopen if you run into this again.


Added Fixed label.

@sigmundch
Copy link
Member

Removed this from the Later milestone.
Added this to the M8 milestone.

@sethladd
Copy link
Contributor Author

Thanks for checking!

This issue was closed.
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

No branches or pull requests

4 participants