Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Debug] Fix same vendor detection in class loader #24570

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
Oct 16, 2017
Merged

[Debug] Fix same vendor detection in class loader #24570

merged 1 commit into from
Oct 16, 2017

Conversation

Jean-Beru
Copy link
Contributor

Q A
Branch? 2.7
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets
License MIT
Doc PR

Fix about same vendor detection in ClassLoader. Actually, detected namespace for Doctrine\ORM\Configuration is Doctrine\ORM instead of Doctrine\. So deprecations are triggered for classes in same namespace.

Copy link
Member

@nicolas-grekas nicolas-grekas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this will redefine @deprecated, @internal, @final as being allowed within the same 1st level vendor, whereas we looked for two levels before. Looks legitimate to me.

@fabpot
Copy link
Member

fabpot commented Oct 16, 2017

Thank you @Jean-Beru.

@fabpot fabpot merged commit d2ab0d8 into symfony:2.7 Oct 16, 2017
fabpot added a commit that referenced this pull request Oct 16, 2017
This PR was merged into the 2.7 branch.

Discussion
----------

[Debug] Fix same vendor detection in class loader

| Q             | A
| ------------- | ---
| Branch?       | 2.7
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

Fix about same vendor detection in ClassLoader. Actually, detected namespace for `Doctrine\ORM\Configuration` is `Doctrine\ORM` instead of `Doctrine\`. So deprecations are triggered for classes in same namespace.

Commits
-------

d2ab0d8 [Debug] Fix same vendor detection in class loader
This was referenced Oct 30, 2017
This was referenced Nov 10, 2017
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.

4 participants