Skip to content

Support result type both#20

Closed
zordius wants to merge 2 commits into
JSONPath-Plus:masterfrom
zordius:support_resultType_both
Closed

Support result type both#20
zordius wants to merge 2 commits into
JSONPath-Plus:masterfrom
zordius:support_resultType_both

Conversation

@zordius

@zordius zordius commented Mar 27, 2014

Copy link
Copy Markdown

it make developers can do more works on the results.

brettz9 added a commit to brettz9/JSONPath that referenced this pull request Dec 10, 2014
…, not null (also change tests to check for strict equality since test was not catching this). Add a test to actually return null, a valid JSON value. Incorporate issue JSONPath-Plus#20 but with "all" instead of "both" as name to allow for other result types (e.g., I hope to add parent and property name). Stop returning false for falsy unwrapped (unless actually false) as could be null, a legitimate JSON value
brettz9 added a commit to brettz9/JSONPath that referenced this pull request Dec 15, 2014
…so consistent with normal result; if desired, we can change to also add pathAsArray as an option.
@zordius zordius closed this May 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant