Skip to content

[PropertyInfo] Use the right context for methods defined in traits #40811

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 1 commit into from
Apr 14, 2021

Conversation

colinodell
Copy link
Contributor

Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets Fix #34191
License MIT
Doc PR

Pull request #40175 only partially fixed #34191 - it solved the problem for properties used in traits but it did not address the same issue with methods.

I have therefore applied the same style of fix and confirmed it works properly with tests.

@chalasr chalasr added this to the 4.4 milestone Apr 14, 2021
@chalasr
Copy link
Member

chalasr commented Apr 14, 2021

The target branch should be 4.4.

@nicolas-grekas nicolas-grekas changed the base branch from 5.x to 4.4 April 14, 2021 17:11
@nicolas-grekas
Copy link
Member

Thank you @colinodell.

@nicolas-grekas nicolas-grekas merged commit 236e61b into symfony:4.4 Apr 14, 2021
@colinodell colinodell deleted the fix-trait-method-context branch April 14, 2021 19:54
This was referenced May 1, 2021
Zales0123 added a commit to Sylius/Sylius that referenced this pull request May 12, 2021
…ith wrong namespaces of some translation entities (GSadee)

This PR was merged into the 1.9 branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | 1.9
| Bug fix?        | yes
| New feature?    | no/
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | probably introduced in symfony/symfony#40811
| License         | MIT

The problem occurs in Swagger UI docs and there is a problem with resolving proper namespaces during serializing translation entities, e.g.:
![image](https://user-images.githubusercontent.com/6140884/117978269-fb8ca500-b331-11eb-8dcc-08b1e346954a.png)


Commits
-------

dc01355 [API] Conflict with symfony/property-info due to problem with wrong namespaces of some translation entities
SirDomin pushed a commit to SirDomin/Sylius that referenced this pull request Jul 7, 2021
…blem with wrong namespaces of some translation entities (GSadee)

This PR was merged into the 1.9 branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | 1.9
| Bug fix?        | yes
| New feature?    | no/
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | probably introduced in symfony/symfony#40811
| License         | MIT

The problem occurs in Swagger UI docs and there is a problem with resolving proper namespaces during serializing translation entities, e.g.:
![image](https://user-images.githubusercontent.com/6140884/117978269-fb8ca500-b331-11eb-8dcc-08b1e346954a.png)

Commits
-------

dc01355 [API] Conflict with symfony/property-info due to problem with wrong namespaces of some translation entities
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Property info cannot resolve trait namespace
5 participants