Skip to content

Allow to mark JSON properties & XML nodes as optional (#102) #107

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 6 commits into from
Sep 4, 2017

Conversation

norberttech
Copy link
Member

  • Add optional expander definition

  • Add hasExpander method for Pattern

  • Allow mark JSON properties & xml nodes as optional

  • Use intuitive matcher type definition in Xml test stock qty node definition

  • Add information about optional Expander to README

  • Remove return type declaration

  • Catch PHPMatcher Exception when parsing pattern for optional values

  • Add getName to PatternExpander interface

  • Remove unncessary ExpanderInitializer dependency in TypePattern

  • Use constants for expander name instead of method

  • Redeclare $expanderDefinitions using constants

  • Add static is($name) method to PatternExpander interface

* Add optional expander definition

* Add hasExpander method for Pattern

* Allow mark JSON properties & xml nodes as optional

* Use intuitive matcher type definition in Xml test stock qty node definition

* Add information about optional Expander to README

* Remove return type declaration

* Catch PHPMatcher Exception when parsing pattern for optional values

* Add getName to PatternExpander interface

* Remove unncessary ExpanderInitializer dependency in TypePattern

* Use constants for expander name instead of method

* Redeclare $expanderDefinitions using constants

* Add static is($name) method to PatternExpander interface
@norberttech norberttech merged commit 79a27d1 into 2.3 Sep 4, 2017
@norberttech norberttech deleted the optional-json branch February 4, 2018 21:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants