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.

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

Path param in prefilter function is not useful #19

Closed
ravishivt opened this issue Jun 6, 2014 · 3 comments
Closed

Path param in prefilter function is not useful #19

ravishivt opened this issue Jun 6, 2014 · 3 comments
Labels

Comments

@ravishivt
Copy link
Contributor

The prefilter function often passes an empty path. This param should include the entire path of where the key param exists in the JSON tree. The current empty or one level path is not enough info to make a decision on whether to filter the diff.

@flitbit
Copy link
Collaborator

flitbit commented Jun 6, 2014

I agree with your comment -- that is the intention of the path param so it seems there is a bug. I'll pull it down and add some tests/investigate.

@seanwoodward
Copy link

@ravishivt do you find this happening when you are comparing arrays containing objects or objects with properties that are arrays containing objects?

@flitbit
Copy link
Collaborator

flitbit commented Aug 13, 2014

I've got an update that I'm testing right now; it was indeed the path related to arrays. The fix will be a breaking change to the structure of changes. Any change below an array did not report paths correctly to the prefilter (was always null). More info in the check-in readme when it is ready (final).

@flitbit flitbit added the bug label Aug 13, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants