Skip to content
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

Get class metadata only when it's needed #3293

Merged
merged 1 commit into from
Dec 2, 2019
Merged

Conversation

norkunas
Copy link
Contributor

@norkunas norkunas commented Dec 2, 2019

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tickets -
License MIT
Doc PR -

Replaces #3292

@norkunas norkunas changed the base branch from master to 2.5 December 2, 2019 13:50
@soyuka soyuka merged commit cfd4e0a into api-platform:2.5 Dec 2, 2019
@soyuka
Copy link
Member

soyuka commented Dec 2, 2019

thanks @norkunas !

@norkunas norkunas deleted the optim branch December 2, 2019 14:44
dunglas added a commit that referenced this pull request Apr 20, 2020
* 2.5:
  Test Location with status code between 300 and 400
  Prevent header Location on POST when status code is not 201 or 30x
  Fix phpcs issues
  Update CHANGELOG.md
  Improve the DQL query in case the values are equals
  Fix cs
  Fix bug Pagination throws 500 when etrying set large value page
  Fix cs, use static::class instead of get_class($this)
  fix cs
  Update path to switch to other API docs.
  missed doctrine type of bottom of filterProperty
  !squash remove type from $fieldType
  Passing custom doctrine type to addWhereByStrategy
  Allow normalization on classes implementing \Traversable interface
  Fixed php-cs-fixer issue
  Do not treat classes implementing \Traversable as collection if defined as resource
  Fix mispelling in hydra spec : hydra:writable to hydra:writeable
  Add a test for exists filter on a collection relation (#3482)
  Use SchemaBuilderInterface in GraphQlExportCommand (#3470)
  Issue-#3293 - fix: changed "hydra:next" apparition condition (#3457]
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.

None yet

2 participants