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.

Sign up for GitHub

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

Jump to bottom

Compiler error "Cannot read property 'name' of null" with two-way-binding + brackets #602

Closed
TehShrike opened this issue May 28, 2017 · 1 comment
Closed

Compiler error "Cannot read property 'name' of null" with two-way-binding + brackets #602

TehShrike opened this issue May 28, 2017 · 1 comment

Comments

@TehShrike
Copy link
Member

Svelte version 1.20.2

Template snippet:

<input type='checkbox' bind:checked='todo["done"]'>

Expected behavior: should compile as happily as if I had used bind:checked='todo.done'

Observed behavior: compiler errors out with Cannot read property 'name' of null

Reproduction: https://svelte.technology/repl?version=1.20.2&gist=6bc409d8cf45766230d012b355ebcf7c
TehShrike added a commit to TehShrike/way-over-the-top that referenced this issue May 28, 2017
@TehShrike
Refactoring the components
93bf1b9 
- in ChannelList, moved the provider data from the template to a data structure
- made a Checkbox component because of this bug: sveltejs/svelte#602
- changed provider-channels to export an array instead of an object with a single array property
- added svelte as an explicit dependency for version pinning
@TehShrike TehShrike mentioned this issue May 28, 2017
Merged
Rich-Harris added a commit that referenced this issue May 28, 2017
@Rich-Harris
support binding to computed member expressions (fixes #602)
2a92b36
@Rich-Harris Rich-Harris mentioned this issue May 28, 2017
Merged
@TehShrike
Copy link
Member Author

TehShrike commented May 28, 2017 •
edited

haha, I was about to comment here noting that Rich confirmed this wasn't a bug, and here's a recent commit from Rich purporting to add the ability to bind to computed properties. Kickass.

Rich-Harris added a commit that referenced this issue Jun 1, 2017
@Rich-Harris
Merge pull request #605 from sveltejs/gh-602
66a1fd7 
support binding to computed member expressions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@TehShrike