Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

fix($compile): Allow literals in isolate scope references #5385

Merged
merged 1 commit into from
Dec 13, 2013

Conversation

tbosch
Copy link
Contributor

@tbosch tbosch commented Dec 12, 2013

When a component uses an isolate scope reference
and the the component is used with an object literal
a new object is created on every evaluation.
Therefore the compiler needs to compare
the values of the parent and the isolate scope
using object equality and not object reference
equality.

Fixes #5296.

@tbosch
Copy link
Contributor Author

tbosch commented Dec 13, 2013

Checked the travis output and there were no errors. Travis says at the end though:

No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.
The build has been terminated

@btford
Copy link
Contributor

btford commented Dec 13, 2013

Add tests for: null and numeric types, then LGTM.

When a component uses an isolate scope reference
and the the component is used with an object literal
a new object is created on every evaluation.
Therefore the compiler needs to compare
the values of the parent and the isolate scope
using object equality and not object reference
equality.

Fixes angular#5296.
@tbosch tbosch merged commit 43072e3 into angular:master Dec 13, 2013
@tbosch tbosch deleted the isolate-literal-binding branch December 13, 2013 00:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using JSON in HTML attribute throws error
2 participants