Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • O openapi-generator
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 3,476
    • Issues 3,476
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 402
    • Merge requests 402
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • OpenAPI Tools
  • openapi-generator
  • Merge requests
  • !12171

[PHP] Replace @ character so we can use Hydra Jsonld models.

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Administrator requested to merge github/fork/warmwaterkruik/php-at-character-hydra into master Apr 19, 2022
  • Overview 0
  • Commits 4
  • Pipelines 0
  • Changes 2

Created by: warmwaterkruik

When using a Hydra schema, where you can have @id and id property in the model, the generated PHP code included duplicate id properties, getter and setter methods. The result was thus an invalid class. The Hydra schema is extensively used in the PHP Api Platform codebase.

This change replaces the @ with at characters, just like it does in the Java generator and Rust generator.

@jebentier @dkarlovi @mandrean @jfastnacht @ybelenko @renepardon

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: github/fork/warmwaterkruik/php-at-character-hydra