Skip to content

Fixed BC Layer in DoctrineChoiceLoader #19241

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
Jun 30, 2016
Merged

Fixed BC Layer in DoctrineChoiceLoader #19241

merged 1 commit into from
Jun 30, 2016

Conversation

Rootie
Copy link
Contributor

@Rootie Rootie commented Jun 30, 2016

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

in the case when the BC Layer is used $idReader is not set to the right value.

in the case when the BC Layer is used $idReader is not set to the right value.
@fabpot
Copy link
Member

fabpot commented Jun 30, 2016

Thank you @Rootie.

@fabpot fabpot merged commit c44d71c into symfony:master Jun 30, 2016
fabpot added a commit that referenced this pull request Jun 30, 2016
This PR was merged into the 3.2-dev branch.

Discussion
----------

Fixed BC Layer in DoctrineChoiceLoader

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

in the case when the BC Layer is used $idReader is not set to the right value.

Commits
-------

c44d71c Fixed BC Layer in DoctrineChoiceLoader
@xabbuh
Copy link
Member

xabbuh commented Jun 30, 2016

Where tests failing before? Otherwise we should add a test case to avoid future regressions.

@HeahDude
Copy link
Contributor

Thanks @Rootie for that PR!

@fabpot fabpot mentioned this pull request Oct 27, 2016
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.

5 participants