Skip to content

Post Processor #16

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

Merged
merged 8 commits into from
Jul 14, 2020
Merged

Post Processor #16

merged 8 commits into from
Jul 14, 2020

Conversation

wol-soft
Copy link
Owner

Add post processors to the model generator

…er the generation process.

Moved serialization logic from model template into a post processor
@wol-soft wol-soft added the enhancement New feature or request label Jul 10, 2020
@wol-soft wol-soft self-assigned this Jul 10, 2020
@coveralls
Copy link

coveralls commented Jul 10, 2020

Pull Request Test Coverage Report for Build 280

  • 376 of 378 (99.47%) changed or added relevant lines in 44 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.4%) to 98.2%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/Model/Property/PropertyProxy.php 0 1 0.0%
src/PropertyProcessor/Property/ReferenceProcessor.php 4 5 80.0%
Totals Coverage Status
Change from base Build 262: 0.4%
Covered Lines: 1582
Relevant Lines: 1611

💛 - Coveralls

@wol-soft wol-soft added this to the 0.17.0 milestone Jul 10, 2020
wol-soft and others added 6 commits July 12, 2020 17:45
Move base validators to a separate function inside the generated models
Add post processor documentation
…wrap the schema into a JsonSchema object to provide additional meta data.

The JsonSchema object is additionally associated with Schema and Property objects to determine the scope of the generated code later (eg. in Post-Processors or to attach debug information to exceptions).
@wol-soft wol-soft merged commit 6547053 into master Jul 14, 2020
@wol-soft wol-soft deleted the PostProcessor branch July 14, 2020 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants