Skip to content

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