-
Notifications
You must be signed in to change notification settings - Fork 347
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
#2454 Handle the parsing of json arrays to handle duplicate array values #2455
Conversation
(both orders and extension/value)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor things...
return 0; | ||
} | ||
|
||
public void ParsedPropertyValue(string memberName, int count) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SetPropertyIndex?
|
||
public void Schedule(string key, Action validation) | ||
public int ParsingPropertyValue(string memberName) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
GetPropertyIndex?
fixes #2465 |
(both orders and extension/value)
Description
Update to json parser to handle the case.
Related issues
Fixes issue #2454
Testing
Unit test included to verify functionality
FirelyTeam Checklist