Skip to content

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