Skip to content
This repository has been archived by the owner on Mar 13, 2018. It is now read-only.

Doc should clarify Path requirements for PathObserver, Path #59

Closed
arthurevans opened this issue Jun 9, 2014 · 2 comments
Closed

Doc should clarify Path requirements for PathObserver, Path #59

arthurevans opened this issue Jun 9, 2014 · 2 comments

Comments

@arthurevans
Copy link

e.g., '0' is not a valid path, but '[0]' is.

See Polymer/polymer#526

@tachyon1337
Copy link

Full agreement. There are some that use the Platform but not polymer itself. This undocumented "valid path" change caused a wtf evening of frustration when switched over to the newest version, a switch in large part prompted by Observe being released to Chrome stable. Not sure why backwards compatibility was not retained.

To clarify with an example: whereas before you could do this:

new PathObserver($scope,'profile.0.name')

getPath in the constructor will now return the path as invalid. So you now have to do something like this:

new PathObserver($scope,'profile[0].name')

@jmesserly
Copy link
Contributor

I think this is a dupe of #6

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

No branches or pull requests

3 participants